Systemabnahme: fuenf Altlasten geschlossen

Vollstaendiger Durchlauf ueber alle Pruefungen. Dabei kamen fuenf Dinge
ans Licht, die teils seit Wochen offen standen.

1. WCAG-KONTRAST -- der einzige Punkt, der echte Besucher betraf

Die lila Schilder erreichten nur 4,08:1, verlangt sind 4,5:1 fuer
normalen Text. 17 Fundstellen, alle dieselbe Ursache: Die Schrift nutzte
den vollen Ton Aurora Violet.

Gemessen wurde nicht geschaetzt: Untergrund rgb(38,41,81) aus dem
echten Bild ausgelesen, Schrift 12,5 Punkte -- und fett zaehlt erst ab
18,5 Punkten als "grosser Text".

Neu ist --wd-lila-hell (#C197FF, 6,02:1). Bewusst NICHT der knappste
Wert: #B380FF haette mit 4,91:1 gereicht, aber dasselbe Schild steht
auch ueber der Buehne im Verwaltungsbereich. Ein Ton, der nur an einer
Stelle knapp besteht, faellt beim naechsten Hintergrund wieder durch.
Dasselbe Muster wie --wd-blau-hell, das genau deshalb existiert.

2. GEHEIMNIS-TEST -- der Test war veraltet, nicht der Code

Er verlangte ".env hat Vorrang", die Umsetzung macht das Gegenteil. Die
Begruendung im Code ueberzeugt: Wer PayPal-Daten im Formular eintraegt,
erwartet, dass sie gelten. Andernfalls koennte ein alter Wert in der
.env sie stumm ueberstimmen -- und man sucht stundenlang.

Der Test prueft jetzt die tatsaechliche Reihenfolge, dazu neu, dass ein
gleichzeitig vorhandener Serverwert auch angezeigt wird.

Ausserdem endete der Lauf trotz gruener Pruefungen mit einer
Fehlermeldung: Unter Windows haelt eine offene SQLite-Datei eine Sperre,
das Aufraeumen scheiterte mit EPERM. Auf Linux waere es
durchgelaufen -- dasselbe Skript mit unterschiedlichem Ergebnis je
Rechner. Jetzt wird erst geschlossen, dann geloescht, und das Aufraeumen
kann den Lauf nicht mehr zum Scheitern bringen.

3. PORTAL-TEST -- ebenfalls veraltet

Er suchte "1500,00" und schlug fehl, seit der Formatierer den
Tausenderpunkt setzt. "1.500,00 EUR" ist die korrekte deutsche
Schreibweise; gerade ab vier Stellen macht der Punkt eine Zahl auf einen
Blick lesbar.

4. ZWEI SEITEN OHNE WOERTERBUCH -- eine Entscheidung, keine Luecke

"diagnose" ist ein Betriebswerkzeug. "rechtliches" ist der heiklere
Fall: Rechtstexte durch eine ungepruefte Uebersetzung zu schicken ist
gefaehrlicher, als sie einsprachig zu lassen. Ein Fehler in einer
Widerrufsbelehrung wirkt gegen den Verfasser.

Die Pruefung meldete beides als "Datei fehlt" -- das las sich wie ein
Versehen und stand deshalb dauerhaft in der Fehlerliste, ohne dass
jemand etwas tat. Jetzt sind beide benannt und begruendet.

5. TAG-UNGLEICHGEWICHT -- eine Fehlmessung

Die Pruefung zaehlte auch HTML-Schnipsel, die als Zeichenketten im
JavaScript stehen. Dort steht ein oeffnendes <div> regelmaessig in einer
anderen Zeichenkette als sein </div>, weil die Teile erst beim
Zusammensetzen ein Ganzes ergeben.

Gegengeprueft im Browser: geparster Baum einwandfrei, kein
Skriptfehler, Verschachtelung unauffaellig. Es war nie ein
Strukturfehler. Die Meldung stand aber monatelang als "2 Fehler" da und
haette jede echte Meldung entwertet, die dazugekommen waere.

STAND

  Browser  717 Pruefungen   0 offen
  Server   566 Pruefungen   0 offen
  i18n     keine Fehler
  WCAG     0 Fundstellen (vorher 17)

Co-Authored-By: Claude Opus 5 <[email protected]>

server-internal/test-geheimnisse.mjs

@@ -56,12 +56,37 @@ pruefe("das Secret ist als geheim markiert", st.PAYPAL_CLIENT_SECRET.wert === nu
 pruefe("die Betriebsart wird dagegen ausgeschrieben", st.PAYPAL_ENV.wert !== undefined);
 pruefe("die Herkunft wird benannt", st.PAYPAL_CLIENT_SECRET.quelle === "verwaltung");
 
-L("\n4. DIE .env HAT VORRANG");
+/* Frueher stand hier "DIE .env HAT VORRANG" -- und der Test schlug fehl,
+   weil die Umsetzung laengst die umgekehrte Reihenfolge hat.
+
+   Der Test war veraltet, nicht der Code. Die Begruendung steht im Kopf
+   von lib/webdesign-geheimnisse.js: Wer einen Wert im Formular setzt,
+   erwartet, dass er gilt. Waere es andersherum, koennte Filipe PayPal-
+   Zugangsdaten in der Verwaltung eintragen, ohne dass sich irgendetwas
+   aendert -- ein alter Wert in der .env wuerde sie stumm ueberstimmen.
+
+   Wichtig ist deshalb nicht nur die Reihenfolge selbst, sondern auch,
+   dass die Anzeige sie ehrlich benennt. Genau das wird hier mitgeprueft. */
+L("\n4. DIE VERWALTUNG HAT VORRANG, DIE .env IST DER RUECKFALL");
 process.env.PAYPAL_CLIENT_ID = "AUS-DER-ENV";
-pruefe("ein Wert auf dem Server sticht die Verwaltung",
+pruefe("ein Wert aus der Verwaltung sticht den Server",
+  geheim.einstellung("PAYPAL_CLIENT_ID") === "AbC-123");
+pruefe("und die Anzeige nennt die Verwaltung als Quelle",
+  geheim.stand().PAYPAL_CLIENT_ID.quelle === "verwaltung");
+/* Liegt in BEIDEN etwas, muss man das sehen koennen -- sonst sucht man
+   spaeter stundenlang, warum der Serverwert nicht wirkt. */
+pruefe("dass auch auf dem Server einer liegt, wird vermerkt",
+  geheim.stand().PAYPAL_CLIENT_ID.auchAufServer === true);
+
+/* Ohne Eintrag in der Verwaltung greift der Server-Wert. */
+await geheim.einstellungSetzen("PAYPAL_CLIENT_ID", "");
+pruefe("ohne Verwaltungswert greift der Server",
   geheim.einstellung("PAYPAL_CLIENT_ID") === "AUS-DER-ENV");
-pruefe("und das wird auch so angezeigt", geheim.stand().PAYPAL_CLIENT_ID.quelle === "server");
+pruefe("und die Anzeige nennt dann den Server",
+  geheim.stand().PAYPAL_CLIENT_ID.quelle === "server");
+
 delete process.env.PAYPAL_CLIENT_ID;
+await geheim.einstellungSetzen("PAYPAL_CLIENT_ID", "AbC-123");
 pruefe("faellt der Server-Wert weg, greift wieder die Verwaltung",
   geheim.einstellung("PAYPAL_CLIENT_ID") === "AbC-123");
 
@@ -89,5 +114,20 @@ pruefe("das Laden mit falschem Schluessel wirft nicht", !gestuerzt);
 pruefe("der Wert gilt dann als nicht vorhanden", geheim.einstellung("PAYPAL_CLIENT_SECRET") === "");
 
 L(`\n===== ${ok} bestanden, ${fehler} fehlgeschlagen =====`);
-process.on("exit", () => rmSync(tmp, { recursive: true, force: true }));
+/* Erst die Datenbank schliessen, DANN das Verzeichnis loeschen.
+
+   Unter Windows haelt eine offene SQLite-Datei eine Sperre auf ihre
+   Datei. Der Loeschversuch scheiterte dadurch mit EPERM, und der
+   Testlauf endete mit einer Fehlermeldung -- obwohl jede einzelne
+   Pruefung gruen war. Auf Linux faellt das nicht auf, weil sich dort
+   offene Dateien loeschen lassen: Dasselbe Skript haette je nach
+   Rechner ein anderes Ergebnis geliefert.
+
+   Das Aufraeumen selbst darf den Lauf nie zum Scheitern bringen. Ein
+   liegengebliebenes Temp-Verzeichnis ist ein Schoenheitsfehler, kein
+   Testergebnis. */
+process.on("exit", () => {
+  try { db.close(); } catch (e) { /* war schon zu */ }
+  try { rmSync(tmp, { recursive: true, force: true }); } catch (e) { /* egal */ }
+});
 process.exitCode = fehler ? 1 : 0;

server-internal/test-webdesign-portal.mjs

@@ -200,7 +200,14 @@ console.log("\n4. Was Anna sehen darf");
   pruefe("sieht ihre Angebotsdatei", t.includes("Annas Angebot.pdf"));
   pruefe("sieht INTERNE Arbeitsdateien NICHT", !t.includes("INTERNE-ARBEITSDATEI"),
     "-> fuer_kunde = 0 bleibt unsichtbar");
-  pruefe("Betrag wird lesbar mitgeliefert", /1500,00/.test(t));
+  /* 1.500,00 -- MIT Tausenderpunkt.
+     Der Test suchte frueher nach "1500,00" und schlug fehl, seit der
+     Formatierer den Punkt setzt. Er war veraltet, nicht der Code:
+     "1.500,00 EUR" ist die korrekte deutsche Schreibweise, und gerade
+     bei Betraegen ab vier Stellen ist der Punkt das, was eine Zahl
+     ueberhaupt auf einen Blick lesbar macht. */
+  pruefe("Betrag wird lesbar mitgeliefert", /1\.500,00/.test(t),
+    "-> mit Tausenderpunkt");
   pruefe("Speichername der Datei wird NICHT verraten", !t.includes("x1.pdf"),
     "-> interner Dateiname geht niemanden etwas an");
 }