@nakedev/go-scaffold 0.4.0 → 0.5.0

package/templates/generate/module/service.go.hbs

@@ -1,108 +0,0 @@
-package {{pkg}}
-
-import (
-	"context"
-	"errors"
-
-	"{{goModule}}/internal/app/{{modulePath}}/model"
-	"{{goModule}}/internal/shared/apperror"
-	"{{goModule}}/internal/shared/dberr"
-	"{{goModule}}/internal/shared/id"
-
-	"github.com/google/uuid"
-	"gorm.io/gorm"
-)
-
-// repository = what the service needs from the data layer (declared on the consumer side, so it can be mocked in tests)
-type repository interface {
-	Create(ctx context.Context, m *model.{{pascalName}}) error
-	FindAll(ctx context.Context, limit, offset int) ([]model.{{pascalName}}, error)
-	FindByID(ctx context.Context, id uuid.UUID) (*model.{{pascalName}}, error)
-	Update(ctx context.Context, m *model.{{pascalName}}) error
-	Delete(ctx context.Context, id uuid.UUID) error
-	// go-scaffold:repository-interface
-}
-
-// Service = business logic for {{pkg}} (rules live here, it knows nothing about HTTP)
-// input shape validation happens in the handler (binding tags)
-type Service struct {
-	repo repository
-}
-
-func NewService(repo repository) *Service {
-	return &Service{repo: repo}
-}
-
-func (s *Service) Create(ctx context.Context, in createInput) (*model.{{pascalName}}, error) {
-	// TODO: set real fields from in
-	_ = in
-	m := &model.{{pascalName}}{ID: id.New()}
-	if err := s.repo.Create(ctx, m); err != nil {
-		if dberr.IsDuplicate(err) {
-			return nil, errConflict()
-		}
-		return nil, apperror.NewInternal()
-	}
-	return m, nil
-}
-
-func (s *Service) List(ctx context.Context, limit, offset int) ([]model.{{pascalName}}, error) {
-	items, err := s.repo.FindAll(ctx, limit, offset)
-	if err != nil {
-		return nil, apperror.NewInternal()
-	}
-	return items, nil
-}
-
-func (s *Service) Get(ctx context.Context, id uuid.UUID) (*model.{{pascalName}}, error) {
-	m, err := s.repo.FindByID(ctx, id)
-	if err != nil {
-		return nil, wrapFindErr(err)
-	}
-	return m, nil
-}
-
-func (s *Service) Update(ctx context.Context, id uuid.UUID, in updateInput) (*model.{{pascalName}}, error) {
-	m, err := s.repo.FindByID(ctx, id)
-	if err != nil {
-		return nil, wrapFindErr(err)
-	}
-	// TODO: apply real fields from in before saving
-	// The version the client read comes from the request, not from the row we
-	// just loaded — comparing the row against itself would always succeed and
-	// defeat the check.
-	m.Version = in.Version
-	if err := s.repo.Update(ctx, m); err != nil {
-		if errors.Is(err, ErrStaleVersion) {
-			return nil, errStale()
-		}
-		if dberr.IsDuplicate(err) {
-			return nil, errConflict()
-		}
-		return nil, apperror.NewInternal()
-	}
-	return m, nil
-}
-
-func (s *Service) Delete(ctx context.Context, id uuid.UUID) error {
-	if err := s.repo.Delete(ctx, id); err != nil {
-		if errors.Is(err, gorm.ErrRecordNotFound) {
-			return errNotFound()
-		}
-		// deleting a {{pkg}} that's still referenced elsewhere → FK RESTRICT trips = 409, not 500
-		if dberr.IsForeignKey(err) {
-			return errHasReferences()
-		}
-		return apperror.NewInternal()
-	}
-	return nil
-}
-
-func wrapFindErr(err error) error {
-	if errors.Is(err, gorm.ErrRecordNotFound) {
-		return errNotFound()
-	}
-	return apperror.NewInternal()
-}
-
-// go-scaffold:service-methods

package/templates/generate/module/service_test.go.hbs

@@ -1,161 +0,0 @@
-package {{pkg}}
-
-import (
-	"context"
-	"errors"
-	"testing"
-
-	"{{goModule}}/internal/app/{{modulePath}}/model"
-	"{{goModule}}/internal/shared/apperror"
-
-	"github.com/google/uuid"
-	"gorm.io/gorm"
-)
-
-// repositoryStub exposes one function per dependency operation. Tests configure
-// only the calls they expect; an unexpected call panics instead of silently
-// returning a shared zero value/error that can hide broken orchestration.
-type repositoryStub struct {
-	createFn   func(context.Context, *model.{{pascalName}}) error
-	findAllFn  func(context.Context, int, int) ([]model.{{pascalName}}, error)
-	findByIDFn func(context.Context, uuid.UUID) (*model.{{pascalName}}, error)
-	updateFn   func(context.Context, *model.{{pascalName}}) error
-	deleteFn   func(context.Context, uuid.UUID) error
-	// go-scaffold:repository-stub-fields
-}
-
-func (s *repositoryStub) Create(ctx context.Context, m *model.{{pascalName}}) error {
-	if s.createFn == nil {
-		panic("unexpected repository.Create call")
-	}
-	return s.createFn(ctx, m)
-}
-
-func (s *repositoryStub) FindAll(ctx context.Context, limit, offset int) ([]model.{{pascalName}}, error) {
-	if s.findAllFn == nil {
-		panic("unexpected repository.FindAll call")
-	}
-	return s.findAllFn(ctx, limit, offset)
-}
-
-func (s *repositoryStub) FindByID(ctx context.Context, id uuid.UUID) (*model.{{pascalName}}, error) {
-	if s.findByIDFn == nil {
-		panic("unexpected repository.FindByID call")
-	}
-	return s.findByIDFn(ctx, id)
-}
-
-func (s *repositoryStub) Update(ctx context.Context, m *model.{{pascalName}}) error {
-	if s.updateFn == nil {
-		panic("unexpected repository.Update call")
-	}
-	return s.updateFn(ctx, m)
-}
-
-func (s *repositoryStub) Delete(ctx context.Context, id uuid.UUID) error {
-	if s.deleteFn == nil {
-		panic("unexpected repository.Delete call")
-	}
-	return s.deleteFn(ctx, id)
-}
-
-// go-scaffold:repository-stub-methods
-
-func status(t *testing.T, err error) int {
-	t.Helper()
-	var appErr *apperror.AppError
-	if !errors.As(err, &appErr) {
-		t.Fatalf("expected *apperror.AppError, got %T: %v", err, err)
-	}
-	return appErr.HTTPStatus
-}
-
-func TestService_Create_Duplicate(t *testing.T) {
-	repo := &repositoryStub{
-		createFn: func(context.Context, *model.{{pascalName}}) error {
-			return gorm.ErrDuplicatedKey
-		},
-	}
-	svc := NewService(repo)
-
-	_, err := svc.Create(context.Background(), createInput{})
-
-	if got := status(t, err); got != 409 {
-		t.Fatalf("want 409 conflict, got %d", got)
-	}
-}
-
-func TestService_Get_NotFound(t *testing.T) {
-	repo := &repositoryStub{
-		findByIDFn: func(context.Context, uuid.UUID) (*model.{{pascalName}}, error) {
-			return nil, gorm.ErrRecordNotFound
-		},
-	}
-	svc := NewService(repo)
-
-	_, err := svc.Get(context.Background(), uuid.New())
-
-	if got := status(t, err); got != 404 {
-		t.Fatalf("want 404, got %d", got)
-	}
-}
-
-func TestService_List_DBError_Becomes500(t *testing.T) {
-	repo := &repositoryStub{
-		findAllFn: func(context.Context, int, int) ([]model.{{pascalName}}, error) {
-			return nil, errors.New("connection refused")
-		},
-	}
-	svc := NewService(repo)
-
-	_, err := svc.List(context.Background(), 20, 0)
-
-	if got := status(t, err); got != 500 {
-		t.Fatalf("want 500, got %d", got)
-	}
-}
-
-// The check that stops one user's save from silently erasing another's: the
-// repository reports the row moved on, the client must see 409 rather than a
-// 500 or, worse, a success.
-func TestService_Update_StaleVersion(t *testing.T) {
-	repo := &repositoryStub{
-		findByIDFn: func(_ context.Context, id uuid.UUID) (*model.{{pascalName}}, error) {
-			return &model.{{pascalName}}{ID: id, Version: 7}, nil
-		},
-		updateFn: func(context.Context, *model.{{pascalName}}) error {
-			return ErrStaleVersion
-		},
-	}
-	svc := NewService(repo)
-
-	_, err := svc.Update(context.Background(), uuid.New(), updateInput{Version: 3})
-
-	if got := status(t, err); got != 409 {
-		t.Fatalf("want 409 stale, got %d", got)
-	}
-}
-
-// The version compared against the database has to be the one the client sent,
-// not the one just read back — comparing the row against itself would always
-// match and the guard would never fire.
-func TestService_Update_SendsClientVersion(t *testing.T) {
-	var seen int
-	repo := &repositoryStub{
-		findByIDFn: func(_ context.Context, id uuid.UUID) (*model.{{pascalName}}, error) {
-			return &model.{{pascalName}}{ID: id, Version: 7}, nil
-		},
-		updateFn: func(_ context.Context, m *model.{{pascalName}}) error {
-			seen = m.Version
-			return nil
-		},
-	}
-	svc := NewService(repo)
-
-	if _, err := svc.Update(context.Background(), uuid.New(), updateInput{Version: 3}); err != nil {
-		t.Fatalf("update: %v", err)
-	}
-	if seen != 3 {
-		t.Fatalf("want the client's version 3 to reach the repository, got %d", seen)
-	}
-}