P3.5: Feed Me integration tests — real class-string event dispatch verified

[sjelfull]

P3.5: Real Feed Me integration tests — class-string event dispatch verified

Adds genuine integration coverage for the Feed Me bake-in shipped in #119. Previously, FeedMeListener was only exercised against synthetic event stubs via Process::trigger(...) — which tested the listener's API but not the integration contract (deferred plugin-load registration → Yii class-string event dispatch → real lifecycle).

What changed

• composer.json — adds craftcms/feed-me ^6.0 to require-dev so CI installs it and the integration test runs everywhere.
• src/services/FeedMeListener.php — idempotency guards $initialized + $registered upgraded from private to private static. This is a real production-correctness fix, not just a test-isolation patch: Yii's Event::on() writes to a process-global static handler registry, so per-instance guards do nothing when the plugin singleton gets recreated (in tests, but also in long-running queue workers reconstructing components). With private, each new instance saw $registered = false and added another pair of handlers to the global registry. After N reconstructions, a single Feed Me feed import would have fired N batches and produced N×rows audit child rows.
• tests/Unit/FeedMeListenerTest.php (new, 9 tests) — moves the original Feature/FeedMeListenerTest.php here and reframes as unit tests for the listener's API surface using synthetic event stubs. These remain valuable for tight feedback on edge cases (missing feed, missing id, idempotent close, etc.).
• tests/Feature/FeedMeIntegrationTest.php (new, 6 tests) — verifies the integration contract:
  1. Listener registers handlers on craft\feedme\services\Process (probed via reflection on Yii's Event::$_events)
  2. Triggering Process::EVENT_BEFORE_PROCESS_FEED opens an audit batch with metadata.source = 'feed-me'
  3. Triggering Process::EVENT_AFTER_PROCESS_FEED closes it
  4. Audit rows written between fire→fire inherit the batch's parentId via the AuditRecorder auto-attach hook
  5. After-without-before is a silent no-op
  6. End-to-end CSV import: a 3-row CSV → real FeedModel saved via Feeds::saveFeed() → driven through FeedImport::execute() (Feed Me's actual queue job) → entries created, batch opened by the listener via real class-string event dispatch, child rows attached with the right parentId, batch closed, cache cleared. This is the full production code path.

Test results

• 178 passed / 1 skipped / 760 assertions (+15 from new E2E test)
• ECS clean
• PHPStan clean (60 files, no errors)

Notable findings

The E2E test surfaced a latent quirk worth flagging:

audit_log.snapshot is double-JSON-encoded. When a snapshot is written, the JSON object gets encoded once into a JSON string, then that string is encoded a second time by the column-attribute writer. PHP-side decode round-trips fine, but it means SQL LIKE queries against snapshot contents are fragile (the stored bytes are "\"feedId\":1" not "feedId":1). The E2E test works around this by fetching BatchCompleted rows and matching in PHP — but the writer should single-encode. Captured as a follow-up issue.

Stacked on

p3.3-cp-ui-batch-children (#120)

[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.}