Skip to content

201_arch-fix-testutil-rename.md

Latest commit

 

History

History
45 lines (39 loc) · 1.18 KB

201_arch-fix-testutil-rename.md

File metadata and controls

45 lines (39 loc) · 1.18 KB
Raw
id 201
title Rename internal/testutil to internal/testsymlink
status ✅
summary Fix the anti-pattern package name. internal/testutil answers "a grab bag"; rename it to internal/testsymlink, which names the question its single symlink.go file answers.
model
depends-on

Rename internal/testutil to internal/testsymlink

Goal

internal/testutil violated the SRP naming rule from the architecture hub. A package named util attracts unrelated code. The package had one non-test file, symlink.go, that creates temporary symlinks for tests. It now lives at internal/testsymlink, which signals the narrow scope.

Tasks

  1. Create internal/testsymlink/.
  2. Move symlink.go and update its package declaration.
  3. Update all imports referencing internal/testutil.
  4. Delete internal/testutil/.
  5. Run go build ./... and go test ./....

Acceptance Criteria

  • internal/testutil/ is gone.
  • internal/testsymlink/ exists.
  • grep -r --include='*.go' 'internal/testutil' returns no results.
  • go build ./... clean.
  • go test ./... passes.
  • go tool golangci-lint run clean.