mail: Vertragstest repariert, qa05/qa09-Gates skippen ohne .git/Root-Modul #400

Merged
patrick merged 1 commits from fix/mail-vertragstest into main 2026-10-28 11:32:15 +01:00
Owner

Behebt die auf main bestehenden Testfehler in mail (Ursachen aus der Analyse, jeweils bestaetigt durch den Testlauf).

  • mailapi/openapi.yaml: Der Anhang-Download dokumentierte nur application/octet-stream; der Handler setzt absichtlich den per Magic-Bytes erkannten Typ (UX-03 AK2). Spec auf */* mit Hinweis geaendert.
  • mailapi/contract_test.go: zwei fehlende Faelle anhangsliste_eigener_mandant (GET /messages/{id}/attachments) und postfaecher_eigener_mandant (GET /mailboxes), damit der Abdeckungs-Check greift.
  • qa09gate: ohne .git (Wurfverzeichnis) skippt der Test, mit REQUIRE_GIT=1 bleibt es ein Fehler.
  • qa05gate: fehlen die Quellen des Root-Moduls (cmd/auditlog-devserver), wird geskippt, mit REQUIRE_GIT=1 Fehler; ein echter Build-Fehler bleibt fatal.

Auswirkung

Nur Tests und Spec-Datei. Die API selbst ist unveraendert. Clients, die den Content-Type des Downloads auswerten, sind nicht betroffen (die Spec beschreibt jetzt das echte Verhalten).

Tests

131, frisches Wurfverzeichnis (ohne .git, nur mail/): go test der Pakete mailapi, qa05gate, qa09gate ok. Die 4 gofmt-Meldungen im Modul (indexworker, mailer, mimeparse) bestehen schon auf main und gehoeren nicht zu diesem PR.

🤖 Generated with Claude Code

Behebt die auf main bestehenden Testfehler in mail (Ursachen aus der Analyse, jeweils bestaetigt durch den Testlauf). - `mailapi/openapi.yaml`: Der Anhang-Download dokumentierte nur `application/octet-stream`; der Handler setzt absichtlich den per Magic-Bytes erkannten Typ (UX-03 AK2). Spec auf `*/*` mit Hinweis geaendert. - `mailapi/contract_test.go`: zwei fehlende Faelle `anhangsliste_eigener_mandant` (GET /messages/{id}/attachments) und `postfaecher_eigener_mandant` (GET /mailboxes), damit der Abdeckungs-Check greift. - `qa09gate`: ohne `.git` (Wurfverzeichnis) skippt der Test, mit `REQUIRE_GIT=1` bleibt es ein Fehler. - `qa05gate`: fehlen die Quellen des Root-Moduls (`cmd/auditlog-devserver`), wird geskippt, mit `REQUIRE_GIT=1` Fehler; ein echter Build-Fehler bleibt fatal. ## Auswirkung Nur Tests und Spec-Datei. Die API selbst ist unveraendert. Clients, die den Content-Type des Downloads auswerten, sind nicht betroffen (die Spec beschreibt jetzt das echte Verhalten). ## Tests 131, frisches Wurfverzeichnis (ohne .git, nur mail/): `go test` der Pakete mailapi, qa05gate, qa09gate ok. Die 4 gofmt-Meldungen im Modul (indexworker, mailer, mimeparse) bestehen schon auf main und gehoeren nicht zu diesem PR. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
patrick added 1 commit 2026-10-28 10:23:04 +01:00
mail: Vertragstest (Anhang-Content-Type, zwei Endpunkt-Faelle), Gate-Tests ohne .git/Root-Modul skippen
CI / Web-Barrierefreiheitspruefung (axe, Tastatur, Reflow, Textabstaende) (push) Canceled after 0s
CI / Web-Barrierefreiheitspruefung (axe, Tastatur, Reflow, Textabstaende) (pull_request) Canceled after 0s
CI / Core-Modul (Build/Vet/Test) (push) Canceled after 0s
CI / DMS Security-Test-Gate (AC5, Merge-Gate) (push) Canceled after 0s
CI / Connect Security-Test-Gate (QA-01 AC4, Merge-Gate) (push) Canceled after 0s
CI / DMS-Modul (Lint/Build/Test/Coverage/Artefakt) (push) Canceled after 0s
CI / DMS-Modul Supply-Chain-Scan (govulncheck) (push) Canceled after 0s
CI / Web (dms-browser) (push) Canceled after 0s
CI / Web (pdf-viewer) (push) Canceled after 0s
CI / Web (shl) (push) Canceled after 0s
Supply-Chain-Scan (Go) / govulncheck (push) Canceled after 0s
Mail-Schnittstellen-Vertragstests / contract-tests (push) Canceled after 0s
CI / Core-Modul (Build/Vet/Test) (pull_request) Canceled after 0s
CI / DMS Security-Test-Gate (AC5, Merge-Gate) (pull_request) Canceled after 0s
CI / Connect Security-Test-Gate (QA-01 AC4, Merge-Gate) (pull_request) Canceled after 0s
CI / DMS-Modul (Lint/Build/Test/Coverage/Artefakt) (pull_request) Canceled after 0s
CI / DMS-Modul Supply-Chain-Scan (govulncheck) (pull_request) Canceled after 0s
CI / Web (dms-browser) (pull_request) Canceled after 0s
CI / Web (pdf-viewer) (pull_request) Canceled after 0s
CI / Web (shl) (pull_request) Canceled after 0s
Core-Schnittstellen-Vertragstests / contract-tests (pull_request) Canceled after 0s
DB-Integrationstests / db-tests (pull_request) Canceled after 0s
Supply-Chain-Scan (Go) / govulncheck (pull_request) Canceled after 0s
Mail-Schnittstellen-Vertragstests / contract-tests (pull_request) Canceled after 0s
Mail-Pflichttest-Gate / pflichttest-gate (pull_request) Canceled after 0s
Supply-Chain-Scan (npm) / npm-audit (pull_request) Canceled after 0s
Core-Pflichttest-Gate / pflichttest-gate (pull_request) Canceled after 0s
RBAC-Rollenhierarchie-Pflichtgate / rbac-policy-invariant (pull_request) Canceled after 0s
63d7e01948
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
patrick merged commit f5b8e7db7c into main 2026-10-28 11:32:15 +01:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: scripte/nexarch#400