)]}'
{"trove/tests/unittests/guestagent/test_agent_heartbeats_models.py":[{"author":{"_account_id":7806,"name":"Sushil Kumar","email":"skm.net@gmail.com","username":"sushilkm"},"change_message_id":"ab133c246bd7e2f0ef1dfcddb706991d8b9773db","unresolved":false,"context_lines":[{"line_number":10,"context_line":"# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied."},{"line_number":11,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":12,"context_line":"# limitations under the License."},{"line_number":13,"context_line":"import uuid"},{"line_number":14,"context_line":"from trove.common import exception"},{"line_number":15,"context_line":"from trove.guestagent.models import AgentHeartBeat"},{"line_number":16,"context_line":"from trove.tests.unittests import trove_testtools"}],"source_content_type":"text/x-python","patch_set":2,"id":"1aca2d91_42fd1a39","line":13,"updated":"2015-05-07 18:49:03.000000000","message":"Not sure, at many other places i saw uuid going down, can we be consistent with order","commit_id":"c0a8853fe7578075ad8fdbd4520982c5089032c0"}],"trove/tests/unittests/guestagent/test_models.py":[{"author":{"_account_id":13355,"name":"Edmond Kotowski","email":"ekotowski@gmail.com","username":"ekotowski"},"change_message_id":"cbc1d63c3a621ca3a4b023e9e7f9c4bbd3f77210","unresolved":false,"context_lines":[{"line_number":61,"context_line":"        self.heartBeat.save()"},{"line_number":62,"context_line":"        self.assertEqual(1, utils.utcnow.call_count)"},{"line_number":63,"context_line":"        # make sure we remove is_valid from the Mock class"},{"line_number":64,"context_line":"        del dbmodels.DatabaseModelBase.is_valid"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"    def test_is_active(self):"},{"line_number":67,"context_line":"        models.AGENT_HEARTBEAT \u003d 10000000000"}],"source_content_type":"text/x-python","patch_set":2,"id":"1aca2d91_e215469b","line":64,"updated":"2015-05-07 18:34:51.000000000","message":"Why would we want to set is_valid to None in the scope of the test vs in tearDown?","commit_id":"c0a8853fe7578075ad8fdbd4520982c5089032c0"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"2d02fde154f7bb81c8651d5ab0f025fb58a4fe44","unresolved":false,"context_lines":[{"line_number":61,"context_line":"        self.heartBeat.save()"},{"line_number":62,"context_line":"        self.assertEqual(1, utils.utcnow.call_count)"},{"line_number":63,"context_line":"        # make sure we remove is_valid from the Mock class"},{"line_number":64,"context_line":"        del dbmodels.DatabaseModelBase.is_valid"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"    def test_is_active(self):"},{"line_number":67,"context_line":"        models.AGENT_HEARTBEAT \u003d 10000000000"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a016987_d0e7d83e","line":64,"in_reply_to":"1aca2d91_e215469b","updated":"2015-05-14 17:29:25.000000000","message":"This was to fix the Mock class (since is_valid got added to Mock), but you\u0027re right - it should be done as patch.  (it\u0027s only used here so I\u0027ll take it out of setUp and tearDown ...)","commit_id":"c0a8853fe7578075ad8fdbd4520982c5089032c0"}],"trove/tests/unittests/guestagent/test_query.py":[{"author":{"_account_id":7806,"name":"Sushil Kumar","email":"skm.net@gmail.com","username":"sushilkm"},"change_message_id":"e5ea699d78113ef1c5c6c6a49037657e401c26e2","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"class QueryTestBase(trove_testtools.TestCase):"},{"line_number":20,"context_line":"    def setUp(self):"},{"line_number":21,"context_line":"        super(QueryTestBase, self).setUp()"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    def tearDown(self):"}],"source_content_type":"text/x-python","patch_set":7,"id":"3afb71cf_1965ad44","line":20,"updated":"2015-06-05 06:41:15.000000000","message":"This class could have been cleaned, I do not see any purpose of this class","commit_id":"4be4daa8e1b7edb204293bb6cd9aff5ae13241a6"}]}