From 1f96dd2ed267cb8635af3c693e0cc0f7fb550176 Mon Sep 17 00:00:00 2001 From: nexxo Date: Tue, 4 Aug 2026 17:42:19 +0200 Subject: [PATCH] Plan: ein Test ohne positive Zusage prueft nichts MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Der Test "nimmt keinen Namen auf, den niemand konfiguriert hat" behauptete nur die Abwesenheit von files. — er waere auch dann gruen gewesen, wenn das Skript ueberhaupt nichts gerendert haette. Die anderen Tests mit not->toContain haben ihre positive Haelfte, dieser hatte sie nicht. Dazu die Testanzahl im Schritt nachgezogen: durch die Vorabkorrektur an der Zertifikatsliste sind es sieben, nicht sechs. Co-Authored-By: Claude Opus 5 --- .../plans/2026-08-04-oeffentliche-seiten-im-tunnel.md | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/docs/superpowers/plans/2026-08-04-oeffentliche-seiten-im-tunnel.md b/docs/superpowers/plans/2026-08-04-oeffentliche-seiten-im-tunnel.md index 0f99698..7355641 100644 --- a/docs/superpowers/plans/2026-08-04-oeffentliche-seiten-im-tunnel.md +++ b/docs/superpowers/plans/2026-08-04-oeffentliche-seiten-im-tunnel.md @@ -487,7 +487,11 @@ it('nimmt keinen Namen auf, den niemand konfiguriert hat', function () { 'VPN_TUNNEL_HOSTS' => '', ], ['admin.clupilot.test', 'files.clupilot.test']); - expect($config)->not->toContain('files.clupilot.test'); + // Beide Hälften: ohne die positive Zusage bestünde dieser Test auch dann, + // wenn das Skript überhaupt nichts renderte — und genau das ist der Fehler, + // den er fangen soll. + expect($config)->toContain('https://admin.clupilot.test:443') + ->and($config)->not->toContain('files.clupilot.test'); }); it('schreibt die geladenen Zertifikate mit, damit die Erneuerung greift', function () { @@ -671,7 +675,7 @@ Ausführbar machen: `chmod +x docker/caddy/vpn-entrypoint.sh` - [ ] **Step 4: Tests laufen lassen** Run: `docker exec -w /var/www/html/.claude/worktrees/eager-elion-c581f4 clupilot-app-1 php artisan test --filter=VpnGatewayConfig` -Expected: PASS, sechs Tests. +Expected: PASS, sieben Tests. - [ ] **Step 5: Voller Testlauf und Commit**