Add check for AZURE_FEDERATED_TOKEN_FILE environment variable in Velero deployment test

kaovilai · kaovilai · commit 835df129ff2b · 2025-07-31T17:00:34.000-04:00
Signed-off-by: Tiger Kaovilai &lt;tkaovila@redhat.com&gt;
diff --git a/internal/controller/velero_test.go b/internal/controller/velero_test.go
@@ -2579,6 +2579,9 @@ func TestDPAReconciler_buildVeleroDeploymentWithAzureWorkloadIdentity(t *testing
 				if !foundClientIDEnvVar {
 					t.Errorf("Expected AZURE_CLIENT_ID environment variable to be set")
 				}
+				if !foundTokenFileEnvVar {
+					t.Errorf("Expected AZURE_FEDERATED_TOKEN_FILE environment variable to be set")
+				}
 			} else {
 			}
 		})
diff --git a/pkg/credentials/stsflow/stsflow.go b/pkg/credentials/stsflow/stsflow.go
@@ -385,4 +385,3 @@ func GetGCPCredentialsFromSecret(clientset kubernetes.Interface, namespace strin
 
 	return string(serviceAccountJSON), nil
 }
-

Original file line number	Diff line number	Diff line change
`@@ -2579,6 +2579,9 @@ func TestDPAReconciler_buildVeleroDeploymentWithAzureWorkloadIdentity(t *testing`
`2579`	`2579`	`if !foundClientIDEnvVar {`
`2580`	`2580`	`t.Errorf("Expected AZURE_CLIENT_ID environment variable to be set")`
`2581`	`2581`	`}`
	`2582`	`+ if !foundTokenFileEnvVar {`
	`2583`	`+ t.Errorf("Expected AZURE_FEDERATED_TOKEN_FILE environment variable to be set")`
	`2584`	`+ }`
`2582`	`2585`	`} else {`
`2583`	`2586`	`}`
`2584`	`2587`	`})`
Original file line number	Diff line number	Diff line change
`@@ -385,4 +385,3 @@ func GetGCPCredentialsFromSecret(clientset kubernetes.Interface, namespace strin`
`385`	`385`
`386`	`386`	`return string(serviceAccountJSON), nil`
`387`	`387`	`}`
`388`		`-`