Fix wrong version being used in imported test results

Author: Su5eD

base/src/main/kotlin/org/sinytra/probe/base/RestAPI.kt

@@ -36,9 +36,9 @@ data class TestEnvironmentDTO(
 @Serializable
 data class TestResponseBody(
     val modid: String?,
-    val iconUrl: String,
+    val iconUrl: String?,
     val projectUrl: String,
-    val version: String,
+    val version: String?,
     val passing: Boolean,
     val environment: TestEnvironmentDTO,
     val createdAt: LocalDateTime,

discord/src/main/kotlin/CheckCompatCommandBase.kt

@@ -84,7 +84,7 @@ abstract class CheckCompatCommandBase {
                 color = if (result.passing) green else red
                 url = link
                 thumbnail {
-                    url = result.iconUrl
+                    url = result.iconUrl ?: ""
                 }
 
                 field {
@@ -101,7 +101,7 @@ abstract class CheckCompatCommandBase {
                 }
                 field {
                     name = "Mod version"
-                    value = result.version
+                    value = result.version ?: "unknown"
                     inline = true
                 }
                 field {

service/src/main/kotlin/org/sinytra/probe/service/Routing.kt

@@ -90,9 +90,9 @@ fun Application.configureRouting(
 
                     val response = TestResponseBody(
                         result.modid,
-                        project.iconUrl ?: "",
+                        project.iconUrl,
                         project.url,
-                        version?.versionNumber ?: "unknown",
+                        version?.versionNumber,
                         result.passing,
                         envDto,
                         result.createdAt,
@@ -133,7 +133,13 @@ fun Application.configureRouting(
                                     val project = platforms.getProject(ProjectPlatform.MODRINTH, res.project.slug)
                                         ?: return@async null
 
-                                    return@async persistence.saveResult(project, res.result!!.output.primaryModid, res.versionNumber, res.result!!.output.success, testEnvironment)
+                                    return@async persistence.saveResult(
+                                        project,
+                                        res.result!!.output.primaryModid,
+                                        res.project.versionId,
+                                        res.result!!.output.success,
+                                        testEnvironment
+                                    )
                                 } catch (ex: Exception) {
                                     LOGGER.error("Error importing result", ex)
                                     return@async null