From b5d7c1eaacf74a00af518059653abb9455d446dc Mon Sep 17 00:00:00 2001 From: Samuel Merritt Date: Mon, 2 Jun 2014 10:14:22 -0600 Subject: [PATCH] Remove unused variable The tempurl functional tests would fail if you put tempurl.methods in the proxy server's disallowed_sections. The test assumed that the presence of the key 'tempurl' implied the presence of the subkey 'methods', but since 9cbf8a3 landed, operators can remove arbitrary things from /info. Normally I'd just change x['methods'] to x.get('methods', []), but it turns out this was setting up a class variable that nobody ever looked out again, so removing the code also works. Change-Id: Ie899d146bc6fff81a5fae77815897244e8ec6bec --- test/functional/tests.py | 1 - 1 file changed, 1 deletion(-) diff --git a/test/functional/tests.py b/test/functional/tests.py index 38381e24cf..250efe2717 100644 --- a/test/functional/tests.py +++ b/test/functional/tests.py @@ -2126,7 +2126,6 @@ class TestTempurlEnv(object): cls.tempurl_enabled = 'tempurl' in cluster_info if not cls.tempurl_enabled: return - cls.tempurl_methods = cluster_info['tempurl']['methods'] cls.tempurl_key = Utils.create_name() cls.tempurl_key2 = Utils.create_name()