)]}'
{"tempest/api/identity/admin/v3/test_projects.py":[{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"2227d5b022009f0dcfbe153c624f5b1c5e83204b","unresolved":false,"context_lines":[{"line_number":18,"context_line":"from tempest.lib import decorators"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"class ProjectsTestJSON(base.BaseIdentityV3AdminTest):"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    @decorators.idempotent_id(\u00270ecf465c-0dc4-4532-ab53-91ffeb74d12d\u0027)"},{"line_number":24,"context_line":"    def test_project_create_with_description(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"dfd5e7cf_4b2b6bea","line":21,"range":{"start_line":21,"start_character":0,"end_line":21,"end_character":53},"updated":"2019-01-10 03:10:19.000000000","message":"Why testcases in this class need \"force_tenant_isolation \u003d True\"?\nor, is there any testcase that need \"force_tenant_isolation \u003d True\" in this file?","commit_id":"afdf14d6b99f5ac85c5e3698bb64db7e6ce4bab2"},{"author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"change_message_id":"e5656e3f4a24db286fbc0becfa89e4ae8018f3c2","unresolved":false,"context_lines":[{"line_number":18,"context_line":"from tempest.lib import decorators"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"class ProjectsTestJSON(base.BaseIdentityV3AdminTest):"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    @decorators.idempotent_id(\u00270ecf465c-0dc4-4532-ab53-91ffeb74d12d\u0027)"},{"line_number":24,"context_line":"    def test_project_create_with_description(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"bfdaf3ff_332ebc43","line":21,"range":{"start_line":21,"start_character":0,"end_line":21,"end_character":53},"in_reply_to":"dfd5e7cf_45aeef98","updated":"2019-01-14 14:30:57.000000000","message":"I\u0027ll try to move all of them so the test id isn\u0027t modified.","commit_id":"afdf14d6b99f5ac85c5e3698bb64db7e6ce4bab2"},{"author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"change_message_id":"c8e9d42e44c4f702e6f445c503cd4b8127dab477","unresolved":false,"context_lines":[{"line_number":18,"context_line":"from tempest.lib import decorators"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"class ProjectsTestJSON(base.BaseIdentityV3AdminTest):"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    @decorators.idempotent_id(\u00270ecf465c-0dc4-4532-ab53-91ffeb74d12d\u0027)"},{"line_number":24,"context_line":"    def test_project_create_with_description(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"dfd5e7cf_45aeef98","line":21,"range":{"start_line":21,"start_character":0,"end_line":21,"end_character":53},"in_reply_to":"dfd5e7cf_4b2b6bea","updated":"2019-01-10 15:37:55.000000000","message":"I thought because they are interop tests we do want to use \"force_tenant_isolation \u003d True\". Is this not the case? If so, I can update these with \"force_tenant_isolation \u003d False\".","commit_id":"afdf14d6b99f5ac85c5e3698bb64db7e6ce4bab2"},{"author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"change_message_id":"3e8585047c956f05b255fbf937b0f9847df144ba","unresolved":false,"context_lines":[{"line_number":127,"context_line":"        u_desc \u003d u_name + \u0027description\u0027"},{"line_number":128,"context_line":"        u_email \u003d u_name + \u0027@testmail.tm\u0027"},{"line_number":129,"context_line":"        u_password \u003d data_utils.rand_password()"},{"line_number":130,"context_line":"        user \u003d self.users_client.create_user("},{"line_number":131,"context_line":"            name\u003du_name, description\u003du_desc, password\u003du_password,"},{"line_number":132,"context_line":"            email\u003du_email, project_id\u003dproject[\u0027id\u0027])[\u0027user\u0027]"},{"line_number":133,"context_line":"        # Delete the User at the end of this method"}],"source_content_type":"text/x-python","patch_set":2,"id":"bfdaf3ff_05a48f56","line":130,"range":{"start_line":130,"start_character":8,"end_line":130,"end_character":44},"updated":"2019-01-14 15:28:48.000000000","message":"because this test creates a user I do not want to update the test case to use \"force_tenant_isolation \u003d False\". We should add a skip check for this test and we can\u0027t move the rest  of the tests in this test case because they are interop tests.","commit_id":"afdf14d6b99f5ac85c5e3698bb64db7e6ce4bab2"},{"author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"change_message_id":"cea852e87889bf6863b074f8e6aa00c65e3e675a","unresolved":false,"context_lines":[{"line_number":127,"context_line":"        u_desc \u003d u_name + \u0027description\u0027"},{"line_number":128,"context_line":"        u_email \u003d u_name + \u0027@testmail.tm\u0027"},{"line_number":129,"context_line":"        u_password \u003d data_utils.rand_password()"},{"line_number":130,"context_line":"        user \u003d self.users_client.create_user("},{"line_number":131,"context_line":"            name\u003du_name, description\u003du_desc, password\u003du_password,"},{"line_number":132,"context_line":"            email\u003du_email, project_id\u003dproject[\u0027id\u0027])[\u0027user\u0027]"},{"line_number":133,"context_line":"        # Delete the User at the end of this method"}],"source_content_type":"text/x-python","patch_set":2,"id":"bfdaf3ff_a0e76aac","line":130,"range":{"start_line":130,"start_character":8,"end_line":130,"end_character":44},"in_reply_to":"bfdaf3ff_05a48f56","updated":"2019-01-17 14:00:48.000000000","message":"I should add skip checks for tests that create a user in another change and rebase this patch on top of it.","commit_id":"afdf14d6b99f5ac85c5e3698bb64db7e6ce4bab2"}]}