Merge pull request #80 from hauner/#78

fixed default handling for non String types

Author: hauner

src/main/groovy/com/github/hauner/openapi/spring/writer/MethodWriter.groovy

@@ -147,17 +147,12 @@ class MethodWriter {
         ps.join (', ')
     }
 
-    private String quote (String content) {
-        '"' + content + '"'
+    private String getDefault (Parameter parameter) {
+        quote(parameter.constraints.default as String)
     }
 
-    private def getDefault(Parameter parameter) {
-        def value = parameter.constraints.default
-        if (value instanceof String) {
-            quote(value)
-        } else {
-            value
-        }
+    private String quote (String content) {
+        '"' + content + '"'
     }
 
 }

src/test/groovy/com/github/hauner/openapi/spring/writer/MethodWriterSpec.groovy

@@ -389,7 +389,7 @@ class MethodWriterSpec extends Specification {
 """
     }
 
-    void "writes simple (optional) parameter with default value" () {
+    void "writes simple (optional) parameter with string default value" () {
         def endpoint = new Endpoint (path: '/foo', method: HttpMethod.GET, responses: [
             '204': [new Response (responseType: new NoneDataType())]
         ], parameters: [
@@ -408,6 +408,25 @@ class MethodWriterSpec extends Specification {
 """
     }
 
+    void "writes simple (optional) parameter with number default value" () {
+        def endpoint = new Endpoint (path: '/foo', method: HttpMethod.GET, responses: [
+            '204': [new Response (responseType: new NoneDataType())]
+        ], parameters: [
+            new QueryParameter(name: 'foo', required: false,
+                dataType: new LongDataType (
+                    constraints: new DataTypeConstraints (defaultValue: 5)))
+        ])
+
+        when:
+        writer.write (target, endpoint)
+
+        then:
+        target.toString () == """\
+    @GetMapping(path = "${endpoint.path}")
+    ResponseEntity<Void> getFoo(@RequestParam(name = "foo", required = false, defaultValue = "5") Long foo);
+"""
+    }
+
     void "writes mapping annotation with multiple result content types" () {
         def endpoint = new Endpoint (path: '/foo', method: HttpMethod.GET, responses: [
             '200'    : [