Files

fs 95723cebd9 test(architecture): enforce defensive reads in shared form partials

Codifies the convention that shared form partials (pages/admin/*/_form.phtml,
modules/**/_form.phtml) either guard variable reads with ?? / isset() / array
access checks, or have every consuming view template define the variable
before requiring the partial.

The convention was already followed by 6 of 7 partials in the codebase. The
seventh (tenants/_form.phtml) regressed silently when a per-theme logo block
introduced bare $canUpdateTenant reads in commit 6e3fc63c — fixed in commit
e29e6c3. This test catches that exact bug shape and any future variant.

Implementation uses token_get_all (no full PHP parser, no expression
evaluation). For each partial it identifies variable references that are not
locally defined or guarded, then walks the consuming templates discovered via
require statements and verifies each variable is present before the require.
On detection of statically unrecognizable constructs (extract(),
dynamic require paths) the test fails loudly with a "review manually" hint
rather than silently passing.

Verified by reverting commit e29e6c3 in the working tree and running the test
— it produces the exact $canUpdateTenant / line 223 finding that prompted the
original fix. Restored after the dry-run.

ALLOWLIST stays empty today. Future legitimate exceptions go in the test
header with per-entry justification, mirroring DetailDrawerFragmentContractTest.

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

2026-04-26 17:19:14 +02:00

App

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

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

Architecture

test(architecture): enforce defensive reads in shared form partials

2026-04-26 17:19:14 +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

harden(rate-limiter): fail secure login scopes with cache fallback

2026-04-25 23:17:46 +02:00

Support

refactor(departments-edit): migrate to actionEditContext (step 2)

2026-04-25 23:52:27 +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