refactor(ScannerRun): Do not use SortedMap for scan results

Only sort on serialization for human readability and reproducibility.

Signed-off-by: Frank Viernau <frank_viernau@epam.com>

Author: fviernau

model/src/main/kotlin/ScannerRun.kt

@@ -21,9 +21,9 @@ package org.ossreviewtoolkit.model
 
 import com.fasterxml.jackson.annotation.JsonIgnore
 import com.fasterxml.jackson.annotation.JsonIgnoreProperties
+import com.fasterxml.jackson.annotation.JsonPropertyOrder
 
 import java.time.Instant
-import java.util.SortedMap
 
 import org.ossreviewtoolkit.model.config.ScannerConfiguration
 import org.ossreviewtoolkit.utils.ort.Environment
@@ -56,7 +56,8 @@ data class ScannerRun(
     /**
      * The [ScanResult]s for all [Package]s.
      */
-    val scanResults: SortedMap<Identifier, List<ScanResult>>,
+    @JsonPropertyOrder(alphabetic = true)
+    val scanResults: Map<Identifier, List<ScanResult>>,
 
     /**
      * The [AccessStatistics] for the scan results storage.
@@ -73,7 +74,7 @@ data class ScannerRun(
             endTime = Instant.EPOCH,
             environment = Environment(),
             config = ScannerConfiguration(),
-            scanResults = sortedMapOf(),
+            scanResults = emptyMap(),
             storageStats = AccessStatistics()
         )
     }

scanner/src/main/kotlin/Scanner.kt

@@ -130,7 +130,7 @@ class Scanner(
             emptyMap()
         }
 
-        val scanResults = (projectResults + packageResults).toSortedMap()
+        val scanResults = projectResults + packageResults
         val storageStats = AccessStatistics() // TODO: Record access statistics.
 
         val endTime = Instant.now()