)]}'
{"tempest/api/network/test_tags.py":[{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"a678b5654d42fd047e924dd84a43d6c61e712148","unresolved":false,"context_lines":[{"line_number":164,"context_line":"            self.assertEqual(tag_names[i], retrieved_tags[0])"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"            # Check that the expected tag exists for each resource."},{"line_number":167,"context_line":"            resource_object \u003d getattr(self, resource[:-1])"},{"line_number":168,"context_line":"            self.tags_client.check_tag_existence("},{"line_number":169,"context_line":"                resource, resource_object[\u0027id\u0027], tag_names[i])"},{"line_number":170,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5f201791_6ac85f21","line":167,"range":{"start_line":167,"start_character":12,"end_line":167,"end_character":58},"updated":"2017-06-23 06:34:49.000000000","message":"this can be moved to after line #161, so you can use resource_object in line #162","commit_id":"ce2c852d176539703c1bb3ba9cbca521c23d560a"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"cece59fbf816e6ba8e4d552893a09aa95a424109","unresolved":false,"context_lines":[{"line_number":164,"context_line":"            self.assertEqual(tag_names[i], retrieved_tags[0])"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"            # Check that the expected tag exists for each resource."},{"line_number":167,"context_line":"            resource_object \u003d getattr(self, resource[:-1])"},{"line_number":168,"context_line":"            self.tags_client.check_tag_existence("},{"line_number":169,"context_line":"                resource, resource_object[\u0027id\u0027], tag_names[i])"},{"line_number":170,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5f201791_f570c6a7","line":167,"range":{"start_line":167,"start_character":12,"end_line":167,"end_character":58},"in_reply_to":"5f201791_6ac85f21","updated":"2017-06-23 14:49:18.000000000","message":"Nice catch, fixed in next patch.","commit_id":"ce2c852d176539703c1bb3ba9cbca521c23d560a"},{"author":{"_account_id":23659,"name":"Jaewoo Park","email":"jaewoopark91@gmail.com","username":"aselius"},"change_message_id":"d6d28d68c1469b5dd8e755ddf46d4755dae588b8","unresolved":false,"context_lines":[{"line_number":163,"context_line":"            resource_object \u003d getattr(self, resource[:-1])"},{"line_number":164,"context_line":"            retrieved_tags \u003d self.tags_client.list_tags("},{"line_number":165,"context_line":"                resource, resource_object[\u0027id\u0027])[\u0027tags\u0027]"},{"line_number":166,"context_line":"            self.assertEqual(1, len(retrieved_tags))"},{"line_number":167,"context_line":"            self.assertEqual(tag_names[i], retrieved_tags[0])"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"            # Check that the expected tag exists for each resource."},{"line_number":170,"context_line":"            self.tags_client.check_tag_existence("}],"source_content_type":"text/x-python","patch_set":3,"id":"5f201791_49ec7ff2","line":167,"range":{"start_line":166,"start_character":12,"end_line":167,"end_character":61},"updated":"2017-06-23 17:20:19.000000000","message":"is it fine to just test for creating just one tag per resource id? or should there be a test for more than one tags also?","commit_id":"d6833c913c5dc2caefe2b9e5c4c734ab6b5a4139"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"482c0eb4c5bd5b89f6986d98a3d39c263bbac6b4","unresolved":false,"context_lines":[{"line_number":163,"context_line":"            resource_object \u003d getattr(self, resource[:-1])"},{"line_number":164,"context_line":"            retrieved_tags \u003d self.tags_client.list_tags("},{"line_number":165,"context_line":"                resource, resource_object[\u0027id\u0027])[\u0027tags\u0027]"},{"line_number":166,"context_line":"            self.assertEqual(1, len(retrieved_tags))"},{"line_number":167,"context_line":"            self.assertEqual(tag_names[i], retrieved_tags[0])"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"            # Check that the expected tag exists for each resource."},{"line_number":170,"context_line":"            self.tags_client.check_tag_existence("}],"source_content_type":"text/x-python","patch_set":3,"id":"5f201791_98cc226b","line":167,"range":{"start_line":166,"start_character":12,"end_line":167,"end_character":61},"in_reply_to":"5f201791_49ec7ff2","updated":"2017-06-23 18:51:34.000000000","message":"That\u0027s accomplished on L192-195 in test below via update_all_tags.","commit_id":"d6833c913c5dc2caefe2b9e5c4c734ab6b5a4139"}]}