From c70ebbaf40f0285a47e45682bb9f782b469dd415 Mon Sep 17 00:00:00 2001 From: Jan Buchar Date: Fri, 8 Nov 2024 13:34:02 +0100 Subject: [PATCH 1/2] Do not expect event handlers to be executed in any order --- tests/unit/test_platform_event_manager.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/unit/test_platform_event_manager.py b/tests/unit/test_platform_event_manager.py index 608da886..abf671bb 100644 --- a/tests/unit/test_platform_event_manager.py +++ b/tests/unit/test_platform_event_manager.py @@ -75,24 +75,24 @@ def event_handler(data: Any) -> None: dummy_persist_state = Mock() - # Test that they all work, and that they're called in order + # Test that they all work event_manager.emit(event=Event.PERSIST_STATE, event_data=dummy_persist_state) await asyncio.sleep(0.1) - assert event_calls[Event.PERSIST_STATE] == [ + assert set(event_calls[Event.PERSIST_STATE]) == { (1, dummy_persist_state), (2, dummy_persist_state), (3, dummy_persist_state), - ] + } event_calls[Event.PERSIST_STATE].clear() # Test that if you remove one, the others stay event_manager.off(event=Event.PERSIST_STATE, listener=handler_persist_state_3) event_manager.emit(event=Event.PERSIST_STATE, event_data=dummy_persist_state) await asyncio.sleep(0.1) - assert event_calls[Event.PERSIST_STATE] == [ + assert set(event_calls[Event.PERSIST_STATE]) == { (1, dummy_persist_state), (2, dummy_persist_state), - ] + } event_calls[Event.PERSIST_STATE].clear() # Test that removing all in bulk works From c288eabd0ed287010813c9c22bbcbf55bb171ade Mon Sep 17 00:00:00 2001 From: Jan Buchar Date: Fri, 8 Nov 2024 13:41:42 +0100 Subject: [PATCH 2/2] Fix noqa directive --- src/apify/log.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/apify/log.py b/src/apify/log.py index e1f29400..698474f6 100644 --- a/src/apify/log.py +++ b/src/apify/log.py @@ -17,7 +17,7 @@ @ignore_docs -class ActorLogFormatter(CrawleeLogFormatter): # noqa: D101 Inherited from parent class +class ActorLogFormatter(CrawleeLogFormatter): # noqa: D101 (Inherited from parent class) pass