Fix conversion of first-level Iterable[T] (#121)

Co-authored-by: Marc Liechti <[email protected]>

Author: idlefella

src/main/scala/com/github/swagger/scala/converter/SwaggerScalaModelConverter.scala

@@ -57,16 +57,8 @@ class SwaggerScalaModelConverter extends ModelResolver(SwaggerScalaModelConverte
           case Some(property) => {
             if (isIterable(cls)) {
               //untidy solution for https://github.com/swagger-akka-http/swagger-scala-module/issues/48
-              val required = property.getRequired
-              if (required != null) {
-                required.remove("traversableAgain")
-                required.remove("empty")
-              }
-              val props = property.getProperties
-              if (props != null) {
-                props.remove("traversableAgain")
-                props.remove("empty")
-              }
+              property.setRequired(null)
+              property.setProperties(null)
             }
             setRequired(`type`)
             property

src/test/scala/com/github/swagger/scala/converter/ModelPropertyParserTest.scala

@@ -366,6 +366,15 @@ class ModelPropertyParserTest extends AnyFlatSpec with Matchers with OptionValue
     nullSafeList(model.value.getRequired) shouldEqual Seq("listOfStrings")
   }
 
+  it should "process scala Iterable[T] classes" in {
+    val converter = ModelConverters.getInstance()
+    val schemas = converter.readAll(classOf[Seq[String]]).asScala.toMap
+    val model = findModel(schemas, "Seq")
+    model should be(defined)
+    model.value.getProperties should be(null)
+    model.value.getRequired should be(null)
+  }
+
   it should "process Array-Model with forced required Scala Option Seq" in {
     val converter = ModelConverters.getInstance()
     val schemas = converter.readAll(classOf[ModelWOptionStringSeq]).asScala.toMap