)]}'
{"troveclient/tests/fakes.py":[{"author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"change_message_id":"be8ffe24fdbfebb2aa3c694908fed5dc26372309","unresolved":false,"context_lines":[{"line_number":587,"context_line":"        r \u003d {\u0027user\u0027: self.get_instances_1234_users()[2][\u0027users\u0027][0]}"},{"line_number":588,"context_line":"        return (200, {}, r)"},{"line_number":589,"context_line":""},{"line_number":590,"context_line":"    def get_instances_1234_users_harry_databases(self, **kw):"},{"line_number":591,"context_line":"        r \u003d {\u0027databases\u0027: [self.get_instances_1234_users()[2][\u0027users\u0027][2]"},{"line_number":592,"context_line":"                           [\"databases\"][0]]}"},{"line_number":593,"context_line":"        return (200, {}, r)"},{"line_number":594,"context_line":""},{"line_number":595,"context_line":"    def get_instance_1234_users_jacob_databases(self, **kw):"},{"line_number":596,"context_line":"        r \u003d {\u0027databases\u0027: [self.get_instances_1234_users()[2][\u0027users\u0027][0]"},{"line_number":597,"context_line":"                           [\"databases\"][0]]}"},{"line_number":598,"context_line":"        return (200, {}, r)"},{"line_number":599,"context_line":""},{"line_number":600,"context_line":"    def delete_instances_1234_users_jacob(self, **kw):"},{"line_number":601,"context_line":"        return (202, {}, None)"}],"source_content_type":"text/x-python","patch_set":8,"id":"5a74a57a_c6e36232","line":598,"range":{"start_line":590,"start_character":4,"end_line":598,"end_character":27},"updated":"2016-11-29 22:33:37.000000000","message":"No need for jacob test case.\nYou can see the test case for this on line 612 (the same name actually).\n\nNo need for harry test case.\nTests pass without the user Harry in the dictionary on line 584.","commit_id":"7be1bb96d58f6527bf3a700b55a31ffe6453c0a2"},{"author":{"_account_id":22694,"name":"jian.song","email":"jiansong19941206@gmail.com","username":"jiansong"},"change_message_id":"b0246e615e40a1cec83b35e4f6116f82b1b2f6b8","unresolved":false,"context_lines":[{"line_number":609,"context_line":"                            optional\u003d[\u0027databases\u0027])"},{"line_number":610,"context_line":"        return (202, {}, self.get_instances_1234_users()[2][\u0027users\u0027][0])"},{"line_number":611,"context_line":""},{"line_number":612,"context_line":"    def get_instances_1234_users_jacob_databases(self, **kw):"},{"line_number":613,"context_line":"        r \u003d {\u0027databases\u0027: ["},{"line_number":614,"context_line":"            self.get_instances_1234_databases()[2][\u0027databases\u0027][0],"},{"line_number":615,"context_line":"            self.get_instances_1234_databases()[2][\u0027databases\u0027][1]]}"},{"line_number":616,"context_line":"        return (200, {}, r)"},{"line_number":617,"context_line":""},{"line_number":618,"context_line":"    def put_instances_1234_users_jacob(self, **kw):"},{"line_number":619,"context_line":"        return (202, {}, None)"}],"source_content_type":"text/x-python","patch_set":8,"id":"5a74a57a_38d50fb9","line":616,"range":{"start_line":612,"start_character":3,"end_line":616,"end_character":27},"updated":"2016-11-30 01:54:41.000000000","message":"I think this name is in the test jacob but his test method and get_instances_1234_users feeling and no relationship. . . I think the name of this method is wrong.","commit_id":"7be1bb96d58f6527bf3a700b55a31ffe6453c0a2"}],"troveclient/tests/test_v1_shell.py":[{"author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"change_message_id":"be8ffe24fdbfebb2aa3c694908fed5dc26372309","unresolved":false,"context_lines":[{"line_number":828,"context_line":"        self.run_command(\u0027user-show-access 1234 jacob\u0027)"},{"line_number":829,"context_line":"        self.assert_called(\u0027GET\u0027, \u0027/instances/1234/users/jacob/databases\u0027)"},{"line_number":830,"context_line":""},{"line_number":831,"context_line":"    def test_user_show_access_harry(self):"},{"line_number":832,"context_line":"        self.run_command(\u0027user-show-access 1234 harry\u0027)"},{"line_number":833,"context_line":"        self.assert_called(\u0027GET\u0027, \u0027/instances/1234/users/harry/databases\u0027)"},{"line_number":834,"context_line":""},{"line_number":835,"context_line":"    def test_user_update_host(self):"},{"line_number":836,"context_line":"        cmd \u003d \u0027user-update-attributes 1234 jacob --new_host 10.0.0.1\u0027"}],"source_content_type":"text/x-python","patch_set":8,"id":"5a74a57a_c6ee02b4","line":833,"range":{"start_line":831,"start_character":4,"end_line":833,"end_character":74},"updated":"2016-11-29 22:33:37.000000000","message":"No need for this test because the code is covered by the function above it.","commit_id":"7be1bb96d58f6527bf3a700b55a31ffe6453c0a2"}]}