)]}'
{"keystone/tests/unit/test_v3_oauth1.py":[{"author":{"_account_id":23659,"name":"Jaewoo Park","email":"jaewoopark91@gmail.com","username":"aselius"},"change_message_id":"c6f378ae14a1a65c03e45d5d4015ecb67a4240e1","unresolved":false,"context_lines":[{"line_number":161,"context_line":"        consumer \u003d self._create_single_consumer()"},{"line_number":162,"context_line":"        consumer_id \u003d consumer[\u0027id\u0027]"},{"line_number":163,"context_line":"        url \u003d self.CONSUMER_URL + \u0027/%s\u0027 % consumer_id"},{"line_number":164,"context_line":"        resp \u003d self.get(url, expected_status\u003dhttp_client.OK)"},{"line_number":165,"context_line":"        self_url \u003d [\u0027http://localhost/v3\u0027, self.CONSUMER_URL,"},{"line_number":166,"context_line":"                    \u0027/\u0027, consumer_id]"},{"line_number":167,"context_line":"        self_url \u003d \u0027\u0027.join(self_url)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f231b9d_f41f8a58","line":164,"range":{"start_line":164,"start_character":57,"end_line":164,"end_character":59},"updated":"2017-06-14 21:31:32.000000000","message":"nit: this might also be redundant (refer to [0], ignore if on purpose) default for get is OK\n\n[0] https://review.openstack.org/#/c/473885/1/keystone/tests/unit/test_v3_endpoint_policy.py","commit_id":"a25d5d0e0fdb8689974db4ded8132fd52ab37786"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"b493f27f95bacf3b3e093bf1539b6d505ac771e9","unresolved":false,"context_lines":[{"line_number":161,"context_line":"        consumer \u003d self._create_single_consumer()"},{"line_number":162,"context_line":"        consumer_id \u003d consumer[\u0027id\u0027]"},{"line_number":163,"context_line":"        url \u003d self.CONSUMER_URL + \u0027/%s\u0027 % consumer_id"},{"line_number":164,"context_line":"        resp \u003d self.get(url, expected_status\u003dhttp_client.OK)"},{"line_number":165,"context_line":"        self_url \u003d [\u0027http://localhost/v3\u0027, self.CONSUMER_URL,"},{"line_number":166,"context_line":"                    \u0027/\u0027, consumer_id]"},{"line_number":167,"context_line":"        self_url \u003d \u0027\u0027.join(self_url)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f201791_985b502d","line":164,"in_reply_to":"5f201791_ad2dc847","updated":"2017-06-21 15:00:48.000000000","message":"Done","commit_id":"a25d5d0e0fdb8689974db4ded8132fd52ab37786"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"6baf50d336c18d62d41d23c1eb01cba1a8a9d0de","unresolved":false,"context_lines":[{"line_number":161,"context_line":"        consumer \u003d self._create_single_consumer()"},{"line_number":162,"context_line":"        consumer_id \u003d consumer[\u0027id\u0027]"},{"line_number":163,"context_line":"        url \u003d self.CONSUMER_URL + \u0027/%s\u0027 % consumer_id"},{"line_number":164,"context_line":"        resp \u003d self.get(url, expected_status\u003dhttp_client.OK)"},{"line_number":165,"context_line":"        self_url \u003d [\u0027http://localhost/v3\u0027, self.CONSUMER_URL,"},{"line_number":166,"context_line":"                    \u0027/\u0027, consumer_id]"},{"line_number":167,"context_line":"        self_url \u003d \u0027\u0027.join(self_url)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f201791_ad2dc847","line":164,"in_reply_to":"7f231b9d_ad3a6395","updated":"2017-06-20 19:31:39.000000000","message":"Oh - yes it was for clarity. I can refactor those bits out. Good catch.","commit_id":"a25d5d0e0fdb8689974db4ded8132fd52ab37786"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"b3e78f66bcebbcadd1c9808ee0361b33033a21af","unresolved":false,"context_lines":[{"line_number":161,"context_line":"        consumer \u003d self._create_single_consumer()"},{"line_number":162,"context_line":"        consumer_id \u003d consumer[\u0027id\u0027]"},{"line_number":163,"context_line":"        url \u003d self.CONSUMER_URL + \u0027/%s\u0027 % consumer_id"},{"line_number":164,"context_line":"        resp \u003d self.get(url, expected_status\u003dhttp_client.OK)"},{"line_number":165,"context_line":"        self_url \u003d [\u0027http://localhost/v3\u0027, self.CONSUMER_URL,"},{"line_number":166,"context_line":"                    \u0027/\u0027, consumer_id]"},{"line_number":167,"context_line":"        self_url \u003d \u0027\u0027.join(self_url)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f231b9d_ad3a6395","line":164,"range":{"start_line":164,"start_character":57,"end_line":164,"end_character":59},"in_reply_to":"7f231b9d_f41f8a58","updated":"2017-06-20 08:02:10.000000000","message":"You are correct - and of course, the previous code used this default. Lance, was it just for clarity that you provided the explicit expected status?","commit_id":"a25d5d0e0fdb8689974db4ded8132fd52ab37786"},{"author":{"_account_id":23659,"name":"Jaewoo Park","email":"jaewoopark91@gmail.com","username":"aselius"},"change_message_id":"c6f378ae14a1a65c03e45d5d4015ecb67a4240e1","unresolved":false,"context_lines":[{"line_number":322,"context_line":"            \u0027/users/%(user_id)s/OS-OAUTH1/access_tokens\u0027"},{"line_number":323,"context_line":"            % {\u0027user_id\u0027: self.user_id}"},{"line_number":324,"context_line":"        )"},{"line_number":325,"context_line":"        resp \u003d self.get(url, expected_status\u003dhttp_client.OK)"},{"line_number":326,"context_line":"        entities \u003d resp.result[\u0027access_tokens\u0027]"},{"line_number":327,"context_line":"        self.assertEqual([], entities)"},{"line_number":328,"context_line":"        self.assertValidListLinks(resp.result[\u0027links\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f231b9d_920f4936","line":325,"range":{"start_line":325,"start_character":29,"end_line":325,"end_character":59},"updated":"2017-06-14 21:31:32.000000000","message":"Ditto","commit_id":"a25d5d0e0fdb8689974db4ded8132fd52ab37786"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"b493f27f95bacf3b3e093bf1539b6d505ac771e9","unresolved":false,"context_lines":[{"line_number":322,"context_line":"            \u0027/users/%(user_id)s/OS-OAUTH1/access_tokens\u0027"},{"line_number":323,"context_line":"            % {\u0027user_id\u0027: self.user_id}"},{"line_number":324,"context_line":"        )"},{"line_number":325,"context_line":"        resp \u003d self.get(url, expected_status\u003dhttp_client.OK)"},{"line_number":326,"context_line":"        entities \u003d resp.result[\u0027access_tokens\u0027]"},{"line_number":327,"context_line":"        self.assertEqual([], entities)"},{"line_number":328,"context_line":"        self.assertValidListLinks(resp.result[\u0027links\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f201791_583db8b9","line":325,"in_reply_to":"7f231b9d_920f4936","updated":"2017-06-21 15:00:48.000000000","message":"Done","commit_id":"a25d5d0e0fdb8689974db4ded8132fd52ab37786"},{"author":{"_account_id":23659,"name":"Jaewoo Park","email":"jaewoopark91@gmail.com","username":"aselius"},"change_message_id":"c6f378ae14a1a65c03e45d5d4015ecb67a4240e1","unresolved":false,"context_lines":[{"line_number":337,"context_line":"              \u0027user_id\u0027: self.user_id,"},{"line_number":338,"context_line":"              \u0027key\u0027: access_token_key_string"},{"line_number":339,"context_line":"        }"},{"line_number":340,"context_line":"        resp \u003d self.get(url, expected_status\u003dhttp_client.OK)"},{"line_number":341,"context_line":"        entity \u003d resp.result[\u0027access_token\u0027]"},{"line_number":342,"context_line":"        self.assertEqual(access_token_key_string, entity[\u0027id\u0027])"},{"line_number":343,"context_line":"        self.assertEqual(self.consumer[\u0027key\u0027], entity[\u0027consumer_id\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f231b9d_720cad40","line":340,"range":{"start_line":340,"start_character":57,"end_line":340,"end_character":59},"updated":"2017-06-14 21:31:32.000000000","message":"Ditto","commit_id":"a25d5d0e0fdb8689974db4ded8132fd52ab37786"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"b493f27f95bacf3b3e093bf1539b6d505ac771e9","unresolved":false,"context_lines":[{"line_number":337,"context_line":"              \u0027user_id\u0027: self.user_id,"},{"line_number":338,"context_line":"              \u0027key\u0027: access_token_key_string"},{"line_number":339,"context_line":"        }"},{"line_number":340,"context_line":"        resp \u003d self.get(url, expected_status\u003dhttp_client.OK)"},{"line_number":341,"context_line":"        entity \u003d resp.result[\u0027access_token\u0027]"},{"line_number":342,"context_line":"        self.assertEqual(access_token_key_string, entity[\u0027id\u0027])"},{"line_number":343,"context_line":"        self.assertEqual(self.consumer[\u0027key\u0027], entity[\u0027consumer_id\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f201791_384afc54","line":340,"in_reply_to":"7f231b9d_720cad40","updated":"2017-06-21 15:00:48.000000000","message":"Done","commit_id":"a25d5d0e0fdb8689974db4ded8132fd52ab37786"},{"author":{"_account_id":23659,"name":"Jaewoo Park","email":"jaewoopark91@gmail.com","username":"aselius"},"change_message_id":"c6f378ae14a1a65c03e45d5d4015ecb67a4240e1","unresolved":false,"context_lines":[{"line_number":361,"context_line":"            % {\u0027id\u0027: self.user_id,"},{"line_number":362,"context_line":"               \u0027key\u0027: self.access_token.key.decode()}"},{"line_number":363,"context_line":"        )"},{"line_number":364,"context_line":"        resp \u003d self.get(url, expected_status\u003dhttp_client.OK)"},{"line_number":365,"context_line":"        entities \u003d resp.result[\u0027roles\u0027]"},{"line_number":366,"context_line":"        self.assertTrue(entities)"},{"line_number":367,"context_line":"        self.assertValidListLinks(resp.result[\u0027links\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f231b9d_34018206","line":364,"range":{"start_line":364,"start_character":57,"end_line":364,"end_character":59},"updated":"2017-06-14 21:31:32.000000000","message":"Ditto","commit_id":"a25d5d0e0fdb8689974db4ded8132fd52ab37786"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"b493f27f95bacf3b3e093bf1539b6d505ac771e9","unresolved":false,"context_lines":[{"line_number":361,"context_line":"            % {\u0027id\u0027: self.user_id,"},{"line_number":362,"context_line":"               \u0027key\u0027: self.access_token.key.decode()}"},{"line_number":363,"context_line":"        )"},{"line_number":364,"context_line":"        resp \u003d self.get(url, expected_status\u003dhttp_client.OK)"},{"line_number":365,"context_line":"        entities \u003d resp.result[\u0027roles\u0027]"},{"line_number":366,"context_line":"        self.assertTrue(entities)"},{"line_number":367,"context_line":"        self.assertValidListLinks(resp.result[\u0027links\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f201791_786994bc","line":364,"in_reply_to":"7f231b9d_34018206","updated":"2017-06-21 15:00:48.000000000","message":"Done","commit_id":"a25d5d0e0fdb8689974db4ded8132fd52ab37786"},{"author":{"_account_id":23659,"name":"Jaewoo Park","email":"jaewoopark91@gmail.com","username":"aselius"},"change_message_id":"c6f378ae14a1a65c03e45d5d4015ecb67a4240e1","unresolved":false,"context_lines":[{"line_number":375,"context_line":"        url \u003d (\u0027/users/%(id)s/OS-OAUTH1/access_tokens/%(key)s/roles/%(role)s\u0027"},{"line_number":376,"context_line":"               % {\u0027id\u0027: self.user_id, \u0027key\u0027: access_token_key,"},{"line_number":377,"context_line":"                  \u0027role\u0027: self.role_id})"},{"line_number":378,"context_line":"        resp \u003d self.get(url, expected_status\u003dhttp_client.OK)"},{"line_number":379,"context_line":"        entity \u003d resp.result[\u0027role\u0027]"},{"line_number":380,"context_line":"        self.assertEqual(self.role_id, entity[\u0027id\u0027])"},{"line_number":381,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7f231b9d_1406861e","line":378,"range":{"start_line":378,"start_character":57,"end_line":378,"end_character":59},"updated":"2017-06-14 21:31:32.000000000","message":"Ditto","commit_id":"a25d5d0e0fdb8689974db4ded8132fd52ab37786"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"b493f27f95bacf3b3e093bf1539b6d505ac771e9","unresolved":false,"context_lines":[{"line_number":375,"context_line":"        url \u003d (\u0027/users/%(id)s/OS-OAUTH1/access_tokens/%(key)s/roles/%(role)s\u0027"},{"line_number":376,"context_line":"               % {\u0027id\u0027: self.user_id, \u0027key\u0027: access_token_key,"},{"line_number":377,"context_line":"                  \u0027role\u0027: self.role_id})"},{"line_number":378,"context_line":"        resp \u003d self.get(url, expected_status\u003dhttp_client.OK)"},{"line_number":379,"context_line":"        entity \u003d resp.result[\u0027role\u0027]"},{"line_number":380,"context_line":"        self.assertEqual(self.role_id, entity[\u0027id\u0027])"},{"line_number":381,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"5f201791_f89a44c3","line":378,"in_reply_to":"7f231b9d_1406861e","updated":"2017-06-21 15:00:48.000000000","message":"Done","commit_id":"a25d5d0e0fdb8689974db4ded8132fd52ab37786"},{"author":{"_account_id":23659,"name":"Jaewoo Park","email":"jaewoopark91@gmail.com","username":"aselius"},"change_message_id":"c6f378ae14a1a65c03e45d5d4015ecb67a4240e1","unresolved":false,"context_lines":[{"line_number":398,"context_line":"            \u0027/users/%(user_id)s/OS-OAUTH1/access_tokens\u0027"},{"line_number":399,"context_line":"            % {\u0027user_id\u0027: self.user_id}"},{"line_number":400,"context_line":"        )"},{"line_number":401,"context_line":"        resp \u003d self.get(url, expected_status\u003dhttp_client.OK)"},{"line_number":402,"context_line":"        self.head(url, expected_status\u003dhttp_client.OK)"},{"line_number":403,"context_line":"        entities \u003d resp.result[\u0027access_tokens\u0027]"},{"line_number":404,"context_line":"        self.assertTrue(entities)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f231b9d_f4066a1c","line":401,"range":{"start_line":401,"start_character":57,"end_line":401,"end_character":59},"updated":"2017-06-14 21:31:32.000000000","message":"Ditto","commit_id":"a25d5d0e0fdb8689974db4ded8132fd52ab37786"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"b493f27f95bacf3b3e093bf1539b6d505ac771e9","unresolved":false,"context_lines":[{"line_number":398,"context_line":"            \u0027/users/%(user_id)s/OS-OAUTH1/access_tokens\u0027"},{"line_number":399,"context_line":"            % {\u0027user_id\u0027: self.user_id}"},{"line_number":400,"context_line":"        )"},{"line_number":401,"context_line":"        resp \u003d self.get(url, expected_status\u003dhttp_client.OK)"},{"line_number":402,"context_line":"        self.head(url, expected_status\u003dhttp_client.OK)"},{"line_number":403,"context_line":"        entities \u003d resp.result[\u0027access_tokens\u0027]"},{"line_number":404,"context_line":"        self.assertTrue(entities)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f201791_d89f88b2","line":401,"in_reply_to":"7f231b9d_f4066a1c","updated":"2017-06-21 15:00:48.000000000","message":"Done","commit_id":"a25d5d0e0fdb8689974db4ded8132fd52ab37786"},{"author":{"_account_id":23659,"name":"Jaewoo Park","email":"jaewoopark91@gmail.com","username":"aselius"},"change_message_id":"c6f378ae14a1a65c03e45d5d4015ecb67a4240e1","unresolved":false,"context_lines":[{"line_number":412,"context_line":"        self.assertResponseStatus(resp, http_client.NO_CONTENT)"},{"line_number":413,"context_line":""},{"line_number":414,"context_line":"        # List access_token should be 0"},{"line_number":415,"context_line":"        resp \u003d self.get(url, expected_status\u003dhttp_client.OK)"},{"line_number":416,"context_line":"        self.head(url, expected_status\u003dhttp_client.OK)"},{"line_number":417,"context_line":"        entities \u003d resp.result[\u0027access_tokens\u0027]"},{"line_number":418,"context_line":"        self.assertEqual([], entities)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f231b9d_b4147245","line":415,"range":{"start_line":415,"start_character":57,"end_line":415,"end_character":59},"updated":"2017-06-14 21:31:32.000000000","message":"Ditto","commit_id":"a25d5d0e0fdb8689974db4ded8132fd52ab37786"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"b493f27f95bacf3b3e093bf1539b6d505ac771e9","unresolved":false,"context_lines":[{"line_number":412,"context_line":"        self.assertResponseStatus(resp, http_client.NO_CONTENT)"},{"line_number":413,"context_line":""},{"line_number":414,"context_line":"        # List access_token should be 0"},{"line_number":415,"context_line":"        resp \u003d self.get(url, expected_status\u003dhttp_client.OK)"},{"line_number":416,"context_line":"        self.head(url, expected_status\u003dhttp_client.OK)"},{"line_number":417,"context_line":"        entities \u003d resp.result[\u0027access_tokens\u0027]"},{"line_number":418,"context_line":"        self.assertEqual([], entities)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f201791_98a910e1","line":415,"in_reply_to":"7f231b9d_b4147245","updated":"2017-06-21 15:00:48.000000000","message":"Done","commit_id":"a25d5d0e0fdb8689974db4ded8132fd52ab37786"}]}