Skip to content

Commit e02e65d

Browse files
committed
add unit test
1 parent 51defc9 commit e02e65d

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

enterprise/coderd/provisionerdaemons_test.go

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -739,7 +739,7 @@ func TestGetProvisionerDaemons(t *testing.T) {
739739
t.Parallel()
740740
dv := coderdtest.DeploymentValues(t)
741741
dv.Experiments = []string{string(codersdk.ExperimentMultiOrganization)}
742-
client, _ := coderdenttest.New(t, &coderdenttest.Options{
742+
client, first := coderdenttest.New(t, &coderdenttest.Options{
743743
Options: &coderdtest.Options{
744744
DeploymentValues: dv,
745745
},
@@ -753,6 +753,7 @@ func TestGetProvisionerDaemons(t *testing.T) {
753753
})
754754
org := coderdenttest.CreateOrganization(t, client, coderdenttest.CreateOrganizationOptions{})
755755
orgAdmin, _ := coderdtest.CreateAnotherUser(t, client, org.ID, rbac.ScopedRoleOrgAdmin(org.ID))
756+
outsideOrg, _ := coderdtest.CreateAnotherUser(t, client, first.OrganizationID)
756757

757758
res, err := orgAdmin.CreateProvisionerKey(context.Background(), org.ID, codersdk.CreateProvisionerKeyRequest{
758759
Name: "my-key",
@@ -800,5 +801,9 @@ func TestGetProvisionerDaemons(t *testing.T) {
800801
assert.Equal(t, buildinfo.Version(), pkDaemons[0].Daemons[0].Version)
801802
assert.Equal(t, proto.CurrentVersion.String(), pkDaemons[0].Daemons[0].APIVersion)
802803
assert.Equal(t, keys[0].ID, pkDaemons[0].Daemons[0].KeyID)
804+
805+
// Verify user outside the org cannot read the provisioners
806+
_, err = outsideOrg.ListProvisionerKeyDaemons(ctx, org.ID)
807+
require.Error(t, err)
803808
})
804809
}

0 commit comments

Comments
 (0)