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.

All cases of `KnownProvenance` as a cast type were investigated
by the same methods as the previous commit.

Signed-off-by: Jens Keim <[email protected]>

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.

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 {