)]}'
{"trove/tests/fakes/swift.py":[{"author":{"_account_id":7806,"name":"Sushil Kumar","email":"skm.net@gmail.com","username":"sushilkm"},"change_message_id":"24f4d41bbb2a7d97b116b249868684c99cb84700","unresolved":false,"context_lines":[{"line_number":14,"context_line":"#    under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from hashlib import md5"},{"line_number":17,"context_line":"from mock import MagicMock, patch"},{"line_number":18,"context_line":"import httplib"},{"line_number":19,"context_line":"import json"},{"line_number":20,"context_line":"import logging"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aca2d91_76ae9786","line":17,"updated":"2015-05-07 18:58:10.000000000","message":"can we redo the order","commit_id":"fbfc1cfab00178b423a8600a5797ecb4b31cd84b"},{"author":{"_account_id":14576,"name":"Petr Malik","email":"pmalik@tesora.com","username":"pmalik"},"change_message_id":"b7d1f9e68c9404b20097077c57bb64b3adf73662","unresolved":false,"context_lines":[{"line_number":203,"context_line":"        pass"},{"line_number":204,"context_line":""},{"line_number":205,"context_line":""},{"line_number":206,"context_line":"class Patcher(object):"},{"line_number":207,"context_line":"    \"\"\"Objects that need to mock global symbols throughout their existence"},{"line_number":208,"context_line":"    should extend this base class."},{"line_number":209,"context_line":"    The object acts as a context manager which, when used in conjunction with"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aca2d91_631729a4","line":206,"updated":"2015-05-06 19:39:02.000000000","message":"This is necessary because although the \u0027SwiftClientStub\u0027 is a standard \u0027object\u0027 it mocks global modules. This is not good at all. This base class makes it into a context manager which keeps track and restores all of them on __exit__.","commit_id":"fbfc1cfab00178b423a8600a5797ecb4b31cd84b"},{"author":{"_account_id":9782,"name":"Morgan Jones","email":"morgan@tesora.com","username":"vgnbkr"},"change_message_id":"5862d99f2203fbe916f7e7cf6d95b25ece822a79","unresolved":false,"context_lines":[{"line_number":203,"context_line":"        pass"},{"line_number":204,"context_line":""},{"line_number":205,"context_line":""},{"line_number":206,"context_line":"class Patcher(object):"},{"line_number":207,"context_line":"    \"\"\"Objects that need to mock global symbols throughout their existence"},{"line_number":208,"context_line":"    should extend this base class."},{"line_number":209,"context_line":"    The object acts as a context manager which, when used in conjunction with"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aca2d91_964ba1b0","line":206,"in_reply_to":"1aca2d91_631729a4","updated":"2015-05-06 20:12:33.000000000","message":"Why not make this a comment so it is preserved","commit_id":"fbfc1cfab00178b423a8600a5797ecb4b31cd84b"}],"trove/tests/unittests/common/test_remote.py":[{"author":{"_account_id":14576,"name":"Petr Malik","email":"pmalik@tesora.com","username":"pmalik"},"change_message_id":"24de91c00ce85c210774c51cb6b63503a3e3071d","unresolved":false,"context_lines":[{"line_number":40,"context_line":"    def test_creation(self):"},{"line_number":41,"context_line":"        swiftclient.client.Connection.get_auth \u003d MagicMock(return_value\u003dNone)"},{"line_number":42,"context_line":"        conn \u003d swiftclient.client.Connection()"},{"line_number":43,"context_line":"        self.assertIsNone(conn.get_auth())"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    def test_create_swift_client(self):"},{"line_number":46,"context_line":"        mock_resp \u003d MagicMock()"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aca2d91_23e25104","side":"PARENT","line":43,"updated":"2015-05-06 19:29:51.000000000","message":"This statement really tests the mock library rather than any Trove code. I changed it to assert that the created object is not None.","commit_id":"b1cf32ba5b0874914d6f8d1413df69e95b06c3f6"},{"author":{"_account_id":7806,"name":"Sushil Kumar","email":"skm.net@gmail.com","username":"sushilkm"},"change_message_id":"24f4d41bbb2a7d97b116b249868684c99cb84700","unresolved":false,"context_lines":[{"line_number":23,"context_line":"from trove.common.context import TroveContext"},{"line_number":24,"context_line":"from trove.tests.fakes.swift import SwiftClientStub"},{"line_number":25,"context_line":"from trove.tests.unittests import trove_testtools"},{"line_number":26,"context_line":"import swiftclient.client"},{"line_number":27,"context_line":"import uuid"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1aca2d91_f6fe276a","line":26,"updated":"2015-05-07 18:58:10.000000000","message":"can we redo the order","commit_id":"fbfc1cfab00178b423a8600a5797ecb4b31cd84b"}],"trove/tests/unittests/common/test_template.py":[{"author":{"_account_id":7806,"name":"Sushil Kumar","email":"skm.net@gmail.com","username":"sushilkm"},"change_message_id":"24f4d41bbb2a7d97b116b249868684c99cb84700","unresolved":false,"context_lines":[{"line_number":16,"context_line":"from trove.datastore.models import DatastoreVersion"},{"line_number":17,"context_line":"from trove.tests.unittests import trove_testtools"},{"line_number":18,"context_line":"from trove.tests.unittests.util import util"},{"line_number":19,"context_line":"import re"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"class TemplateTest(trove_testtools.TestCase):"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aca2d91_d60fab86","line":19,"updated":"2015-05-07 18:58:10.000000000","message":"can we improve on ordering of imports","commit_id":"fbfc1cfab00178b423a8600a5797ecb4b31cd84b"}]}