)]}'
{"tests/test_http.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8a5f8ebd2a11a398783a3d0207c9d41a5a965015","unresolved":false,"context_lines":[{"line_number":249,"context_line":"        try:"},{"line_number":250,"context_line":"            headers \u003d {\u0027header1\u0027: \u0027value1\\xa5\\xa6\u0027}"},{"line_number":251,"context_line":"            body \u003d \u0027examplebody\\xa5\\xa6\u0027"},{"line_number":252,"context_line":"            http_client_object \u003d http.HTTPClient(self.endpoint,"},{"line_number":253,"context_line":"                                                 token\u003d\u0027fake-token\u0027)"},{"line_number":254,"context_line":"            http_client_object.log_curl_request(\u0027GET\u0027,"},{"line_number":255,"context_line":"                                                \u0027/api/v1/\\xa5\u0027,"}],"source_content_type":"text/x-python","patch_set":11,"id":"da9df570_6204d67e","line":252,"updated":"2014-09-21 20:54:54.000000000","message":"Re-use self.client.","commit_id":"5033284dd7ecd95a23649f1690d6a99d9e1a1d5b"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"df1e7df2129d856ab806b14c9e734a2b3f8c61dd","unresolved":false,"context_lines":[{"line_number":249,"context_line":"        try:"},{"line_number":250,"context_line":"            headers \u003d {\u0027header1\u0027: \u0027value1\\xa5\\xa6\u0027}"},{"line_number":251,"context_line":"            body \u003d \u0027examplebody\\xa5\\xa6\u0027"},{"line_number":252,"context_line":"            http_client_object \u003d http.HTTPClient(self.endpoint,"},{"line_number":253,"context_line":"                                                 token\u003d\u0027fake-token\u0027)"},{"line_number":254,"context_line":"            http_client_object.log_curl_request(\u0027GET\u0027,"},{"line_number":255,"context_line":"                                                \u0027/api/v1/\\xa5\u0027,"}],"source_content_type":"text/x-python","patch_set":11,"id":"da9df570_7afff634","line":252,"in_reply_to":"da9df570_6204d67e","updated":"2014-09-23 08:37:52.000000000","message":"Done","commit_id":"5033284dd7ecd95a23649f1690d6a99d9e1a1d5b"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8a5f8ebd2a11a398783a3d0207c9d41a5a965015","unresolved":false,"context_lines":[{"line_number":267,"context_line":"                                                 token\u003d\u0027fake-token\u0027)"},{"line_number":268,"context_line":"            http_client_object.log_curl_request(\u0027GET\u0027, \u0027/api/v1/\u0027, headers,"},{"line_number":269,"context_line":"                                                body, None)"},{"line_number":270,"context_line":"            mock_log.assert_called()"},{"line_number":271,"context_line":"            self.assertTrue(mock_log.call_args[0],"},{"line_number":272,"context_line":"                            \u0027LOG.debug called with no arguments\u0027)"},{"line_number":273,"context_line":"            hd_regex \u003d \".*\\s-H\\s+\u0027\\s*%s\\s*:\\s*%s\\s*\u0027.*\" % (hd_name, hd_val)"}],"source_content_type":"text/x-python","patch_set":11,"id":"da9df570_a2e79ee3","line":270,"updated":"2014-09-21 20:54:54.000000000","message":"Not a method on Mock.","commit_id":"5033284dd7ecd95a23649f1690d6a99d9e1a1d5b"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"df1e7df2129d856ab806b14c9e734a2b3f8c61dd","unresolved":false,"context_lines":[{"line_number":267,"context_line":"                                                 token\u003d\u0027fake-token\u0027)"},{"line_number":268,"context_line":"            http_client_object.log_curl_request(\u0027GET\u0027, \u0027/api/v1/\u0027, headers,"},{"line_number":269,"context_line":"                                                body, None)"},{"line_number":270,"context_line":"            mock_log.assert_called()"},{"line_number":271,"context_line":"            self.assertTrue(mock_log.call_args[0],"},{"line_number":272,"context_line":"                            \u0027LOG.debug called with no arguments\u0027)"},{"line_number":273,"context_line":"            hd_regex \u003d \".*\\s-H\\s+\u0027\\s*%s\\s*:\\s*%s\\s*\u0027.*\" % (hd_name, hd_val)"}],"source_content_type":"text/x-python","patch_set":11,"id":"da9df570_3a9f2e54","line":270,"in_reply_to":"da9df570_a2e79ee3","updated":"2014-09-23 08:37:52.000000000","message":"Done","commit_id":"5033284dd7ecd95a23649f1690d6a99d9e1a1d5b"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8a5f8ebd2a11a398783a3d0207c9d41a5a965015","unresolved":false,"context_lines":[{"line_number":292,"context_line":"                                                 token\u003d\u0027fake-token\u0027)"},{"line_number":293,"context_line":"            http_client_object.log_curl_request(\u0027GET\u0027, \u0027/api/v1/\u0027, headers,"},{"line_number":294,"context_line":"                                                None, None)"},{"line_number":295,"context_line":"            mock_log.assert_called()"},{"line_number":296,"context_line":"            self.assertTrue(len(mock_log.call_args[0]) \u003e 0,"},{"line_number":297,"context_line":"                            \u0027LOG.debug called with no arguments\u0027)"},{"line_number":298,"context_line":"            key_regex \u003d \".*\\s--key\\s+(\u0027%s\u0027|%s).*\" % (key, key)"}],"source_content_type":"text/x-python","patch_set":11,"id":"da9df570_42c1922d","line":295,"updated":"2014-09-21 20:54:54.000000000","message":"Not a method on Mock.","commit_id":"5033284dd7ecd95a23649f1690d6a99d9e1a1d5b"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"df1e7df2129d856ab806b14c9e734a2b3f8c61dd","unresolved":false,"context_lines":[{"line_number":292,"context_line":"                                                 token\u003d\u0027fake-token\u0027)"},{"line_number":293,"context_line":"            http_client_object.log_curl_request(\u0027GET\u0027, \u0027/api/v1/\u0027, headers,"},{"line_number":294,"context_line":"                                                None, None)"},{"line_number":295,"context_line":"            mock_log.assert_called()"},{"line_number":296,"context_line":"            self.assertTrue(len(mock_log.call_args[0]) \u003e 0,"},{"line_number":297,"context_line":"                            \u0027LOG.debug called with no arguments\u0027)"},{"line_number":298,"context_line":"            key_regex \u003d \".*\\s--key\\s+(\u0027%s\u0027|%s).*\" % (key, key)"}],"source_content_type":"text/x-python","patch_set":11,"id":"da9df570_5aa3129e","line":295,"in_reply_to":"da9df570_42c1922d","updated":"2014-09-23 08:37:52.000000000","message":"Done","commit_id":"5033284dd7ecd95a23649f1690d6a99d9e1a1d5b"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8a5f8ebd2a11a398783a3d0207c9d41a5a965015","unresolved":false,"context_lines":[{"line_number":308,"context_line":"                            matchers.MatchesRegex(cacert_regex),"},{"line_number":309,"context_line":"                            \u0027no --caert option in curl command\u0027)"},{"line_number":310,"context_line":""},{"line_number":311,"context_line":"    def test_log_curl_request_with_some_certs(self):"},{"line_number":312,"context_line":"        with mock.patch(\u0027glanceclient.common.http.LOG.debug\u0027) as mock_log:"},{"line_number":313,"context_line":"            headers \u003d {\u0027header1\u0027: \u0027value1\u0027}"},{"line_number":314,"context_line":"            key \u003d \u0027key1\u0027"}],"source_content_type":"text/x-python","patch_set":11,"id":"da9df570_42d8b21d","line":311,"updated":"2014-09-21 20:54:54.000000000","message":"You could probably easily combine this test with the previous one into a private method used by both and pass in the cert_file value (None and not None per test).  Would reduce the copy/paste.","commit_id":"5033284dd7ecd95a23649f1690d6a99d9e1a1d5b"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"df1e7df2129d856ab806b14c9e734a2b3f8c61dd","unresolved":false,"context_lines":[{"line_number":308,"context_line":"                            matchers.MatchesRegex(cacert_regex),"},{"line_number":309,"context_line":"                            \u0027no --caert option in curl command\u0027)"},{"line_number":310,"context_line":""},{"line_number":311,"context_line":"    def test_log_curl_request_with_some_certs(self):"},{"line_number":312,"context_line":"        with mock.patch(\u0027glanceclient.common.http.LOG.debug\u0027) as mock_log:"},{"line_number":313,"context_line":"            headers \u003d {\u0027header1\u0027: \u0027value1\u0027}"},{"line_number":314,"context_line":"            key \u003d \u0027key1\u0027"}],"source_content_type":"text/x-python","patch_set":11,"id":"da9df570_fde200d4","line":311,"in_reply_to":"da9df570_42d8b21d","updated":"2014-09-23 08:37:52.000000000","message":"Done","commit_id":"5033284dd7ecd95a23649f1690d6a99d9e1a1d5b"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8a5f8ebd2a11a398783a3d0207c9d41a5a965015","unresolved":false,"context_lines":[{"line_number":320,"context_line":"                                                 token\u003d\u0027fake-token\u0027)"},{"line_number":321,"context_line":"            http_client_object.log_curl_request(\u0027GET\u0027, \u0027/api/v1/\u0027, headers,"},{"line_number":322,"context_line":"                                                None, None)"},{"line_number":323,"context_line":"            mock_log.assert_called()"},{"line_number":324,"context_line":"            self.assertTrue(len(mock_log.call_args[0]) \u003e 0,"},{"line_number":325,"context_line":"                            \u0027LOG.debug called with no arguments\u0027)"},{"line_number":326,"context_line":"            key_regex \u003d \".*\\s--key\\s+(\u0027%s\u0027|%s).*\" % (key, key)"}],"source_content_type":"text/x-python","patch_set":11,"id":"da9df570_02c78a46","line":323,"updated":"2014-09-21 20:54:54.000000000","message":"Not a method on Mock.","commit_id":"5033284dd7ecd95a23649f1690d6a99d9e1a1d5b"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"df1e7df2129d856ab806b14c9e734a2b3f8c61dd","unresolved":false,"context_lines":[{"line_number":320,"context_line":"                                                 token\u003d\u0027fake-token\u0027)"},{"line_number":321,"context_line":"            http_client_object.log_curl_request(\u0027GET\u0027, \u0027/api/v1/\u0027, headers,"},{"line_number":322,"context_line":"                                                None, None)"},{"line_number":323,"context_line":"            mock_log.assert_called()"},{"line_number":324,"context_line":"            self.assertTrue(len(mock_log.call_args[0]) \u003e 0,"},{"line_number":325,"context_line":"                            \u0027LOG.debug called with no arguments\u0027)"},{"line_number":326,"context_line":"            key_regex \u003d \".*\\s--key\\s+(\u0027%s\u0027|%s).*\" % (key, key)"}],"source_content_type":"text/x-python","patch_set":11,"id":"da9df570_7aa0d690","line":323,"in_reply_to":"da9df570_02c78a46","updated":"2014-09-23 08:37:52.000000000","message":"Done","commit_id":"5033284dd7ecd95a23649f1690d6a99d9e1a1d5b"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8a5f8ebd2a11a398783a3d0207c9d41a5a965015","unresolved":false,"context_lines":[{"line_number":344,"context_line":"                                                 token\u003d\u0027fake-token\u0027)"},{"line_number":345,"context_line":"            http_client_object.log_curl_request(\u0027GET\u0027, \u0027/api/v1/\u0027, headers,"},{"line_number":346,"context_line":"                                                None, None)"},{"line_number":347,"context_line":"            mock_log.assert_called()"},{"line_number":348,"context_line":"            self.assertTrue(mock_log.call_args[0],"},{"line_number":349,"context_line":"                            \u0027LOG.debug called with no arguments\u0027)"},{"line_number":350,"context_line":"            self.assertThat(mock_log.call_args[0][0],"}],"source_content_type":"text/x-python","patch_set":11,"id":"da9df570_e2c9a65b","line":347,"updated":"2014-09-21 20:54:54.000000000","message":"Not a method on Mock.","commit_id":"5033284dd7ecd95a23649f1690d6a99d9e1a1d5b"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"df1e7df2129d856ab806b14c9e734a2b3f8c61dd","unresolved":false,"context_lines":[{"line_number":344,"context_line":"                                                 token\u003d\u0027fake-token\u0027)"},{"line_number":345,"context_line":"            http_client_object.log_curl_request(\u0027GET\u0027, \u0027/api/v1/\u0027, headers,"},{"line_number":346,"context_line":"                                                None, None)"},{"line_number":347,"context_line":"            mock_log.assert_called()"},{"line_number":348,"context_line":"            self.assertTrue(mock_log.call_args[0],"},{"line_number":349,"context_line":"                            \u0027LOG.debug called with no arguments\u0027)"},{"line_number":350,"context_line":"            self.assertThat(mock_log.call_args[0][0],"}],"source_content_type":"text/x-python","patch_set":11,"id":"da9df570_9aad7a8b","line":347,"in_reply_to":"da9df570_e2c9a65b","updated":"2014-09-23 08:37:52.000000000","message":"Done","commit_id":"5033284dd7ecd95a23649f1690d6a99d9e1a1d5b"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8a5f8ebd2a11a398783a3d0207c9d41a5a965015","unresolved":false,"context_lines":[{"line_number":352,"context_line":"                            \u0027no -k option in curl command\u0027)"},{"line_number":353,"context_line":""},{"line_number":354,"context_line":"    def test_log_curl_request_with_token_header(self):"},{"line_number":355,"context_line":"        with mock.patch(\u0027glanceclient.common.http.LOG.debug\u0027) as mock_log:"},{"line_number":356,"context_line":"            fake_token \u003d \u0027fake-token\u0027"},{"line_number":357,"context_line":"            headers \u003d {\u0027X-Auth-Token\u0027: fake_token}"},{"line_number":358,"context_line":"            http_client_object \u003d http.HTTPClient(self.endpoint,"}],"source_content_type":"text/x-python","patch_set":11,"id":"da9df570_c2b582ce","line":355,"updated":"2014-09-21 20:54:54.000000000","message":"Prefer the method decorator style rather than context, but it\u0027s a style nit.","commit_id":"5033284dd7ecd95a23649f1690d6a99d9e1a1d5b"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"df1e7df2129d856ab806b14c9e734a2b3f8c61dd","unresolved":false,"context_lines":[{"line_number":352,"context_line":"                            \u0027no -k option in curl command\u0027)"},{"line_number":353,"context_line":""},{"line_number":354,"context_line":"    def test_log_curl_request_with_token_header(self):"},{"line_number":355,"context_line":"        with mock.patch(\u0027glanceclient.common.http.LOG.debug\u0027) as mock_log:"},{"line_number":356,"context_line":"            fake_token \u003d \u0027fake-token\u0027"},{"line_number":357,"context_line":"            headers \u003d {\u0027X-Auth-Token\u0027: fake_token}"},{"line_number":358,"context_line":"            http_client_object \u003d http.HTTPClient(self.endpoint,"}],"source_content_type":"text/x-python","patch_set":11,"id":"da9df570_5a54f253","line":355,"in_reply_to":"da9df570_c2b582ce","updated":"2014-09-23 08:37:52.000000000","message":"Yup, that\u0027s true, decorators are more elegant. Done.","commit_id":"5033284dd7ecd95a23649f1690d6a99d9e1a1d5b"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8a5f8ebd2a11a398783a3d0207c9d41a5a965015","unresolved":false,"context_lines":[{"line_number":359,"context_line":"                                                 identity_headers\u003dheaders)"},{"line_number":360,"context_line":"            http_client_object.log_curl_request(\u0027GET\u0027, \u0027/api/v1/\u0027, headers,"},{"line_number":361,"context_line":"                                                None, None)"},{"line_number":362,"context_line":"            mock_log.assert_called()"},{"line_number":363,"context_line":"            self.assertTrue(mock_log.call_args[0],"},{"line_number":364,"context_line":"                            \u0027LOG.debug called with no arguments\u0027)"},{"line_number":365,"context_line":"            token_regex \u003d \u0027.*%s.*\u0027 % fake_token"}],"source_content_type":"text/x-python","patch_set":11,"id":"da9df570_a2b07ede","line":362,"updated":"2014-09-21 20:54:54.000000000","message":"assert_called() isn\u0027t a method on mock.Mock.\n\nUse assert_called_once_with instead.","commit_id":"5033284dd7ecd95a23649f1690d6a99d9e1a1d5b"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"df1e7df2129d856ab806b14c9e734a2b3f8c61dd","unresolved":false,"context_lines":[{"line_number":359,"context_line":"                                                 identity_headers\u003dheaders)"},{"line_number":360,"context_line":"            http_client_object.log_curl_request(\u0027GET\u0027, \u0027/api/v1/\u0027, headers,"},{"line_number":361,"context_line":"                                                None, None)"},{"line_number":362,"context_line":"            mock_log.assert_called()"},{"line_number":363,"context_line":"            self.assertTrue(mock_log.call_args[0],"},{"line_number":364,"context_line":"                            \u0027LOG.debug called with no arguments\u0027)"},{"line_number":365,"context_line":"            token_regex \u003d \u0027.*%s.*\u0027 % fake_token"}],"source_content_type":"text/x-python","patch_set":11,"id":"da9df570_7a51b661","line":362,"in_reply_to":"da9df570_a2b07ede","updated":"2014-09-23 08:37:52.000000000","message":"Done","commit_id":"5033284dd7ecd95a23649f1690d6a99d9e1a1d5b"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"fcbd7f371c5efdd16ae4bb3b38d9712c65628da8","unresolved":false,"context_lines":[{"line_number":261,"context_line":"        headers \u003d {hd_name: hd_val}"},{"line_number":262,"context_line":"        body \u003d \u0027examplebody\u0027"},{"line_number":263,"context_line":"        self.client.log_curl_request(\u0027GET\u0027, \u0027/api/v1/\u0027, headers, body, None)"},{"line_number":264,"context_line":"        mock_log.assert_called_once()"},{"line_number":265,"context_line":"        self.assertTrue(mock_log.call_args[0],"},{"line_number":266,"context_line":"                        \u0027LOG.debug called with no arguments\u0027)"},{"line_number":267,"context_line":"        hd_regex \u003d \".*\\s-H\\s+\u0027\\s*%s\\s*:\\s*%s\\s*\u0027.*\" % (hd_name, hd_val)"}],"source_content_type":"text/x-python","patch_set":12,"id":"da9df570_ebe05f98","line":264,"updated":"2014-09-23 21:24:36.000000000","message":"This isn\u0027t a valid method, use assert_called_once_with:\n\nhttps://docs.python.org/3/library/unittest.mock.html","commit_id":"6c6ba7e065b6af8ca6352111634c3cebae3ae5dc"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"ea48c08aeb5925646d782f3a0ac60681070adc2b","unresolved":false,"context_lines":[{"line_number":261,"context_line":"        headers \u003d {hd_name: hd_val}"},{"line_number":262,"context_line":"        body \u003d \u0027examplebody\u0027"},{"line_number":263,"context_line":"        self.client.log_curl_request(\u0027GET\u0027, \u0027/api/v1/\u0027, headers, body, None)"},{"line_number":264,"context_line":"        mock_log.assert_called_once()"},{"line_number":265,"context_line":"        self.assertTrue(mock_log.call_args[0],"},{"line_number":266,"context_line":"                        \u0027LOG.debug called with no arguments\u0027)"},{"line_number":267,"context_line":"        hd_regex \u003d \".*\\s-H\\s+\u0027\\s*%s\\s*:\\s*%s\\s*\u0027.*\" % (hd_name, hd_val)"}],"source_content_type":"text/x-python","patch_set":12,"id":"da9df570_851097a9","line":264,"in_reply_to":"da9df570_ebe05f98","updated":"2014-09-24 09:34:25.000000000","message":"Done","commit_id":"6c6ba7e065b6af8ca6352111634c3cebae3ae5dc"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"fcbd7f371c5efdd16ae4bb3b38d9712c65628da8","unresolved":false,"context_lines":[{"line_number":280,"context_line":"                                             token\u003d\u0027fake-token\u0027)"},{"line_number":281,"context_line":"        http_client_object.log_curl_request(\u0027GET\u0027, \u0027/api/v1/\u0027, headers, None,"},{"line_number":282,"context_line":"                                            None)"},{"line_number":283,"context_line":"        mock_log.assert_called_once()"},{"line_number":284,"context_line":"        self.assertTrue(len(mock_log.call_args[0]) \u003e 0,"},{"line_number":285,"context_line":"                        \u0027LOG.debug called with no arguments\u0027)"},{"line_number":286,"context_line":""}],"source_content_type":"text/x-python","patch_set":12,"id":"da9df570_0bef6388","line":283,"updated":"2014-09-23 21:24:36.000000000","message":"ditto","commit_id":"6c6ba7e065b6af8ca6352111634c3cebae3ae5dc"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"ea48c08aeb5925646d782f3a0ac60681070adc2b","unresolved":false,"context_lines":[{"line_number":280,"context_line":"                                             token\u003d\u0027fake-token\u0027)"},{"line_number":281,"context_line":"        http_client_object.log_curl_request(\u0027GET\u0027, \u0027/api/v1/\u0027, headers, None,"},{"line_number":282,"context_line":"                                            None)"},{"line_number":283,"context_line":"        mock_log.assert_called_once()"},{"line_number":284,"context_line":"        self.assertTrue(len(mock_log.call_args[0]) \u003e 0,"},{"line_number":285,"context_line":"                        \u0027LOG.debug called with no arguments\u0027)"},{"line_number":286,"context_line":""}],"source_content_type":"text/x-python","patch_set":12,"id":"da9df570_a5267b14","line":283,"in_reply_to":"da9df570_0bef6388","updated":"2014-09-24 09:34:25.000000000","message":"Done","commit_id":"6c6ba7e065b6af8ca6352111634c3cebae3ae5dc"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"fcbd7f371c5efdd16ae4bb3b38d9712c65628da8","unresolved":false,"context_lines":[{"line_number":314,"context_line":"                                             token\u003d\u0027fake-token\u0027)"},{"line_number":315,"context_line":"        http_client_object.log_curl_request(\u0027GET\u0027, \u0027/api/v1/\u0027, headers, None,"},{"line_number":316,"context_line":"                                            None)"},{"line_number":317,"context_line":"        mock_log.assert_called_once()"},{"line_number":318,"context_line":"        self.assertTrue(mock_log.call_args[0],"},{"line_number":319,"context_line":"                        \u0027LOG.debug called with no arguments\u0027)"},{"line_number":320,"context_line":"        self.assertThat(mock_log.call_args[0][0],"}],"source_content_type":"text/x-python","patch_set":12,"id":"da9df570_2bf2676f","line":317,"updated":"2014-09-23 21:24:36.000000000","message":"ditto","commit_id":"6c6ba7e065b6af8ca6352111634c3cebae3ae5dc"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"ea48c08aeb5925646d782f3a0ac60681070adc2b","unresolved":false,"context_lines":[{"line_number":314,"context_line":"                                             token\u003d\u0027fake-token\u0027)"},{"line_number":315,"context_line":"        http_client_object.log_curl_request(\u0027GET\u0027, \u0027/api/v1/\u0027, headers, None,"},{"line_number":316,"context_line":"                                            None)"},{"line_number":317,"context_line":"        mock_log.assert_called_once()"},{"line_number":318,"context_line":"        self.assertTrue(mock_log.call_args[0],"},{"line_number":319,"context_line":"                        \u0027LOG.debug called with no arguments\u0027)"},{"line_number":320,"context_line":"        self.assertThat(mock_log.call_args[0][0],"}],"source_content_type":"text/x-python","patch_set":12,"id":"da9df570_c521bffd","line":317,"in_reply_to":"da9df570_2bf2676f","updated":"2014-09-24 09:34:25.000000000","message":"Done","commit_id":"6c6ba7e065b6af8ca6352111634c3cebae3ae5dc"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"fcbd7f371c5efdd16ae4bb3b38d9712c65628da8","unresolved":false,"context_lines":[{"line_number":329,"context_line":"                                             identity_headers\u003dheaders)"},{"line_number":330,"context_line":"        http_client_object.log_curl_request(\u0027GET\u0027, \u0027/api/v1/\u0027, headers, None,"},{"line_number":331,"context_line":"                                            None)"},{"line_number":332,"context_line":"        mock_log.assert_called_once()"},{"line_number":333,"context_line":"        self.assertTrue(mock_log.call_args[0],"},{"line_number":334,"context_line":"                        \u0027LOG.debug called with no arguments\u0027)"},{"line_number":335,"context_line":"        token_regex \u003d \u0027.*%s.*\u0027 % fake_token"}],"source_content_type":"text/x-python","patch_set":12,"id":"da9df570_6bec6f90","line":332,"updated":"2014-09-23 21:24:36.000000000","message":"ditto","commit_id":"6c6ba7e065b6af8ca6352111634c3cebae3ae5dc"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"ea48c08aeb5925646d782f3a0ac60681070adc2b","unresolved":false,"context_lines":[{"line_number":329,"context_line":"                                             identity_headers\u003dheaders)"},{"line_number":330,"context_line":"        http_client_object.log_curl_request(\u0027GET\u0027, \u0027/api/v1/\u0027, headers, None,"},{"line_number":331,"context_line":"                                            None)"},{"line_number":332,"context_line":"        mock_log.assert_called_once()"},{"line_number":333,"context_line":"        self.assertTrue(mock_log.call_args[0],"},{"line_number":334,"context_line":"                        \u0027LOG.debug called with no arguments\u0027)"},{"line_number":335,"context_line":"        token_regex \u003d \u0027.*%s.*\u0027 % fake_token"}],"source_content_type":"text/x-python","patch_set":12,"id":"da9df570_e52083f9","line":332,"in_reply_to":"da9df570_6bec6f90","updated":"2014-09-24 09:34:25.000000000","message":"Done","commit_id":"6c6ba7e065b6af8ca6352111634c3cebae3ae5dc"},{"author":{"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},"change_message_id":"0296f17656a1fd7f7228768c55aad8b8175cc1b5","unresolved":false,"context_lines":[{"line_number":284,"context_line":"        self.assertTrue(mock_log.call_args[0],"},{"line_number":285,"context_line":"                        \u0027LOG.debug called with no arguments\u0027)"},{"line_number":286,"context_line":""},{"line_number":287,"context_line":"        needles \u003d [(\u0027key\u0027, key), (\u0027cert\u0027, cert), (\u0027cacert\u0027, cacert)]"},{"line_number":288,"context_line":"        for option, value in needles:"},{"line_number":289,"context_line":"            if value:"},{"line_number":290,"context_line":"                regex \u003d \".*\\s--%s\\s+(\u0027%s\u0027|%s).*\" % (option, value, value)"}],"source_content_type":"text/x-python","patch_set":13,"id":"5a890539_9427a162","line":287,"updated":"2014-12-03 13:52:35.000000000","message":"Why not use a dict here, and do\n\n    needles \u003d {\u0027key\u0027: key, \u0027cert\u0027: cert, \u0027cacert\u0027: cacert}\n    for option, value in needles.items():","commit_id":"3ec6137aa2beb3e3a33923fe6110a85edfdd8558"}]}