)]}'
{"trove/tests/unittests/guestagent/test_dbaas.py":[{"author":{"_account_id":8214,"name":"Auston McReynolds","email":"amcreynolds@ebaysf.com","username":"amcreynolds"},"change_message_id":"e528e9c49485b2528836d19b511aa0c6ac121df4","unresolved":false,"context_lines":[{"line_number":963,"context_line":"    def test_end_install_or_restart(self):"},{"line_number":964,"context_line":""},{"line_number":965,"context_line":"        self.baseDbStatus \u003d BaseDbStatus()"},{"line_number":966,"context_line":"        self.baseDbStatus._get_actual_db_status \u003d \\"},{"line_number":967,"context_line":"            Mock(return_value\u003drd_instance.ServiceStatuses.SHUTDOWN)"},{"line_number":968,"context_line":""},{"line_number":969,"context_line":"        self.baseDbStatus.end_install_or_restart()"}],"source_content_type":"text/x-python","patch_set":9,"id":"AAAAS3%2F%2FO7I%3D","line":966,"updated":"2013-10-01 20:28:40.000000000","message":"Backslash continuations are generally frowned upon. Could you remove these (there are a couple).","commit_id":"8990101d80f65c633adde06c5f2a80eae8482957"},{"author":{"_account_id":8491,"name":"Illia Khudoshyn","email":"ikhudoshyn@mirantis.com","username":"ikhudoshyn"},"change_message_id":"0e210c5dc5be7844c084172a20c63cd066b4de2c","unresolved":false,"context_lines":[{"line_number":963,"context_line":"    def test_end_install_or_restart(self):"},{"line_number":964,"context_line":""},{"line_number":965,"context_line":"        self.baseDbStatus \u003d BaseDbStatus()"},{"line_number":966,"context_line":"        self.baseDbStatus._get_actual_db_status \u003d \\"},{"line_number":967,"context_line":"            Mock(return_value\u003drd_instance.ServiceStatuses.SHUTDOWN)"},{"line_number":968,"context_line":""},{"line_number":969,"context_line":"        self.baseDbStatus.end_install_or_restart()"}],"source_content_type":"text/x-python","patch_set":9,"id":"AAAAS3%2F%2FNks%3D","line":966,"in_reply_to":"AAAAS3%2F%2FO7I%3D","updated":"2013-10-02 08:03:09.000000000","message":"Done","commit_id":"8990101d80f65c633adde06c5f2a80eae8482957"},{"author":{"_account_id":8214,"name":"Auston McReynolds","email":"amcreynolds@ebaysf.com","username":"amcreynolds"},"change_message_id":"e528e9c49485b2528836d19b511aa0c6ac121df4","unresolved":false,"context_lines":[{"line_number":1041,"context_line":"                         (rd_instance.ServiceStatuses.SHUTDOWN, 10))"},{"line_number":1042,"context_line":""},{"line_number":1043,"context_line":""},{"line_number":1044,"context_line":"class MySqlAppStatusTest(testtools.TestCase):"},{"line_number":1045,"context_line":""},{"line_number":1046,"context_line":"    def setUp(self):"},{"line_number":1047,"context_line":"        super(MySqlAppStatusTest, self).setUp()"}],"source_content_type":"text/x-python","patch_set":9,"id":"AAAAS3%2F%2FO7A%3D","line":1044,"updated":"2013-10-01 20:28:40.000000000","message":"Given that quite a few of the methods moved to the base class, some of these MySqlAppStatus() tests are now rendered redundant. You could consider removing tests that don\u0027t exercise an overloaded method?","commit_id":"8990101d80f65c633adde06c5f2a80eae8482957"},{"author":{"_account_id":8491,"name":"Illia Khudoshyn","email":"ikhudoshyn@mirantis.com","username":"ikhudoshyn"},"change_message_id":"0e210c5dc5be7844c084172a20c63cd066b4de2c","unresolved":false,"context_lines":[{"line_number":1041,"context_line":"                         (rd_instance.ServiceStatuses.SHUTDOWN, 10))"},{"line_number":1042,"context_line":""},{"line_number":1043,"context_line":""},{"line_number":1044,"context_line":"class MySqlAppStatusTest(testtools.TestCase):"},{"line_number":1045,"context_line":""},{"line_number":1046,"context_line":"    def setUp(self):"},{"line_number":1047,"context_line":"        super(MySqlAppStatusTest, self).setUp()"}],"source_content_type":"text/x-python","patch_set":9,"id":"AAAAS3%2F%2FNkg%3D","line":1044,"in_reply_to":"AAAAS3%2F%2FO7A%3D","updated":"2013-10-02 08:03:09.000000000","message":"Done","commit_id":"8990101d80f65c633adde06c5f2a80eae8482957"},{"author":{"_account_id":739,"name":"Michael Basnight","email":"mbasnight@gmail.com","username":"hubcap"},"change_message_id":"1919170e509847f2a22f84b33115b187a85dd460","unresolved":false,"context_lines":[{"line_number":942,"context_line":"        InstanceServiceStatus.find_by(instance_id\u003dself.FAKE_ID).delete()"},{"line_number":943,"context_line":"        dbaas.CONF.guest_id \u003d None"},{"line_number":944,"context_line":""},{"line_number":945,"context_line":"    def test_being_install(self):"},{"line_number":946,"context_line":""},{"line_number":947,"context_line":"        self.baseDbStatus \u003d BaseDbStatus()"},{"line_number":948,"context_line":""}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAAS3%2F%2FMG8%3D","line":945,"updated":"2013-10-02 16:56:49.000000000","message":"begin","commit_id":"6924bcfe1585f7d3f787f1868d0194f057352481"},{"author":{"_account_id":8491,"name":"Illia Khudoshyn","email":"ikhudoshyn@mirantis.com","username":"ikhudoshyn"},"change_message_id":"7c7b5dcb3c51846a2c8db786598dbee9cafbac20","unresolved":false,"context_lines":[{"line_number":942,"context_line":"        InstanceServiceStatus.find_by(instance_id\u003dself.FAKE_ID).delete()"},{"line_number":943,"context_line":"        dbaas.CONF.guest_id \u003d None"},{"line_number":944,"context_line":""},{"line_number":945,"context_line":"    def test_being_install(self):"},{"line_number":946,"context_line":""},{"line_number":947,"context_line":"        self.baseDbStatus \u003d BaseDbStatus()"},{"line_number":948,"context_line":""}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAAS3%2F%2FL78%3D","line":945,"in_reply_to":"AAAAS3%2F%2FMG8%3D","updated":"2013-10-02 17:52:13.000000000","message":"Done","commit_id":"6924bcfe1585f7d3f787f1868d0194f057352481"}]}