Remove unnecessary query parameters and tests for them

Author: adamtheturtle

pyproject.toml

@@ -277,7 +277,6 @@ dependencies = [
     "requests-mock",
     "tzdata; sys_platform == 'win32'",
     "vws-auth-tools",
-    "werkzeug",
 ]
 
 [project.optional-dependencies]
@@ -319,7 +318,10 @@ dev = [
     "types-PyYAML==6.0.12.9",
     "types-Pillow==9.5.0.4",
     "types-requests==2.30.0.0",
-    "urllib3==2.0.2",
+    # We need urllib3 < 2.0.0 for pdm to work.
+    # After we can bump urllib3, we can remove types-urllib.
+    "types-urllib3==1.26.25.13",
+    "urllib3<2.0.0",
     "vulture==2.7",
     "vws-python==2023.3.25",
 ]

src/mock_vws/_flask_server/vwq.py

@@ -15,12 +15,10 @@
 from werkzeug.datastructures import Headers
 
 from mock_vws._query_tools import (
-    ActiveMatchingTargetsDeleteProcessing,
     get_query_match_response_text,
 )
 from mock_vws._query_validators import run_query_validators
 from mock_vws._query_validators.exceptions import (
-    DeletedTargetMatched,
     ValidatorException,
 )
 from mock_vws.database import VuforiaDatabase
@@ -55,8 +53,6 @@ class VWQSettings(BaseSettings):
 
     vwq_host: str = ""
     target_manager_base_url: str
-    deletion_processing_seconds: float = 3.0
-    deletion_recognition_seconds: float = 2.0
     query_image_matcher: _ImageMatcherChoice = _ImageMatcherChoice.AVERAGE_HASH
 
 
@@ -128,21 +124,14 @@ def query() -> Response:
     )
     date = email.utils.formatdate(None, localtime=False, usegmt=True)
 
-    try:
-        response_text = get_query_match_response_text(
-            request_headers=dict(request.headers),
-            request_body=request_body,
-            request_method=request.method,
-            request_path=request.path,
-            databases=databases,
-            query_processes_deletion_seconds=settings.deletion_processing_seconds,
-            query_recognizes_deletion_seconds=(
-                settings.deletion_recognition_seconds
-            ),
-            query_match_checker=query_match_checker,
-        )
-    except ActiveMatchingTargetsDeleteProcessing as exc:
-        raise DeletedTargetMatched from exc
+    response_text = get_query_match_response_text(
+        request_headers=dict(request.headers),
+        request_body=request_body,
+        request_method=request.method,
+        request_path=request.path,
+        databases=databases,
+        query_match_checker=query_match_checker,
+    )
 
     headers = {
         "Content-Type": "application/json",

src/mock_vws/_query_tools.py

@@ -24,20 +24,12 @@
     from mock_vws.image_matchers import ImageMatcher
 
 
-class ActiveMatchingTargetsDeleteProcessing(Exception):
-    """
-    There is at least one active target which matches and was recently deleted.
-    """
-
-
 def get_query_match_response_text(
     request_headers: dict[str, str],
     request_body: bytes,
     request_method: str,
     request_path: str,
     databases: set[VuforiaDatabase],
-    query_processes_deletion_seconds: int | float,
-    query_recognizes_deletion_seconds: int | float,
     query_match_checker: ImageMatcher,
 ) -> str:
     """
@@ -47,21 +39,11 @@ def get_query_match_response_text(
         request_body: The body of the request.
         request_method: The HTTP method of the request.
         databases: All Vuforia databases.
-        query_recognizes_deletion_seconds: The number of seconds after a target
-            has been deleted that the query endpoint will still recognize the
-            target for.
-        query_processes_deletion_seconds: The number of seconds after a target
-            deletion is recognized that the query endpoint will return a 500
-            response on a match.
         query_match_checker: A callable which takes two image values and
             returns whether they match.
 
     Returns:
         The response text for a query endpoint request.
-
-    Raises:
-        ActiveMatchingTargetsDeleteProcessing: There is at least one active
-            target which matches and was recently deleted.
     """
     body_file = io.BytesIO(request_body)
 
@@ -88,15 +70,7 @@ def get_query_match_response_text(
     assert image_part is not None
     image_value = image_part.raw
     gmt = ZoneInfo("GMT")
-    now = datetime.datetime.now(tz=gmt)
-
-    processing_timedelta = datetime.timedelta(
-        seconds=query_processes_deletion_seconds,
-    )
-
-    recognition_timedelta = datetime.timedelta(
-        seconds=query_recognizes_deletion_seconds,
-    )
+    datetime.datetime.now(tz=gmt)
 
     database = get_database_matching_client_keys(
         request_headers=request_headers,
@@ -125,28 +99,7 @@ def get_query_match_response_text(
         and target.status == TargetStatuses.SUCCESS.value
     ]
 
-    deletion_not_recognized_matches = [
-        target
-        for target in matching_targets
-        if target.active_flag
-        and target.delete_date
-        and (now - target.delete_date) < recognition_timedelta
-    ]
-
-    active_matching_targets_delete_processing = [
-        target
-        for target in matching_targets
-        if target.active_flag
-        and target.delete_date
-        and (now - target.delete_date)
-        < (recognition_timedelta + processing_timedelta)
-        and target not in deletion_not_recognized_matches
-    ]
-
-    if active_matching_targets_delete_processing:
-        raise ActiveMatchingTargetsDeleteProcessing
-
-    all_quality_matches = not_deleted_matches + deletion_not_recognized_matches
+    all_quality_matches = not_deleted_matches
     minimum_rating = 0
     matches = [
         match

src/mock_vws/_requests_mock_server/decorators.py

@@ -43,8 +43,6 @@ def __init__(
         duplicate_match_checker: ImageMatcher = _AVERAGE_HASH_MATCHER,
         query_match_checker: ImageMatcher = _AVERAGE_HASH_MATCHER,
         processing_time_seconds: int | float = 2,
-        query_recognizes_deletion_seconds: int | float = 2,
-        query_processes_deletion_seconds: int | float = 3,
         target_tracking_rater: TargetTrackingRater = _BRISQUE_TRACKING_RATER,
         *,
         real_http: bool = False,
@@ -62,12 +60,6 @@ def __init__(
                 In the real Vuforia Web Services, this is not deterministic.
             base_vwq_url: The base URL for the VWQ API.
             base_vws_url: The base URL for the VWS API.
-            query_recognizes_deletion_seconds: The number
-                of seconds after a target has been deleted that the query
-                endpoint will still recognize the target for.
-            query_processes_deletion_seconds: The number of
-                seconds after a target deletion is recognized that the query
-                endpoint will return a 500 response on a match.
             query_match_checker: A callable which takes two image values and
                 returns whether they will match in a query request.
             duplicate_match_checker: A callable which takes two image values
@@ -104,12 +96,6 @@ def __init__(
 
         self._mock_vwq_api = MockVuforiaWebQueryAPI(
             target_manager=self._target_manager,
-            query_processes_deletion_seconds=(
-                query_processes_deletion_seconds
-            ),
-            query_recognizes_deletion_seconds=(
-                query_recognizes_deletion_seconds
-            ),
             query_match_checker=query_match_checker,
         )
 

src/mock_vws/_requests_mock_server/mock_web_query_api.py

@@ -14,12 +14,10 @@
 
 from mock_vws._mock_common import Route
 from mock_vws._query_tools import (
-    ActiveMatchingTargetsDeleteProcessing,
     get_query_match_response_text,
 )
 from mock_vws._query_validators import run_query_validators
 from mock_vws._query_validators.exceptions import (
-    DeletedTargetMatched,
     ValidatorException,
 )
 
@@ -82,19 +80,11 @@ class MockVuforiaWebQueryAPI:
     def __init__(
         self,
         target_manager: TargetManager,
-        query_recognizes_deletion_seconds: int | float,
-        query_processes_deletion_seconds: int | float,
         query_match_checker: ImageMatcher,
     ) -> None:
         """
         Args:
             target_manager: The target manager which holds all databases.
-            query_recognizes_deletion_seconds: The number of seconds after a
-                target has been deleted that the query endpoint will still
-                recognize the target for.
-            query_processes_deletion_seconds: The number of seconds after a
-                target deletion is recognized that the query endpoint will
-                return a 500 response on a match.
             query_match_checker: A callable which takes two image values and
                 returns whether they match.
 
@@ -103,12 +93,6 @@ def __init__(
         """
         self.routes: set[Route] = _ROUTES
         self._target_manager = target_manager
-        self._query_processes_deletion_seconds = (
-            query_processes_deletion_seconds
-        )
-        self._query_recognizes_deletion_seconds = (
-            query_recognizes_deletion_seconds
-        )
         self._query_match_checker = query_match_checker
 
     @route(path_pattern="/v1/query", http_methods={POST})
@@ -133,26 +117,14 @@ def query(
             context.status_code = exc.status_code
             return exc.response_text
 
-        try:
-            response_text = get_query_match_response_text(
-                request_headers=request.headers,
-                request_body=request.body,
-                request_method=request.method,
-                request_path=request.path,
-                databases=self._target_manager.databases,
-                query_processes_deletion_seconds=(
-                    self._query_processes_deletion_seconds
-                ),
-                query_recognizes_deletion_seconds=(
-                    self._query_recognizes_deletion_seconds
-                ),
-                query_match_checker=self._query_match_checker,
-            )
-        except ActiveMatchingTargetsDeleteProcessing:
-            deleted_target_matched_exception = DeletedTargetMatched()
-            context.headers = deleted_target_matched_exception.headers
-            context.status_code = deleted_target_matched_exception.status_code
-            return deleted_target_matched_exception.response_text
+        response_text = get_query_match_response_text(
+            request_headers=request.headers,
+            request_body=request.body,
+            request_method=request.method,
+            request_path=request.path,
+            databases=self._target_manager.databases,
+            query_match_checker=self._query_match_checker,
+        )
 
         date = email.utils.formatdate(None, localtime=False, usegmt=True)
         context.headers = {