)]}'
{"troveclient/tests/test_root.py":[{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"63d6b3db1f1066f7c6b4133ed235dfc86843b37b","unresolved":false,"context_lines":[{"line_number":55,"context_line":"        self._resp.status_code \u003d 400"},{"line_number":56,"context_line":"        self.assertRaises(Exception, self.root.delete, 1234)"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    def test_root_enabled(self):"},{"line_number":59,"context_line":"        self.root.api.client.get \u003d self._get_mock_method()"},{"line_number":60,"context_line":"        self._resp.status_code \u003d 200"},{"line_number":61,"context_line":"        self._resp.body \u003d {\u0027rootEnabled\u0027: True}"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a8ec9b2_d554f022","line":58,"updated":"2016-09-19 16:58:20.000000000","message":"I don\u0027t see the value of the test.\n\nis_root_enabled() calls _is_root_enabled(). This test is effectively verifying whether the Mock() routines are working or not.\n\nWhat good is that?\n\nOn the other hand, if you verified that _is_root_enabled() the new mock was called with the right parameters or something, it may be useful.\n\nas such, I don\u0027t know that it does anything useful.","commit_id":"913a615e2841f5680d1ba47b578bb869fc20f752"}]}