)]}'
{"glance/tests/unit/test_auth.py":[{"author":{"_account_id":6493,"name":"John Bresnahan","email":"john@bresnahan.me","username":"jbresnah"},"change_message_id":"90e55101ca0f7119a79a11dd99fb02d10ab2dcc4","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"import json"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"import stubout"},{"line_number":21,"context_line":"import webob"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"from glance.api import authorization"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAN3%2F%2F4os%3D","line":20,"updated":"2013-03-07 17:20:24.000000000","message":"That empty line is not needed.","commit_id":"24cc1394c5648ae48e1a49750f5e1c51e02e0f75"},{"author":{"_account_id":6493,"name":"John Bresnahan","email":"john@bresnahan.me","username":"jbresnah"},"change_message_id":"90e55101ca0f7119a79a11dd99fb02d10ab2dcc4","unresolved":false,"context_lines":[{"line_number":102,"context_line":"                \"roles\": [{"},{"line_number":103,"context_line":"                \"tenantId\": \"1\","},{"line_number":104,"context_line":"                \"id\": \"1\","},{"line_number":105,"context_line":"                \"name\": \"Admin\"}], \"name\": \"joeadmin\"}}}"},{"line_number":106,"context_line":""},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"class TestKeystoneAuthPlugin(utils.BaseTestCase):"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAN3%2F%2F4o0%3D","line":105,"updated":"2013-03-07 17:20:24.000000000","message":"I prefer the previous formatting as it gives a better visual representation.","commit_id":"24cc1394c5648ae48e1a49750f5e1c51e02e0f75"},{"author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"change_message_id":"cea239a5e4298cb2d1a7dca7ee6e24b9724eb51b","unresolved":false,"context_lines":[{"line_number":171,"context_line":"            resp.status \u003d 400"},{"line_number":172,"context_line":"            return FakeResponse(resp), \"\""},{"line_number":173,"context_line":""},{"line_number":174,"context_line":"        self.stubs.Set("},{"line_number":175,"context_line":"            auth.KeystoneStrategy, \u0027_do_request\u0027, fake_do_request)"},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"        bad_creds \u003d {"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAN3%2F%2F0ro%3D","line":174,"updated":"2013-03-11 12:07:07.000000000","message":"Hey,\n\nAre these kind of changes really needed? Are these lines longer than 79 characters? (Applied to similar changes below)\n\nIf so, try not to break the line at the beginning \"(\" but at the second or third argument.","commit_id":"391e16299bf5706723047bb189ab7867a4d0ee5e"},{"author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"change_message_id":"2b5d685d5954da07c334cd73132ef0e9ebdd77b8","unresolved":false,"context_lines":[{"line_number":157,"context_line":"            except exception.MissingCredentialError:"},{"line_number":158,"context_line":"                continue  # Expected"},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"    def test_auth_params_empty_cred(self):"},{"line_number":161,"context_line":"        creds \u003d {\u0027username\u0027: \u0027test\u0027,"},{"line_number":162,"context_line":"                 \u0027password\u0027: \u0027pass\u0027,"},{"line_number":163,"context_line":"                 \u0027strategy\u0027: \u0027keystone\u0027,"}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAAOH%2F%2F7ps%3D","line":160,"updated":"2013-03-20 14:34:24.000000000","message":"Hey,\n\nSorry for not saying this before but, could you please add a small comment here saying that it should fail because tenant is missing? It might not be obvious for other contributors!\n\nCheers!","commit_id":"29e826c6e85c6484ef975ba5bf1044d82378c24d"}]}