From 7d08b0735fdffe4066e7fa3d303da61aed00a7e8 Mon Sep 17 00:00:00 2001 From: Tim Burke Date: Tue, 21 Mar 2017 08:46:13 -0700 Subject: [PATCH] Clean up unused function Also, use `==` instead of `is` for string comparison. Change-Id: I12040d56afc239a7b7be1ae37b27cce708bf6f10 Related-Change: I1ccb2665b6cd2887659e548e55a26aa00de879e3 --- test/unit/common/middleware/test_subrequest_logging.py | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/test/unit/common/middleware/test_subrequest_logging.py b/test/unit/common/middleware/test_subrequest_logging.py index 1754215082..8cd002c23c 100644 --- a/test/unit/common/middleware/test_subrequest_logging.py +++ b/test/unit/common/middleware/test_subrequest_logging.py @@ -36,7 +36,7 @@ class FakeFilter(object): def __call__(self, env, start_response): path = SUB_PUT_POST_PATH - if env['REQUEST_METHOD'] is 'GET': + if env['REQUEST_METHOD'] == 'GET': path = SUB_GET_PATH # Make a subrequest that will be logged @@ -52,9 +52,6 @@ class FakeFilter(object): resp = sub_req.get_response(self.app) close_if_possible(resp.app_iter) - def _start_response(status, headers, exc_info=None): - return start_response(status, headers, exc_info) - return self.app(env, start_response)