move cache update code into executor

this should not change behaviour, as it is a code move

Author: benclifford

parsl/executors/status_handling.py

@@ -241,3 +241,8 @@ def workers_per_node(self) -> Union[int, float]:
 
     def _should_poll(self, now: float) -> bool:
         return now >= self._last_poll_time + self.status_polling_interval
+
+    def _refresh_poll_mutable_status_if_time(self, now):
+        if self._should_poll(now):
+            self._poller_mutable_status = self.status()
+            self._last_poll_time = now

parsl/jobs/job_status_poller.py

@@ -37,9 +37,7 @@ def __init__(self, executor: BlockProviderExecutor, dfk: Optional["parsl.dataflo
     def poll(self, now: float) -> None:
         previous_status = self.executor._poller_mutable_status
 
-        if self.executor._should_poll(now):
-            self.executor._poller_mutable_status = self._executor.status()
-            self.executor._last_poll_time = now
+        self.executor._refresh_poll_mutable_status_if_time(now)
 
         if previous_status != self.executor._poller_mutable_status:
             # short circuit the case where the two objects are identical so