)]}'
{"neutron/tests/tempest/api/admin/test_shared_network_extension.py":[{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"44d0edf71d7c0528e94a93bd94eaa9bbfaf14f14","unresolved":false,"context_lines":[{"line_number":50,"context_line":"        self.assertNotEmpty(items)"},{"line_number":51,"context_line":"        self.assertTrue(all(n[\u0027shared\u0027] \u003d\u003d shared for n in items))"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    def _list_subnets_ids(self, user, shared):"},{"line_number":54,"context_line":"        body \u003d user.list_subnets(shared\u003dshared)"},{"line_number":55,"context_line":"        return [subnet[\u0027id\u0027] for subnet in body[\u0027subnets\u0027]]"},{"line_number":56,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"1f013ff3_c7bb44e3","line":53,"range":{"start_line":53,"start_character":32,"end_line":53,"end_character":36},"updated":"2017-05-15 07:25:51.000000000","message":"nit: consider \u0027client\u0027","commit_id":"d95940f344f6b8ee32e2b192506d5521d77a04d5"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"68bb79a2f8f2a993040446add0d3dcbf580bb64f","unresolved":false,"context_lines":[{"line_number":50,"context_line":"        self.assertNotEmpty(items)"},{"line_number":51,"context_line":"        self.assertTrue(all(n[\u0027shared\u0027] \u003d\u003d shared for n in items))"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    def _list_subnets_ids(self, user, shared):"},{"line_number":54,"context_line":"        body \u003d user.list_subnets(shared\u003dshared)"},{"line_number":55,"context_line":"        return [subnet[\u0027id\u0027] for subnet in body[\u0027subnets\u0027]]"},{"line_number":56,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"1f013ff3_fceea0b2","line":53,"range":{"start_line":53,"start_character":32,"end_line":53,"end_character":36},"in_reply_to":"1f013ff3_c7bb44e3","updated":"2017-05-16 16:17:00.000000000","message":"yeah. In L120 below there is a method _list_shared_networks where they use user to refer to the client. I was trying to be consistent. But you are right, client is better :-)","commit_id":"d95940f344f6b8ee32e2b192506d5521d77a04d5"}]}