test(application): link custom-metadata tests to TC-APPLICATION-CLI-07 scenarios [PYSDK-130][PYSDK-147]

Add record_property("tested-item-id", ...) traceability from the pytest tests
to the TC-APPLICATION-CLI-07 test-case scenarios, SWR-APPLICATION-2-17, and the
platform/application SIS.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: arne-aignx

tests/aignostics/application/cli_test.py

@@ -1161,8 +1161,9 @@ def test_cli_run_update_metadata_not_dict(runner: CliRunner) -> None:
 
 
 @pytest.mark.unit
-def test_cli_run_update_metadata_forwards_checksum(runner: CliRunner) -> None:
+def test_cli_run_update_metadata_forwards_checksum(runner: CliRunner, record_property) -> None:
     """Check run update-metadata forwards --checksum to the service as a keyword argument."""
+    record_property("tested-item-id", "TC-APPLICATION-CLI-07-01, SWR-APPLICATION-2-17, SPEC-APPLICATION-SERVICE")
     with patch("aignostics.application._cli.Service") as mock_service_cls:
         result = runner.invoke(
             cli,
@@ -1184,8 +1185,9 @@ def test_cli_run_update_metadata_forwards_checksum(runner: CliRunner) -> None:
 
 
 @pytest.mark.unit
-def test_cli_run_update_metadata_default_enrich_sdk_metadata_true(runner: CliRunner) -> None:
+def test_cli_run_update_metadata_default_enrich_sdk_metadata_true(runner: CliRunner, record_property) -> None:
     """Check run update-metadata passes enrich_sdk_metadata=True to the service by default."""
+    record_property("tested-item-id", "TC-APPLICATION-CLI-07-04, SWR-APPLICATION-2-17, SPEC-APPLICATION-SERVICE")
     with patch("aignostics.application._cli.Service") as mock_service_cls:
         result = runner.invoke(cli, ["application", "run", "update-metadata", "run-123", '{"key": "value"}'])
 
@@ -1196,8 +1198,11 @@ def test_cli_run_update_metadata_default_enrich_sdk_metadata_true(runner: CliRun
 
 
 @pytest.mark.unit
-def test_cli_run_update_metadata_no_enrich_sdk_metadata_reaches_service_false(runner: CliRunner) -> None:
+def test_cli_run_update_metadata_no_enrich_sdk_metadata_reaches_service_false(
+    runner: CliRunner, record_property
+) -> None:
     """Check run update-metadata --no-enrich-sdk-metadata reaches the service as enrich_sdk_metadata=False."""
+    record_property("tested-item-id", "TC-APPLICATION-CLI-07-03, SWR-APPLICATION-2-17, SPEC-APPLICATION-SERVICE")
     with patch("aignostics.application._cli.Service") as mock_service_cls:
         result = runner.invoke(
             cli,
@@ -1218,8 +1223,9 @@ def test_cli_run_update_metadata_no_enrich_sdk_metadata_reaches_service_false(ru
 
 
 @pytest.mark.unit
-def test_cli_run_update_metadata_concurrency_conflict_exits_3(runner: CliRunner) -> None:
+def test_cli_run_update_metadata_concurrency_conflict_exits_3(runner: CliRunner, record_property) -> None:
     """Check run update-metadata exits with code 3 when the service raises ConcurrencyConflictError."""
+    record_property("tested-item-id", "TC-APPLICATION-CLI-07-02, SWR-APPLICATION-2-17, SPEC-APPLICATION-SERVICE")
     with patch("aignostics.application._cli.Service") as mock_service_cls:
         mock_service_cls.return_value.application_run_update_custom_metadata.side_effect = ConcurrencyConflictError(
             "stale checksum"
@@ -1231,8 +1237,9 @@ def test_cli_run_update_metadata_concurrency_conflict_exits_3(runner: CliRunner)
 
 
 @pytest.mark.unit
-def test_cli_run_update_item_metadata_forwards_checksum(runner: CliRunner) -> None:
+def test_cli_run_update_item_metadata_forwards_checksum(runner: CliRunner, record_property) -> None:
     """Check run update-item-metadata forwards --checksum to the service as a keyword argument."""
+    record_property("tested-item-id", "TC-APPLICATION-CLI-07-06, SWR-APPLICATION-2-17, SPEC-APPLICATION-SERVICE")
     with patch("aignostics.application._cli.Service") as mock_service_cls:
         result = runner.invoke(
             cli,
@@ -1255,8 +1262,14 @@ def test_cli_run_update_item_metadata_forwards_checksum(runner: CliRunner) -> No
 
 
 @pytest.mark.unit
-def test_cli_run_update_item_metadata_no_enrich_sdk_metadata_reaches_service_false(runner: CliRunner) -> None:
+def test_cli_run_update_item_metadata_no_enrich_sdk_metadata_reaches_service_false(
+    runner: CliRunner, record_property
+) -> None:
     """Check run update-item-metadata --no-enrich-sdk-metadata reaches the service as enrich_sdk_metadata=False."""
+    record_property(
+        "tested-item-id",
+        "TC-APPLICATION-CLI-07-03, TC-APPLICATION-CLI-07-06, SWR-APPLICATION-2-17, SPEC-APPLICATION-SERVICE",
+    )
     with patch("aignostics.application._cli.Service") as mock_service_cls:
         result = runner.invoke(
             cli,
@@ -1278,8 +1291,12 @@ def test_cli_run_update_item_metadata_no_enrich_sdk_metadata_reaches_service_fal
 
 
 @pytest.mark.unit
-def test_cli_run_update_item_metadata_concurrency_conflict_exits_3(runner: CliRunner) -> None:
+def test_cli_run_update_item_metadata_concurrency_conflict_exits_3(runner: CliRunner, record_property) -> None:
     """Check run update-item-metadata exits with code 3 when the service raises ConcurrencyConflictError."""
+    record_property(
+        "tested-item-id",
+        "TC-APPLICATION-CLI-07-02, TC-APPLICATION-CLI-07-06, SWR-APPLICATION-2-17, SPEC-APPLICATION-SERVICE",
+    )
     with patch("aignostics.application._cli.Service") as mock_service_cls:
         mock_service_cls.return_value.application_run_update_item_custom_metadata.side_effect = (
             ConcurrencyConflictError("stale checksum")
@@ -1293,8 +1310,9 @@ def test_cli_run_update_item_metadata_concurrency_conflict_exits_3(runner: CliRu
 
 
 @pytest.mark.unit
-def test_cli_run_dump_metadata_show_checksum(runner: CliRunner) -> None:
+def test_cli_run_dump_metadata_show_checksum(runner: CliRunner, record_property) -> None:
     """Check run dump-metadata --show-checksum wraps output with the checksum."""
+    record_property("tested-item-id", "TC-APPLICATION-CLI-07-05, SWR-APPLICATION-2-17, SPEC-APPLICATION-SERVICE")
     mock_run_data = MagicMock()
     mock_run_data.custom_metadata = {"key": "value"}
     mock_run_data.custom_metadata_checksum = "checksum-abc"
@@ -1312,8 +1330,9 @@ def test_cli_run_dump_metadata_show_checksum(runner: CliRunner) -> None:
 
 
 @pytest.mark.unit
-def test_cli_run_dump_item_metadata_show_checksum(runner: CliRunner) -> None:
+def test_cli_run_dump_item_metadata_show_checksum(runner: CliRunner, record_property) -> None:
     """Check run dump-item-metadata --show-checksum wraps output with the checksum."""
+    record_property("tested-item-id", "TC-APPLICATION-CLI-07-05, SWR-APPLICATION-2-17, SPEC-APPLICATION-SERVICE")
     mock_item = MagicMock()
     mock_item.external_id = "item-ext-id"
     mock_item.custom_metadata = {"key": "value"}

tests/aignostics/application/service_test.py

@@ -521,8 +521,9 @@ def test_application_run_update_custom_metadata_success(mock_get_client: MagicMo
 
 @pytest.mark.unit
 @patch("aignostics.application._service.Service._get_platform_client")
-def test_application_run_update_custom_metadata_forwards_checksum(mock_get_client: MagicMock) -> None:
+def test_application_run_update_custom_metadata_forwards_checksum(mock_get_client: MagicMock, record_property) -> None:
     """Test that custom_metadata_checksum is forwarded to the platform layer."""
+    record_property("tested-item-id", "TC-APPLICATION-CLI-07-01, SWR-APPLICATION-2-17, SPEC-APPLICATION-SERVICE")
     mock_client = MagicMock()
     mock_run = MagicMock()
     mock_client.run.return_value = mock_run
@@ -540,8 +541,11 @@ def test_application_run_update_custom_metadata_forwards_checksum(mock_get_clien
 
 @pytest.mark.unit
 @patch("aignostics.application._service.Service._get_platform_client")
-def test_application_run_update_custom_metadata_forwards_enrich_sdk_metadata_false(mock_get_client: MagicMock) -> None:
+def test_application_run_update_custom_metadata_forwards_enrich_sdk_metadata_false(
+    mock_get_client: MagicMock, record_property
+) -> None:
     """Test that enrich_sdk_metadata=False is forwarded to the platform layer."""
+    record_property("tested-item-id", "TC-APPLICATION-CLI-07-03, SWR-APPLICATION-2-17, SPEC-APPLICATION-SERVICE")
     mock_client = MagicMock()
     mock_run = MagicMock()
     mock_client.run.return_value = mock_run
@@ -560,9 +564,10 @@ def test_application_run_update_custom_metadata_forwards_enrich_sdk_metadata_fal
 @pytest.mark.unit
 @patch("aignostics.application._service.Service._get_platform_client")
 def test_application_run_update_custom_metadata_static_forwards_enrich_sdk_metadata(
-    mock_get_client: MagicMock,
+    mock_get_client: MagicMock, record_property
 ) -> None:
     """Test that the static wrapper forwards enrich_sdk_metadata."""
+    record_property("tested-item-id", "TC-APPLICATION-CLI-07-03, SWR-APPLICATION-2-17, SPEC-APPLICATION-SERVICE")
     mock_client = MagicMock()
     mock_run = MagicMock()
     mock_client.run.return_value = mock_run
@@ -579,8 +584,11 @@ def test_application_run_update_custom_metadata_static_forwards_enrich_sdk_metad
 
 @pytest.mark.unit
 @patch("aignostics.application._service.Service._get_platform_client")
-def test_application_run_update_custom_metadata_static_forwards_checksum(mock_get_client: MagicMock) -> None:
+def test_application_run_update_custom_metadata_static_forwards_checksum(
+    mock_get_client: MagicMock, record_property
+) -> None:
     """Test that the static wrapper forwards custom_metadata_checksum."""
+    record_property("tested-item-id", "TC-APPLICATION-CLI-07-01, SWR-APPLICATION-2-17, SPEC-APPLICATION-SERVICE")
     mock_client = MagicMock()
     mock_run = MagicMock()
     mock_client.run.return_value = mock_run
@@ -597,8 +605,11 @@ def test_application_run_update_custom_metadata_static_forwards_checksum(mock_ge
 
 @pytest.mark.unit
 @patch("aignostics.application._service.Service._get_platform_client")
-def test_application_run_update_custom_metadata_concurrency_conflict(mock_get_client: MagicMock) -> None:
+def test_application_run_update_custom_metadata_concurrency_conflict(
+    mock_get_client: MagicMock, record_property
+) -> None:
     """Test that a 412 ApiException is mapped to ConcurrencyConflictError."""
+    record_property("tested-item-id", "TC-APPLICATION-CLI-07-02, SWR-APPLICATION-2-17, SPEC-APPLICATION-SERVICE")
     mock_client = MagicMock()
     mock_run = MagicMock()
     mock_run.update_custom_metadata.side_effect = ApiException(
@@ -619,8 +630,11 @@ def test_application_run_update_custom_metadata_concurrency_conflict(mock_get_cl
 
 @pytest.mark.unit
 @patch("aignostics.application._service.Service._get_platform_client")
-def test_application_run_update_custom_metadata_non_412_api_exception_unchanged(mock_get_client: MagicMock) -> None:
+def test_application_run_update_custom_metadata_non_412_api_exception_unchanged(
+    mock_get_client: MagicMock, record_property
+) -> None:
     """Test that a non-412 ApiException keeps the pre-existing behavior (RuntimeError)."""
+    record_property("tested-item-id", "TC-APPLICATION-CLI-07-02, SWR-APPLICATION-2-17, SPEC-APPLICATION-SERVICE")
     mock_client = MagicMock()
     mock_run = MagicMock()
     mock_run.update_custom_metadata.side_effect = ApiException(
@@ -636,8 +650,9 @@ def test_application_run_update_custom_metadata_non_412_api_exception_unchanged(
 
 
 @pytest.mark.unit
-def test_concurrency_conflict_error_is_value_error() -> None:
+def test_concurrency_conflict_error_is_value_error(record_property) -> None:
     """Test that ConcurrencyConflictError subclasses ValueError."""
+    record_property("tested-item-id", "TC-APPLICATION-CLI-07-02, SWR-APPLICATION-2-17, SPEC-APPLICATION-SERVICE")
     assert issubclass(ConcurrencyConflictError, ValueError)
 
 
@@ -683,9 +698,13 @@ def test_application_run_update_item_custom_metadata_success(mock_get_client: Ma
 @pytest.mark.unit
 @patch("aignostics.application._service.Service._get_platform_client")
 def test_application_run_update_item_custom_metadata_forwards_enrich_sdk_metadata_false(
-    mock_get_client: MagicMock,
+    mock_get_client: MagicMock, record_property
 ) -> None:
     """Test that enrich_sdk_metadata=False is forwarded for item metadata updates."""
+    record_property(
+        "tested-item-id",
+        "TC-APPLICATION-CLI-07-03, TC-APPLICATION-CLI-07-06, SWR-APPLICATION-2-17, SPEC-APPLICATION-SERVICE",
+    )
     mock_client = MagicMock()
     mock_run = MagicMock()
     mock_client.run.return_value = mock_run
@@ -706,9 +725,13 @@ def test_application_run_update_item_custom_metadata_forwards_enrich_sdk_metadat
 @pytest.mark.unit
 @patch("aignostics.application._service.Service._get_platform_client")
 def test_application_run_update_item_custom_metadata_static_forwards_enrich_sdk_metadata(
-    mock_get_client: MagicMock,
+    mock_get_client: MagicMock, record_property
 ) -> None:
     """Test that the item static wrapper forwards enrich_sdk_metadata."""
+    record_property(
+        "tested-item-id",
+        "TC-APPLICATION-CLI-07-03, TC-APPLICATION-CLI-07-06, SWR-APPLICATION-2-17, SPEC-APPLICATION-SERVICE",
+    )
     mock_client = MagicMock()
     mock_run = MagicMock()
     mock_client.run.return_value = mock_run
@@ -725,8 +748,14 @@ def test_application_run_update_item_custom_metadata_static_forwards_enrich_sdk_
 
 @pytest.mark.unit
 @patch("aignostics.application._service.Service._get_platform_client")
-def test_application_run_update_item_custom_metadata_concurrency_conflict(mock_get_client: MagicMock) -> None:
+def test_application_run_update_item_custom_metadata_concurrency_conflict(
+    mock_get_client: MagicMock, record_property
+) -> None:
     """Test that a 412 ApiException for an item update is mapped to ConcurrencyConflictError."""
+    record_property(
+        "tested-item-id",
+        "TC-APPLICATION-CLI-07-02, TC-APPLICATION-CLI-07-06, SWR-APPLICATION-2-17, SPEC-APPLICATION-SERVICE",
+    )
     mock_client = MagicMock()
     mock_run = MagicMock()
     mock_run.update_item_custom_metadata.side_effect = ApiException(

tests/aignostics/platform/resources/runs_test.py

@@ -1319,8 +1319,9 @@ def test_results_omits_none_filters(app_run, mock_api) -> None:
 
 
 @pytest.mark.unit
-def test_update_custom_metadata_forwards_checksum(app_run, mock_api) -> None:
+def test_update_custom_metadata_forwards_checksum(app_run, mock_api, record_property) -> None:
     """update_custom_metadata forwards custom_metadata_checksum on the update request."""
+    record_property("tested-item-id", "TC-APPLICATION-CLI-07-01, SWR-APPLICATION-2-17, SPEC-PLATFORM-SERVICE")
     with patch(_PATCH_BUILD_RUN_SDK_METADATA, return_value={}), patch(_PATCH_VALIDATE_RUN_SDK_METADATA):
         app_run.update_custom_metadata({"key": "value"}, custom_metadata_checksum="abc123")
 
@@ -1330,8 +1331,9 @@ def test_update_custom_metadata_forwards_checksum(app_run, mock_api) -> None:
 
 
 @pytest.mark.unit
-def test_update_custom_metadata_enrich_default_calls_sdk_metadata_builders(app_run, mock_api) -> None:
+def test_update_custom_metadata_enrich_default_calls_sdk_metadata_builders(app_run, mock_api, record_property) -> None:
     """Default enrich_sdk_metadata=True merges and validates SDK metadata."""
+    record_property("tested-item-id", "TC-APPLICATION-CLI-07-04, SWR-APPLICATION-2-17, SPEC-PLATFORM-SERVICE")
     mock_api.put_run_custom_metadata_v1_runs_run_id_custom_metadata_put.return_value = None
     with (
         patch(_PATCH_BUILD_RUN_SDK_METADATA, return_value={"schema_version": "0.0.1"}) as mock_build,
@@ -1348,8 +1350,9 @@ def test_update_custom_metadata_enrich_default_calls_sdk_metadata_builders(app_r
 
 
 @pytest.mark.unit
-def test_update_custom_metadata_no_enrich_skips_sdk_metadata_builders(app_run, mock_api) -> None:
+def test_update_custom_metadata_no_enrich_skips_sdk_metadata_builders(app_run, mock_api, record_property) -> None:
     """enrich_sdk_metadata=False skips build/validate and forwards custom_metadata verbatim."""
+    record_property("tested-item-id", "TC-APPLICATION-CLI-07-03, SWR-APPLICATION-2-17, SPEC-PLATFORM-SERVICE")
     with (
         patch(_PATCH_BUILD_RUN_SDK_METADATA) as mock_build,
         patch(_PATCH_VALIDATE_RUN_SDK_METADATA) as mock_validate,
@@ -1365,8 +1368,9 @@ def test_update_custom_metadata_no_enrich_skips_sdk_metadata_builders(app_run, m
 
 
 @pytest.mark.unit
-def test_update_item_custom_metadata_forwards_checksum(app_run, mock_api) -> None:
+def test_update_item_custom_metadata_forwards_checksum(app_run, mock_api, record_property) -> None:
     """update_item_custom_metadata forwards custom_metadata_checksum on the update request."""
+    record_property("tested-item-id", "TC-APPLICATION-CLI-07-06, SWR-APPLICATION-2-17, SPEC-PLATFORM-SERVICE")
     with patch(_PATCH_BUILD_ITEM_SDK_METADATA, return_value={}), patch(_PATCH_VALIDATE_ITEM_SDK_METADATA):
         app_run.update_item_custom_metadata("item-ext-id", {"key": "value"}, custom_metadata_checksum="xyz789")
 
@@ -1378,8 +1382,12 @@ def test_update_item_custom_metadata_forwards_checksum(app_run, mock_api) -> Non
 
 
 @pytest.mark.unit
-def test_update_item_custom_metadata_no_enrich_skips_sdk_metadata_builders(app_run, mock_api) -> None:
+def test_update_item_custom_metadata_no_enrich_skips_sdk_metadata_builders(app_run, mock_api, record_property) -> None:
     """enrich_sdk_metadata=False skips build/validate for items and forwards metadata verbatim."""
+    record_property(
+        "tested-item-id",
+        "TC-APPLICATION-CLI-07-03, TC-APPLICATION-CLI-07-06, SWR-APPLICATION-2-17, SPEC-PLATFORM-SERVICE",
+    )
     with (
         patch(_PATCH_BUILD_ITEM_SDK_METADATA) as mock_build,
         patch(_PATCH_VALIDATE_ITEM_SDK_METADATA) as mock_validate,