)]}'
{"nova/tests/test_db_api.py":[{"author":{"_account_id":5511,"name":"Nikola Dipanov","email":"ndipanov@redhat.com","username":"ndipanov"},"change_message_id":"91b21fda291e9ad2d24d7bacc3ee6cc1512a3265","unresolved":false,"context_lines":[{"line_number":1176,"context_line":"        _compare(bw_usages[2], expected_bw_usages[2])"},{"line_number":1177,"context_line":"        timeutils.clear_time_override()"},{"line_number":1178,"context_line":""},{"line_number":1179,"context_line":"    def _test_decorator_wraps_helper(self, decorator):"},{"line_number":1180,"context_line":"        def test_func():"},{"line_number":1181,"context_line":"            \"\"\"Test docstring.\"\"\""},{"line_number":1182,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPH%2F%2F684%3D","line":1179,"updated":"2013-05-20 08:53:02.000000000","message":"These tests are actually testing the functools.wraps method of the python standard library and as such seem not very useful to me.\n\nThere is some value (I guess) in leaving it to make sure functools.wraps is used consistently, but then this should not be in the db_api test class but on the base TestCase class as it will be used in tests of other modules that have decorators.\n\nIf it were up to me I\u0027d vote for not testing this at all as useless tests are a bad thing, but I appreciate there may be people who will dissagree.","commit_id":"19aaaf5ee166ae0276a4bc00d8bfba1fb3c7fc57"}]}