)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":21273,"name":"Ilya Chukhnakov","email":"Ilya.chukhnakov@gmail.com","username":"ichukhnakov"},"change_message_id":"f8520ab2b964019ff0ffff4558bc3317d2547aef","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add test that ensures subnet is usable after update"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adds a simple test to make sure a port can be created"},{"line_number":10,"context_line":"on a subnet after disabling DHCP."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Related-Bug: #1573443"},{"line_number":13,"context_line":"Change-Id: I94f1a59e52b109984904fd9e56e3c1a3f1c270fe"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"bab6814e_a0637840","line":10,"range":{"start_line":10,"start_character":12,"end_line":10,"end_character":27},"updated":"2016-05-19 09:43:19.000000000","message":"nit: it is disabled in the test from the start and then you enable it.","commit_id":"d2eb6f70ac111d672a0f5ae85565921f82e28108"}],"neutron/tests/tempest/api/test_ports.py":[{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"2aa9fd8e5f7d1ff61c0e7b6e645f18343e0b631e","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        self.assertEqual(\u0027d2\u0027, body[\u0027description\u0027])"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    @test.idempotent_id(\u0027c72c1c0c-2193-4aca-bbb4-b1442640c123\u0027)"},{"line_number":44,"context_line":"    def test_change_dhcp_flag_then_create_port(self):"},{"line_number":45,"context_line":"        s \u003d self.create_subnet(self.network, enable_dhcp\u003dFalse)"},{"line_number":46,"context_line":"        self.create_port(self.network)"},{"line_number":47,"context_line":"        self.client.update_subnet(s[\u0027id\u0027], enable_dhcp\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":2,"id":"bab6814e_2b83fd05","line":44,"updated":"2016-05-19 00:31:15.000000000","message":"If you\u0027re simply checking that create_port doesn\u0027t explode, might be worth add a note or rename the test method to be self-explanatory","commit_id":"d2eb6f70ac111d672a0f5ae85565921f82e28108"},{"author":{"_account_id":7448,"name":"Carl Baldwin","email":"carl@ecbaldwin.net","username":"carl-baldwin"},"change_message_id":"8e40ae4fcdf3ca34c609d5263af1c97e1c1a13d9","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        self.assertEqual(\u0027d2\u0027, body[\u0027description\u0027])"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    @test.idempotent_id(\u0027c72c1c0c-2193-4aca-bbb4-b1442640c123\u0027)"},{"line_number":44,"context_line":"    def test_change_dhcp_flag_then_create_port(self):"},{"line_number":45,"context_line":"        s \u003d self.create_subnet(self.network, enable_dhcp\u003dFalse)"},{"line_number":46,"context_line":"        self.create_port(self.network)"},{"line_number":47,"context_line":"        self.client.update_subnet(s[\u0027id\u0027], enable_dhcp\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":2,"id":"bab6814e_34012a06","line":44,"in_reply_to":"bab6814e_2b83fd05","updated":"2016-05-20 19:09:20.000000000","message":"Yeah, tests without assertions always raise my eyebrows","commit_id":"d2eb6f70ac111d672a0f5ae85565921f82e28108"}]}