)]}'
{"networking_ovn/tests/unit/ml2/test_mech_driver.py":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"fb80b6ef59e8d573a38ce4ae746e250c14e1ac31","unresolved":false,"context_lines":[{"line_number":1749,"context_line":"            new_port \u003d plugin.update_port(ctx, port_id, {\"port\": {}})"},{"line_number":1750,"context_line":"            # NOTE(mjozefcz): This is only one difference between neutron"},{"line_number":1751,"context_line":"            # and ovn test. This class is runned against real OVN mechanism"},{"line_number":1752,"context_line":"            # driver that adds information to the new_port dict."},{"line_number":1753,"context_line":"            new_port.pop(\u0027network\u0027)"},{"line_number":1754,"context_line":"            self.assertEqual(port[\"port\"], new_port)"},{"line_number":1755,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"7faddb67_3bf79a8f","line":1752,"updated":"2019-08-08 08:50:01.000000000","message":"We are overwriting the test here, not sure if it\u0027s ideal but I would say let\u0027s leave a FIXME note here so we can remove it later or fix/adapt the test in neutron.\n\nI see it was failing due to a mock_call_count we perhaps could change neutron to verify the calls using assert_has_calls() instead ? Just an idea.","commit_id":"9eb4b19ec85fc7ad6ac752dbc4d565783ac71cd5"},{"author":{"_account_id":24791,"name":"Maciej Jozefczyk","email":"jeicam.pl@gmail.com","username":"maciej.jozefczyk"},"change_message_id":"31bfacdcf5f4a3ac04cbada3ec57099128c92497","unresolved":false,"context_lines":[{"line_number":1749,"context_line":"            new_port \u003d plugin.update_port(ctx, port_id, {\"port\": {}})"},{"line_number":1750,"context_line":"            # NOTE(mjozefcz): This is only one difference between neutron"},{"line_number":1751,"context_line":"            # and ovn test. This class is runned against real OVN mechanism"},{"line_number":1752,"context_line":"            # driver that adds information to the new_port dict."},{"line_number":1753,"context_line":"            new_port.pop(\u0027network\u0027)"},{"line_number":1754,"context_line":"            self.assertEqual(port[\"port\"], new_port)"},{"line_number":1755,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"7faddb67_9bec8e90","line":1752,"in_reply_to":"7faddb67_3bf79a8f","updated":"2019-08-08 08:59:53.000000000","message":"Hmm, I\u0027ve added the related fixme info.\nIt is failing because of different dicts in L1754. Our ml2 mechanism driver adds \u0027network\u0027 key to the new_port dict.","commit_id":"9eb4b19ec85fc7ad6ac752dbc4d565783ac71cd5"}]}