)]}'
{"mistral_tempest_tests/services/base.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f4641e8400f971b30eb4cef2673fd3eebecf17da","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"fa69d971_0f3d5edc","updated":"2016-01-07 00:52:28.000000000","message":"Looks good, but this need refactor for service client, auth and base tests class part. Which i hope you have plan after merging plugin.","commit_id":"8e4026855560e42e3d4c00af29efcf2954ba0074"},{"author":{"_account_id":8592,"name":"Anastasia Kuznetsova","email":"akuznetsova@mirantis.com","username":"nastya-kuz"},"change_message_id":"4c6829414fbd898dcbd715fe43b70e96b322d253","unresolved":false,"context_lines":[{"line_number":226,"context_line":"        return resp, json.loads(body)"},{"line_number":227,"context_line":""},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"class AuthProv(auth.KeystoneV2AuthProvider):"},{"line_number":230,"context_line":"    def __init__(self):"},{"line_number":231,"context_line":"        self.alt_part \u003d None"},{"line_number":232,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"1a7b0d38_45ba09c2","line":229,"updated":"2015-12-24 08:26:33.000000000","message":"I am not sure that we need to move this class. It is a \u0027mock\u0027 for authorization in tempest to have an ability to run API tests without OpenStack locally. So I guess that in the context of implementing bp we do not need it here, because it is future tempest plugin, it always use authorization.","commit_id":"8e4026855560e42e3d4c00af29efcf2954ba0074"},{"author":{"_account_id":18238,"name":"Hardik Parekh","email":"hardik.parekh047@gmail.com","username":"hparekh"},"change_message_id":"880a23ee66c2a40ac50ec2c7e15e7757e39fa797","unresolved":false,"context_lines":[{"line_number":226,"context_line":"        return resp, json.loads(body)"},{"line_number":227,"context_line":""},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"class AuthProv(auth.KeystoneV2AuthProvider):"},{"line_number":230,"context_line":"    def __init__(self):"},{"line_number":231,"context_line":"        self.alt_part \u003d None"},{"line_number":232,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"1a7b0d38_a09b9b71","line":229,"in_reply_to":"1a7b0d38_45ba09c2","updated":"2015-12-24 08:54:49.000000000","message":"Hi,\n\nI am planning to add tempest plugin in such a way that we\ncan run this functional test from tempest repository as well as from mistral repository \n(like we do currently I will modify run-functional test script later once all things get merged and stable.)\n\nSo In second case, (running from mistral repo) we need this mocked class.","commit_id":"8e4026855560e42e3d4c00af29efcf2954ba0074"},{"author":{"_account_id":8592,"name":"Anastasia Kuznetsova","email":"akuznetsova@mirantis.com","username":"nastya-kuz"},"change_message_id":"4062ea4588995739162b7b1a401f9b89e592152d","unresolved":false,"context_lines":[{"line_number":226,"context_line":"        return resp, json.loads(body)"},{"line_number":227,"context_line":""},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"class AuthProv(auth.KeystoneV2AuthProvider):"},{"line_number":230,"context_line":"    def __init__(self):"},{"line_number":231,"context_line":"        self.alt_part \u003d None"},{"line_number":232,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"1a7b0d38_8c9d2c0c","line":229,"in_reply_to":"1a7b0d38_a09b9b71","updated":"2015-12-28 14:22:55.000000000","message":"Ok, I see.","commit_id":"8e4026855560e42e3d4c00af29efcf2954ba0074"},{"author":{"_account_id":8592,"name":"Anastasia Kuznetsova","email":"akuznetsova@mirantis.com","username":"nastya-kuz"},"change_message_id":"4c6829414fbd898dcbd715fe43b70e96b322d253","unresolved":false,"context_lines":[{"line_number":255,"context_line":""},{"line_number":256,"context_line":"        if \u0027WITHOUT_AUTH\u0027 in os.environ:"},{"line_number":257,"context_line":"            cls.mgr \u003d mock.MagicMock()"},{"line_number":258,"context_line":"            cls.mgr.auth_provider \u003d AuthProv()"},{"line_number":259,"context_line":"        else:"},{"line_number":260,"context_line":"            cls.creds \u003d creds.get_configured_credentials("},{"line_number":261,"context_line":"                credential_type\u003d\u0027user\u0027"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a7b0d38_c5c6f948","line":258,"updated":"2015-12-24 08:26:33.000000000","message":"Please, see comment above, we do not need it here too.","commit_id":"8e4026855560e42e3d4c00af29efcf2954ba0074"}]}