Skip to content

Commit bc6c44a

Browse files
committed
consistent api
1 parent 11612d1 commit bc6c44a

File tree

3 files changed

+9
-9
lines changed

3 files changed

+9
-9
lines changed

coderd/database/dbfake/dbfake.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -38,15 +38,15 @@ func Workspace(t testing.TB, db database.Store, seed database.Workspace) databas
3838
return dbgen.Workspace(t, db, seed)
3939
}
4040

41-
func TemplateWithVersion(t testing.TB, db database.Store, tpl database.Template, tv database.TemplateVersion, resources ...*proto.Resource) (database.Template, database.TemplateVersion) {
41+
func TemplateWithVersion(t testing.TB, db database.Store, tpl database.Template, tv database.TemplateVersion, job database.ProvisionerJob, resources ...*proto.Resource) (database.Template, database.TemplateVersion) {
4242
t.Helper()
4343

4444
template := dbgen.Template(t, db, tpl)
4545

4646
tv.TemplateID = dbgen.TakeFirst(tv.TemplateID, uuid.NullUUID{UUID: template.ID, Valid: true})
4747
tv.OrganizationID = dbgen.TakeFirst(tv.OrganizationID, template.OrganizationID)
4848
tv.CreatedBy = dbgen.TakeFirst(tv.CreatedBy, template.CreatedBy)
49-
version := TemplateVersion(t, db, tv, database.ProvisionerJob{}, resources...)
49+
version := TemplateVersion(t, db, tv, job, resources...)
5050

5151
err := db.UpdateTemplateActiveVersionByID(dbgen.Ctx, database.UpdateTemplateActiveVersionByIDParams{
5252
ID: template.ID,
@@ -79,10 +79,10 @@ func TemplateVersion(t testing.TB, db database.Store, tv database.TemplateVersio
7979
return templateVersion
8080
}
8181

82-
func TemplateVersionWithParams(t testing.TB, db database.Store, tv database.TemplateVersion, params []database.TemplateVersionParameter) (database.TemplateVersion, []database.TemplateVersionParameter) {
82+
func TemplateVersionWithParams(t testing.TB, db database.Store, tv database.TemplateVersion, job database.ProvisionerJob, params []database.TemplateVersionParameter) (database.TemplateVersion, []database.TemplateVersionParameter) {
8383
t.Helper()
8484

85-
version := TemplateVersion(t, db, tv, database.ProvisionerJob{})
85+
version := TemplateVersion(t, db, tv, job)
8686
tvps := make([]database.TemplateVersionParameter, 0, len(params))
8787

8888
for _, param := range params {

coderd/workspaces_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -350,7 +350,7 @@ func TestResolveAutostart(t *testing.T) {
350350
template, version1 := dbfake.TemplateWithVersion(t, db, database.Template{
351351
CreatedBy: owner.UserID,
352352
OrganizationID: owner.OrganizationID,
353-
}, database.TemplateVersion{})
353+
}, database.TemplateVersion{}, database.ProvisionerJob{})
354354

355355
params := []database.TemplateVersionParameter{{
356356
Name: "param",
@@ -361,7 +361,7 @@ func TestResolveAutostart(t *testing.T) {
361361
TemplateID: uuid.NullUUID{UUID: template.ID, Valid: true},
362362
OrganizationID: template.OrganizationID,
363363
CreatedBy: owner.UserID,
364-
}, []database.TemplateVersionParameter{{
364+
}, database.ProvisionerJob{}, []database.TemplateVersionParameter{{
365365
Name: "param",
366366
DefaultValue: "",
367367
Required: true,
@@ -411,7 +411,7 @@ func TestResolveAutostart(t *testing.T) {
411411
TemplateID: uuid.NullUUID{UUID: template.ID, Valid: true},
412412
OrganizationID: owner.OrganizationID,
413413
CreatedBy: owner.UserID,
414-
}, params)
414+
}, database.ProvisionerJob{}, params)
415415

416416
err = ownerClient.UpdateActiveTemplateVersion(ctx, template.ID, codersdk.UpdateActiveTemplateVersion{
417417
ID: version3.ID,

enterprise/coderd/workspaces_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1090,7 +1090,7 @@ func TestResolveAutostart(t *testing.T) {
10901090
template, version1 := dbfake.TemplateWithVersion(t, db, database.Template{
10911091
CreatedBy: owner.UserID,
10921092
OrganizationID: owner.OrganizationID,
1093-
}, database.TemplateVersion{})
1093+
}, database.TemplateVersion{}, database.ProvisionerJob{})
10941094

10951095
ctx, cancel := context.WithTimeout(context.Background(), testutil.WaitLong)
10961096
defer cancel()
@@ -1104,7 +1104,7 @@ func TestResolveAutostart(t *testing.T) {
11041104
TemplateID: uuid.NullUUID{UUID: template.ID, Valid: true},
11051105
OrganizationID: template.OrganizationID,
11061106
CreatedBy: owner.UserID,
1107-
}, []database.TemplateVersionParameter{{
1107+
}, database.ProvisionerJob{}, []database.TemplateVersionParameter{{
11081108
Name: "param",
11091109
DefaultValue: "",
11101110
Required: true,

0 commit comments

Comments
 (0)