From 1561fbb3cc40a3978108563cdf05bff3ccbdc37b Mon Sep 17 00:00:00 2001 From: Javeme Date: Mon, 22 Feb 2016 14:23:02 +0800 Subject: [PATCH] Py3: Replace filter()/map() if a list is needed -- "cls.scenarios = filter(f, cls.scenarios)" It's better to set a list to cls.scenarios but filter()/map() returns an iterator on Python 3. This patch we replaced the filter()/map() with a list comprehension in test_server.py. Change-Id: I16640e564d41c6a1c29e229049467ae788bb1437 --- oslo_messaging/tests/rpc/test_server.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/oslo_messaging/tests/rpc/test_server.py b/oslo_messaging/tests/rpc/test_server.py index 3dc158f45..5b747a561 100644 --- a/oslo_messaging/tests/rpc/test_server.py +++ b/oslo_messaging/tests/rpc/test_server.py @@ -455,9 +455,9 @@ class TestMultipleServers(test_utils.BaseTestCase, ServerSetupMixin): return scenario for f in [filter_fanout_call, filter_same_topic_and_server]: - cls.scenarios = filter(f, cls.scenarios) + cls.scenarios = [i for i in cls.scenarios if f(i)] for m in [fanout_to_servers, single_topic_multi_endpoints]: - cls.scenarios = map(m, cls.scenarios) + cls.scenarios = [m(i) for i in cls.scenarios] def __init__(self, *args): super(TestMultipleServers, self).__init__(*args)