)]}'
{"neutron_fwaas/tests/unit/services/firewall/freescale/test_fwaas_plugin.py":[{"author":{"_account_id":490,"name":"Sumit Naiksatam","email":"sumitnaiksatam@gmail.com","username":"snaiksat"},"change_message_id":"1b5a66c910b5d2a162c7b31255d500bc1f38075c","unresolved":false,"context_lines":[{"line_number":95,"context_line":"            self.plugin.delete_firewall(self.ctx, fw_id)"},{"line_number":96,"context_line":"            self.clnt.delete_firewall.assert_called_once_with(fw_id)"},{"line_number":97,"context_line":"            self.callbacks.firewall_deleted(self.ctx, fw_id)"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"    def test_create_update_delete_firewall_rule(self):"},{"line_number":100,"context_line":"        \"\"\"Testing create, update and delete firewall rule.\"\"\""},{"line_number":101,"context_line":"        ctx \u003d context.get_admin_context()"}],"source_content_type":"text/x-python","patch_set":2,"id":"dac4157f_9d0dc792","line":98,"updated":"2015-04-21 20:00:44.000000000","message":"why are we overriding this for the freescale plugin?","commit_id":"d3a00def8933818f65bdb2e7f369276153c1baf0"},{"author":{"_account_id":13702,"name":"yushiro","username":"yushiro","inactive":true},"change_message_id":"b3818a754fd6e589bef0ceede342a00ad6cb9fff","unresolved":false,"context_lines":[{"line_number":95,"context_line":"            self.plugin.delete_firewall(self.ctx, fw_id)"},{"line_number":96,"context_line":"            self.clnt.delete_firewall.assert_called_once_with(fw_id)"},{"line_number":97,"context_line":"            self.callbacks.firewall_deleted(self.ctx, fw_id)"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"    def test_create_update_delete_firewall_rule(self):"},{"line_number":100,"context_line":"        \"\"\"Testing create, update and delete firewall rule.\"\"\""},{"line_number":101,"context_line":"        ctx \u003d context.get_admin_context()"}],"source_content_type":"text/x-python","patch_set":2,"id":"dac4157f_0b92809e","line":98,"in_reply_to":"dac4157f_9d0dc792","updated":"2015-04-22 05:23:21.000000000","message":"Hi, sumit\n\nI\u0027m sorry I don\u0027t know the style in such case.\nPlease let me know if I am mistaken.\nIf it is not overwritten,  this test failed when service-plugin is freescale.\nBecause the behavior of mock method is different between\ndefault and freescale-plugin.\n\n* default:\n\nwhen calling firewall() with arg \"do_delete\u003dTrue\", the firewall will be deleted\nafter firewall() by using  self._delete() at L.286\n\n* freescale-plugin:\n\nwhen calling firewall() with arg \"do_delete\u003dTrue\", firewall will not be deleted\nafter firewall() by using  self._delete() at L.286\nThough the firewall status is \"PENDING_DELETE\", the firewall still exist\nafter calling self._delete()","commit_id":"d3a00def8933818f65bdb2e7f369276153c1baf0"}]}