DREI BLOECKE AUS DEM PERFEKTIONSLAUF.
1. HOCHLADEN MIT FORTSCHRITT UND ABBRUCH
Alle fuenf Wege (Dateien, Chat-Anhang, Wissen-PDF, Aufgaben-Anhang,
Profilbild) benutzten `fetch`. Das kann beim SENDEN nicht sagen, wie
weit es ist -- sichtbar war "wird hochgeladen …", von der ersten bis
zur letzten Sekunde gleich. Bei 40 MB im Mobilfunknetz zwei Minuten.
Wer das sieht, drueckt noch einmal und laedt dieselbe Datei doppelt.
Neu: workspace/assets/js/hochladen.js (XMLHttpRequest, das Einzige,
was `upload.onprogress` kann) samt gemeinsamer Anzeige.
Drei Ausgaenge: fertig / abgebrochen / schiefgegangen -- und ein
Abbruch ist KEIN Fehler und bekommt keine rote Meldung.
DABEI AUFGEFALLEN: KEINE EINZIGE PRUEFUNG im Haus laedt eine Datei
ueber die Oberflaeche hoch. Der ganze Umbau waere gruen gewesen,
ohne dass ein Byte je den Weg der Nutzer gegangen waere.
Neu: server/pruef-hochladen.mjs -- 18/0, mit echter Datei.
Zwei Irrtuemer beim Bauen, beide gemessen statt vermutet:
Ohne Drosselung gibt es auf localhost EINEN Fortschritt-Stand.
Das sah nach Befund aus und war keiner. Jetzt 2 MBit/s ueber
CDP -- derselbe Verlauf wie bei den Modis im Mobilfunk, 57
gemessene Zwischenstaende.
Gewartet wurde auf den Dateinamen "irgendwo im Dokument" -- der
stand auch im Fortschrittsbalken. Die Bedingung war erfuellt,
bevor etwas angekommen war.
2. LEERZUSTAENDE
Elf von 18 Brettern fielen auf "Noch kein Eintrag in diesem
Bereich" zurueck. Am ersten Tag ist ALLES leer -- wer da achtzehn
Bretter oeffnet und achtzehnmal denselben Satz liest, lernt nichts
ueber die Bretter, sondern dass das System kaputt ist. Jeder Satz
sagt jetzt, was hier hingehoert UND was der naechste Schritt ist.
Neu: server/pruef-leerzustand.mjs -- 13/0, leitet die Bretter aus
BEREICHE ab; ein neunzehntes ohne Satz macht sie rot.
3. ABMELDEN UND KONTRASTMODUS
Abmelden war am Handy ein 44-Pixel-Zeichen neben Glocke und Suche,
sofort wirksam. Teurer als es aussieht: Zum Wiederanmelden braucht
man den Zugangscode, und den gibt es EINMAL. Jetzt mit Rueckfrage,
die genau das sagt -- und dazu, dass Zumachen reicht (12 Stunden).
Kontrastmodus: 68 Regeln zeigen einen Zustand NUR ueber Farbe
(35x aria-pressed, 33x data-an). Der Modus ersetzt alle Farben und
entfernt box-shadow -- gedrueckt sah aus wie nicht gedrueckt.
14 CSS-Dateien hatten gar keinen Block. Statt 14 Bloecke zu pflegen
eine Regel in gate.css, die den ZUSTAND trifft statt die Datei.
Gemessen mit forcedColors: active -- vorher ununterscheidbar,
jetzt `solid 2px Highlight`.
Was seinen Zustand als WORT traegt (.marke-status, .t-stufe,
.spalte), braucht nichts -- nachgesehen, nicht vermutet.
PRUEFUNGEN, DIE AUF confirm() WARTETEN: Fuenf Dateien benutzten
`seite.once("dialog", d => d.accept())`. Playwright faengt confirm()
selbst ab, einen <dialog> nicht -- pruef-chat-anhaenge meldete acht
Fehler, keiner davon im Code. Neu: server/helfer-nachfrage.mjs, der
beide Wege kennt (auch den Notnagel fuer Safari vor 15.4).
pruef-chat-anhaenge, -ausbau, -optik und pruef-code wieder gruen.
hilfeAufraeumen bleibt ausgeschaltet -- das loescht echte Daten und
ist Filipes Entscheidung.
319 lines
14 KiB
JavaScript
319 lines
14 KiB
JavaScript
/* =====================================================================
|
||
pruef-hochladen.mjs — Hochladen, das man sehen und abbrechen kann.
|
||
|
||
Filipe, 19.09.2026: „Ladezustände … Bestätigungen … Es darf keine
|
||
Funktionen geben, bei denen man sich fragt: Was soll ich hier jetzt
|
||
machen?"
|
||
|
||
---------------------------------------------------------------------
|
||
WARUM ES DIESE PRÜFUNG GEBEN MUSS
|
||
|
||
Am 19.09.2026 wurden alle fünf Hochladewege von `fetch` auf
|
||
`XMLHttpRequest` umgebaut -- `fetch` kann beim SENDEN nicht sagen,
|
||
wie weit es ist. Danach fiel beim Nachsehen auf:
|
||
|
||
**Keine einzige Prüfung im Haus lädt eine Datei über die
|
||
Oberfläche hoch.**
|
||
|
||
Gesucht wurde nach `feld-datei`, `n-datei`, `sb-datei`,
|
||
`anhang-feld` -- ein Treffer, und der prüft nur, dass das Feld
|
||
existiert. Der komplette Umbau wäre also grün gewesen, ohne dass
|
||
ein einziges Byte je den Weg gegangen wäre, den die Nutzer gehen.
|
||
|
||
Das ist die dritte Sorte Fehler aus den Hausregeln: nicht ein
|
||
übersprungener Test, nicht ein grüner, der das Falsche prüft --
|
||
sondern ein Weg, für den es überhaupt keinen Test gab.
|
||
|
||
---------------------------------------------------------------------
|
||
WAS GEPRÜFT WIRD, UND ZWAR AM ECHTEN BILDSCHIRM
|
||
|
||
1. Die Datei kommt WIRKLICH an -- in der Liste und auf der Platte.
|
||
2. Der Balken erscheint, und sein Text ÄNDERT sich unterwegs.
|
||
(Nur „er ist da" wäre zu wenig: Genau das konnte die alte
|
||
Fassung auch. Der Unterschied ist die Bewegung.)
|
||
3. Abbrechen bricht wirklich ab -- die Datei kommt NICHT an.
|
||
4. Und der Abbruch erzeugt KEINE rote Fehlermeldung. Wer selbst
|
||
abbricht, hat nichts falsch gemacht; eine rote Meldung dafür
|
||
lehrt nur, rote Meldungen zu übersehen.
|
||
|
||
---------------------------------------------------------------------
|
||
DIE LEITUNG WIRD GEDROSSELT, SONST MISST DIESE PRÜFUNG NICHTS
|
||
|
||
Erster Anlauf war eine 12-MB-Datei ohne Drosselung. Ergebnis: EIN
|
||
einziger Fortschritt-Stand, kein Prozentsatz. Das sah nach einem
|
||
Befund aus und war keiner -- auf `localhost` gibt der Browser die
|
||
Daten in einem Atemzug ans Betriebssystem, und das meldet sofort
|
||
"gesendet". Es gab schlicht nichts zu sehen.
|
||
|
||
Jetzt wird über CDP auf 2 MBit/s gedrosselt. Das erzeugt genau den
|
||
Verlauf, den Filipes Modis im Mobilfunknetz haben -- also den Fall,
|
||
für den der Balken überhaupt gebaut wurde. 1,5 MB brauchen dabei
|
||
rund sechs Sekunden: lang genug für 57 gemessene Zwischenstände,
|
||
kurz genug für einen Prüflauf, den man abwartet.
|
||
|
||
ZWEITER IRRTUM, gleich daneben: Gewartet wurde zuerst auf
|
||
`document.body.textContent.includes(name)` -- und der Dateiname
|
||
stand auch im Fortschrittsbalken. Die Bedingung war erfüllt, bevor
|
||
ein einziges Byte angekommen war. Gewartet wird jetzt auf die
|
||
LISTE.
|
||
|
||
AUFRUF: node server/pruef-hochladen.mjs
|
||
===================================================================== */
|
||
|
||
import { mkdtempSync, rmSync, readdirSync } from "node:fs";
|
||
import { tmpdir } from "node:os";
|
||
import { join } from "node:path";
|
||
|
||
const ordner = mkdtempSync(join(tmpdir(), "ws-hochlade-"));
|
||
process.env.WORKSPACE_DB = join(ordner, "workspace.db");
|
||
|
||
import { notbremse } from "./helfer-notbremse.mjs";
|
||
const { portMussFreiSein } = await import("./helfer-port.mjs");
|
||
const PORT = await portMussFreiSein(4327, "die Hochladepruefung");
|
||
|
||
process.env.PORT = String(PORT);
|
||
process.env.SITE_ACCESS_SECRET = "lokaler-test";
|
||
process.env.SITE_PUBLIC_LAUNCH_AT = "2020-01-01T00:00:00+01:00";
|
||
|
||
const express = (await import("express")).default;
|
||
const ec = express.response.cookie;
|
||
express.response.cookie = function (n, w, o) { return ec.call(this, n, w, { ...(o || {}), secure: false }); };
|
||
await import("./index.js");
|
||
notbremse(300_000, "pruef-hochladen");
|
||
await new Promise((r) => setTimeout(r, 1000));
|
||
const BASIS = `http://127.0.0.1:${PORT}`;
|
||
|
||
process.on("uncaughtException", (f) => { console.error("ABSTURZ:", f); process.exit(7); });
|
||
process.on("unhandledRejection", (f) => { console.error("ABSTURZ (Versprechen):", f); process.exit(7); });
|
||
|
||
let fehler = 0, geprueft = 0;
|
||
const ok = (b, t) => { geprueft++; console.log((b ? " ok " : " FEHL ") + t); if (!b) fehler++; };
|
||
|
||
/* ---- Bestand ------------------------------------------------------- */
|
||
const { DatabaseSync } = await import("node:sqlite");
|
||
const { scryptSync, randomBytes } = await import("node:crypto");
|
||
await fetch(BASIS + "/workspace/api/anmelden", {
|
||
method: "POST", headers: { "Content-Type": "application/json" },
|
||
body: JSON.stringify({ rolle: "admin", code: "x" }) }).catch(() => {});
|
||
for (let i = 0; i < 60; i++) {
|
||
try {
|
||
const p = new DatabaseSync(process.env.WORKSPACE_DB);
|
||
try { p.prepare("SELECT 1 FROM personen LIMIT 1").get(); break; } finally { p.close(); }
|
||
} catch { await new Promise((r) => setTimeout(r, 250)); }
|
||
}
|
||
const d = new DatabaseSync(process.env.WORKSPACE_DB);
|
||
const jetzt = new Date().toISOString();
|
||
const salt = randomBytes(16).toString("hex");
|
||
const hash = scryptSync("CODE-DOGI-0001", salt, 64,
|
||
{ N: 32768, r: 8, p: 1, maxmem: 96 * 1024 * 1024 }).toString("hex");
|
||
d.prepare("INSERT INTO personen (name, rolle, code_hash, code_salt, code_n, aktiv, erstellt)"
|
||
+ " VALUES (?,?,?,?,?,1,?)").run("Filipe", "admin", hash, salt, 32768, jetzt);
|
||
d.close();
|
||
|
||
const pw = await import("file:///C:/Users/qciga/Documents/Obelix/Analyse/node_modules/playwright/index.mjs");
|
||
const browser = await pw.chromium.launch();
|
||
const ctx = await browser.newContext({ viewport: { width: 1280, height: 900 } });
|
||
const seite = await ctx.newPage();
|
||
|
||
/* DIE LEITUNG DROSSELN -- sonst misst diese Prüfung nichts.
|
||
------------------------------------------------------------------
|
||
Auf `localhost` sind 12 MB in einem Atemzug weg: Der Browser gibt
|
||
sie dem Betriebssystem, das meldet sofort "gesendet", und es gibt
|
||
GENAU EIN Fortschritt-Ereignis bei 100 %. Die Prüfung wäre rot --
|
||
nicht weil der Code kaputt ist, sondern weil es nichts zu sehen gab.
|
||
|
||
Gedrosselt entsteht derselbe Verlauf wie bei Filipes Modis im
|
||
Mobilfunknetz -- also genau der Fall, für den der Balken gebaut
|
||
wurde. 2 MBit/s ist ein schlechtes, aber realistisches Netz. */
|
||
const cdp = await ctx.newCDPSession(seite);
|
||
await cdp.send("Network.enable");
|
||
await cdp.send("Network.emulateNetworkConditions", {
|
||
offline: false, latency: 40,
|
||
downloadThroughput: 4 * 1024 * 1024 / 8,
|
||
uploadThroughput: 2 * 1024 * 1024 / 8,
|
||
});
|
||
const konsole = [];
|
||
seite.on("pageerror", (f) => konsole.push("Absturz: " + f.message));
|
||
|
||
await seite.goto(BASIS + "/workspace/", { waitUntil: "networkidle" });
|
||
await seite.click('.rolle[data-rolle="admin"]');
|
||
await seite.fill("#code", "CODE-DOGI-0001");
|
||
await seite.click("button[type=submit]");
|
||
await seite.waitForURL("**/start.html", { timeout: 25000 });
|
||
await seite.goto(BASIS + "/workspace/dateien.html", { waitUntil: "networkidle" });
|
||
await seite.waitForTimeout(700);
|
||
|
||
console.log("\n=== Hochladen: sieht man es, und kann man es abbrechen? ===\n");
|
||
|
||
ok(await seite.$("#feld-datei") !== null, "die Seite hat ein Dateifeld");
|
||
ok(await seite.evaluate(() => typeof window.hochladen === "function"),
|
||
"hochladen() ist geladen");
|
||
ok(await seite.evaluate(() => typeof window.hochladeAnzeige === "function"),
|
||
"und die Anzeige dazu auch");
|
||
|
||
/* ---------------------------------------------------------------------
|
||
1+2. EINE ECHTE DATEI, MIT BEWEGUNG
|
||
------------------------------------------------------------------ */
|
||
|
||
/** Legt eine Datei im Browser an und hängt sie ins Feld -- so, wie es
|
||
* ein Mensch mit dem Dateiauswahlfenster täte. `setInputFiles` mit
|
||
* einem Puffer geht denselben Weg wie ein echter Griff zur Platte. */
|
||
/* Gedrosselt auf 2 MBit/s braucht 1,5 MB rund sechs Sekunden -- lang
|
||
genug für viele Fortschritt-Ereignisse, kurz genug für einen
|
||
Prüflauf, den man abwartet. */
|
||
const GROSS = 1536 * 1024;
|
||
|
||
/* Die Texte MITSCHREIBEN, waehrend es laeuft. Nur nachher zu schauen
|
||
waere sinnlos -- dann ist der Balken schon weg. */
|
||
await seite.evaluate(() => {
|
||
window.__gesehen = [];
|
||
const ziel = document.getElementById("fortschritt");
|
||
new MutationObserver(() => {
|
||
const t = ziel.textContent.trim();
|
||
if (t && window.__gesehen[window.__gesehen.length - 1] !== t) window.__gesehen.push(t);
|
||
}).observe(ziel, { childList: true, subtree: true, characterData: true });
|
||
});
|
||
|
||
await seite.setInputFiles("#feld-datei", {
|
||
name: "Kampagne Oktober.pdf",
|
||
mimeType: "application/pdf",
|
||
buffer: Buffer.alloc(GROSS, 7),
|
||
});
|
||
|
||
/* Warten, bis die Datei in der Liste steht -- nicht auf eine feste
|
||
Zeit. Eine feste Zahl waere wieder "eine Rechnung von gestern". */
|
||
/* AUF DIE LISTE WARTEN, nicht auf irgendeinen Text im Dokument.
|
||
Erster Anlauf war `document.body.textContent.includes(name)` -- und
|
||
der Name stand auch im Fortschrittsbalken. Die Bedingung war damit
|
||
erfüllt, BEVOR ein einziges Byte angekommen war, und die Prüfung
|
||
maß einen Balken, der noch lief. */
|
||
await seite.waitForFunction(
|
||
() => [...document.querySelectorAll(".datei")]
|
||
.some((e) => e.textContent.includes("Kampagne Oktober.pdf")),
|
||
{ timeout: 120_000 });
|
||
|
||
const gesehen = await seite.evaluate(() => window.__gesehen);
|
||
ok(gesehen.length > 0, "der Balken war überhaupt zu sehen");
|
||
/* DIE ZAHL GEHOERT IN DIE BEDINGUNG: Bei einer leeren Liste waere
|
||
"alle Texte waren verschieden" trivialerweise wahr. */
|
||
ok(gesehen.length >= 2,
|
||
`und sein Text hat sich unterwegs geändert (${gesehen.length} verschiedene Stände)`);
|
||
const mitProzent = gesehen.filter((t) => /\d+\s*%/.test(t));
|
||
ok(mitProzent.length > 0,
|
||
`es stand ein Prozentsatz da (${mitProzent.length}× — z. B. „${mitProzent[0] || "—"}")`);
|
||
ok(gesehen.some((t) => /Abbrechen/.test(t)), "und ein Abbrechen-Knopf daneben");
|
||
|
||
/* Die Datei ist wirklich da -- nicht nur in der Liste, auch auf der Platte. */
|
||
const abgelegt = (() => {
|
||
try { return readdirSync(join(ordner, "dateien")); } catch { return []; }
|
||
})();
|
||
ok(abgelegt.length >= 1, `die Datei liegt wirklich auf der Platte (${abgelegt.length})`);
|
||
|
||
const listeNachher = await seite.evaluate(() =>
|
||
[...document.querySelectorAll(".datei")].length);
|
||
ok(listeNachher === 1, `und genau einmal in der Liste (${listeNachher})`);
|
||
|
||
/* ---------------------------------------------------------------------
|
||
3+4. ABBRECHEN
|
||
------------------------------------------------------------------ */
|
||
console.log("\n— Abbrechen —");
|
||
|
||
/* Warten, bis die Anzeige des ERSTEN Laufs weg ist. Sonst klickt der
|
||
Abbruch auf einen Knopf, der im selben Moment ersetzt wird --
|
||
Playwright meldet dann "element was detached from the DOM". Das sah
|
||
nach einem Befund aus und war eine Wettlaufsituation in der
|
||
Prüfung selbst. */
|
||
await seite.waitForSelector(".hochlade__abbruch", { state: "detached", timeout: 15_000 })
|
||
.catch(() => {});
|
||
await seite.evaluate(() => { window.__gesehen = []; });
|
||
await seite.setInputFiles("#feld-datei", {
|
||
name: "Wird abgebrochen.pdf",
|
||
mimeType: "application/pdf",
|
||
buffer: Buffer.alloc(GROSS, 3),
|
||
});
|
||
|
||
/* Auf den Abbruchknopf warten und ihn druecken, waehrend es laeuft. */
|
||
let abbruchDa = true;
|
||
try {
|
||
await seite.waitForSelector(".hochlade__abbruch", { timeout: 10_000 });
|
||
} catch { abbruchDa = false; }
|
||
ok(abbruchDa, "während es läuft, steht ein Abbrechen-Knopf da");
|
||
|
||
if (abbruchDa) {
|
||
await seite.click(".hochlade__abbruch");
|
||
await seite.waitForTimeout(1500);
|
||
|
||
const listeDanach = await seite.evaluate(() =>
|
||
[...document.querySelectorAll(".datei")].map((e) => e.textContent).join(" | "));
|
||
ok(!listeDanach.includes("Wird abgebrochen"),
|
||
"die abgebrochene Datei ist NICHT in der Liste gelandet");
|
||
|
||
const aufPlatte = (() => {
|
||
try { return readdirSync(join(ordner, "dateien")).length; } catch { return 0; }
|
||
})();
|
||
ok(aufPlatte === abgelegt.length,
|
||
`und auch nicht auf der Platte (${aufPlatte}, vorher ${abgelegt.length})`);
|
||
|
||
/* DER WICHTIGSTE TEIL: kein roter Fehler fuer etwas, das der Nutzer
|
||
selbst getan hat. */
|
||
const rot = await seite.evaluate(() =>
|
||
document.getElementById("fehler")?.textContent?.trim() || "");
|
||
ok(rot === "", `keine rote Fehlermeldung nach dem eigenen Abbruch (stand: „${rot}")`);
|
||
|
||
const schluss = await seite.evaluate(() => window.__gesehen.join(" ‖ "));
|
||
ok(/[Aa]bgebrochen/.test(schluss),
|
||
"stattdessen ein ruhiger Satz, dass abgebrochen wurde");
|
||
}
|
||
|
||
/* ---------------------------------------------------------------------
|
||
DIE GEGENPROBEN
|
||
------------------------------------------------------------------ */
|
||
console.log("\n --- Gegenproben (jede MUSS anschlagen) ---");
|
||
|
||
/* Ohne `beiFortschritt` darf KEIN Prozentsatz erscheinen -- sonst
|
||
misst die Prüfung oben etwas, das auch ohne den Umbau dastünde. */
|
||
const ohneMelder = await seite.evaluate(async () => {
|
||
const p = document.createElement("p");
|
||
document.body.append(p);
|
||
const a = window.hochladeAnzeige(p, "Probe.pdf", null);
|
||
a.fortschritt(undefined);
|
||
const text = p.textContent;
|
||
a.weg();
|
||
p.remove();
|
||
return text;
|
||
});
|
||
ok(!/\d+\s*%/.test(ohneMelder),
|
||
`ohne bekannte Größe steht KEINE erfundene Prozentzahl da („${ohneMelder.trim()}")`);
|
||
|
||
const mitMelder = await seite.evaluate(async () => {
|
||
const p = document.createElement("p");
|
||
document.body.append(p);
|
||
const a = window.hochladeAnzeige(p, "Probe.pdf", null);
|
||
a.fortschritt(0.42, 42, 100);
|
||
const text = p.textContent;
|
||
a.weg();
|
||
p.remove();
|
||
return text;
|
||
});
|
||
ok(/42\s*%/.test(mitMelder), `mit bekannter Größe schon („${mitMelder.trim()}")`);
|
||
|
||
/* Und die Anzeige verschwindet wirklich. */
|
||
const wegDanach = await seite.evaluate(() => {
|
||
const p = document.createElement("p");
|
||
document.body.append(p);
|
||
const a = window.hochladeAnzeige(p, "x", null);
|
||
a.weg();
|
||
const r = { versteckt: p.hidden, leer: p.textContent === "" };
|
||
p.remove();
|
||
return r;
|
||
});
|
||
ok(wegDanach.versteckt && wegDanach.leer, "und sie räumt sich hinterher auf");
|
||
|
||
ok(konsole.length === 0, `keine Fehler in der Konsole${konsole.length ? ": " + konsole[0] : ""}`);
|
||
|
||
/* ---- Ende ---------------------------------------------------------- */
|
||
await browser.close();
|
||
console.log(`\n${fehler ? fehler + " FEHLER" : "Alles in Ordnung"} — ${geprueft} Prüfungen\n`);
|
||
try { rmSync(ordner, { recursive: true, force: true }); } catch { /* egal */ }
|
||
process.exit(fehler ? 1 : 0);
|