)]}'
{"neutron/tests/unit/nec/test_nec_plugin.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a2ff744428970c984d11f6236d9ce41862dffba1","unresolved":false,"context_lines":[{"line_number":80,"context_line":"        # NOTE: `test_config\u0027 is global, and most tests don\u0027t set"},{"line_number":81,"context_line":"        # test_config[\u0027config_files\u0027] but read this in setUp()."},{"line_number":82,"context_line":"        # So clean test_config[\u0027config_files\u0027] ASAP, to avoid side effects"},{"line_number":83,"context_line":"        # on other tests which are running at the same time."},{"line_number":84,"context_line":"        self._clean_nec_ini()"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"        self.plugin \u003d manager.NeutronManager.get_plugin()"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAP3%2F%2FzmU%3D","line":83,"updated":"2013-07-11 05:50:36.000000000","message":"Please file a problem with global test_config as a bug? I think this issue is not a part of this patch, but it is worth filing it.","commit_id":"3273b07a0bd775191ca12e6face36b3e87eb6343"},{"author":{"_account_id":1894,"name":"Ryota MIBU","email":"r-mibu@cq.jp.nec.com","username":"r-mibu"},"change_message_id":"4f5efb1a0ad46cbfae1e1da56ee0fa33b407083c","unresolved":false,"context_lines":[{"line_number":80,"context_line":"        # NOTE: `test_config\u0027 is global, and most tests don\u0027t set"},{"line_number":81,"context_line":"        # test_config[\u0027config_files\u0027] but read this in setUp()."},{"line_number":82,"context_line":"        # So clean test_config[\u0027config_files\u0027] ASAP, to avoid side effects"},{"line_number":83,"context_line":"        # on other tests which are running at the same time."},{"line_number":84,"context_line":"        self._clean_nec_ini()"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"        self.plugin \u003d manager.NeutronManager.get_plugin()"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAP3%2F%2Fzbc%3D","line":83,"in_reply_to":"AAAAP3%2F%2FzmU%3D","updated":"2013-07-11 07:17:03.000000000","message":"will do","commit_id":"3273b07a0bd775191ca12e6face36b3e87eb6343"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a2ff744428970c984d11f6236d9ce41862dffba1","unresolved":false,"context_lines":[{"line_number":396,"context_line":""},{"line_number":397,"context_line":"        net \u003d None"},{"line_number":398,"context_line":"        ctx \u003d mock.ANY"},{"line_number":399,"context_line":"        with self.network(do_delete\u003dFalse) as network:"},{"line_number":400,"context_line":"            net \u003d network[\u0027network\u0027]"},{"line_number":401,"context_line":"            net_ref \u003d self._show(\u0027networks\u0027, net[\u0027id\u0027])"},{"line_number":402,"context_line":"            self.assertEqual(net_ref[\u0027network\u0027][\u0027status\u0027], \u0027ERROR\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAP3%2F%2FzmA%3D","line":399,"updated":"2013-07-11 05:50:36.000000000","message":"There is not so many case where do_delete\u003dFalse is required. It is helpful to add a comment why do_delete\u003dFalse is needed here.","commit_id":"3273b07a0bd775191ca12e6face36b3e87eb6343"},{"author":{"_account_id":1894,"name":"Ryota MIBU","email":"r-mibu@cq.jp.nec.com","username":"r-mibu"},"change_message_id":"4f5efb1a0ad46cbfae1e1da56ee0fa33b407083c","unresolved":false,"context_lines":[{"line_number":396,"context_line":""},{"line_number":397,"context_line":"        net \u003d None"},{"line_number":398,"context_line":"        ctx \u003d mock.ANY"},{"line_number":399,"context_line":"        with self.network(do_delete\u003dFalse) as network:"},{"line_number":400,"context_line":"            net \u003d network[\u0027network\u0027]"},{"line_number":401,"context_line":"            net_ref \u003d self._show(\u0027networks\u0027, net[\u0027id\u0027])"},{"line_number":402,"context_line":"            self.assertEqual(net_ref[\u0027network\u0027][\u0027status\u0027], \u0027ERROR\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAP3%2F%2FzWI%3D","line":399,"in_reply_to":"AAAAP3%2F%2FzmA%3D","updated":"2013-07-11 07:17:03.000000000","message":"Done","commit_id":"3273b07a0bd775191ca12e6face36b3e87eb6343"}],"neutron/tests/unit/nec/test_security_group.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a2ff744428970c984d11f6236d9ce41862dffba1","unresolved":false,"context_lines":[{"line_number":30,"context_line":"PLUGIN_NAME \u003d test_nec_plugin.PLUGIN_NAME"},{"line_number":31,"context_line":"OFC_MANAGER \u003d (\u0027neutron.plugins.nec.nec_plugin.ofc_manager.OFCManager\u0027)"},{"line_number":32,"context_line":"AGENT_NAME \u003d (\u0027neutron.plugins.nec.agent.nec_neutron_agent.NECNeutronAgent\u0027)"},{"line_number":33,"context_line":"NOTIFIER \u003d (\u0027neutron.plugins.nec.nec_plugin.NECPluginV2AgentNotifierApi\u0027)"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"class NecSecurityGroupsTestCase(test_sg.SecurityGroupDBTestCase):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAP3%2F%2Fzko%3D","line":33,"updated":"2013-07-11 05:50:36.000000000","message":"The parenthesis at L.31-33 is unnecessary. It is a good chance to clean up them.","commit_id":"3273b07a0bd775191ca12e6face36b3e87eb6343"},{"author":{"_account_id":1894,"name":"Ryota MIBU","email":"r-mibu@cq.jp.nec.com","username":"r-mibu"},"change_message_id":"4f5efb1a0ad46cbfae1e1da56ee0fa33b407083c","unresolved":false,"context_lines":[{"line_number":30,"context_line":"PLUGIN_NAME \u003d test_nec_plugin.PLUGIN_NAME"},{"line_number":31,"context_line":"OFC_MANAGER \u003d (\u0027neutron.plugins.nec.nec_plugin.ofc_manager.OFCManager\u0027)"},{"line_number":32,"context_line":"AGENT_NAME \u003d (\u0027neutron.plugins.nec.agent.nec_neutron_agent.NECNeutronAgent\u0027)"},{"line_number":33,"context_line":"NOTIFIER \u003d (\u0027neutron.plugins.nec.nec_plugin.NECPluginV2AgentNotifierApi\u0027)"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"class NecSecurityGroupsTestCase(test_sg.SecurityGroupDBTestCase):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAP3%2F%2Fzbw%3D","line":33,"in_reply_to":"AAAAP3%2F%2Fzko%3D","updated":"2013-07-11 07:17:03.000000000","message":"Done","commit_id":"3273b07a0bd775191ca12e6face36b3e87eb6343"}]}