refactor: replace final string comparison with type-safe enum check

- Replace container platform check string comparison with enum comparison
- Change `args.artifact_type == "container"` to `config.artifact_type == ArtifactType.CONTAINER`
- Add comprehensive test coverage for enum comparisons and value access
- Completes strangler fig migration of artifact type logic to type-safe enums

This final change eliminates the last remaining string-based artifact type
comparison in the container platform validation logic, completing the
migration to type-safe enum comparisons throughout the orchestrator.

Author: bluesentinelsec

entrypoint/entrypoint/orchestrator.py

@@ -204,7 +204,7 @@ def invoke_sbomgen(args) -> int:
         sbomgen_args.append("--skip-files")
         sbomgen_args.append(args.skip_files)
 
-    if args.artifact_type == "container":
+    if config.artifact_type == ArtifactType.CONTAINER:
 
         if args.platform:
             platform_arg = args.platform.lower()

entrypoint/tests/test_data_model.py

@@ -78,3 +78,16 @@ def test_scan_config_binary_type_comparison():
 def test_scan_config_archive_type_comparison():
     config = ScanConfig(artifact_type=ArtifactType.ARCHIVE)
     assert config.artifact_type == ArtifactType.ARCHIVE
+
+def test_scan_config_container_platform_check():
+    config = ScanConfig(artifact_type=ArtifactType.CONTAINER)
+    assert config.artifact_type == ArtifactType.CONTAINER
+
+def test_scan_config_artifact_type_value():
+    config = ScanConfig(artifact_type=ArtifactType.REPOSITORY)
+    assert config.artifact_type.value == "repository"
+
+def test_scan_config_display_mapping():
+    config = ScanConfig(artifact_type=ArtifactType.REPOSITORY)
+    display_type = "repository" if config.artifact_type == ArtifactType.REPOSITORY else config.artifact_type.value
+    assert display_type == "repository"