Skip to content

Commit 7bc1460

Browse files
committed
fix: Ensure terraform tests have a cache path and logger
1 parent 54547a4 commit 7bc1460

File tree

2 files changed

+9
-2
lines changed

2 files changed

+9
-2
lines changed

provisioner/terraform/parse_test.go

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,9 @@ import (
1212
"github.com/stretchr/testify/assert"
1313
"github.com/stretchr/testify/require"
1414

15+
"cdr.dev/slog"
16+
"cdr.dev/slog/sloggers/slogtest"
17+
1518
"github.com/coder/coder/provisioner/terraform"
1619
"github.com/coder/coder/provisionersdk"
1720
"github.com/coder/coder/provisionersdk/proto"
@@ -33,6 +36,8 @@ func TestParse(t *testing.T) {
3336
ServeOptions: &provisionersdk.ServeOptions{
3437
Listener: server,
3538
},
39+
CachePath: t.TempDir(),
40+
Logger: slogtest.Make(t, nil).Leveled(slog.LevelDebug),
3641
})
3742
assert.NoError(t, err)
3843
}()
@@ -175,7 +180,8 @@ func TestParse(t *testing.T) {
175180
DefaultDestination: &proto.ParameterDestination{
176181
Scheme: proto.ParameterDestination_PROVISIONER_VARIABLE,
177182
},
178-
}},
183+
},
184+
},
179185
},
180186
},
181187
},

provisioner/terraform/provision_test.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,8 @@ func setupProvisioner(t *testing.T) (context.Context, proto.DRPCProvisionerClien
3535
ServeOptions: &provisionersdk.ServeOptions{
3636
Listener: server,
3737
},
38-
Logger: slogtest.Make(t, nil).Leveled(slog.LevelDebug),
38+
CachePath: t.TempDir(),
39+
Logger: slogtest.Make(t, nil).Leveled(slog.LevelDebug),
3940
})
4041
assert.NoError(t, err)
4142
}()

0 commit comments

Comments
 (0)