Skip to content

Commit dc4546e

Browse files
committed
address linter complaints in template{create,edit}_test.go
1 parent 161eb54 commit dc4546e

File tree

2 files changed

+9
-5
lines changed

2 files changed

+9
-5
lines changed

enterprise/cli/templatecreate_test.go

+7-4
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import (
88
"github.com/coder/coder/v2/cli/clitest"
99
"github.com/coder/coder/v2/coderd/coderdtest"
1010
"github.com/coder/coder/v2/coderd/database"
11+
"github.com/coder/coder/v2/coderd/rbac"
1112
"github.com/coder/coder/v2/codersdk"
1213
"github.com/coder/coder/v2/enterprise/coderd/coderdenttest"
1314
"github.com/coder/coder/v2/enterprise/coderd/license"
@@ -37,6 +38,7 @@ func TestTemplateCreate(t *testing.T) {
3738
IncludeProvisionerDaemon: true,
3839
},
3940
})
41+
templateAdmin, _ := coderdtest.CreateAnotherUser(t, client, user.OrganizationID, rbac.RoleTemplateAdmin())
4042

4143
source := clitest.CreateTemplateVersionSource(t, &echo.Responses{
4244
Parse: echo.ParseComplete,
@@ -51,13 +53,13 @@ func TestTemplateCreate(t *testing.T) {
5153
"-y",
5254
)
5355

54-
clitest.SetupConfig(t, client, conf)
56+
clitest.SetupConfig(t, templateAdmin, conf)
5557

5658
err := inv.Run()
5759
require.NoError(t, err)
5860

5961
ctx := testutil.Context(t, testutil.WaitMedium)
60-
template, err := client.TemplateByName(ctx, user.OrganizationID, "new")
62+
template, err := templateAdmin.TemplateByName(ctx, user.OrganizationID, "new")
6163
require.NoError(t, err)
6264
require.True(t, template.RequireActiveVersion)
6365
})
@@ -70,7 +72,7 @@ func TestTemplateCreate(t *testing.T) {
7072
string(codersdk.ExperimentTemplateUpdatePolicies),
7173
}
7274

73-
client, _ := coderdenttest.New(t, &coderdenttest.Options{
75+
client, admin := coderdenttest.New(t, &coderdenttest.Options{
7476
LicenseOptions: &coderdenttest.LicenseOptions{
7577
Features: license.Features{},
7678
},
@@ -79,14 +81,15 @@ func TestTemplateCreate(t *testing.T) {
7981
IncludeProvisionerDaemon: true,
8082
},
8183
})
84+
templateAdmin, _ := coderdtest.CreateAnotherUser(t, client, admin.OrganizationID, rbac.RoleTemplateAdmin())
8285

8386
inv, conf := newCLI(t, "templates",
8487
"create", "new",
8588
"--require-active-version",
8689
"-y",
8790
)
8891

89-
clitest.SetupConfig(t, client, conf)
92+
clitest.SetupConfig(t, templateAdmin, conf)
9093

9194
err := inv.Run()
9295
require.Error(t, err)

enterprise/cli/templateedit_test.go

+2-1
Original file line numberDiff line numberDiff line change
@@ -77,6 +77,7 @@ func TestTemplateEdit(t *testing.T) {
7777
IncludeProvisionerDaemon: true,
7878
},
7979
})
80+
templateAdmin, _ := coderdtest.CreateAnotherUser(t, client, owner.OrganizationID, rbac.RoleTemplateAdmin())
8081

8182
version := coderdtest.CreateTemplateVersion(t, client, owner.OrganizationID, nil)
8283
_ = coderdtest.AwaitTemplateVersionJobCompleted(t, client, version.ID)
@@ -89,7 +90,7 @@ func TestTemplateEdit(t *testing.T) {
8990
"-y",
9091
)
9192

92-
clitest.SetupConfig(t, client, conf)
93+
clitest.SetupConfig(t, templateAdmin, conf)
9394

9495
err := inv.Run()
9596
require.Error(t, err)

0 commit comments

Comments
 (0)