Commita361673

authored

fix: Include 'CODER' env var prefix on group mappings (#7015)

* fix: Include 'CODER' env var prefix on group mappings* Unit test to force CODER prefix

1 parent00d468b commita361673Copy full SHA for a361673

File tree

-0

lines changed

-0

lines changed

Lines changed: 7 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,7 @@`
`1`	`1`	`package codersdk_test`
`2`	`2`
`3`	`3`	`import (`
	`4`	`+"strings"`
`4`	`5`	`"testing"`
`5`	`6`
`6`	`7`	`"github.com/stretchr/testify/require"`
`@@ -101,6 +102,12 @@ func TestDeploymentValues_HighlyConfigurable(t *testing.T) {`
`101`	`102`	`t.Errorf("Option %q is excluded but has an env name",opt.Name)`
`102`	`103`	`}`
`103`	`104`
	`105`	`+// Also check all env vars are prefixed with CODER_`
	`106`	`+constprefix="CODER_"`
	`107`	`+ifopt.Env!=""&&!strings.HasPrefix(opt.Env,prefix) {`
	`108`	`+t.Errorf("Option %q has an env name (%q) that is not prefixed with %s",opt.Name,opt.Env,prefix)`
	`109`	`+}`
	`110`	`+`
`104`	`111`	`delete(excludes,opt.Name)`
`105`	`112`	`}`
`106`	`113`

Comments

(0)