)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"2a82dc5e7cfe2ce0852954812fbb27508b4777d8","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Refactor TestShowNetwork: Use TestNetwork in TestShowNetwork"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There are severail problems in TestShowNetwork:"},{"line_number":10,"context_line":"1. NETWORK_ITEM is not used."},{"line_number":11,"context_line":"2. Some redundant code."},{"line_number":12,"context_line":"3. The param n_mock is used to fake find_attr(), but the name is confusing."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a8ffd7b_db401b0a","line":9,"updated":"2015-12-01 13:39:29.000000000","message":"nit...typo on \u0027severail\u0027","commit_id":"ee9192ef00a61f3497b305c491951f18cf46012a"}],"openstackclient/tests/network/v2/test_network.py":[{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"4d8c861a69b4d1acd32f8f786f7c3e560da36619","unresolved":false,"context_lines":[{"line_number":478,"context_line":"        # Get the command object to test"},{"line_number":479,"context_line":"        self.cmd \u003d network.ShowNetwork(self.app, self.namespace)"},{"line_number":480,"context_line":""},{"line_number":481,"context_line":"    def test_show_no_options(self, find_attr):"},{"line_number":482,"context_line":"        arglist \u003d ["},{"line_number":483,"context_line":"            FAKE_NAME,"},{"line_number":484,"context_line":"        ]"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_90801848","line":481,"updated":"2015-12-06 02:57:40.000000000","message":"Yeh, n_mock was a bad name, but find_mock would of been better.","commit_id":"0cc1492ccdf44312ea50630d92ac121ea8e579a8"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"81df028a2935e880d2fac324cac3a7e246086f61","unresolved":false,"context_lines":[{"line_number":478,"context_line":"        # Get the command object to test"},{"line_number":479,"context_line":"        self.cmd \u003d network.ShowNetwork(self.app, self.namespace)"},{"line_number":480,"context_line":""},{"line_number":481,"context_line":"    def test_show_no_options(self, find_attr):"},{"line_number":482,"context_line":"        arglist \u003d ["},{"line_number":483,"context_line":"            FAKE_NAME,"},{"line_number":484,"context_line":"        ]"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_d3622aad","line":481,"in_reply_to":"7a740942_90801848","updated":"2015-12-06 07:14:10.000000000","message":"Yes, it is better. But this will be removed anyway in the end, so no need to change, I think. :)\n\nhttps://review.openstack.org/#/c/253348/","commit_id":"0cc1492ccdf44312ea50630d92ac121ea8e579a8"}]}