fix invalid response type: array of strings (swaggo#173)

* fix invalid response type: array of strings

* fix lint

operation.go

@@ -443,14 +443,24 @@ func (operation *Operation) ParseResponseComment(commentLine string) error {
 	}
 
 	if schemaType == "array" {
-		response.Schema.Items = &spec.SchemaOrArray{
-			Schema: &spec.Schema{
-				SchemaProps: spec.SchemaProps{
-					Ref: spec.Ref{Ref: jsonreference.MustCreateRef("#/definitions/" + refType)},
+		refType = TransToValidSchemeType(refType)
+		if IsPrimitiveType(refType) {
+			response.Schema.Items = &spec.SchemaOrArray{
+				Schema: &spec.Schema{
+					SchemaProps: spec.SchemaProps{
+						Type: spec.StringOrArray{refType},
+					},
 				},
-			},
+			}
+		} else {
+			response.Schema.Items = &spec.SchemaOrArray{
+				Schema: &spec.Schema{
+					SchemaProps: spec.SchemaProps{
+						Ref: spec.Ref{Ref: jsonreference.MustCreateRef("#/definitions/" + refType)},
+					},
+				},
+			}
 		}
-
 	}
 
 	if operation.Responses == nil {

parser_test.go

@@ -188,6 +188,15 @@ func TestParseSimpleApi(t *testing.T) {
                             "$ref": "#/definitions/web.APIError"
                         }
                     },
+                    "401": {
+                        "description": "Unauthorized",
+                        "schema": {
+                            "type": "array",
+                            "items": {
+                                "type": "string"
+                            }
+                        }
+                    },
                     "404": {
                         "description": "Can not find ID",
                         "schema": {

schema.go

@@ -4,11 +4,18 @@ import "fmt"
 
 // CheckSchemaType TODO: NEEDS COMMENT INFO
 func CheckSchemaType(typeName string) {
+	if !IsPrimitiveType(typeName) {
+		panic(fmt.Errorf("%s is not basic types", typeName))
+	}
+}
+
+// IsPrimitiveType determine whether the type name is a primitive type
+func IsPrimitiveType(typeName string) bool {
 	switch typeName {
 	case "string", "number", "integer", "boolean", "array", "object":
-
+		return true
 	default:
-		panic(fmt.Errorf("%s is not basic types", typeName))
+		return false
 	}
 }
 

testdata/simple/api/api.go

@@ -50,6 +50,7 @@ func GetStructArrayByString(c *gin.Context) {
 // @Param   file formData file true  "this is a test file"
 // @Success 200 {string} string "ok"
 // @Failure 400 {object} web.APIError "We need ID!!"
+// @Failure 401 {array} string
 // @Failure 404 {object} web.APIError "Can not find ID"
 // @Router /file/upload [post]
 func Upload(ctx *gin.Context) {