chore(model): Use checkNotNull instead of suppressing unsafe calls

Signed-off-by: Sebastian Schuberth <[email protected]>

Author: sschuberth

model/src/main/kotlin/ScannerRun.kt

@@ -241,8 +241,7 @@ data class ScannerRun(
             }
         }
 
-        @Suppress("UnsafeCallOnNullableType")
-        val packageProvenance = resolutionResult.packageProvenance!!
+        val packageProvenance = checkNotNull(resolutionResult.packageProvenance)
 
         val scanResultsByPath = resolutionResult.getKnownProvenancesWithoutVcsPath().mapValues { (_, provenance) ->
             scanResultsByProvenance[provenance].orEmpty()
@@ -265,8 +264,7 @@ data class ScannerRun(
             it.packageProvenanceResolutionIssue == null && it.nestedProvenanceResolutionIssue == null
         } ?: return null
 
-        @Suppress("UnsafeCallOnNullableType")
-        val packageProvenance = resolutionResult.packageProvenance!!
+        val packageProvenance = checkNotNull(resolutionResult.packageProvenance)
 
         val fileListsByPath = resolutionResult.getKnownProvenancesWithoutVcsPath().mapValues { (_, provenance) ->
             // If there was an issue creating at least one file list, then return null instead of an incomplete file