)]}'
{"tempest/api/identity/admin/v3/test_groups_negative.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f85f2b9053f80b9c2492acd61c385282dc76306f","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright 2015 RIL."},{"line_number":2,"context_line":"# All Rights Reserved."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba8a016a_fb4c4d9f","line":1,"updated":"2015-11-20 06:32:01.000000000","message":"you might want to update this","commit_id":"70c9cbc91def2f6cc3d28f8ce57cf6631b199068"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"7cab93306045de3ffa5de448a119f9e390ba3a5f","unresolved":false,"context_lines":[{"line_number":57,"context_line":"    @test.attr(type\u003d[\u0027negative\u0027])"},{"line_number":58,"context_line":"    @test.idempotent_id(\u002724c49279-45dd-4155-865a-cb738c2385aa\u0027)"},{"line_number":59,"context_line":"    def test_add_list_delete_user_non_existent_group(self):"},{"line_number":60,"context_line":"        users \u003d []"},{"line_number":61,"context_line":"        name \u003d data_utils.rand_name(\u0027User\u0027)"},{"line_number":62,"context_line":"        user \u003d self.client.create_user(name)[\u0027user\u0027]"},{"line_number":63,"context_line":"        users.append(user)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a8ffd7b_aa95f540","line":60,"updated":"2015-11-24 15:56:48.000000000","message":"Why do you need this variable ? It seems unused.","commit_id":"70c9cbc91def2f6cc3d28f8ce57cf6631b199068"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"496be754d8add3af9bd5e4d8b411413cac069168","unresolved":false,"context_lines":[{"line_number":20,"context_line":"from tempest.api.identity import base"},{"line_number":21,"context_line":"from tempest.common.utils import data_utils"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"from tempest_lib import exceptions as lib_exc"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"class GroupsV3NegativeTestJSON(base.BaseIdentityV3AdminTest):"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a8ffd7b_7350ca32","line":23,"updated":"2015-12-03 02:53:48.000000000","message":"import order are wrong. it should be\n\nimport uuid\n\nfrom tempest_lib import exceptions as lib_exc\n\nfrom tempest.api.identity import base\nfrom tempest.common.utils import data_utils\nfrom tempest import test","commit_id":"1e8843644a8869b36d5bc2dcc0aa4b880d2b427d"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"32c1eba6344d53e97385e961b0dac78805b2157b","unresolved":false,"context_lines":[{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    @test.attr(type\u003d[\u0027negative\u0027])"},{"line_number":59,"context_line":"    @test.idempotent_id(\u002724c49279-45dd-4155-865a-cb738c2385aa\u0027)"},{"line_number":60,"context_line":"    def test_add_list_delete_user_non_existent_group(self):"},{"line_number":61,"context_line":"        name \u003d data_utils.rand_name(\u0027User\u0027)"},{"line_number":62,"context_line":"        user \u003d self.client.create_user(name)[\u0027user\u0027]"},{"line_number":63,"context_line":"        self.addCleanup(self.client.delete_user, user[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_62a1b393","line":60,"updated":"2015-12-14 09:05:26.000000000","message":"I\u0027m not sure why these three needs to be squashed together in one test?\nIt\u0027s three different API calls, each with it\u0027s own assert. Could you please split this into three separate ones?","commit_id":"f91378da441849ccc8e4702bb8e4d0d761955630"}]}