Fix unit test assertion

Emyrk · Emyrk · commit 4e23ea740065 · 2022-06-08T18:55:46.000-05:00
diff --git a/cli/autostart_test.go b/cli/autostart_test.go
@@ -109,7 +109,7 @@ func TestAutostart(t *testing.T) {
 		clitest.SetupConfig(t, client, root)
 
 		err := cmd.Execute()
-		require.ErrorContains(t, err, "status code 403: Forbidden", "unexpected error")
+		require.ErrorContains(t, err, "status code 404", "unexpected error")
 	})
 
 	t.Run("Disable_NotFound", func(t *testing.T) {
@@ -126,7 +126,7 @@ func TestAutostart(t *testing.T) {
 		clitest.SetupConfig(t, client, root)
 
 		err := cmd.Execute()
-		require.ErrorContains(t, err, "status code 403: Forbidden", "unexpected error")
+		require.ErrorContains(t, err, "status code 404:", "unexpected error")
 	})
 
 	t.Run("Enable_DefaultSchedule", func(t *testing.T) {
diff --git a/cli/ttl_test.go b/cli/ttl_test.go
@@ -149,7 +149,7 @@ func TestTTL(t *testing.T) {
 		clitest.SetupConfig(t, client, root)
 
 		err := cmd.Execute()
-		require.ErrorContains(t, err, "status code 403: Forbidden", "unexpected error")
+		require.ErrorContains(t, err, "status code 404:", "unexpected error")
 	})
 
 	t.Run("Unset_NotFound", func(t *testing.T) {
@@ -166,7 +166,7 @@ func TestTTL(t *testing.T) {
 		clitest.SetupConfig(t, client, root)
 
 		err := cmd.Execute()
-		require.ErrorContains(t, err, "status code 403: Forbidden", "unexpected error")
+		require.ErrorContains(t, err, "status code 404:", "unexpected error")
 	})
 
 	t.Run("TemplateMaxTTL", func(t *testing.T) {
diff --git a/coderd/files_test.go b/coderd/files_test.go
@@ -50,7 +50,7 @@ func TestDownload(t *testing.T) {
 		_, _, err := client.Download(context.Background(), "something")
 		var apiErr *codersdk.Error
 		require.ErrorAs(t, err, &apiErr)
-		require.Equal(t, http.StatusForbidden, apiErr.StatusCode())
+		require.Equal(t, http.StatusNotFound, apiErr.StatusCode())
 	})
 
 	t.Run("Insert", func(t *testing.T) {
diff --git a/coderd/organizations_test.go b/coderd/organizations_test.go
@@ -30,7 +30,7 @@ func TestOrganizationByUserAndName(t *testing.T) {
 		_, err := client.OrganizationByName(context.Background(), codersdk.Me, "nothing")
 		var apiErr *codersdk.Error
 		require.ErrorAs(t, err, &apiErr)
-		require.Equal(t, http.StatusForbidden, apiErr.StatusCode())
+		require.Equal(t, http.StatusNotFound, apiErr.StatusCode())
 	})
 
 	t.Run("NoMember", func(t *testing.T) {
diff --git a/coderd/workspaces_test.go b/coderd/workspaces_test.go
@@ -295,7 +295,7 @@ func TestWorkspaceByOwnerAndName(t *testing.T) {
 		// Then:
 		// When we call without includes_deleted, we don't expect to get the workspace back
 		_, err = client.WorkspaceByOwnerAndName(context.Background(), workspace.OwnerName, workspace.Name, codersdk.WorkspaceByOwnerAndNameParams{})
-		require.ErrorContains(t, err, "403")
+		require.ErrorContains(t, err, "404")
 
 		// Then:
 		// When we call with includes_deleted, we should get the workspace back
