feat: improve validation for integer's named values (#274)

Part of
#253.

Author: krzema12

src/jvmMain/kotlin/it/krzeminski/githubactionstyping/parsing/TypesManifestParsing.kt

@@ -21,7 +21,7 @@ data class ApiItem(
     val allowedValues: List<String>? = null,
     val separator: String? = null,
     @SerialName("named-values")
-    val namedValues: Map<String, Int>? = null,
+    val namedValues: Map<String, String>? = null,
     @SerialName("list-item")
     val listItem: ApiItem? = null,
 )

src/jvmMain/kotlin/it/krzeminski/githubactionstyping/validation/types/Integer.kt

@@ -13,6 +13,9 @@ fun ApiItem.validateInteger(): ItemValidationResult {
     if (this.namedValues?.keys?.any { it.isBlank() } == true) {
         return ItemValidationResult.Invalid("Named value names must not be empty.")
     }
+    if (this.namedValues?.values?.any { it.toIntOrNull() == null } == true) {
+        return ItemValidationResult.Invalid("Named values must be integer.")
+    }
     if (this.name != null && this.name.isBlank()) {
         return ItemValidationResult.Invalid("Name must not be empty.")
     }

src/jvmTest/kotlin/it/krzeminski/githubactionstyping/validation/ManifestValidationTest.kt

@@ -17,12 +17,12 @@ class ManifestValidationTest : FunSpec({
                     "integer-input" to ApiItem(type = "integer"),
                     "integer-with-named-values-input" to ApiItem(
                         type = "integer",
-                        namedValues = mapOf("foo" to 1, "bar" to 2)
+                        namedValues = mapOf("foo" to "1", "bar" to "2")
                     ),
                     "integer-with-named-values-and-custom-item-name-input" to ApiItem(
                         type = "integer",
                         name = "SomeItemName",
-                        namedValues = mapOf("foo" to 1, "bar" to 2)
+                        namedValues = mapOf("foo" to "1", "bar" to "2")
                     ),
                     "float-input" to ApiItem(type = "float"),
                 ),
@@ -481,11 +481,11 @@ class ManifestValidationTest : FunSpec({
             // given
             val manifest = TypesManifest(
                 inputs = mapOf(
-                    "string-input" to ApiItem(type = "string", namedValues = mapOf("foo" to 1)),
-                    "boolean-input" to ApiItem(type = "boolean", namedValues = mapOf("foo" to 1)),
-                    "float-input" to ApiItem(type = "float", namedValues = mapOf("foo" to 1)),
-                    "list-input" to ApiItem(type = "list", separator = ",", listItem = ApiItem(type = "string"), namedValues = mapOf("foo" to 1)),
-                    "enum-input" to ApiItem(type = "enum", allowedValues = listOf("foo", "bar"), namedValues = mapOf("foo" to 1)),
+                    "string-input" to ApiItem(type = "string", namedValues = mapOf("foo" to "1")),
+                    "boolean-input" to ApiItem(type = "boolean", namedValues = mapOf("foo" to "1")),
+                    "float-input" to ApiItem(type = "float", namedValues = mapOf("foo" to "1")),
+                    "list-input" to ApiItem(type = "list", separator = ",", listItem = ApiItem(type = "string"), namedValues = mapOf("foo" to "1")),
+                    "enum-input" to ApiItem(type = "enum", allowedValues = listOf("foo", "bar"), namedValues = mapOf("foo" to "1")),
                 ),
             )