)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"80516ab464d5d7bb9a0adaa3bd87e04e448bea8e","unresolved":false,"context_lines":[{"line_number":11,"context_line":"project scoped tokens but was missed from domain scoped tokens."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Add a service catalog to the domain token fixture (and fix a test that"},{"line_number":14,"context_line":"for some reason checked that this token didn\u0027t have a service catalog)"},{"line_number":15,"context_line":"and add a domain scoped test."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I8484f4a26a5695ef7ae962918ad442fe20bd2caa"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAATn%2F9Ho0%3D","line":14,"updated":"2013-12-16 17:43:34.000000000","message":"suggest splitting out this change. Also, the test was checking that the service catalog was empty and not that it didn\u0027t have one.","commit_id":"17ecaa0f029c7bc568b8bc998156ef43d0c07587"}],"keystoneclient/tests/v3/client_fixtures.py":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"80516ab464d5d7bb9a0adaa3bd87e04e448bea8e","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        \u0027catalog\u0027: [{"},{"line_number":42,"context_line":"            \u0027endpoints\u0027: [{"},{"line_number":43,"context_line":"                \u0027url\u0027:"},{"line_number":44,"context_line":"                \u0027http://public.com:8776/v1/225da22d3ce34b15877ea70b2a575f58\u0027,"},{"line_number":45,"context_line":"                \u0027region\u0027: \u0027RegionOne\u0027,"},{"line_number":46,"context_line":"                \u0027interface\u0027: \u0027public\u0027"},{"line_number":47,"context_line":"            }, {"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F9HVc%3D","line":44,"updated":"2013-12-16 17:43:34.000000000","message":"when I get a domain-scoped token, the endpoints have \"None\" for the tenant-id, so suggest use that throughout these changes to be more realistic.","commit_id":"17ecaa0f029c7bc568b8bc998156ef43d0c07587"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"80516ab464d5d7bb9a0adaa3bd87e04e448bea8e","unresolved":false,"context_lines":[{"line_number":106,"context_line":"            \u0027type\u0027: \u0027ec2\u0027"},{"line_number":107,"context_line":"        }, {"},{"line_number":108,"context_line":"            \u0027endpoints\u0027: [{"},{"line_number":109,"context_line":"                \u0027url\u0027: \u0027http://public.com:5000/v3\u0027,"},{"line_number":110,"context_line":"                \u0027region\u0027: \u0027RegionOne\u0027,"},{"line_number":111,"context_line":"                \u0027interface\u0027: \u0027public\u0027"},{"line_number":112,"context_line":"            }, {"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F9HcA%3D","line":109,"updated":"2013-12-16 17:43:34.000000000","message":"I thought that we didn\u0027t want to have the version on the identity endpoints?","commit_id":"17ecaa0f029c7bc568b8bc998156ef43d0c07587"}],"keystoneclient/tests/v3/test_client.py":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"80516ab464d5d7bb9a0adaa3bd87e04e448bea8e","unresolved":false,"context_lines":[{"line_number":161,"context_line":"        self.stub_auth(json\u003dclient_fixtures.PROJECT_SCOPED_TOKEN)"},{"line_number":162,"context_line":"        cl \u003d client.Client(username\u003d\u0027exampleuser\u0027,"},{"line_number":163,"context_line":"                           password\u003d\u0027password\u0027,"},{"line_number":164,"context_line":"                           project_name\u003d\u0027exampleproject\u0027,"},{"line_number":165,"context_line":"                           auth_url\u003dself.TEST_URL)"},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"        self.assertEqual(cl.management_url, first_url)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F9HIs%3D","line":164,"updated":"2013-12-16 17:43:34.000000000","message":"why change tenant_name to project_name here? was the code changed so that tenant_name doesn\u0027t work anymore?","commit_id":"17ecaa0f029c7bc568b8bc998156ef43d0c07587"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"80516ab464d5d7bb9a0adaa3bd87e04e448bea8e","unresolved":false,"context_lines":[{"line_number":172,"context_line":""},{"line_number":173,"context_line":"    @httpretty.activate"},{"line_number":174,"context_line":"    def test_management_url_is_updated_with_domain(self):"},{"line_number":175,"context_line":"        second \u003d copy.deepcopy(client_fixtures.DOMAIN_SCOPED_TOKEN)"},{"line_number":176,"context_line":"        first_url \u003d \u0027http://admin:35357/v3\u0027"},{"line_number":177,"context_line":"        second_url \u003d \"http://secondurl:%d/v3\u0027\""},{"line_number":178,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F9HHw%3D","line":175,"updated":"2013-12-16 17:43:34.000000000","message":"rather than copy-paste code, move the common code to a function.","commit_id":"17ecaa0f029c7bc568b8bc998156ef43d0c07587"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"047f7ee836b9a3b96074f156df9ac9e745268d0b","unresolved":false,"context_lines":[{"line_number":136,"context_line":"                          username\u003d\u0027exampleuser\u0027,"},{"line_number":137,"context_line":"                          password\u003d\u0027password\u0027)"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"    def _management_url_is_udpated(self, fixture, **kwargs):"},{"line_number":140,"context_line":"        second \u003d copy.deepcopy(fixture)"},{"line_number":141,"context_line":"        first_url \u003d \u0027http://admin:35357/v3\u0027"},{"line_number":142,"context_line":"        second_url \u003d \"http://secondurl:%d/v3\u0027\""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATn%2F80RA%3D","line":139,"updated":"2013-12-20 01:04:39.000000000","message":"change \"udpated\" to \"updated\"","commit_id":"c841778de3136ab816cd3a975c8a431dcfbf29e0"},{"author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"change_message_id":"22853e759187c3f5b170c228cdf61e34b081ca11","unresolved":false,"context_lines":[{"line_number":136,"context_line":"                          username\u003d\u0027exampleuser\u0027,"},{"line_number":137,"context_line":"                          password\u003d\u0027password\u0027)"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"    def _management_url_is_udpated(self, fixture, **kwargs):"},{"line_number":140,"context_line":"        second \u003d copy.deepcopy(fixture)"},{"line_number":141,"context_line":"        first_url \u003d \u0027http://admin:35357/v3\u0027"},{"line_number":142,"context_line":"        second_url \u003d \"http://secondurl:%d/v3\u0027\""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATn%2F80N0%3D","line":139,"in_reply_to":"AAAATn%2F80RA%3D","updated":"2013-12-20 01:15:53.000000000","message":"heh, I managed to miss that not just when writing the function, but c\u0026p the mistake twice in the actual tests.","commit_id":"c841778de3136ab816cd3a975c8a431dcfbf29e0"}]}