Back

Code quality

Test Data Factory Cleanup: Retire Builders That Hide Product Changes

Test Data Factory Cleanup: Retire Builders That Hide Product Changes starts with a test helper that creates product states users can no longer create. The test data factory may look quiet, but quiet is not the same as unused. It can still support a rare workflow, a contract, a rollback path, or an owner who no longer sits near the team doing the cleanup.

Use this note when you need to reduce stale software surface area without turning deletion into the first real test. The useful result is a small decision record: current owner, current purpose, evidence reviewed, reversible first step, caveats, and the rule that prevents the same test data factory from returning.

What makes this cleanup risky

The risk is not age. The risk is losing a dependency that is visible only during unusual conditions. In test data factory cleanup, the review should start by naming the exact behavior the test data factory still enables and the exact behavior that has replaced it.

Review areaWhat to inspectCleanup signal
Current ownerTeam, service, data owner, or support pathSomeone can approve a keep or remove decision
Runtime evidencecall sites, assertions, validation rules, database constraints, migration tests, and fixture defaultsRecent use is absent or explained
Replacement pathnamed current scenarioThe new path handles the same real cases
Rollback or historyBackup, audit, archive, or recreation planA wrong decision is recoverable
Creation pathHow new items are createdA prevention rule can stop recurrence

A cleanup candidate with no owner should not be treated as safe. It should be treated as an ownership bug that must be resolved before the final removal.

Evidence checks that fit the subject

Collect several signals before acting:

  • Inspect call sites, assertions, validation rules, database constraints, migration tests, and fixture defaults.
  • Confirm the replacement path, not just the absence of recent edits.
  • Review the longest business, reporting, incident, or customer cycle that could still use the test data factory.
  • Ask the owner to choose keep, narrow, archive, disable, remove, or investigate.

A focused review sample can keep the conversation concrete:

rg "legacySubscriptionFactory|withGrandfatheredPlan" tests src fixtures

Treat the output as a candidate list, not a deletion command. It proves one slice of behavior and must be paired with ownership, dependency review, and a rollback plan.

Prefer a reversible first move

Good cleanup usually happens in stages. First stop creating new test data factory records or references. Then narrow the scope, disable the stale path, or archive the visible surface while watching for unexpected use. Remove only after the waiting window matches how the system is actually used.

Do not rush when the test data factory touches security response, customer commitments, billing, compliance, incident recovery, or low-frequency operational work. Also slow down when the replacement changed semantics rather than only names; similar labels can hide different behavior.

Prevention rule

new builders should state the product rule they represent and the removal condition for temporary states. Add the rule where the test data factory is created, not only in a cleanup spreadsheet. The next cleanup should begin with owner and sunset context already attached.

Key takeaways

  • Stale test data factory cleanup needs evidence about use, ownership, replacement, and reversibility.
  • Recent silence is helpful, but it is not enough by itself.
  • The best first move is usually narrowing, disabling, or archiving before final removal.
  • Prevention belongs in the creation path so the same stale item does not return.