Files

fs 6146b0bd00 test(architecture): catch missing container bindings statically

Codifies the rule that every class referenced via `$c->get(X::class)`
inside a container registrar must itself be bound via
`$container->set(X::class, ...)` somewhere — in the same registrar,
in core/App/registerContainer.php, or in a module container
registrar.

Catches the bug pattern from cd2c9c5: a registrar's factory closure
pulled UserSettingsGateway via $c->get() but no matching ->set() ever
ran. Unit tests with mocked dependencies passed; the failure only
surfaced at runtime ("Service not bound: ...") when the affected
admin/settings/user-lifecycle page tried to construct the dashboard
service against the live container.

Implementation is purely static — token / regex parsing of registrar
files plus class_exists() / interface_exists() gating to avoid false
positives on non-class identifiers. No factory invocation, so the
test does not require DB / session / HTTP and is fast.

Verified by reverting the cd2c9c5 binding line in the working tree
and running the test — it produces the exact "UserSettingsGateway
referenced in core/App/Container/Registrars/UserRegistrar.php"
finding. Restored after the dry-run.

Allowlist starts with two framework-bootstrap classes (AppContainer,
ModuleRegistry) that are bound outside the registrar mechanism.
Each future addition needs an inline justification.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

2026-04-27 08:54:21 +02:00

App

fix(routing): prevent login redirect loops on alias targets

2026-04-26 10:20:28 +02:00

Architecture

test(architecture): catch missing container bindings statically

2026-04-27 08:54:21 +02:00

Console

feat(modules): harden module platform with event constants, validation checks, and CLI improvements

2026-04-14 08:23:04 +02:00

Domain/Taxonomy

refactor(audit): extract audit domain into self-contained module

2026-03-25 21:12:49 +01:00

Http

fix(routing): prevent login redirect loops on alias targets

2026-04-26 10:20:28 +02:00

I18n

feat: module-owned i18n with merge-at-boot translation loading

2026-03-19 22:08:34 +01:00

Repository

refactor(tests): remove redundant tests and fix assertion style

2026-03-25 18:46:49 +01:00

Service

fix(user-lifecycle): track last-run state in core, not in audit log

2026-04-26 21:07:57 +02:00

Support

refactor(action-context): remove unused fragment aggregator + building block

2026-04-26 17:34:53 +02:00

Unit

fix(flash): make flash messages one-shot per request

2026-04-15 18:45:45 +02:00

bootstrap.php

refactor: rename lib/ to core/ for clearer core-module separation

2026-04-13 23:20:42 +02:00

README.md

Clarify test suite taxonomy

2026-03-19 20:22:10 +01:00

README.md

Tests

Ziel

Die Tests sind nach Schutzwirkung organisiert, nicht nur nach technischer Ebene. Wichtige Architektur- und Security-Invarianten sollen schnell gezielt laufen koennen, ohne dass jede Aenderung immer die komplette Suite im Kopf behalten muss.

PHPUnit-Suiten

CoreCore: gesamte Test-Suite.
Unit: kleine verhaltensorientierte Unit-Tests ohne breites Orchestrierungs-Setup.
Service: Business-Logik, Policies, Gateways und Orchestrierungs-Tests unter tests/Service/.
Repository: Query- und Repository-Tests unter tests/Repository/.
Domain: fachliche Taxonomie- und Domain-Tests unter tests/Domain/.
Architecture: alle Architektur-Contracts unter tests/Architecture/.
ArchitectureCore: Core-Boundaries, Module-/Repository-Contracts, Taxonomie- und Sync-Contracts.
ArchitectureSecurity: CSRF, PRG, File-Storage, Crypto, Logging sowie AuthZ-nahe Contracts.
ArchitectureUI: UI-Contracts fuer Listen, Detailseiten, Runtime-Komponenten und A11y.
ArchitectureModules: modulbezogene Struktur- und Isolations-Contracts.
ArchitectureMeta: Agent-/Skill-/Meta-Contracts fuer das Repository-Setup.

Regeln fuer neue Architecture-Tests

Nur stabile Invarianten testen, keine einmaligen Migrationsdetails.
Guards oder Risiken explizit abdecken, zum Beispiel GR-CORE-*, GR-SEC-*, GR-TEST-*.
Keine Duplikate neben bereits breiten Contracts anlegen.
String-Assertions nur verwenden, wenn kein robusterer Contract moeglich ist.
Monolithische Sammeltests vermeiden; lieber kleine thematische Contracts.

Delete vs. Refactor vs. Keep

keep: testet einen echten plattformweiten Contract oder Security-Guard.
refactor: Schutz ist wichtig, aber der Test ist zu datei-, markup- oder implementation-detailnah.
delete: dupliziert bestehende Schutzwirkung oder prueft nur historische Altlasten.

Regeln fuer neue Service-Tests

Services auf Guards, Orchestrierung, Normalisierung, Fehlerpfade und Seiteneffekte testen.
Keine eigenen Tests fuer reine find/list/get/set-Delegation ohne zusaetzliche Regel.
Thin Wrapper auf gemeinsame Primitive zentral testen, nicht pro Gateway erneut.
Mockability-, Reflection- und Interface-Existenz-Tests ohne Risiko-Schutz vermeiden.
Kleine Normalisierungs-Matrizen mit DataProvidern oder zusammengezogenen Contract-Dateien verdichten.

Typische Low-Value-Muster

Gateway-Test prueft nur encrypt/decrypt erneut, obwohl das Basisverhalten schon zentral abgesichert ist.
Test bestaetigt nur, dass ein Service eins zu eins an Repository oder Gateway weiterreicht.
Test prueft nur Methodensignatur, Reflection oder dass eine Klasse mockbar ist.
Mehrere Minidateien decken denselben Settings-/Fallback-Mechanismus mit identischem Mock-Muster ab.

Naechste Audit-Kandidaten

methodenweiser Low-value-Pass in tests/Service/User/UserAccountServiceTest.php
methodenweiser Low-value-Pass in tests/Service/Org/DepartmentServiceTest.php
methodenweiser Low-value-Pass in tests/Service/Access/RoleServiceTest.php
verbleibende kleine Settings-Gateway-Tests auf weitere Verdichtung pruefen