Progress towards moving helper to external

Author: adamtheturtle

src/vws/_wait_for_target_processed.py

@@ -0,0 +1,33 @@
+"""
+This is tooling for waiting for a target to be processed.
+
+It lives here rather than in ``vws.py`` to support Windows.
+"""
+
+from typing import Optional
+
+from wrapt_timeout_decorator import timeout
+
+from vws import VWS
+from vws.exceptions import TargetProcessingTimeout
+
+
+def foobar(
+    vws_client: VWS,
+    timeout_seconds: Optional[float],
+    seconds_between_requests: float,
+    target_id: str,
+):
+
+    @timeout(
+        dec_timeout=timeout_seconds,
+        timeout_exception=TargetProcessingTimeout,
+        use_signals=False,
+    )
+    def decorated() -> None:
+        vws_client._wait_for_target_processed(
+            target_id=target_id,
+            seconds_between_requests=seconds_between_requests,
+        )
+
+    decorated()

src/vws/vws.py

@@ -12,11 +12,9 @@
 
 import requests
 from requests import Response
-from wrapt_timeout_decorator import timeout
 from vws_auth_tools import authorization_header, rfc_1123_date
 
 from vws._result_codes import raise_for_result_code
-from vws.exceptions import TargetProcessingTimeout
 from vws.reports import (
     DatabaseSummaryReport,
     TargetRecord,
@@ -320,19 +318,8 @@ def wait_for_target_processed(
             ~vws.exceptions.RequestTimeTooSkewed: There is an error with the
                 time sent to Vuforia.
         """
-
-        @timeout(
-            dec_timeout=timeout_seconds,
-            timeout_exception=TargetProcessingTimeout,
-            use_signals=False,
-        )
-        def decorated() -> None:
-            self._wait_for_target_processed(
-                target_id=target_id,
-                seconds_between_requests=seconds_between_requests,
-            )
-
-        decorated()
+        from _wait_for_target_processed import foobar
+        foobar(vws_client=self)
 
     def list_targets(self) -> List[str]:
         """