Update log test

Author: Pijukatel

src/apify/_actor.py

@@ -288,6 +288,7 @@ async def init(self) -> None:
         This method should be called immediately before performing any additional Actor actions, and it should be
         called only once.
         """
+        self.log.info('Initializing Actor...')
         if self._configuration:
             # Set explicitly the configuration in the service locator
             service_locator.set_configuration(self.configuration)
@@ -315,7 +316,6 @@ async def init(self) -> None:
         if self._configure_logging:
             _configure_logging()
 
-        self.log.info('Initializing Actor...')
         self.log.info('System info', extra=get_system_info())
 
         await self.event_manager.__aenter__()

tests/unit/actor/test_actor_log.py

@@ -37,7 +37,7 @@ async def test_actor_logs_messages_correctly(caplog: pytest.LogCaptureFixture) -
             raise RuntimeError('Dummy RuntimeError')
 
     # Updated expected number of log records (an extra record is now captured)
-    assert len(caplog.records) == 14
+    assert len(caplog.records) == 15
 
     # Record 0: Extra Pytest context log
     assert caplog.records[0].levelno == logging.DEBUG
@@ -51,54 +51,58 @@ async def test_actor_logs_messages_correctly(caplog: pytest.LogCaptureFixture) -
     assert caplog.records[2].levelno == logging.INFO
     assert caplog.records[2].message == 'Initializing Actor...'
 
+    # Record 2: Initializing Actor...
+    assert caplog.records[3].levelno == logging.DEBUG
+    assert caplog.records[3].message.startswith('Storage client set to')
+
     # Record 3: System info
-    assert caplog.records[3].levelno == logging.INFO
-    assert caplog.records[3].message == 'System info'
+    assert caplog.records[4].levelno == logging.INFO
+    assert caplog.records[4].message == 'System info'
 
     # Record 4: Event manager initialized
-    assert caplog.records[4].levelno == logging.DEBUG
-    assert caplog.records[4].message == 'Event manager initialized'
+    assert caplog.records[5].levelno == logging.DEBUG
+    assert caplog.records[5].message == 'Event manager initialized'
 
     # Record 5: Charging manager initialized
-    assert caplog.records[5].levelno == logging.DEBUG
-    assert caplog.records[5].message == 'Charging manager initialized'
+    assert caplog.records[6].levelno == logging.DEBUG
+    assert caplog.records[6].message == 'Charging manager initialized'
 
     # Record 6: Debug message
-    assert caplog.records[6].levelno == logging.DEBUG
-    assert caplog.records[6].message == 'Debug message'
+    assert caplog.records[7].levelno == logging.DEBUG
+    assert caplog.records[7].message == 'Debug message'
 
     # Record 7: Info message
-    assert caplog.records[7].levelno == logging.INFO
-    assert caplog.records[7].message == 'Info message'
+    assert caplog.records[8].levelno == logging.INFO
+    assert caplog.records[8].message == 'Info message'
 
     # Record 8: Warning message
-    assert caplog.records[8].levelno == logging.WARNING
-    assert caplog.records[8].message == 'Warning message'
+    assert caplog.records[9].levelno == logging.WARNING
+    assert caplog.records[9].message == 'Warning message'
 
     # Record 9: Error message
-    assert caplog.records[9].levelno == logging.ERROR
-    assert caplog.records[9].message == 'Error message'
+    assert caplog.records[10].levelno == logging.ERROR
+    assert caplog.records[10].message == 'Error message'
 
     # Record 10: Exception message with traceback (ValueError)
-    assert caplog.records[10].levelno == logging.ERROR
-    assert caplog.records[10].message == 'Exception message'
-    assert caplog.records[10].exc_info is not None
-    assert caplog.records[10].exc_info[0] is ValueError
-    assert isinstance(caplog.records[10].exc_info[1], ValueError)
-    assert str(caplog.records[10].exc_info[1]) == 'Dummy ValueError'
+    assert caplog.records[11].levelno == logging.ERROR
+    assert caplog.records[11].message == 'Exception message'
+    assert caplog.records[11].exc_info is not None
+    assert caplog.records[11].exc_info[0] is ValueError
+    assert isinstance(caplog.records[11].exc_info[1], ValueError)
+    assert str(caplog.records[11].exc_info[1]) == 'Dummy ValueError'
 
     # Record 11: Multiline log message
-    assert caplog.records[11].levelno == logging.INFO
-    assert caplog.records[11].message == 'Multi\nline\nlog\nmessage'
+    assert caplog.records[12].levelno == logging.INFO
+    assert caplog.records[12].message == 'Multi\nline\nlog\nmessage'
 
     # Record 12: Actor failed with an exception (RuntimeError)
-    assert caplog.records[12].levelno == logging.ERROR
-    assert caplog.records[12].message == 'Actor failed with an exception'
-    assert caplog.records[12].exc_info is not None
-    assert caplog.records[12].exc_info[0] is RuntimeError
-    assert isinstance(caplog.records[12].exc_info[1], RuntimeError)
-    assert str(caplog.records[12].exc_info[1]) == 'Dummy RuntimeError'
+    assert caplog.records[13].levelno == logging.ERROR
+    assert caplog.records[13].message == 'Actor failed with an exception'
+    assert caplog.records[13].exc_info is not None
+    assert caplog.records[13].exc_info[0] is RuntimeError
+    assert isinstance(caplog.records[13].exc_info[1], RuntimeError)
+    assert str(caplog.records[13].exc_info[1]) == 'Dummy RuntimeError'
 
     # Record 13: Exiting Actor
-    assert caplog.records[13].levelno == logging.INFO
-    assert caplog.records[13].message == 'Exiting Actor'
+    assert caplog.records[14].levelno == logging.INFO
+    assert caplog.records[14].message == 'Exiting Actor'