)]}'
{"tempest/api/compute/admin/test_simple_tenant_usage.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"1396bddfa8a5507b4d0c4b0d41b244c81a77a3f5","unresolved":false,"context_lines":[{"line_number":59,"context_line":"                return True"},{"line_number":60,"context_line":"            except e.InvalidHTTPResponseBody:"},{"line_number":61,"context_line":"                return False"},{"line_number":62,"context_line":"        result \u003d test.call_until_true(is_valid, duration, 1)"},{"line_number":63,"context_line":"        self.assertEqual(result, True,"},{"line_number":64,"context_line":"                         \"%s not return valid response in %s secs\" % ("},{"line_number":65,"context_line":"                             getattr(func, \u0027__name__\u0027), str(duration)))"}],"source_content_type":"text/x-python","patch_set":1,"id":"9abb7d3a_09ef1888","line":62,"updated":"2016-05-30 08:20:11.000000000","message":"I suggest:\n\n   self.assertTrue(test.call_until_true(is_valid, duration, 1))","commit_id":"9cafaa200224c893fa9bb7fdf4bc29874a83b426"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"1396bddfa8a5507b4d0c4b0d41b244c81a77a3f5","unresolved":false,"context_lines":[{"line_number":62,"context_line":"        result \u003d test.call_until_true(is_valid, duration, 1)"},{"line_number":63,"context_line":"        self.assertEqual(result, True,"},{"line_number":64,"context_line":"                         \"%s not return valid response in %s secs\" % ("},{"line_number":65,"context_line":"                             getattr(func, \u0027__name__\u0027), str(duration)))"},{"line_number":66,"context_line":"        return self.resp"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"    @test.idempotent_id(\u0027062c8ae9-9912-4249-8b51-e38d664e926e\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9abb7d3a_a9a6cc50","line":65,"updated":"2016-05-30 08:20:11.000000000","message":"There\u0027s no need to cast to str(). %s already does this.\n\nNo need for gettattr(func, \u0027__name__\u0027), func.__name__ works and is more direct.","commit_id":"9cafaa200224c893fa9bb7fdf4bc29874a83b426"}]}