Phase 1.6 — Tests: migration + integration + enum coverage (+19 tests)

[sjelfull]

Part of the v3 refactor stack for FRE-56. Closes FRE-129.

Diff: 5 files changed, 284 insertions(+)
Tests: 114 → 133 (+19)
Verified on fresh DB: DROP DATABASE; pest green

Stack navigation

↑ Builds on #94 ui-improvements-diff-rendering (P1.5)
↓ Followed by #96 decompose-elements-users-groups (P2.1)

Review notes

• Single-commit branch — review the commit, not just the diff
• BC preserved — old AuditService API still works via @deprecated delegators
• This is a draft — not a request to merge yet. Marked ready when the stack is approved end-to-end.

[sjelfull]

📚 Stack navigation

Part of the FRE-56 v3 refactor — 12-branch stack via git-spice. Merge in order: bottom-up, each PR rebases the next.

	#	Stage	Title
	#90	P1.1	Foundation — enum + JSON migration
	#91	P1.2	Core services — recorder, diff, registry
	#92	P1.3	15 field handlers
	#93	P1.4	Project config tracking
	#94	P1.5	UI — diff renderer + 20 templates
▶	#95	P1.6	Tests — +19 coverage
	#96	P2.1	Decompose — Element/User/UserGroup handlers
	#97	P2.2	Decompose — 5 more handlers
	#98	P2.3	Repoint event wiring
	#99	P2.4	Tests — migrate to enum
	#111	P2.5	CI — PHP 8.2/8.3/8.4 matrix
	#110	P3.0	Docs — escape hatch (filter-records, getting-started, events)

Quick jump

← #94 P1.5 • #96 P2.1 →

---

_{📌 Reviewing: focus on the single commit. BC preserved via @deprecated delegators. Tests: 49 → 131 across the full stack, fresh-DB-verified per branch. CI: ECS + PHPStan green per branch.}

[sjelfull]

This change is part of the following stack:

• Phase 1.1 — Foundation: AuditEvent enum + JSON snapshot migration #90
  • Phase 1.2 — Core services: AuditRecorder + FieldDiffService + HandlerRegistry #91
    • Phase 1.3 — Field handlers: 15 handlers for native + third-party fields #92
      • Phase 1.4 — Project config tracking: 8 config areas #93
        • Phase 1.5 — UI: DiffRenderer + 20 Twig templates + asset bundle #94
          • Phase 1.6 — Tests: migration + integration + enum coverage (+19 tests) #95 ◀
            • Phase 2.1 — Decompose AuditService: ElementHandler, UserHandler, UserGroupHandler #96
              • Phase 2.2 — Decompose AuditService: Schema/Route/Backup/Plugin/Settings handlers #97
                • Phase 2.3 — Repoint event wiring from AuditService to handler services #98
                  • Phase 2.4 — Migrate Pest tests from AuditModel::EVENT_* to AuditEvent enum #99
                  • Phase 2.5 — CI: PHP 8.2/8.3/8.4 matrix + v3 branch trigger #111
                  • Phase 3.0 — Docs: ship escape hatch (filter-records + getting-started + events-and-recording) #110
                  • P2.6: drop @deprecated AuditService delegators, move @internal helpers to AuditRecorder #112
                  • P2.7: drop @deprecated EVENT_/USER_ string constants from AuditModel, migrate to AuditEvent::Xxx->value #113
                  • P2.8: strip section-divider banners and boilerplate file/class headers #114
                  • P2.9: convert tests to exercise real Craft APIs instead of synthetic event objects #115
                  • P3.1: batch processing foundation — BatchService + auto-attach + 3 enum cases + 13 tests #116
                  • P3.2: migrate ResaveElements to BatchService — kills cache-key concurrency bug #117
                  • P3.4: Feed Me bake-in — auto-batch each feed import via EVENT_BEFORE/AFTER_PROCESS_FEED #119
                  • P3.3: CP UI — expandable indented batch children in audit log index #120
                  • P3.5: Feed Me integration tests — real class-string event dispatch verified #121
                  • P3.6: Fix snapshot column double-JSON-encoding (FRE-239) #122
                  • P3.7: Stack-wide comment cleanup pass #123

_{Change managed by git-spice.}