fix: tests after review

Author: tkcontiant

internal/controller/postgres_controller_test.go

@@ -73,7 +73,6 @@ var _ = Describe("PostgresReconciler", func() {
 		pg = mockpg.NewMockPG(mockCtrl)
 		pg.EXPECT().AlterDatabaseOwner(gomock.Any(), gomock.Any()).Return(nil).AnyTimes()
 		pg.EXPECT().ReassignDatabaseOwner(gomock.Any(), gomock.Any(), gomock.Any(), gomock.Any()).Return(nil).AnyTimes()
-		pg.EXPECT().ReassignDatabaseOwner(gomock.Any(), gomock.Any(), gomock.Any(), gomock.Any()).Return(nil).AnyTimes()
 		cl = k8sClient
 		// Create runtime scheme
 		sc = scheme.Scheme

pkg/postgres/aws.go

@@ -78,11 +78,3 @@ func (c *awspg) DropRole(role, newOwner, database string, logger logr.Logger) er
 
 	return c.pg.DropRole(role, newOwner, database, logger)
 }
-
-func (c *awspg) AlterDatabaseOwner(dbName, owner string) error {
-	return c.pg.AlterDatabaseOwner(dbName, owner)
-}
-
-func (c *awspg) ReassignDatabaseOwner(dbName, currentOwner, newOwner string, logger logr.Logger) error {
-	return c.pg.ReassignDatabaseOwner(dbName, currentOwner, newOwner, logger)
-}

pkg/postgres/azure.go

@@ -48,11 +48,3 @@ func (azpg *azurepg) DropRole(role, newOwner, database string, logger logr.Logge
 	// Delegate to parent implementation to perform the actual drop
 	return azpg.pg.DropRole(role, newOwner, database, logger)
 }
-
-func (azpg *azurepg) AlterDatabaseOwner(dbName, owner string) error {
-	return azpg.pg.AlterDatabaseOwner(dbName, owner)
-}
-
-func (azpg *azurepg) ReassignDatabaseOwner(dbName, currentOwner, newOwner string, logger logr.Logger) error {
-	return azpg.pg.ReassignDatabaseOwner(dbName, currentOwner, newOwner, logger)
-}

pkg/postgres/gcp.go

@@ -83,11 +83,3 @@ func (c *gcppg) DropRole(role, newOwner, database string, logger logr.Logger) er
 	}
 	return nil
 }
-
-func (c *gcppg) AlterDatabaseOwner(dbName, owner string) error {
-	return c.pg.AlterDatabaseOwner(dbName, owner)
-}
-
-func (c *gcppg) ReassignDatabaseOwner(dbName, currentOwner, newOwner string, logger logr.Logger) error {
-	return c.pg.ReassignDatabaseOwner(dbName, currentOwner, newOwner, logger)
-}