)]}'
{"tempest/api/identity/admin/v3/test_domains_negative.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"7b488a5ff9331ce93d044f9e4d505197492aaa33","unresolved":false,"context_lines":[{"line_number":48,"context_line":"    @test.attr(type\u003d[\u0027negative\u0027])"},{"line_number":49,"context_line":"    @test.idempotent_id(\u00272ee842ab-0997-4424-be57-9a954c086f13\u0027)"},{"line_number":50,"context_line":"    def test_create_domain_by_unauthorized_user(self):"},{"line_number":51,"context_line":"        # Non-admin user should not be authorized to create a project"},{"line_number":52,"context_line":"        d_name \u003d data_utils.rand_name(\u0027domain\u0027)"},{"line_number":53,"context_line":"        self.assertRaises("},{"line_number":54,"context_line":"            lib_exc.Forbidden, self.non_admin_client.create_domain,"}],"source_content_type":"text/x-python","patch_set":1,"id":"da85f559_befe7b03","line":51,"updated":"2015-11-13 16:00:50.000000000","message":"This depends on Keystone\u0027s policy.json. IMO we should not add test that depends on cloud policy.","commit_id":"c9aa998f24b648420f6760d9a9e07baa9e9b9b37"},{"author":{"_account_id":19112,"name":"Wei Liu","email":"wei.liu@easystack.cn","username":"liuwei5018"},"change_message_id":"012a8ef23f5bc97d66ff138a9314e622757a773b","unresolved":false,"context_lines":[{"line_number":48,"context_line":"    @test.attr(type\u003d[\u0027negative\u0027])"},{"line_number":49,"context_line":"    @test.idempotent_id(\u00272ee842ab-0997-4424-be57-9a954c086f13\u0027)"},{"line_number":50,"context_line":"    def test_create_domain_by_unauthorized_user(self):"},{"line_number":51,"context_line":"        # Non-admin user should not be authorized to create a project"},{"line_number":52,"context_line":"        d_name \u003d data_utils.rand_name(\u0027domain\u0027)"},{"line_number":53,"context_line":"        self.assertRaises("},{"line_number":54,"context_line":"            lib_exc.Forbidden, self.non_admin_client.create_domain,"}],"source_content_type":"text/x-python","patch_set":1,"id":"da85f559_6a3fc025","line":51,"in_reply_to":"da85f559_061dda65","updated":"2015-11-16 03:33:22.000000000","message":"Thanks. Jordan.","commit_id":"c9aa998f24b648420f6760d9a9e07baa9e9b9b37"},{"author":{"_account_id":19112,"name":"Wei Liu","email":"wei.liu@easystack.cn","username":"liuwei5018"},"change_message_id":"cfad1342989369c147eb26bf43ec8996bf443c22","unresolved":false,"context_lines":[{"line_number":48,"context_line":"    @test.attr(type\u003d[\u0027negative\u0027])"},{"line_number":49,"context_line":"    @test.idempotent_id(\u00272ee842ab-0997-4424-be57-9a954c086f13\u0027)"},{"line_number":50,"context_line":"    def test_create_domain_by_unauthorized_user(self):"},{"line_number":51,"context_line":"        # Non-admin user should not be authorized to create a project"},{"line_number":52,"context_line":"        d_name \u003d data_utils.rand_name(\u0027domain\u0027)"},{"line_number":53,"context_line":"        self.assertRaises("},{"line_number":54,"context_line":"            lib_exc.Forbidden, self.non_admin_client.create_domain,"}],"source_content_type":"text/x-python","patch_set":1,"id":"da85f559_fdb87777","line":51,"in_reply_to":"da85f559_befe7b03","updated":"2015-11-14 03:10:46.000000000","message":"I see, and will abandon this commit. But I still have concern. Since the authorization is also the very important part of keystone. If we did not add such test based on default policy configuration, how could tempest cover the authorization functions?","commit_id":"c9aa998f24b648420f6760d9a9e07baa9e9b9b37"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"12156dba51dedc88477a85797927b3ad4fdf5b1c","unresolved":false,"context_lines":[{"line_number":48,"context_line":"    @test.attr(type\u003d[\u0027negative\u0027])"},{"line_number":49,"context_line":"    @test.idempotent_id(\u00272ee842ab-0997-4424-be57-9a954c086f13\u0027)"},{"line_number":50,"context_line":"    def test_create_domain_by_unauthorized_user(self):"},{"line_number":51,"context_line":"        # Non-admin user should not be authorized to create a project"},{"line_number":52,"context_line":"        d_name \u003d data_utils.rand_name(\u0027domain\u0027)"},{"line_number":53,"context_line":"        self.assertRaises("},{"line_number":54,"context_line":"            lib_exc.Forbidden, self.non_admin_client.create_domain,"}],"source_content_type":"text/x-python","patch_set":1,"id":"da85f559_061dda65","line":51,"in_reply_to":"da85f559_fdb87777","updated":"2015-11-15 13:20:59.000000000","message":"Tempest should be able to test any sort of openstack cloud with any sort of cloud policy (policy.json), that\u0027s why I think this particular test is not a good fit for tempest. Maybe you could try to add it as a functional test to Keystone (I see keystone has a keystone/tests/functional) directory.","commit_id":"c9aa998f24b648420f6760d9a9e07baa9e9b9b37"}]}