ignore unknown formats

Author: hauner

src/main/kotlin/io/openapiprocessor/core/converter/DataTypeConverter.kt

@@ -213,7 +213,7 @@ class DataTypeConverter(
         }
 
         var typeFormat = schemaInfo.getType()
-        if (schemaInfo.getFormat() != null) {
+        if (isSupportedFormat(schemaInfo.getFormat())) {
             typeFormat += ":" + schemaInfo.getFormat()
         }
 
@@ -254,6 +254,19 @@ class DataTypeConverter(
         }
     }
 
+    private fun isSupportedFormat(format: String?): Boolean {
+        if(format == null)
+            return false
+
+        return format in listOf(
+            "int32",
+            "int64",
+            "float",
+            "double",
+            "date",
+            "date-time")
+    }
+
     private fun createStringDataType(info: SchemaInfo, constraints: DataTypeConstraints, dataTypes: DataTypes): DataType {
         if (!info.isEnum()) {
             return StringDataType(constraints, info.getDeprecated())

src/test/kotlin/io/openapiprocessor/core/converter/DataTypeConverterPrimitiveSpec.kt

@@ -0,0 +1,48 @@
+/*
+ * Copyright © 2020 https://github.com/openapi-processor/openapi-processor-core
+ * PDX-License-Identifier: Apache-2.0
+ */
+
+package io.openapiprocessor.core.converter
+
+import io.kotest.core.spec.IsolationMode
+import io.kotest.core.spec.style.StringSpec
+import io.kotest.data.forAll
+import io.kotest.data.row
+import io.kotest.matchers.shouldBe
+import io.mockk.every
+import io.mockk.mockk
+import io.openapiprocessor.core.converter.mapping.MappingFinder
+import io.openapiprocessor.core.model.DataTypes
+import io.openapiprocessor.core.parser.RefResolver
+import io.openapiprocessor.core.parser.Schema
+
+class DataTypeConverterPrimitiveSpec: StringSpec({
+    isolationMode = IsolationMode.InstancePerTest
+
+    val converter = DataTypeConverter(ApiOptions(), MappingFinder())
+    val resolver = mockk<RefResolver>()
+
+    "ignores unknown primitive data type format" {
+        forAll(
+            row("string","unknown", "String"),
+            row("integer","unknown", "Integer"),
+            row("number","unknown", "Float"),
+            row("boolean","unknown", "Boolean")
+        ) { type, format, dataTypeName ->
+            val schema = mockk<Schema>(relaxed = true)
+            every { schema.getRef() } returns null
+            every { schema.getType() } returns type
+            every { schema.getFormat() } returns format
+
+            // when:
+            val info = SchemaInfo("", "foo", schema = schema, resolver = resolver)
+            val datatype = converter.convert(info, DataTypes())
+
+            // then:
+            datatype.getName() shouldBe dataTypeName
+        }
+
+    }
+
+})