)]}'
{"keystoneauth1/tests/unit/identity/test_identity_v3_oidc.py":[{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"166948c7d5bf8ce6b8eb1246306efc3899fccd6f","unresolved":false,"context_lines":[{"line_number":155,"context_line":"        self.assertEqual(encoded_payload, last_req.body)"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"# This is a special case, as we do not need all the other openid parameters,"},{"line_number":159,"context_line":"# like client_id, client_secret, etc."},{"line_number":160,"context_line":"class OIDCTokenTests(utils.TestCase):"},{"line_number":161,"context_line":"    def setUp(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"3aaa91ec_001081a7","line":158,"updated":"2016-06-22 23:39:45.000000000","message":"Please add a NOTE(\u003cyour-nickname\u003e) at the beginning of this comment","commit_id":"f3f5732dd3e5189e31e4a1dceb5b84fe11bf16cb"},{"author":{"_account_id":91,"name":"Alvaro","email":"aloga@ifca.unican.es","username":"aloga"},"change_message_id":"97ff1988603394b554333cbe02588332940ef97c","unresolved":false,"context_lines":[{"line_number":155,"context_line":"        self.assertEqual(encoded_payload, last_req.body)"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"# This is a special case, as we do not need all the other openid parameters,"},{"line_number":159,"context_line":"# like client_id, client_secret, etc."},{"line_number":160,"context_line":"class OIDCTokenTests(utils.TestCase):"},{"line_number":161,"context_line":"    def setUp(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"3aaa91ec_42aa8271","line":158,"in_reply_to":"3aaa91ec_001081a7","updated":"2016-06-23 07:43:34.000000000","message":"Done","commit_id":"f3f5732dd3e5189e31e4a1dceb5b84fe11bf16cb"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"166948c7d5bf8ce6b8eb1246306efc3899fccd6f","unresolved":false,"context_lines":[{"line_number":163,"context_line":""},{"line_number":164,"context_line":"        self.session \u003d session.Session()"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"        self.AUTH_URL \u003d \u0027http://keystone:5000/v3\u0027"},{"line_number":167,"context_line":"        self.IDENTITY_PROVIDER \u003d \u0027bluepages\u0027"},{"line_number":168,"context_line":"        self.PROTOCOL \u003d \u0027oidc\u0027"},{"line_number":169,"context_line":"        self.PROJECT_NAME \u003d \u0027foo project\u0027"},{"line_number":170,"context_line":"        self.ACCESS_TOKEN \u003d uuid.uuid4().hex"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"        self.FEDERATION_AUTH_URL \u003d \u0027%s/%s\u0027 % ("},{"line_number":173,"context_line":"            self.AUTH_URL,"},{"line_number":174,"context_line":"            \u0027OS-FEDERATION/identity_providers/bluepages/protocols/oidc/auth\u0027)"},{"line_number":175,"context_line":""},{"line_number":176,"context_line":"        self.plugin \u003d oidc.OidcAccessToken("},{"line_number":177,"context_line":"            self.AUTH_URL,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3aaa91ec_20d2ddd2","line":174,"range":{"start_line":166,"start_character":9,"end_line":174,"end_character":77},"updated":"2016-06-22 23:39:45.000000000","message":"I\u0027d be fine with inheriting from the base class and not needing to redefine all these","commit_id":"f3f5732dd3e5189e31e4a1dceb5b84fe11bf16cb"},{"author":{"_account_id":91,"name":"Alvaro","email":"aloga@ifca.unican.es","username":"aloga"},"change_message_id":"97ff1988603394b554333cbe02588332940ef97c","unresolved":false,"context_lines":[{"line_number":163,"context_line":""},{"line_number":164,"context_line":"        self.session \u003d session.Session()"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"        self.AUTH_URL \u003d \u0027http://keystone:5000/v3\u0027"},{"line_number":167,"context_line":"        self.IDENTITY_PROVIDER \u003d \u0027bluepages\u0027"},{"line_number":168,"context_line":"        self.PROTOCOL \u003d \u0027oidc\u0027"},{"line_number":169,"context_line":"        self.PROJECT_NAME \u003d \u0027foo project\u0027"},{"line_number":170,"context_line":"        self.ACCESS_TOKEN \u003d uuid.uuid4().hex"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"        self.FEDERATION_AUTH_URL \u003d \u0027%s/%s\u0027 % ("},{"line_number":173,"context_line":"            self.AUTH_URL,"},{"line_number":174,"context_line":"            \u0027OS-FEDERATION/identity_providers/bluepages/protocols/oidc/auth\u0027)"},{"line_number":175,"context_line":""},{"line_number":176,"context_line":"        self.plugin \u003d oidc.OidcAccessToken("},{"line_number":177,"context_line":"            self.AUTH_URL,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3aaa91ec_427442fc","line":174,"range":{"start_line":166,"start_character":9,"end_line":174,"end_character":77},"in_reply_to":"3aaa91ec_20d2ddd2","updated":"2016-06-23 07:43:34.000000000","message":"This is done like this on purpose, the test class does not inherit from the base tests, but from the utils.TestCase class.\n\nContinuing in this patch series https://review.openstack.org/#/c/330464 implements the discovery document support and its testing in the base class, as this is common for the OpenID different grant types (password, auth code), but not for this plugin.","commit_id":"f3f5732dd3e5189e31e4a1dceb5b84fe11bf16cb"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"1a721be5822717a54e005b9fe5eca53f21893ea4","unresolved":false,"context_lines":[{"line_number":163,"context_line":""},{"line_number":164,"context_line":"        self.session \u003d session.Session()"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"        self.AUTH_URL \u003d \u0027http://keystone:5000/v3\u0027"},{"line_number":167,"context_line":"        self.IDENTITY_PROVIDER \u003d \u0027bluepages\u0027"},{"line_number":168,"context_line":"        self.PROTOCOL \u003d \u0027oidc\u0027"},{"line_number":169,"context_line":"        self.PROJECT_NAME \u003d \u0027foo project\u0027"},{"line_number":170,"context_line":"        self.ACCESS_TOKEN \u003d uuid.uuid4().hex"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"        self.FEDERATION_AUTH_URL \u003d \u0027%s/%s\u0027 % ("},{"line_number":173,"context_line":"            self.AUTH_URL,"},{"line_number":174,"context_line":"            \u0027OS-FEDERATION/identity_providers/bluepages/protocols/oidc/auth\u0027)"},{"line_number":175,"context_line":""},{"line_number":176,"context_line":"        self.plugin \u003d oidc.OidcAccessToken("},{"line_number":177,"context_line":"            self.AUTH_URL,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3aaa91ec_f9debc6b","line":174,"range":{"start_line":166,"start_character":9,"end_line":174,"end_character":77},"in_reply_to":"3aaa91ec_427442fc","updated":"2016-06-23 10:04:41.000000000","message":"Nice, thanks for clarifying","commit_id":"f3f5732dd3e5189e31e4a1dceb5b84fe11bf16cb"},{"author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"change_message_id":"09f26fd70ee7988b12ffa863b807572f74bfb8d3","unresolved":false,"context_lines":[{"line_number":50,"context_line":"    def setUp(self):"},{"line_number":51,"context_line":"        super(OIDCPasswordTests, self).setUp()"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"        self.plugin \u003d oidc.OidcPassword("},{"line_number":54,"context_line":"            self.AUTH_URL,"},{"line_number":55,"context_line":"            self.IDENTITY_PROVIDER,"},{"line_number":56,"context_line":"            self.PROTOCOL,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_00231aa8","line":53,"range":{"start_line":53,"start_character":8,"end_line":53,"end_character":12},"updated":"2016-06-24 01:52:34.000000000","message":"typically we do this with a property or something to show future subclassers you need to implement it. But it\u0027s not a big deal for tests.","commit_id":"280cde2ff97a936c9a718d1401b611bb19daa6a4"}]}
