refactor: Update as KnownProvenance casts

In some instances, the casting to `KnownProvenance` is not explicitly required by receiving functions, here a more strict casting to `RemoteProvenance` is cleaner. Signed-off-by: Jens Keim <jens.keim@forvia.com>
oss-review-toolkit · Nov 26, 2024 · 2d70747 · 2d70747
1 parent 78e4be6
commit 2d70747
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/helper-cli/src/main/kotlin/utils/Extensions.kt b/helper-cli/src/main/kotlin/utils/Extensions.kt
@@ -219,8 +219,8 @@ internal fun OrtResult.getViolatedRulesByLicense(
 /**
  * Return the [Provenance] of the first scan result matching the given [id] or null if there is no match.
  */
-internal fun OrtResult.getScannedProvenance(id: Identifier): KnownProvenance? =
-    getScanResultsForId(id).firstNotNullOfOrNull { it.provenance as? KnownProvenance }
+internal fun OrtResult.getScannedProvenance(id: Identifier): RemoteProvenance? =
+    getScanResultsForId(id).firstNotNullOfOrNull { it.provenance as? RemoteProvenance }
 
 /**
  * Return the [SourceCodeOrigin] for this provenance.

diff --git a/model/src/main/kotlin/ScannerRun.kt b/model/src/main/kotlin/ScannerRun.kt
@@ -167,8 +167,8 @@ data class ScannerRun(
         provenances.associateBy { it.id }
     }
 
-    private val scanResultsByProvenance: Map<KnownProvenance, List<ScanResult>> by lazy {
-        scanResults.groupBy { it.provenance as KnownProvenance }
+    private val scanResultsByProvenance: Map<RemoteProvenance, List<ScanResult>> by lazy {
+        scanResults.groupBy { it.provenance as RemoteProvenance }
     }
 
     private val scanResultsById: Map<Identifier, List<ScanResult>> by lazy {