Add build checks

.github/workflows/build.yml

@@ -0,0 +1,19 @@
+name: Build
+
+on:
+  workflow_dispatch:
+  pull_request:
+  push:
+    branches:
+      - main
+
+jobs:
+  build:
+    runs-on: macos-latest
+    steps:
+      - name: Checkout sources
+        uses: actions/checkout@v4
+      - name: Build project
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+        run: ./gradlew build -PallWarningsAsErrors=true

gradle/libs.versions.toml

@@ -1,7 +1,7 @@
 [versions]
-kotlin = "2.0.0"
-kotlinDatetime = "0.6.0"
-kotlinSerialization = "1.6.3"
+kotlin = "2.1.10"
+kotlinDatetime = "0.6.2"
+kotlinSerialization = "1.8.0"
 
 [libraries]
 kotlin-datetime = { group = "org.jetbrains.kotlinx", name = "kotlinx-datetime", version.ref = "kotlinDatetime" }

library/build.gradle.kts

@@ -31,6 +31,13 @@ kotlin {
             }
         }
     }
+
+    compilerOptions {
+        val allWarningsAsErrorsArgument = "allWarningsAsErrors"
+        if (project.hasProperty(allWarningsAsErrorsArgument)) {
+            allWarningsAsErrors = (project.property(allWarningsAsErrorsArgument) == "true")
+        }
+    }
 }
 
 publishing {

library/src/commonMain/kotlin/dk/spilpind/sms/api/RequestSerializerInterceptor.kt

@@ -39,8 +39,8 @@ object RequestSerializerInterceptor : JsonTransformingSerializer<Request>(Reques
     ) : Exception(message)
 
     @Suppress("DEPRECATION") // Accept
-    private val contextActionClassMap = Context.values().associate { context ->
-        context.contextKey to Action.values().mapNotNull { action ->
+    private val contextActionClassMap = Context.entries.associate { context ->
+        context.contextKey to Action.entries.mapNotNull { action ->
             val actionClass: KSerializer<out ContextAction>? = when (context) {
                 Context.System -> null
                 Context.Authentication -> when (action) {

library/src/commonMain/kotlin/dk/spilpind/sms/api/ResponseSerializerInterceptor.kt

@@ -61,7 +61,7 @@ object ResponseSerializerInterceptor : JsonTransformingSerializer<Response>(Resp
     }
 
     @Suppress("DEPRECATION") // Accepted
-    private val reactionContextClassMap = Reaction.values().associate { reaction ->
+    private val reactionContextClassMap = Reaction.entries.associate { reaction ->
         reaction.reactionKey to when (reaction) {
             Reaction.Informed -> createContextMap { context ->
                 when (context) {
@@ -234,7 +234,7 @@ object ResponseSerializerInterceptor : JsonTransformingSerializer<Response>(Resp
         mapper: (Context) -> KSerializer<out ReactionData>?
     ): ReactionClassMap {
         return ReactionClassMap.ContextBased(
-            serializerMap = Context.values().mapNotNull { context ->
+            serializerMap = Context.entries.mapNotNull { context ->
                 val serializer = mapper(context) ?: return@mapNotNull null
                 Pair(context, serializer)
             }.toMap()

library/src/commonMain/kotlin/dk/spilpind/sms/api/core/Status.kt

@@ -71,7 +71,7 @@ data class Status(
 
         override fun deserialize(decoder: Decoder): Type {
             val typeName = decoder.decodeString()
-            return Type.values().find { type ->
+            return Type.entries.find { type ->
                 type.name.lowercase() == typeName
             } ?: Type.Unknown
         }

library/src/commonMain/kotlin/dk/spilpind/sms/core/TimeHelper.kt

@@ -32,14 +32,14 @@ object TimeHelper {
     fun LocalDateTime.toInstant(): Instant = toInstant(timeZone)
 
     /**
-     * Parses [isoDateString] to a date time object. This can e.g. parse values from [currentDateTimeString]
+     * Parses [isoDateString] to a date time object. This can e.g. parse a value returned by [LocalDateTime.toString]
      */
     fun parse(isoDateString: String): LocalDateTime {
         return if (isoDateString.length >= 24) {
             // Javascript Date.toISOString() for instance returns this
-            isoDateString.toInstant().toLocalDateTime(timeZone)
+            Instant.parse(isoDateString).toLocalDateTime(timeZone)
         } else {
-            isoDateString.toLocalDateTime()
+            LocalDateTime.parse(isoDateString)
         }
     }
 

library/src/commonMain/kotlin/dk/spilpind/sms/core/model/ModelHelper.kt

@@ -36,7 +36,7 @@ object ModelHelper {
             created = created
         )
 
-        val type = Event.Type.values().firstOrNull { type -> type.typeId == typeId }
+        val type = Event.Type.entries.firstOrNull { type -> type.typeId == typeId }
             ?: throw IllegalStateException("Got unknown event type: $typeId")
 
         return when (type) {
@@ -72,7 +72,7 @@ object ModelHelper {
         tournamentId = tournamentId,
         teamAId = teamAId,
         teamBId = teamBId,
-        state = Game.State.values().firstOrNull { state -> state.identifier == gameState }
+        state = Game.State.entries.firstOrNull { state -> state.identifier == gameState }
             ?: throw IllegalStateException("Got unknown game state: $gameState"),
         teamAPoints = teamAPoints,
         teamBPoints = teamBPoints,

library/src/commonMain/kotlin/dk/spilpind/sms/core/model/PendingRequest.kt

@@ -134,7 +134,7 @@ sealed interface PendingRequest {
          * Finds a [Type] based on the provided parameters or throw an [IllegalArgumentException] if not found
          */
         fun findType(contextIdentifier: String, requestIdentifier: String): Type {
-            val context = PendingRequestContext.values().firstOrNull { context ->
+            val context = PendingRequestContext.entries.firstOrNull { context ->
                 context.context.contextKey == contextIdentifier
             }
 
@@ -160,7 +160,7 @@ sealed interface PendingRequest {
                 }
                 null -> throw IllegalArgumentException(
                     "Context \"$contextIdentifier\" of pending request not found. Available contexts: ${
-                        PendingRequestContext.values().map { availableContext -> availableContext.context.contextKey }
+                        PendingRequestContext.entries.map { availableContext -> availableContext.context.contextKey }
                     }"
                 )
             }

library/src/commonMain/kotlin/dk/spilpind/sms/core/model/UserRole.kt

@@ -131,7 +131,7 @@ sealed interface UserRole {
          * Finds a [ContextRole] based on the provided parameters or throw an [IllegalArgumentException] if not found
          */
         fun findContext(contextIdentifier: String, roleIdentifier: String): ContextRole {
-            val context = RoleContext.values().firstOrNull { context ->
+            val context = RoleContext.entries.firstOrNull { context ->
                 context.context.contextKey == contextIdentifier
             }
 
@@ -155,7 +155,7 @@ sealed interface UserRole {
                 }
                 null -> throw IllegalArgumentException(
                     "Context \"$contextIdentifier\" not found. Available contexts: "
-                            + "${RoleContext.values().map { availableContext -> availableContext.context.contextKey }}"
+                            + "${RoleContext.entries.map { availableContext -> availableContext.context.contextKey }}"
                 )
             }
         }