Fix linting issues

Author: padiazg

build.go

@@ -158,7 +158,7 @@ func makeAllPathsMap(paths *Paths) pathsMap {
 		pathMap[keySecurity] = makeSecurityMap(&path.Security)
 		pathMap[keyRequestBody] = makeRequestBodyMap(&path.RequestBody)
 		pathMap[keyResponses] = makeResponsesMap(&path.Responses)
-		pathMap[keyParameters] = makeParametersMap(&path.Parameters)
+		pathMap[keyParameters] = makeParametersMap(path.Parameters)
 
 		allPaths[path.Route][strings.ToLower(path.HTTPMethod)] = pathMap
 	}
@@ -348,11 +348,14 @@ func isStrEmpty(s string) bool {
 	return s == emptyStr
 }
 
-func makeParametersMap(parameters *Parameters) []map[string]interface{} {
+func makeParametersMap(parameters Parameters) []map[string]interface{} {
 	parametersMap := []map[string]interface{}{}
 
-	for _, param := range *parameters {
-		paramMap := make(map[string]interface{})
+	for i := 0; i < len(parameters); i++ {
+		var (
+			param    = &parameters[i]
+			paramMap = make(map[string]interface{})
+		)
 
 		paramMap[keyName] = param.Name
 		paramMap[keyIn] = param.In

build_test.go

@@ -232,9 +232,12 @@ func TestOAS_BuildStream(t *testing.T) {
 }
 
 func Test_makeParametersMap(t *testing.T) {
+	t.Parallel()
+
 	type args struct {
-		parameters *Parameters
+		parameters Parameters
 	}
+
 	tests := []struct {
 		name string
 		args args
@@ -243,7 +246,7 @@ func Test_makeParametersMap(t *testing.T) {
 		{
 			name: "success-minimal",
 			args: args{
-				parameters: &Parameters{{
+				parameters: Parameters{{
 					Name:        "id",
 					In:          "path",
 					Description: "test",
@@ -262,7 +265,7 @@ func Test_makeParametersMap(t *testing.T) {
 		{
 			name: "success-full",
 			args: args{
-				parameters: &Parameters{{
+				parameters: Parameters{{
 					Name:        "id",
 					In:          "path",
 					Description: "test",
@@ -279,16 +282,19 @@ func Test_makeParametersMap(t *testing.T) {
 				"in":          "path",
 				"description": "test",
 				"required":    true,
-				"schema": map[string]interface{}{"name": "id", "type": "integer",
+				"schema": map[string]interface{}{
+					"name": "id",
+					"type": "integer",
 					"properties": map[string]interface{}{
 						"id": map[string]interface{}{"type": "integer"},
-					}},
+					},
+				},
 			}},
 		},
 		{
 			name: "success-ref",
 			args: args{
-				parameters: &Parameters{{
+				parameters: Parameters{{
 					Name:        "id",
 					In:          "path",
 					Description: "test",
@@ -307,7 +313,7 @@ func Test_makeParametersMap(t *testing.T) {
 		{
 			name: "success-xml-entry",
 			args: args{
-				parameters: &Parameters{{
+				parameters: Parameters{{
 					Name:        "id",
 					In:          "path",
 					Description: "test",
@@ -325,9 +331,13 @@ func Test_makeParametersMap(t *testing.T) {
 		},
 	}
 	for _, tt := range tests {
-		t.Run(tt.name, func(t *testing.T) {
-			if got := makeParametersMap(tt.args.parameters); !reflect.DeepEqual(got, tt.want) {
-				t.Errorf("makeParametersMap() = %+v, want %+v", got, tt.want)
+		trn := tt
+
+		t.Run(trn.name, func(t *testing.T) {
+			t.Parallel()
+
+			if got := makeParametersMap(trn.args.parameters); !reflect.DeepEqual(got, trn.want) {
+				t.Errorf("makeParametersMap() = %+v, want %+v", got, trn.want)
 			}
 		})
 	}

models.go

@@ -211,12 +211,15 @@ type SecurityScope struct {
 	Description string `yaml:"description,omitempty"`
 }
 
-// Parameters
+// Parameters is a slice of Parameter objects.
 type Parameters []Parameter
 
+// Parameter represents OAS parameter object.
 type Parameter struct {
-	// If in is "path", the name field MUST correspond to a template expression occurring within the path field in the Paths Object. See Path Templating for further information.
-	// If in is "header" and the name field is "Accept", "Content-Type" or "Authorization", the parameter definition SHALL be ignored.
+	// If in is "path", the name field MUST correspond to a template expression occurring within
+	// the path field in the Paths Object. See Path Templating for further information.
+	// If in is "header" and the name field is "Accept", "Content-Type" or "Authorization",
+	//  the parameter definition SHALL be ignored.
 	// For all other cases, the name corresponds to the parameter name used by the in property.
 	Name            string `yaml:"name,omitempty"`
 	In              string `yaml:"in,omitempty"` // "query", "header", "path" or "cookie".