)]}'
{"tempest/api/identity/admin/v2/test_tenant_negative.py":[{"author":{"_account_id":19587,"name":"Joshua White","email":"joshua.l.white@intel.com","username":"jwhite"},"change_message_id":"765e5049f391490a6f6d5fb89a91c3747b3fcd0b","unresolved":false,"context_lines":[{"line_number":128,"context_line":"    def test_update_non_existent_tenant(self):"},{"line_number":129,"context_line":"        # Attempt to update a non existent tenant should fail"},{"line_number":130,"context_line":"        self.assertRaises(lib_exc.NotFound, self.tenants_client.update_tenant,"},{"line_number":131,"context_line":"                          str(data_utils.rand_uuid_hex()))"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    @test.attr(type\u003d[\u0027negative\u0027])"},{"line_number":134,"context_line":"    @test.idempotent_id(\u002741704dc5-c5f7-4f79-abfa-76e6fedc570b\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a122d0e_2830b8e7","line":131,"range":{"start_line":131,"start_character":30,"end_line":131,"end_character":56},"updated":"2016-04-19 20:40:00.000000000","message":"Current error message is specific to rand_uuid but does not acknowledge rand_uuid_hex.  Maybe could add a line before in the checks for --\u003e if uuid.uuid4().hex.  What do you think?","commit_id":"e63f7e4672bc22112e09b8cf7860c5eb8794932d"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"770aba32be8d54f21e7fb0e5a401cb522a136aeb","unresolved":false,"context_lines":[{"line_number":128,"context_line":"    def test_update_non_existent_tenant(self):"},{"line_number":129,"context_line":"        # Attempt to update a non existent tenant should fail"},{"line_number":130,"context_line":"        self.assertRaises(lib_exc.NotFound, self.tenants_client.update_tenant,"},{"line_number":131,"context_line":"                          str(data_utils.rand_uuid_hex()))"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    @test.attr(type\u003d[\u0027negative\u0027])"},{"line_number":134,"context_line":"    @test.idempotent_id(\u002741704dc5-c5f7-4f79-abfa-76e6fedc570b\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a122d0e_43751730","line":131,"range":{"start_line":131,"start_character":30,"end_line":131,"end_character":56},"in_reply_to":"1a122d0e_2830b8e7","updated":"2016-04-19 21:20:12.000000000","message":"Nice idea, will add it into error message.\nThanks!","commit_id":"e63f7e4672bc22112e09b8cf7860c5eb8794932d"}],"tempest/hacking/checks.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"c81bcb3abf9393df7a37a115e11a9b542c9af998","unresolved":false,"context_lines":[{"line_number":246,"context_line":""},{"line_number":247,"context_line":"    T113"},{"line_number":248,"context_line":"    \"\"\""},{"line_number":249,"context_line":"    if \u0027tempest/lib/\u0027 in filename:"},{"line_number":250,"context_line":"        return"},{"line_number":251,"context_line":""},{"line_number":252,"context_line":"    if \u0027uuid.uuid4()\u0027 not in logical_line:"},{"line_number":253,"context_line":"        return"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a122d0e_c4a0bef6","line":250,"range":{"start_line":249,"start_character":0,"end_line":250,"end_character":14},"updated":"2016-04-21 06:53:55.000000000","message":"I\u0027m actually unsure why we should skip to check under the tempest/lib.\nBut I think this rule is for \"tests\" not \"library\" as the message said. I don\u0027t have a strong objection for it :)","commit_id":"d079c8925d26787c788c92d526fdacfc1132f957"}]}