removing unnecessary changes

dannykopping · dannykopping · commit 769ae1d24c70 · 2025-03-17T12:12:02.000Z
Signed-off-by: Danny Kopping &lt;dannykopping@gmail.com&gt;
diff --git a/enterprise/coderd/groups_test.go b/enterprise/coderd/groups_test.go
@@ -825,7 +825,6 @@ func TestGroup(t *testing.T) {
 		// TODO (sasswart): this test seems to have drifted from its original intention. evaluate and remove/fix
 		t.Parallel()
 
-		// TODO: we should not be returning the prebuilds user in Group, and this is not returned in dbmem.
 		if !dbtestutil.WillUsePostgres() {
 			t.Skip("This test requires postgres")
 		}
diff --git a/enterprise/coderd/roles_test.go b/enterprise/coderd/roles_test.go
@@ -7,8 +7,6 @@ import (
 	"slices"
 	"testing"
 
-	"github.com/coder/coder/v2/coderd/database/dbtestutil"
-
 	"github.com/google/uuid"
 	"github.com/stretchr/testify/require"
 
@@ -335,12 +333,6 @@ func TestCustomOrganizationRole(t *testing.T) {
 	// Verify deleting a custom role cascades to all members
 	t.Run("DeleteRoleCascadeMembers", func(t *testing.T) {
 		t.Parallel()
-
-		// TODO: we should not be returning the prebuilds user in OrganizationMembers, and this is not returned in dbmem.
-		if !dbtestutil.WillUsePostgres() {
-			t.Skip("This test requires postgres")
-		}
-
 		owner, first := coderdenttest.New(t, &coderdenttest.Options{
 			LicenseOptions: &coderdenttest.LicenseOptions{
 				Features: license.Features{
diff --git a/enterprise/coderd/templates_test.go b/enterprise/coderd/templates_test.go
@@ -924,7 +924,6 @@ func TestTemplateACL(t *testing.T) {
 	t.Run("everyoneGroup", func(t *testing.T) {
 		t.Parallel()
 
-		// TODO: we should not be returning the prebuilds user in TemplateACL, and this is not returned in dbmem.
 		if !dbtestutil.WillUsePostgres() {
 			t.Skip("This test requires postgres")
 		}

Original file line number	Diff line number	Diff line change
`@@ -825,7 +825,6 @@ func TestGroup(t *testing.T) {`
`825`	`825`	`// TODO (sasswart): this test seems to have drifted from its original intention. evaluate and remove/fix`
`826`	`826`	`t.Parallel()`
`827`	`827`
`828`		`- // TODO: we should not be returning the prebuilds user in Group, and this is not returned in dbmem.`
`829`	`828`	`if !dbtestutil.WillUsePostgres() {`
`830`	`829`	`t.Skip("This test requires postgres")`
`831`	`830`	`}`
Original file line number	Diff line number	Diff line change
`@@ -924,7 +924,6 @@ func TestTemplateACL(t *testing.T) {`
`924`	`924`	`t.Run("everyoneGroup", func(t *testing.T) {`
`925`	`925`	`t.Parallel()`
`926`	`926`
`927`		`- // TODO: we should not be returning the prebuilds user in TemplateACL, and this is not returned in dbmem.`
`928`	`927`	`if !dbtestutil.WillUsePostgres() {`
`929`	`928`	`t.Skip("This test requires postgres")`
`930`	`929`	`}`