p34709852
/
monkey
forked from p15670423/monkey
1
0
Fork 1
Code Issues Pull Requests Projects Releases Wiki Activity

UT: Define functions instead of lambda expressions in PyPubSubEventQueue tests since flake8 complains

This commit is contained in:
Shreya Malviya 2022-08-10 15:41:03 +05:30
parent edad0d8cf2
commit e905f6eddf
1 changed files with 11 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
monkey/tests/unit_tests/common/event_queue/test_pypubsub_event_queue.py
View File

@ -36,12 +36,18 @@ def subscriber_2_calls():
@pytest.fixture @pytest.fixture
def subscriber_1(subscriber_1_calls): def subscriber_1(subscriber_1_calls):
return lambda event, topic=pub.AUTO_TOPIC: subscriber_1_calls.append(topic.getName()) def fn(event, topic=pub.AUTO_TOPIC):
subscriber_1_calls.append(topic.getName())
return fn
@pytest.fixture @pytest.fixture
def subscriber_2(subscriber_2_calls): def subscriber_2(subscriber_2_calls):
return lambda event, topic=pub.AUTO_TOPIC: subscriber_2_calls.append(topic.getName()) def fn(event, topic=pub.AUTO_TOPIC):
subscriber_2_calls.append(topic.getName())
return fn
@pytest.mark.usefixtures("subscriber_1", "subscriber_2", "subscriber_1_calls", "subscriber_2_calls") @pytest.mark.usefixtures("subscriber_1", "subscriber_2", "subscriber_1_calls", "subscriber_2_calls")
@ -57,7 +63,9 @@ def test_topic_subscription(subscriber_1, subscriber_2, subscriber_1_calls, subs
def test_subscribe_all(): def test_subscribe_all():
subscriber_calls = [] subscriber_calls = []
subscriber = lambda topic=pub.AUTO_TOPIC: subscriber_calls.append(topic.getName())
def subscriber(topic=pub.AUTO_TOPIC):
subscriber_calls.append(topic.getName())
pypubsub_event_queue.subscribe_all(subscriber) pypubsub_event_queue.subscribe_all(subscriber)
pypubsub_event_queue.publish(EventType) pypubsub_event_queue.publish(EventType)