Fix unit tests for yaml merge functions (#1204)

common/utils/fixtures/tags_a.yaml

@@ -1,2 +1 @@
-tags:
-  - a
+tags: [a]

common/utils/fixtures/tags_ab.yaml

@@ -1,3 +1,3 @@
 tags:
   - a
-  - b
+  - b

common/utils/fixtures/tags_b.yaml

@@ -1,2 +1,2 @@
 tags:
-  - b
+  - b

common/utils/yaml_test.go

@@ -68,7 +68,6 @@ func TestMergeYAMLWithSlices(t *testing.T) {
 		"slices value, merge slices":             {oldValues: tagsA, newValues: tagsB, expectedResult: tagsAB, expectError: false},
 		"slices value inverted, merge slices":    {oldValues: tagsB, newValues: tagsA, expectedResult: tagsAB, expectError: false},
 	}
-
 	for name, tc := range tests {
 		t.Run(name, func(t *testing.T) {
 			got, err := MergeYAMLWithSlices(tc.oldValues, tc.newValues)
@@ -80,10 +79,25 @@ func TestMergeYAMLWithSlices(t *testing.T) {
 
 			var gotYAML map[string]interface{}
 			var expectedYAML map[string]interface{}
-
-			gotMap := yaml.Unmarshal([]byte(got), &gotYAML)
-			expectedMap := yaml.Unmarshal([]byte(tc.expectedResult), &expectedYAML)
-			assert.Equal(t, gotMap, expectedMap, "expected %v, got %v", expectedMap, gotMap)
+			err = yaml.Unmarshal([]byte(got), &gotYAML)
+			require.NoError(t, err, "unexpected error: %v", err)
+			err = yaml.Unmarshal([]byte(tc.expectedResult), &expectedYAML)
+			require.NoError(t, err, "unexpected error: %v", err)
+			assertMapsEqual(t, expectedYAML, gotYAML)
 		})
 	}
 }
+
+func assertMapsEqual(t *testing.T, expected, actual map[string]interface{}) {
+	assert.Equal(t, len(expected), len(actual), "expected %v, got %v", expected, actual)
+	for k, v := range expected {
+		switch v.(type) {
+		case map[string]interface{}:
+			assertMapsEqual(t, v.(map[string]interface{}), actual[k].(map[string]interface{}))
+		case []interface{}:
+			assert.ElementsMatch(t, v.([]interface{}), actual[k].([]interface{}), "expected %v, got %v", v, actual[k])
+		default:
+			assert.Equal(t, v, actual[k], "expected %v, got %v", v, actual[k])
+		}
+	}
+}