fix: test set environment without have returns

matttrach · matttrach · commit 73a2d97214c9 · 2025-11-12T20:49:15.000-06:00
Signed-off-by: matttrach &lt;matt.trachier@suse.com&gt;
diff --git a/test/tests/util.go b/test/tests/util.go
@@ -323,10 +323,7 @@ func GetRetryableTerraformErrors() map[string]string {
 func SetAcmeServer(t *testing.T) string {
 	acmeserver := os.Getenv("ACME_SERVER_URL")
 	if acmeserver == "" {
-		err := t.Setenv("ACME_SERVER_URL", "https://acme-staging-v02.api.letsencrypt.org/directory")
-		if err != nil {
-			t.Logf("Error setting environment variable: %v", err)
-		}
+		t.Setenv("ACME_SERVER_URL", "https://acme-staging-v02.api.letsencrypt.org/directory")
 	}
 	return acmeserver
 }

Original file line number	Diff line number	Diff line change
`@@ -323,10 +323,7 @@ func GetRetryableTerraformErrors() map[string]string {`
`323`	`323`	`func SetAcmeServer(t *testing.T) string {`
`324`	`324`	`acmeserver := os.Getenv("ACME_SERVER_URL")`
`325`	`325`	`if acmeserver == "" {`
`326`		`- err := t.Setenv("ACME_SERVER_URL", "https://acme-staging-v02.api.letsencrypt.org/directory")`
`327`		`- if err != nil {`
`328`		`- t.Logf("Error setting environment variable: %v", err)`
`329`		`- }`
	`326`	`+ t.Setenv("ACME_SERVER_URL", "https://acme-staging-v02.api.letsencrypt.org/directory")`
`330`	`327`	`}`
`331`	`328`	`return acmeserver`
`332`	`329`	`}`