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

@@ -24,9 +24,7 @@ def poll(self) -> None:
         now = time.time()
         previous_status = self.executor._poller_mutable_status
 
-        if now >= self._executor._last_poll_time + self._executor.status_polling_interval:
-            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