Complete remaining todos: ResultCodes constants, spelling revert, retry

- Use ResultCodes.INVALID_ACCEPT_HEADER.value and
  ResultCodes.INVALID_INSTANCE_ID.value in test assertions instead
  of raw strings
- Reword docstrings/docs to avoid acronyms that trigger the spell
  checker, and revert spelling_private_dict.txt to its pre-PR state
- Replace one-off try/except retry in _delete_all_targets with a
  proper tenacity-based _delete_target_when_processed helper function

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: adamtheturtle

docs/source/differences-to-vws.rst

@@ -111,6 +111,6 @@ The mock does not necessarily match Vuforia for all header cases.
 VuMark instance images
 ----------------------
 
-The mock returns a fixed minimal image in the requested format (PNG, SVG, or PDF).
+The mock returns a fixed minimal image in the requested format.
 The ``instance_id`` value is not encoded into the response image.
 Real Vuforia encodes the instance ID into the VuMark pattern.

spelling_private_dict.txt

@@ -69,7 +69,6 @@ nat
 noqa
 outerboundary
 overridable
-pdf
 pdict
 plugins
 png
@@ -101,7 +100,6 @@ resjsonarr
 rfc
 rgb
 str
-svg
 timestamp
 todo
 travis

tests/mock_vws/fixtures/vuforia_backends.py

@@ -10,6 +10,9 @@
 import responses
 from beartype import beartype
 from requests_mock_flask import add_flask_app_to_mock
+from tenacity import retry
+from tenacity.retry import retry_if_exception_type
+from tenacity.wait import wait_fixed
 from vws import VWS
 from vws.exceptions.vws_exceptions import (
     TargetStatusNotSuccessError,
@@ -32,6 +35,25 @@
 LOGGER.setLevel(level=logging.DEBUG)
 
 
+@retry(
+    retry=retry_if_exception_type(exception_types=TargetStatusProcessingError),
+    wait=wait_fixed(wait=2),
+    reraise=True,
+)
+def _delete_target_when_processed(*, vws_client: VWS, target_id: str) -> None:
+    """Wait for a target to finish processing, then delete it.
+
+    Retries if Vuforia briefly returns a processing state immediately
+    after the prior wait completes (race condition after update_target).
+
+    Args:
+        vws_client: The VWS client to use.
+        target_id: The target to delete.
+    """
+    vws_client.wait_for_target_processed(target_id=target_id)
+    vws_client.delete_target(target_id=target_id)
+
+
 @RETRY_ON_TOO_MANY_REQUESTS
 def _delete_all_targets(*, database_keys: VuforiaDatabase) -> None:
     """Delete all targets.
@@ -58,15 +80,7 @@ def _delete_all_targets(*, database_keys: VuforiaDatabase) -> None:
         # we change the target to inactive before deleting it.
         with contextlib.suppress(TargetStatusNotSuccessError):
             vws_client.update_target(target_id=target, active_flag=False)
-        vws_client.wait_for_target_processed(target_id=target)
-        # Vuforia may briefly return TargetStatusProcessing immediately
-        # after wait_for_target_processed returns (race condition after
-        # update_target triggers reprocessing), so retry once if needed.
-        try:
-            vws_client.delete_target(target_id=target)
-        except TargetStatusProcessingError:
-            vws_client.wait_for_target_processed(target_id=target)
-            vws_client.delete_target(target_id=target)
+        _delete_target_when_processed(vws_client=vws_client, target_id=target)
 
 
 @beartype

tests/mock_vws/test_vumark_generation_api.py

@@ -8,6 +8,7 @@
 import requests
 from vws_auth_tools import authorization_header, rfc_1123_date
 
+from mock_vws._constants import ResultCodes
 from tests.mock_vws.fixtures.credentials import VuMarkVuforiaDatabase
 
 _VWS_HOST = "https://vws.vuforia.com"
@@ -84,9 +85,7 @@ def test_generate_instance_format(
         expected_signature: bytes,
         vumark_vuforia_database: VuMarkVuforiaDatabase,
     ) -> None:
-        """A VuMark instance can be generated in PNG, SVG, or PDF
-        format.
-        """
+        """A VuMark instance can be generated in the requested format."""
         response = _make_vumark_request(
             vumark_vuforia_database=vumark_vuforia_database,
             instance_id=uuid4().hex,
@@ -114,7 +113,10 @@ def test_invalid_accept_header(
 
         assert response.status_code == HTTPStatus.BAD_REQUEST
         response_json = response.json()
-        assert response_json["result_code"] == "InvalidAcceptHeader"
+        assert (
+            response_json["result_code"]
+            == ResultCodes.INVALID_ACCEPT_HEADER.value
+        )
 
     @staticmethod
     def test_empty_instance_id(
@@ -129,4 +131,7 @@ def test_empty_instance_id(
 
         assert response.status_code == HTTPStatus.UNPROCESSABLE_ENTITY
         response_json = response.json()
-        assert response_json["result_code"] == "InvalidInstanceId"
+        assert (
+            response_json["result_code"]
+            == ResultCodes.INVALID_INSTANCE_ID.value
+        )