fix: generalize Azure AD comment per code review

dlevy-msft-sql · dlevy-msft-sql · commit bb3bb71692b9 · 2026-01-27T18:55:11.000-06:00
diff --git a/cmd/modern/e2e_test.go b/cmd/modern/e2e_test.go
@@ -73,7 +73,7 @@ func hasLiveConnection() bool {
 }
 
 // hasSQLAuthCredentials returns true if SQL authentication credentials are available.
-// For Azure AD/Entra authentication (service principal), we need different handling.
+// For Azure AD/Entra authentication methods, we need different handling.
 func hasSQLAuthCredentials() bool {
 	return os.Getenv("SQLCMDUSER") != "" && os.Getenv("SQLCMDPASSWORD") != ""
 }

Original file line number	Diff line number	Diff line change
`@@ -73,7 +73,7 @@ func hasLiveConnection() bool {`
`73`	`73`	`}`
`74`	`74`
`75`	`75`	`// hasSQLAuthCredentials returns true if SQL authentication credentials are available.`
`76`		`-// For Azure AD/Entra authentication (service principal), we need different handling.`
	`76`	`+// For Azure AD/Entra authentication methods, we need different handling.`
`77`	`77`	`func hasSQLAuthCredentials() bool {`
`78`	`78`	`return os.Getenv("SQLCMDUSER") != "" && os.Getenv("SQLCMDPASSWORD") != ""`
`79`	`79`	`}`