Skip to content

Commit 61e86f6

Browse files
committed
make more use of dbgen
1 parent d09b757 commit 61e86f6

File tree

3 files changed

+15
-9
lines changed

3 files changed

+15
-9
lines changed

coderd/database/dbauthz/dbauthz_test.go

+1-2
Original file line numberDiff line numberDiff line change
@@ -960,8 +960,7 @@ func (s *MethodTestSuite) TestOrganization() {
960960
TemplateVersionID: workspaceBuild.TemplateVersionID,
961961
Name: "test",
962962
}
963-
preset, err := db.InsertPreset(context.Background(), insertPresetParams)
964-
require.NoError(s.T(), err)
963+
preset := dbgen.Preset(s.T(), db, insertPresetParams)
965964
insertPresetParametersParams := database.InsertPresetParametersParams{
966965
TemplateVersionPresetID: preset.ID,
967966
Names: []string{"test"},

coderd/database/dbgen/dbgen.go

+11
Original file line numberDiff line numberDiff line change
@@ -1195,6 +1195,17 @@ func Preset(t testing.TB, db database.Store, seed database.InsertPresetParams) d
11951195
return preset
11961196
}
11971197

1198+
func PresetParameter(t testing.TB, db database.Store, seed database.InsertPresetParametersParams) []database.TemplateVersionPresetParameter {
1199+
parameters, err := db.InsertPresetParameters(genCtx, database.InsertPresetParametersParams{
1200+
TemplateVersionPresetID: takeFirst(seed.TemplateVersionPresetID, uuid.New()),
1201+
Names: takeFirstSlice(seed.Names, []string{testutil.GetRandomName(t)}),
1202+
Values: takeFirstSlice(seed.Values, []string{testutil.GetRandomName(t)}),
1203+
})
1204+
1205+
require.NoError(t, err, "insert preset parameters")
1206+
return parameters
1207+
}
1208+
11981209
func provisionerJobTiming(t testing.TB, db database.Store, seed database.ProvisionerJobTiming) database.ProvisionerJobTiming {
11991210
timing, err := db.InsertProvisionerJobTimings(genCtx, database.InsertProvisionerJobTimingsParams{
12001211
JobID: takeFirst(seed.JobID, uuid.New()),

coderd/presets_test.go

+3-7
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import (
88
"github.com/coder/coder/v2/coderd/coderdtest"
99
"github.com/coder/coder/v2/coderd/database"
1010
"github.com/coder/coder/v2/coderd/database/dbauthz"
11+
"github.com/coder/coder/v2/coderd/database/dbgen"
1112
"github.com/coder/coder/v2/coderd/httpmw"
1213
"github.com/coder/coder/v2/coderd/rbac"
1314
"github.com/coder/coder/v2/codersdk"
@@ -86,16 +87,12 @@ func TestTemplateVersionPresets(t *testing.T) {
8687
user := coderdtest.CreateFirstUser(t, client)
8788
version := coderdtest.CreateTemplateVersion(t, client, user.OrganizationID, nil)
8889

89-
// nolint:gocritic // This is a test
90-
provisionerCtx := dbauthz.AsProvisionerd(ctx)
91-
9290
// Insert all presets for this test case
9391
for _, givenPreset := range tc.presets {
94-
dbPreset, err := db.InsertPreset(provisionerCtx, database.InsertPresetParams{
92+
dbPreset := dbgen.Preset(t, db, database.InsertPresetParams{
9593
Name: givenPreset.Name,
9694
TemplateVersionID: version.ID,
9795
})
98-
require.NoError(t, err)
9996

10097
if len(givenPreset.Parameters) > 0 {
10198
var presetParameterNames []string
@@ -104,12 +101,11 @@ func TestTemplateVersionPresets(t *testing.T) {
104101
presetParameterNames = append(presetParameterNames, presetParameter.Name)
105102
presetParameterValues = append(presetParameterValues, presetParameter.Value)
106103
}
107-
_, err = db.InsertPresetParameters(provisionerCtx, database.InsertPresetParametersParams{
104+
dbgen.PresetParameter(t, db, database.InsertPresetParametersParams{
108105
TemplateVersionPresetID: dbPreset.ID,
109106
Names: presetParameterNames,
110107
Values: presetParameterValues,
111108
})
112-
require.NoError(t, err)
113109
}
114110
}
115111

0 commit comments

Comments
 (0)