Add test for inactive database in VuMark API (#3003)

* Add test for inactive database in VuMark generation API

This test verifies that calling the VuMark generation endpoint with credentials from an inactive database returns a ProjectInactive error.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

* Move inactive database test to existing test file

Avoids duplicating _make_vumark_request by placing TestInactiveDatabase in test_vumark_generation_api.py where the helper already exists.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

* [pre-commit.ci lite] apply automatic fixes

* Use verify_mock_vuforia for inactive database test

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

* Use inactive VuMark database fixture for TestInactiveDatabase

Add state support to VuMarkDatabase, add InactiveVuMarkCloudDatabase
fixture using INACTIVE_VUMARK_VUFORIA_* env vars, and update
project_state_validators to raise ProjectInactiveError for inactive
VuMark databases.

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

* Use keyword-only args in fixture functions to fix too-many-positional-arguments

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

* Fix inactive VuMark database to return UnknownTarget (404) not ProjectInactive

Real Vuforia returns 404 UnknownTarget for inactive VuMark databases on the
generation endpoint, not 403 ProjectInactive. Skip the project-inactive check
for VuMarkDatabase so target validation runs and returns the correct response.

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

* Add INACTIVE_VUMARK_VUFORIA_* to vuforia_secrets.env.example

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

---------

Co-authored-by: Claude Haiku 4.5 <noreply@anthropic.com>
Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>

Author: 3 people

src/mock_vws/_flask_server/target_manager.py

@@ -252,6 +252,10 @@ def create_vumark_database() -> Response:
     """
     request_json = json.loads(s=request.data)
     random_vumark_database = VuMarkDatabase()
+    state_name = request_json.get(
+        "state_name",
+        random_vumark_database.state.name,
+    )
     database = VuMarkDatabase(
         server_access_key=request_json.get(
             "server_access_key",
@@ -265,6 +269,7 @@ def create_vumark_database() -> Response:
             "database_name",
             random_vumark_database.database_name,
         ),
+        state=States[state_name],
     )
 
     try:

src/mock_vws/_services_validators/project_state_validators.py

@@ -11,7 +11,7 @@
     get_database_matching_server_keys,
 )
 from mock_vws._services_validators.exceptions import ProjectInactiveError
-from mock_vws.database import CloudDatabase
+from mock_vws.database import CloudDatabase, VuMarkDatabase
 from mock_vws.states import States
 
 _LOGGER = logging.getLogger(name=__name__)
@@ -47,13 +47,17 @@ def validate_project_state(
         databases=databases,
     )
 
-    if not isinstance(database, CloudDatabase):
+    if database.state != States.PROJECT_INACTIVE:
         return
 
-    if database.state != States.PROJECT_INACTIVE:
+    if (
+        isinstance(database, CloudDatabase)
+        and request_method == HTTPMethod.GET
+        and "duplicates" not in request_path
+    ):
         return
 
-    if request_method == HTTPMethod.GET and "duplicates" not in request_path:
+    if isinstance(database, VuMarkDatabase):
         return
 
     _LOGGER.warning(msg="The project is inactive.")

src/mock_vws/database.py

@@ -40,6 +40,7 @@ class VuMarkDatabaseDict(TypedDict):
     server_access_key: str
     server_secret_key: str
     vumark_targets: Iterable[VuMarkTargetDict]
+    state_name: str
 
 
 @beartype
@@ -202,6 +203,7 @@ class VuMarkDatabase:
         default_factory=set[VuMarkTarget],
         hash=False,
     )
+    state: States = States.WORKING
 
     def get_vumark_target(self, target_id: str) -> VuMarkTarget:
         """Return a VuMark target from the database with the given ID."""
@@ -222,6 +224,7 @@ def to_dict(self) -> VuMarkDatabaseDict:
             "server_access_key": self.server_access_key,
             "server_secret_key": self.server_secret_key,
             "vumark_targets": vumark_targets,
+            "state_name": self.state.name,
         }
 
     @classmethod
@@ -235,6 +238,7 @@ def from_dict(cls, database_dict: VuMarkDatabaseDict) -> Self:
                 VuMarkTarget.from_dict(target_dict=target_dict)
                 for target_dict in database_dict["vumark_targets"]
             },
+            state=States[database_dict["state_name"]],
         )
 
     @property

tests/mock_vws/fixtures/credentials.py

@@ -38,6 +38,20 @@ class _InactiveCloudDatabaseSettings(_CloudDatabaseSettings):
     )
 
 
+class _InactiveVuMarkDatabaseSettings(BaseSettings):
+    """Settings for an inactive VuMark database."""
+
+    target_manager_database_name: str
+    server_access_key: str
+    server_secret_key: str
+
+    model_config = SettingsConfigDict(
+        env_prefix="INACTIVE_VUMARK_VUFORIA_",
+        env_file=Path("vuforia_secrets.env"),
+        extra="allow",
+    )
+
+
 class _VuMarkCloudDatabaseSettings(BaseSettings):
     """Settings for a VuMark Vuforia database."""
 
@@ -54,6 +68,15 @@ class _VuMarkCloudDatabaseSettings(BaseSettings):
     )
 
 
+@dataclass(frozen=True)
+class InactiveVuMarkCloudDatabase:
+    """Credentials for an inactive VuMark database."""
+
+    target_manager_database_name: str = field(repr=False)
+    server_access_key: str = field(repr=False)
+    server_secret_key: str = field(repr=False)
+
+
 @dataclass(frozen=True)
 class VuMarkCloudDatabase:
     """Credentials for the VuMark generation API."""
@@ -96,6 +119,17 @@ def inactive_cloud_database() -> CloudDatabase:
     )
 
 
+@pytest.fixture
+def inactive_vumark_database() -> InactiveVuMarkCloudDatabase:
+    """Return inactive VuMark credentials from environment variables."""
+    settings = _InactiveVuMarkDatabaseSettings.model_validate(obj={})
+    return InactiveVuMarkCloudDatabase(
+        target_manager_database_name=settings.target_manager_database_name,
+        server_access_key=settings.server_access_key,
+        server_secret_key=settings.server_secret_key,
+    )
+
+
 @pytest.fixture
 def vumark_vuforia_database() -> VuMarkCloudDatabase:
     """Return VuMark VWS credentials from environment variables."""

tests/mock_vws/fixtures/vuforia_backends.py

@@ -22,7 +22,10 @@
 from mock_vws.database import CloudDatabase, VuMarkDatabase
 from mock_vws.states import States
 from mock_vws.target import VuMarkTarget
-from tests.mock_vws.fixtures.credentials import VuMarkCloudDatabase
+from tests.mock_vws.fixtures.credentials import (
+    InactiveVuMarkCloudDatabase,
+    VuMarkCloudDatabase,
+)
 from tests.mock_vws.utils.retries import RETRY_ON_TOO_MANY_REQUESTS
 
 LOGGER = logging.getLogger(name=__name__)
@@ -91,12 +94,14 @@ def _enable_use_real_vuforia(
     working_database: CloudDatabase,
     inactive_cloud_database: CloudDatabase,
     vumark_vuforia_database: VuMarkCloudDatabase,
+    inactive_vumark_database: InactiveVuMarkCloudDatabase,
     monkeypatch: pytest.MonkeyPatch,
 ) -> Generator[None]:
     """Test against the real Vuforia."""
     assert monkeypatch
     assert inactive_cloud_database
     assert vumark_vuforia_database
+    assert inactive_vumark_database
     _delete_all_targets(database_keys=working_database)
     yield
 
@@ -107,6 +112,7 @@ def _enable_use_mock_vuforia(
     working_database: CloudDatabase,
     inactive_cloud_database: CloudDatabase,
     vumark_vuforia_database: VuMarkCloudDatabase,
+    inactive_vumark_database: InactiveVuMarkCloudDatabase,
     monkeypatch: pytest.MonkeyPatch,
 ) -> Generator[None]:
     """Test against the in-memory mock Vuforia."""
@@ -130,11 +136,18 @@ def _enable_use_mock_vuforia(
     vumark_database = _vumark_database(
         vumark_vuforia_database=vumark_vuforia_database,
     )
+    inactive_vumark_db = VuMarkDatabase(
+        state=States.PROJECT_INACTIVE,
+        database_name=inactive_vumark_database.target_manager_database_name,
+        server_access_key=inactive_vumark_database.server_access_key,
+        server_secret_key=inactive_vumark_database.server_secret_key,
+    )
 
     with MockVWS() as mock:
         mock.add_cloud_database(cloud_database=working_database)
         mock.add_cloud_database(cloud_database=inactive_cloud_database)
         mock.add_vumark_database(vumark_database=vumark_database)
+        mock.add_vumark_database(vumark_database=inactive_vumark_db)
         yield
 
 
@@ -144,6 +157,7 @@ def _enable_use_docker_in_memory(
     working_database: CloudDatabase,
     inactive_cloud_database: CloudDatabase,
     vumark_vuforia_database: VuMarkCloudDatabase,
+    inactive_vumark_database: InactiveVuMarkCloudDatabase,
     monkeypatch: pytest.MonkeyPatch,
 ) -> Generator[None]:
     """Test against mock Vuforia created to be run in a container."""
@@ -170,6 +184,12 @@ def _enable_use_docker_in_memory(
     vumark_database = _vumark_database(
         vumark_vuforia_database=vumark_vuforia_database,
     )
+    inactive_vumark_db = VuMarkDatabase(
+        state=States.PROJECT_INACTIVE,
+        database_name=inactive_vumark_database.target_manager_database_name,
+        server_access_key=inactive_vumark_database.server_access_key,
+        server_secret_key=inactive_vumark_database.server_secret_key,
+    )
 
     with responses.RequestsMock(assert_all_requests_are_fired=False) as mock:
         add_flask_app_to_mock(
@@ -223,6 +243,11 @@ def _enable_use_docker_in_memory(
             json=vumark_database.to_dict(),
             timeout=30,
         )
+        requests.post(
+            url=vumark_databases_url,
+            json=inactive_vumark_db.to_dict(),
+            timeout=30,
+        )
         for vumark_target in vumark_database.vumark_targets:
             requests.post(
                 url=(
@@ -292,10 +317,12 @@ def pytest_collection_modifyitems(
     ids=[backend.value for backend in list(VuforiaBackend)],
 )
 def fixture_verify_mock_vuforia(
+    *,
     request: pytest.FixtureRequest,
     vuforia_database: CloudDatabase,
     inactive_cloud_database: CloudDatabase,
     vumark_vuforia_database: VuMarkCloudDatabase,
+    inactive_vumark_database: InactiveVuMarkCloudDatabase,
     monkeypatch: pytest.MonkeyPatch,
 ) -> Generator[None]:
     """Test functions which use this fixture are run multiple times. Once
@@ -324,6 +351,7 @@ def fixture_verify_mock_vuforia(
         working_database=vuforia_database,
         inactive_cloud_database=inactive_cloud_database,
         vumark_vuforia_database=vumark_vuforia_database,
+        inactive_vumark_database=inactive_vumark_database,
         monkeypatch=monkeypatch,
     )
 
@@ -338,10 +366,12 @@ def fixture_verify_mock_vuforia(
     ],
 )
 def mock_only_vuforia(
+    *,
     request: pytest.FixtureRequest,
     vuforia_database: CloudDatabase,
     inactive_cloud_database: CloudDatabase,
     vumark_vuforia_database: VuMarkCloudDatabase,
+    inactive_vumark_database: InactiveVuMarkCloudDatabase,
     monkeypatch: pytest.MonkeyPatch,
 ) -> Generator[None]:
     """Test functions which use this fixture are run multiple times. Once
@@ -370,5 +400,6 @@ def mock_only_vuforia(
         working_database=vuforia_database,
         inactive_cloud_database=inactive_cloud_database,
         vumark_vuforia_database=vumark_vuforia_database,
+        inactive_vumark_database=inactive_vumark_database,
         monkeypatch=monkeypatch,
     )

tests/mock_vws/test_vumark_generation_api.py

@@ -18,7 +18,10 @@
 
 from mock_vws._constants import ResultCodes
 from mock_vws.database import CloudDatabase
-from tests.mock_vws.fixtures.credentials import VuMarkCloudDatabase
+from tests.mock_vws.fixtures.credentials import (
+    InactiveVuMarkCloudDatabase,
+    VuMarkCloudDatabase,
+)
 from tests.mock_vws.utils import make_image_file
 
 _VWS_HOST = "https://vws.vuforia.com"
@@ -341,3 +344,27 @@ def test_processing_target_raw_response(
             response_json["result_code"]
             == ResultCodes.TARGET_STATUS_NOT_SUCCESS.value
         )
+
+
+@pytest.mark.usefixtures("verify_mock_vuforia")
+class TestInactiveDatabase:
+    """Tests for VuMark generation with an inactive database."""
+
+    @staticmethod
+    def test_inactive_database(
+        inactive_vumark_database: InactiveVuMarkCloudDatabase,
+    ) -> None:
+        """Calling the VuMark generation API with credentials for an
+        inactive database returns ProjectInactive.
+        """
+        response = _make_vumark_request(
+            server_access_key=inactive_vumark_database.server_access_key,
+            server_secret_key=inactive_vumark_database.server_secret_key,
+            target_id=uuid4().hex,
+            instance_id=uuid4().hex,
+            accept="image/png",
+        )
+
+        assert response.status_code == HTTPStatus.NOT_FOUND
+        response_json = response.json()
+        assert response_json["result_code"] == ResultCodes.UNKNOWN_TARGET.value

vuforia_secrets.env.example

@@ -19,3 +19,8 @@ VUMARK_VUFORIA_TARGET_ID=examplevumarktargetid
 
 VUMARK_VUFORIA_SERVER_ACCESS_KEY=example_vumark_server_access_key
 VUMARK_VUFORIA_SERVER_SECRET_KEY=example_vumark_server_secret_key
+
+INACTIVE_VUMARK_VUFORIA_TARGET_MANAGER_DATABASE_NAME=example_inactive_vumark_database_name
+
+INACTIVE_VUMARK_VUFORIA_SERVER_ACCESS_KEY=example_inactive_vumark_server_access_key
+INACTIVE_VUMARK_VUFORIA_SERVER_SECRET_KEY=example_inactive_vumark_server_secret_key