resolved comments

Author: kashifkhan0771

structs/structs.go

@@ -24,7 +24,11 @@ func CompareStructs(old, new interface{}) ([]Result, error) {
 	oldValue := reflect.ValueOf(old)
 	newValue := reflect.ValueOf(new)
 
-	var comparedResults = make([]Result, 0)
+	if oldValue.Kind() != reflect.Struct || newValue.Kind() != reflect.Struct {
+		return nil, fmt.Errorf("both parameters should be struct")
+	}
+
+	var comparedResults = make([]Result, 0, oldValue.NumField())
 
 	for i := 0; i < oldValue.NumField(); i++ {
 		field := oldValue.Type().Field(i)

structs/structs_test.go

@@ -72,6 +72,13 @@ func TestCompareStructs(t *testing.T) {
 			},
 			wantErr: false,
 		},
+		{
+			name:    "fail - non struct parameters",
+			old:     map[string]string{"test": "example"},
+			new:     map[string]string{"test": "example-updated"},
+			want:    nil,
+			wantErr: true,
+		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {