From 8e491d88fba50f91e03df06c78038450d778231e Mon Sep 17 00:00:00 2001 From: Danny Kopping Date: Thu, 13 Mar 2025 20:07:29 +0000 Subject: [PATCH] removing unnecessary changes Signed-off-by: Danny Kopping --- enterprise/coderd/groups_test.go | 1 - enterprise/coderd/roles_test.go | 8 -------- enterprise/coderd/templates_test.go | 1 - 3 files changed, 10 deletions(-) diff --git a/enterprise/coderd/groups_test.go b/enterprise/coderd/groups_test.go index a6c9212b95..2f6660c9eb 100644 --- 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 index b2d7da47a6..57b66a3682 100644 --- 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 index e66adebca4..424d427932 100644 --- 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") }