Skip to content

Commit 34bded6

Browse files
committed
fix(testutil) errcheck linter
1 parent c7223fd commit 34bded6

1 file changed

Lines changed: 14 additions & 4 deletions

File tree

stackit/internal/testutil/testutil_test.go

Lines changed: 14 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -151,8 +151,13 @@ func TestConfigBuilderProviderConfig(t *testing.T) {
151151
}
152152

153153
func TestConfigBuilderProviderConfigEnvVar(t *testing.T) {
154-
os.Setenv(CdnCustomEndpoint.envVarName, "http://expected.example.com")
155-
defer os.Unsetenv(CdnCustomEndpoint.envVarName)
154+
os.Setenv(CdnCustomEndpoint.envVarName, "http://expected.example.com") // nolint:errcheck // test would fail
155+
defer func() {
156+
err := os.Unsetenv(CdnCustomEndpoint.envVarName)
157+
if err != nil {
158+
t.Fatalf("unset env: %v", err)
159+
}
160+
}()
156161
got := NewConfigBuilder().BuildProviderConfig()
157162
want := `provider "stackit" {
158163
default_region = "eu01"
@@ -228,8 +233,13 @@ func TestConfigBuilderClientOptions(t *testing.T) {
228233
}
229234

230235
func TestConfigBuilderClientOptionsEnvVar(t *testing.T) {
231-
os.Setenv(CdnCustomEndpoint.envVarName, "http://cdn.example.com")
232-
defer os.Unsetenv(CdnCustomEndpoint.envVarName)
236+
os.Setenv(CdnCustomEndpoint.envVarName, "http://cdn.example.com") // nolint:errcheck // test would fail
237+
defer func() {
238+
err := os.Unsetenv(CdnCustomEndpoint.envVarName)
239+
if err != nil {
240+
t.Fatalf("unset env: %v", err)
241+
}
242+
}()
233243
opts := NewConfigBuilder().BuildClientOptions(CdnCustomEndpoint)
234244
got := sdkConf.Configuration{}
235245
for _, opt := range opts {

0 commit comments

Comments
 (0)