fixup! Add integration test for CODER_TOKEN static auth

cmoog · cmoog · commit ccf389319067 · 2020-11-03T09:45:34.000-06:00
diff --git a/ci/integration/statictokens_test.go b/ci/integration/statictokens_test.go
@@ -27,16 +27,23 @@ func TestStaticAuth(t *testing.T) {
 			tcli.GetResult(&result),
 		)
 
-		c.Run(ctx, "rm -rf ~/.config/coder")
+		// remove loging credentials
+		c.Run(ctx, "rm -rf ~/.config/coder").Assert(t,
+			tcli.Success(),
+		)
 
-		cmd := exec.CommandContext(ctx, "sh", "-c", fmt.Sprintf("export CODER_URL=%s && export CODER_TOKEN=$(cat) && coder envs ls", os.Getenv("CODER_URL")))
+		// make requests with token environment variable authentication
+		cmd := exec.CommandContext(ctx, "sh", "-c",
+			fmt.Sprintf("export CODER_URL=%s && export CODER_TOKEN=$(cat) && coder envs ls", os.Getenv("CODER_URL")),
+		)
 		cmd.Stdin = strings.NewReader(string(result.Stdout))
 		c.RunCmd(cmd).Assert(t,
 			tcli.Success(),
 		)
 
+		// should error when the environment variabels aren't set
 		c.Run(ctx, "coder envs ls").Assert(t,
-			tcli.Success(),
+			tcli.Error(),
 		)
 	})
 }

Original file line number	Diff line number	Diff line change
`@@ -27,16 +27,23 @@ func TestStaticAuth(t *testing.T) {`
`27`	`27`	`tcli.GetResult(&result),`
`28`	`28`	`)`
`29`	`29`
`30`		`- c.Run(ctx, "rm -rf ~/.config/coder")`
	`30`	`+ // remove loging credentials`
	`31`	`+ c.Run(ctx, "rm -rf ~/.config/coder").Assert(t,`
	`32`	`+ tcli.Success(),`
	`33`	`+ )`
`31`	`34`
`32`		`- cmd := exec.CommandContext(ctx, "sh", "-c", fmt.Sprintf("export CODER_URL=%s && export CODER_TOKEN=$(cat) && coder envs ls", os.Getenv("CODER_URL")))`
	`35`	`+ // make requests with token environment variable authentication`
	`36`	`+ cmd := exec.CommandContext(ctx, "sh", "-c",`
	`37`	`+ fmt.Sprintf("export CODER_URL=%s && export CODER_TOKEN=$(cat) && coder envs ls", os.Getenv("CODER_URL")),`
	`38`	`+ )`
`33`	`39`	`cmd.Stdin = strings.NewReader(string(result.Stdout))`
`34`	`40`	`c.RunCmd(cmd).Assert(t,`
`35`	`41`	`tcli.Success(),`
`36`	`42`	`)`
`37`	`43`
	`44`	`+ // should error when the environment variabels aren't set`
`38`	`45`	`c.Run(ctx, "coder envs ls").Assert(t,`
`39`		`- tcli.Success(),`
	`46`	`+ tcli.Error(),`
`40`	`47`	`)`
`41`	`48`	`})`
`42`	`49`	`}`