reduce sonarcloud issues

Signed-off-by: Atif Ali <[email protected]>

Author: aali309

pkg/sync/sync_context_test.go

@@ -2230,29 +2230,29 @@ Forbidden: updates to statefulset spec for fields other than 'replicas', 'ordina
 		{
 			name: "multiple volumeClaimTemplate change",
 			currentSpec: map[string]interface{}{
-				"serviceName": "postgresql-svc",
+				"serviceName": postgresqlSvc,
 				"selector": map[string]interface{}{
 					"matchLabels": map[string]interface{}{
 						"app": "postgresql",
 					},
 				},
 				"volumeClaimTemplates": []interface{}{
-					templateWithStorage("static-files", "1Gi"),
-					templateWithStorage("dexconfig", "1Gi"),
-					templateWithStorage("argocd-dex-server-tls", "1Gi"),
+					templateWithStorage(staticFiles, "1Gi"),
+					templateWithStorage(dexconfig, "1Gi"),
+					templateWithStorage(argocdDexServerTLS, "1Gi"),
 				},
 			},
 			desiredSpec: map[string]interface{}{
-				"serviceName": "postgresql-svc",
+				"serviceName": postgresqlSvc,
 				"selector": map[string]interface{}{
 					"matchLabels": map[string]interface{}{
 						"app": "postgresql",
 					},
 				},
 				"volumeClaimTemplates": []interface{}{
-					templateWithStorage("static-files", "2Gi"),
-					templateWithStorage("dexconfig", "3Gi"),
-					templateWithStorage("argocd-dex-server-tls", "4Gi"),
+					templateWithStorage(staticFiles, "2Gi"),
+					templateWithStorage(dexconfig, "3Gi"),
+					templateWithStorage(argocdDexServerTLS, "4Gi"),
 				},
 			},
 			expectedMessage: `attempting to change immutable fields:
@@ -2271,16 +2271,16 @@ Forbidden: updates to statefulset spec for fields other than 'replicas', 'ordina
 		{
 			name: "multiple field changes",
 			currentSpec: map[string]interface{}{
-				"serviceName": "postgresql-svc",
+				"serviceName": postgresqlSvc,
 				"selector": map[string]interface{}{
 					"matchLabels": map[string]interface{}{
 						"app": "postgresql",
 					},
 				},
 				"volumeClaimTemplates": []interface{}{
-					templateWithStorage("static-files", "1Gi"),
-					templateWithStorage("dexconfig", "1Gi"),
-					templateWithStorage("argocd-dex-server-tls", "1Gi"),
+					templateWithStorage(staticFiles, "1Gi"),
+					templateWithStorage(dexconfig, "1Gi"),
+					templateWithStorage(argocdDexServerTLS, "1Gi"),
 				},
 			},
 			desiredSpec: map[string]interface{}{
@@ -2291,9 +2291,9 @@ Forbidden: updates to statefulset spec for fields other than 'replicas', 'ordina
 					},
 				},
 				"volumeClaimTemplates": []interface{}{
-					templateWithStorage("static-files", "2Gi"),
-					templateWithStorage("dexconfig", "1Gi"),
-					templateWithStorage("argocd-dex-server-tls", "1Gi"),
+					templateWithStorage(staticFiles, "2Gi"),
+					templateWithStorage(dexconfig, "1Gi"),
+					templateWithStorage(argocdDexServerTLS, "1Gi"),
 				},
 			},
 			expectedMessage: `attempting to change immutable fields: