removing unnecessary changes

Signed-off-by: Danny Kopping <dannykopping@gmail.com>

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")
 		}

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{

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")
 		}