Remove unused ListAll() from objecttype repository (#297)

Author: akajla09

.gitignore

@@ -3,3 +3,4 @@ apirunner.conf
 .DS_Store
 bin/
 dist/
+.idea/

pkg/authz/objecttype/mysql.go

@@ -112,34 +112,6 @@ func (repo MySQLRepository) GetByTypeId(ctx context.Context, typeId string) (Mod
 	return &objectType, nil
 }
 
-func (repo MySQLRepository) ListAll(ctx context.Context) ([]Model, error) {
-	models := make([]Model, 0)
-	objectTypes := make([]ObjectType, 0)
-	err := repo.DB.SelectContext(
-		ctx,
-		&objectTypes,
-		`
-			SELECT id, typeId, definition, createdAt, updatedAt, deletedAt
-			FROM objectType
-			WHERE
-				deletedAt IS NULL
-		`,
-	)
-
-	if err != nil {
-		if errors.Is(err, sql.ErrNoRows) {
-			return models, nil
-		}
-		return models, errors.Wrap(err, "error listing all object types")
-	}
-
-	for i := range objectTypes {
-		models = append(models, &objectTypes[i])
-	}
-
-	return models, nil
-}
-
 func (repo MySQLRepository) List(ctx context.Context, listParams service.ListParams) ([]Model, *service.Cursor, *service.Cursor, error) {
 	models := make([]Model, 0)
 	objectTypes := make([]ObjectType, 0)

pkg/authz/objecttype/postgres.go

@@ -118,35 +118,6 @@ func (repo PostgresRepository) GetByTypeId(ctx context.Context, typeId string) (
 	return &objectType, nil
 }
 
-func (repo PostgresRepository) ListAll(ctx context.Context) ([]Model, error) {
-	models := make([]Model, 0)
-	objectTypes := make([]ObjectType, 0)
-	err := repo.DB.SelectContext(
-		ctx,
-		&objectTypes,
-		`
-			SELECT id, type_id, definition, created_at, updated_at, deleted_at
-			FROM object_type
-			WHERE
-				deleted_at IS NULL
-		`,
-	)
-
-	if err != nil {
-		if errors.Is(err, sql.ErrNoRows) {
-			return models, nil
-		}
-
-		return models, errors.Wrap(err, "error listing all object types")
-	}
-
-	for i := range objectTypes {
-		models = append(models, &objectTypes[i])
-	}
-
-	return models, nil
-}
-
 func (repo PostgresRepository) List(ctx context.Context, listParams service.ListParams) ([]Model, *service.Cursor, *service.Cursor, error) {
 	models := make([]Model, 0)
 	objectTypes := make([]ObjectType, 0)

pkg/authz/objecttype/repository.go

@@ -27,7 +27,6 @@ type ObjectTypeRepository interface {
 	Create(ctx context.Context, objectType Model) (int64, error)
 	GetById(ctx context.Context, id int64) (Model, error)
 	GetByTypeId(ctx context.Context, typeId string) (Model, error)
-	ListAll(ctx context.Context) ([]Model, error)
 	List(ctx context.Context, listParams service.ListParams) ([]Model, *service.Cursor, *service.Cursor, error)
 	UpdateByTypeId(ctx context.Context, typeId string, objectType Model) error
 	DeleteByTypeId(ctx context.Context, typeId string) error

pkg/authz/objecttype/sqlite.go

@@ -118,34 +118,6 @@ func (repo SQLiteRepository) GetByTypeId(ctx context.Context, typeId string) (Mo
 	return &objectType, nil
 }
 
-func (repo SQLiteRepository) ListAll(ctx context.Context) ([]Model, error) {
-	models := make([]Model, 0)
-	objectTypes := make([]ObjectType, 0)
-	err := repo.DB.SelectContext(
-		ctx,
-		&objectTypes,
-		`
-			SELECT id, typeId, definition, createdAt, updatedAt, deletedAt
-			FROM objectType
-			WHERE
-				deletedAt IS NULL
-		`,
-	)
-
-	if err != nil {
-		if errors.Is(err, sql.ErrNoRows) {
-			return models, nil
-		}
-		return models, errors.Wrap(err, "error listing all object types")
-	}
-
-	for i := range objectTypes {
-		models = append(models, &objectTypes[i])
-	}
-
-	return models, nil
-}
-
 func (repo SQLiteRepository) List(ctx context.Context, listParams service.ListParams) ([]Model, *service.Cursor, *service.Cursor, error) {
 	models := make([]Model, 0)
 	objectTypes := make([]ObjectType, 0)