)]}'
{"trove/tests/unittests/guestagent/test_mongodb_manager.py":[{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"852b1778a3828a205a92ac8936de21fc2e04d63d","unresolved":false,"context_lines":[{"line_number":82,"context_line":"        mock_app.start_db \u003d MagicMock(return_value\u003dNone)"},{"line_number":83,"context_line":"        mock_app.clear_storage \u003d MagicMock(return_value\u003dNone)"},{"line_number":84,"context_line":"        os.path.exists \u003d MagicMock(return_value\u003dis_db_installed)"},{"line_number":85,"context_line":"        utils.execute_with_timeout \u003d MagicMock(return_value\u003dNone)"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"        # invocation"},{"line_number":88,"context_line":"        self.manager.prepare(context\u003dself.context, databases\u003dNone,"}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_40a4e6a2","line":85,"updated":"2014-08-06 17:24:45.000000000","message":"We should probably patch utils.execute_with_timeout back to the original after the unit tests have run.\n\nEither that or use the \"with patch.object(\u0027utils\u0027, \u0027execute_with_timeout\u0027) context manager.","commit_id":"10055b2449b23ce7d7a0ba5f16bfa0fef2988ce1"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"7fb9fc83e17f960272e5b99b9e9c02004505f54d","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"import testtools"},{"line_number":18,"context_line":"from mock import MagicMock"},{"line_number":19,"context_line":"from mock import patch"},{"line_number":20,"context_line":"from trove.common import utils"},{"line_number":21,"context_line":"from trove.common.context import TroveContext"},{"line_number":22,"context_line":"from trove.guestagent import volume"}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_8a49b468","line":19,"updated":"2014-08-06 21:35:05.000000000","message":"FYI, it\u0027s acceptable to do this:\n\n  from mock import patch, MagicMock","commit_id":"d2c938510890e6a83a9758bb2e277bfaba35df3b"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"497600cb400cae18d5aafc2d259c579ee52991fa","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"import testtools"},{"line_number":18,"context_line":"from mock import MagicMock"},{"line_number":19,"context_line":"from mock import patch"},{"line_number":20,"context_line":"from trove.common import utils"},{"line_number":21,"context_line":"from trove.common.context import TroveContext"},{"line_number":22,"context_line":"from trove.guestagent import volume"}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_e3f05968","line":19,"in_reply_to":"baada198_57a38643","updated":"2014-08-08 14:32:30.000000000","message":"So H301 is more strict than the \u0027official\u0027 pep8?  http://legacy.python.org/dev/peps/pep-0008/#imports","commit_id":"d2c938510890e6a83a9758bb2e277bfaba35df3b"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"7452abc035deaffc93893b53b2bbfb74559ca66d","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"import testtools"},{"line_number":18,"context_line":"from mock import MagicMock"},{"line_number":19,"context_line":"from mock import patch"},{"line_number":20,"context_line":"from trove.common import utils"},{"line_number":21,"context_line":"from trove.common.context import TroveContext"},{"line_number":22,"context_line":"from trove.guestagent import volume"}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_57a38643","line":19,"in_reply_to":"baada198_8a49b468","updated":"2014-08-08 11:15:17.000000000","message":"that would violate H301","commit_id":"d2c938510890e6a83a9758bb2e277bfaba35df3b"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"fc7c610d98497a394a4dea6eb1685a6831cd81d0","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"import testtools"},{"line_number":18,"context_line":"from mock import MagicMock"},{"line_number":19,"context_line":"from mock import patch"},{"line_number":20,"context_line":"from trove.common import utils"},{"line_number":21,"context_line":"from trove.common.context import TroveContext"},{"line_number":22,"context_line":"from trove.guestagent import volume"}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_59d7344f","line":19,"in_reply_to":"baada198_e3f05968","updated":"2014-08-08 15:01:13.000000000","message":"Yes, but I am told that the intent is to enable it sometime soon.","commit_id":"d2c938510890e6a83a9758bb2e277bfaba35df3b"}]}