)]}'
{"tests/test_http.py":[{"author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"change_message_id":"e98e87247804fa0f2e2bf5b5a3fff0acb06a08c0","unresolved":false,"context_lines":[{"line_number":16,"context_line":"import httplib"},{"line_number":17,"context_line":"import socket"},{"line_number":18,"context_line":"import StringIO"},{"line_number":19,"context_line":"from urlparse import ParseResult"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"import mox"},{"line_number":22,"context_line":"import testtools"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAN3%2F%2F6Nw%3D","line":19,"updated":"2013-03-06 21:23:35.000000000","message":"hey,\n\ndon\u0027t import classes, just import modules and access everything using them.\n\nThis should be:\n\nimport urlparse\n\nurlparse.ParseResult","commit_id":"62f57362f0b9a2ed71e5336d7f400740102044b0"},{"author":{"_account_id":1267,"name":"Alexei Ababilov","display_name":"Alexei Ababilov","email":"ilovegnulinux@gmail.com","username":"aababilov"},"change_message_id":"7658baaf25d9ffe0cf2162405f132404b62b5b24","unresolved":false,"context_lines":[{"line_number":91,"context_line":"            \"\"\""},{"line_number":92,"context_line":"            endpoint \u003d \u0027http://example.com:9292\u0027"},{"line_number":93,"context_line":"            client \u003d http.HTTPClient(endpoint, token\u003du\u0027abc123\u0027)"},{"line_number":94,"context_line":"           # m \u003d mox.Mox()"},{"line_number":95,"context_line":"            httplib.HTTPConnection.request("},{"line_number":96,"context_line":"                mox.IgnoreArg(),"},{"line_number":97,"context_line":"                mox.IgnoreArg(),"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAN3%2F%2F0wU%3D","line":94,"updated":"2013-03-11 11:03:13.000000000","message":"Could you drop this comment, please?","commit_id":"62f57362f0b9a2ed71e5336d7f400740102044b0"},{"author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"change_message_id":"02a7bd0faa128cddecd4ba6ac915ed100bfeba53","unresolved":false,"context_lines":[{"line_number":83,"context_line":"                                                    headers\u003dheaders)"},{"line_number":84,"context_line":"        self.assertEqual(resp, fake)"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"    def test_connection_refused_row_request(self):"},{"line_number":87,"context_line":"            \"\"\""},{"line_number":88,"context_line":"            Should receive a CommunicationError if connection refused."},{"line_number":89,"context_line":"            And the error should list the host and port that refused the"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAN3%2F%2Fy1A%3D","line":86,"updated":"2013-03-12 12:13:30.000000000","message":"raw*","commit_id":"c742490ede91a84080376ef4ed4b4d5129b31ad3"},{"author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"change_message_id":"02a7bd0faa128cddecd4ba6ac915ed100bfeba53","unresolved":false,"context_lines":[{"line_number":106,"context_line":"                            (comm_err.message, endpoint))"},{"line_number":107,"context_line":"                self.assertTrue(endpoint in comm_err.message, fail_msg)"},{"line_number":108,"context_line":"            finally:"},{"line_number":109,"context_line":"                self.mock.UnsetStubs()"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"    def test_parse_endpoint(self):"},{"line_number":112,"context_line":"        endpoint \u003d \u0027http://example.com:9292\u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAN3%2F%2Fy1E%3D","line":109,"updated":"2013-03-12 12:13:30.000000000","message":"Hey,\n\nThis is already called in the tearDown method. Is there a good reason for calling it here?","commit_id":"c742490ede91a84080376ef4ed4b4d5129b31ad3"},{"author":{"_account_id":6493,"name":"John Bresnahan","email":"john@bresnahan.me","username":"jbresnah"},"change_message_id":"ccab5e9c0bc39e7284a413034eee7ac15c12e5fc","unresolved":false,"context_lines":[{"line_number":89,"context_line":"            And the error should list the host and port that refused the"},{"line_number":90,"context_line":"            connection"},{"line_number":91,"context_line":"            \"\"\""},{"line_number":92,"context_line":"            endpoint \u003d \u0027http://example.com:9292\u0027"},{"line_number":93,"context_line":"            client \u003d http.HTTPClient(endpoint, token\u003du\u0027abc123\u0027)"},{"line_number":94,"context_line":"            httplib.HTTPConnection.request("},{"line_number":95,"context_line":"                mox.IgnoreArg(),"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAN3%2F%2FyGU%3D","line":92,"updated":"2013-03-12 18:13:51.000000000","message":"Should this be indented so far?","commit_id":"ca1dff779aad28e5d272120134ae3c6980ff8edc"},{"author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"change_message_id":"fe77fa6f0ebc2b98b705f92c56e9db3292b1c959","unresolved":false,"context_lines":[{"line_number":89,"context_line":"            And the error should list the host and port that refused the"},{"line_number":90,"context_line":"            connection"},{"line_number":91,"context_line":"            \"\"\""},{"line_number":92,"context_line":"            endpoint \u003d \u0027http://example.com:9292\u0027"},{"line_number":93,"context_line":"            client \u003d http.HTTPClient(endpoint, token\u003du\u0027abc123\u0027)"},{"line_number":94,"context_line":"            httplib.HTTPConnection.request("},{"line_number":95,"context_line":"                mox.IgnoreArg(),"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAN3%2F%2FyDY%3D","line":92,"in_reply_to":"AAAAN3%2F%2FyGU%3D","updated":"2013-03-12 18:28:33.000000000","message":"It\u0027s definitely over-indented, we should start running flake8 or something similar.","commit_id":"ca1dff779aad28e5d272120134ae3c6980ff8edc"},{"author":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"change_message_id":"5e76ec16ff53ef282f3d66185c10d27ef338185a","unresolved":false,"context_lines":[{"line_number":118,"context_line":"        test_client \u003d http.HTTPClient(endpoint, token\u003du\u0027adc123\u0027)"},{"line_number":119,"context_line":"        actual \u003d test_client.get_connection_class(\u0027https\u0027)"},{"line_number":120,"context_line":"        self.assertTrue("},{"line_number":121,"context_line":"            True, isinstance(actual, http.VerifiedHTTPSConnection))"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"    def test_get_connections_kwargs_http(self):"},{"line_number":124,"context_line":"        endpoint \u003d \u0027http://example.com:9292\u0027"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAOH%2F%2FsJs%3D","line":121,"updated":"2013-04-02 22:44:50.000000000","message":"You should drop the True from the arguments here. All this line does is assert True is True in its current form.","commit_id":"d70728ee6f5ac7d67ec61bb358e55840b64f93e5"},{"author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"change_message_id":"99946fd68eda7a9c1edd034cdaa8763a00699c4c","unresolved":false,"context_lines":[{"line_number":118,"context_line":"        test_client \u003d http.HTTPClient(endpoint, token\u003du\u0027adc123\u0027)"},{"line_number":119,"context_line":"        expected \u003d \"glanceclient.common.http.VerifiedHTTPSConnection\""},{"line_number":120,"context_line":"        actual \u003d str(test_client.get_connection_class(\u0027https\u0027))"},{"line_number":121,"context_line":"        self.assertRegexpMatches(actual, expected)"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"    def test_get_connections_kwargs_http(self):"},{"line_number":124,"context_line":"        endpoint \u003d \u0027http://example.com:9292\u0027"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAOX%2F%2F9rM%3D","line":121,"updated":"2013-04-04 11:11:19.000000000","message":"Hey,\n\nWhat Brian meant was this line should look like:\n\nself.assertTrue(isinstance(actual, http.VerifiedHTTPSConnection))\n\nCheers!","commit_id":"42b66744a9725d405fa89ed69db4e5995c3bf4dc"}]}