Required field in Schema annotation ignored in Kotlin. Fixes #2021.

bnasslahsen · bnasslahsen · commit 039fb60ce21c · 2023-02-04T16:38:28.000+01:00
diff --git a/springdoc-openapi-kotlin/src/main/java/org/springdoc/kotlin/SpringDocKotlinConfiguration.kt b/springdoc-openapi-kotlin/src/main/java/org/springdoc/kotlin/SpringDocKotlinConfiguration.kt
@@ -40,8 +40,8 @@ class SpringDocKotlinConfiguration(objectMapperProvider: ObjectMapperProvider) {
 			.addRequestWrapperToIgnore(Continuation::class.java)
 			.replaceWithSchema(ByteArray::class.java, ByteArraySchema())
 			.addDeprecatedType(Deprecated::class.java)
-		objectMapperProvider.jsonMapper().registerModule(SpringDocRequiredModule())
 		objectMapperProvider.jsonMapper().registerModule(KotlinModule.Builder().build())
+		objectMapperProvider.jsonMapper().registerModule(SpringDocRequiredModule())
 	}
 
 	/**
diff --git a/springdoc-openapi-kotlin/src/test/resources/results/app9.json b/springdoc-openapi-kotlin/src/test/resources/results/app9.json
@@ -45,9 +45,6 @@
   "components": {
     "schemas": {
       "DemoRequest": {
-        "required": [
-          "nonNullableWithDefault"
-        ],
         "type": "object",
         "properties": {
           "nonNullableWithDefault": {

Original file line number	Diff line number	Diff line change
`@@ -40,8 +40,8 @@ class SpringDocKotlinConfiguration(objectMapperProvider: ObjectMapperProvider) {`
`40`	`40`	`.addRequestWrapperToIgnore(Continuation::class.java)`
`41`	`41`	`.replaceWithSchema(ByteArray::class.java, ByteArraySchema())`
`42`	`42`	`.addDeprecatedType(Deprecated::class.java)`
`43`		`- objectMapperProvider.jsonMapper().registerModule(SpringDocRequiredModule())`
`44`	`43`	`objectMapperProvider.jsonMapper().registerModule(KotlinModule.Builder().build())`
	`44`	`+ objectMapperProvider.jsonMapper().registerModule(SpringDocRequiredModule())`
`45`	`45`	`}`
`46`	`46`
`47`	`47`	`/**`