)]}'
{"manilaclient/tests/functional/test_share_networks.py":[{"author":{"_account_id":18058,"name":"Lucio Seki","email":"lseki@redhat.com","username":"lseki"},"change_message_id":"196929513c7175b97dfe5395ddf01213caffb0eb","unresolved":false,"context_lines":[{"line_number":125,"context_line":"            self.assertEqual(self.neutron_net_id, get[\u0027neutron_net_id\u0027])"},{"line_number":126,"context_line":"            self.assertEqual(self.neutron_subnet_id, get[\u0027neutron_subnet_id\u0027])"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    def _get_update_expected_data(self, net_data, net_creation_data):"},{"line_number":129,"context_line":"        default_return_value \u003d ("},{"line_number":130,"context_line":"            None if utils.share_network_subnets_are_supported() else \u0027None\u0027)"},{"line_number":131,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_92c3d238","line":128,"range":{"start_line":128,"start_character":8,"end_line":128,"end_character":33},"updated":"2019-10-23 14:38:44.000000000","message":"nit: sounds like it gets and updates something. You may rename it to _get_expected_update_data","commit_id":"eafe938e3b0dd9332692bcc44621ebddecdf992b"},{"author":{"_account_id":18058,"name":"Lucio Seki","email":"lseki@redhat.com","username":"lseki"},"change_message_id":"196929513c7175b97dfe5395ddf01213caffb0eb","unresolved":false,"context_lines":[{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    def _get_update_expected_data(self, net_data, net_creation_data):"},{"line_number":129,"context_line":"        default_return_value \u003d ("},{"line_number":130,"context_line":"            None if utils.share_network_subnets_are_supported() else \u0027None\u0027)"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"        expected_nn_id \u003d ("},{"line_number":133,"context_line":"            default_return_value"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_929c326b","line":130,"range":{"start_line":130,"start_character":0,"end_line":130,"end_character":76},"updated":"2019-10-23 14:38:44.000000000","message":"nit: can we have a NOTE explaining why do we use \u0027None\u0027 string instead of None type when subnets are not supported?","commit_id":"eafe938e3b0dd9332692bcc44621ebddecdf992b"}]}