minor modifications

Author: pcrespov

packages/pytest-simcore/src/pytest_simcore/helpers/playwright.py

@@ -196,7 +196,6 @@ def __call__(self, message: str) -> None:
 class SocketIONodeProgressCompleteWaiter:
     node_id: str
     logger: logging.Logger
-    _last_call_return: bool = False
     _current_progress: dict[NodeProgressType, float] = field(
         default_factory=defaultdict
     )
@@ -224,24 +223,24 @@ def __call__(self, message: str) -> bool:
                         self._current_progress[
                             node_progress_event.progress_type
                         ] = new_progress
+
                         self.logger.info(
-                            "current startup progress: %s",
+                            "Current startup progress [expected number of states=%d]: %s",
+                            NodeProgressType.required_types_for_started_service(),
                             f"{json.dumps({k:round(v,1) for k,v in self._current_progress.items()})}",
                         )
 
-                self._last_call_return = all(
-                    progress_type in self._current_progress
-                    for progress_type in NodeProgressType.required_types_for_started_service()
-                ) and all(
+                return self.has_progress_on_all_expected_node_states() and all(
                     round(progress, 1) == 1.0
                     for progress in self._current_progress.values()
                 )
-                return self._last_call_return
-        self._last_call_return = False
         return False
 
-    def get_last_call_return(self):
-        return self._last_call_return
+    def has_progress_on_all_expected_node_states(self):
+        return all(
+            progress_type in self._current_progress
+            for progress_type in NodeProgressType.required_types_for_started_service()
+        )
 
     def get_current_progress(self):
         return self._current_progress.values()
@@ -346,9 +345,9 @@ def expected_service_running(
                 yield service_running
 
         except PlaywrightTimeoutError:
-            if waiter.get_last_call_return() is False:
+            if waiter.has_progress_on_all_expected_node_states():
                 ctx.logger.warning(
-                    "⚠️  Progress bar didn't receive 100 percent: %s ⚠️",  # https://github.com/ITISFoundation/osparc-simcore/issues/6449
+                    "⚠️ Progress bar didn't receive 100 percent but all states are there: %s ⚠️",  # https://github.com/ITISFoundation/osparc-simcore/issues/6449
                     waiter.get_current_progress(),
                 )
             else: