Files
werkbaum/frontend/tests/ids.test.js
T
mhoennigandClaude Opus 5 3018db9f61 notation: ohne Titel vertritt die Knoten-ID ihn (SPEC §1, D60)
`- #US-123` ergab bisher nichts — leeres Label, Zeile ignoriert, ID nicht
vergeben. Wo die Kennung schon der Name ist (Ticket-Referenzen, §11), war das
verkehrt herum: Wer den Titel nicht danebenschrieb, verlor den ganzen Knoten.
Jetzt wird `#id` das Label, mit Doppelpunkt geschrieben wie ohne.

Das `#` bleibt im Label — es sagt „hier steht die Adresse, weil es keinen
Titel gibt". Der `#`-Umschalter setzt bei so einem Knoten nichts davor
(sonst `#US-123: #US-123`), und Tooltip wie aria-label lassen die ID weg, die
schon im Titel steht.

Verhaltensänderung: Die ID ist damit vergeben — `- #auth` gefolgt von
`- [ ] Echt #auth` gibt jetzt zwei Knoten und eine duplicateId-Warnung.

SPEC §1, llms.md, D60; 6 neue Tests, der alte auf die neue Regel umgeschrieben.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-08-24 13:31:39 +02:00

156 lines
6.3 KiB
JavaScript

import { describe, it, expect } from 'vitest';
import { parse } from '../src/parser.js';
import { renderTreeHtml } from '../src/render.js';
const t = key => key;
const roots = txt => parse(txt).roots;
const render = txt => renderTreeHtml(roots(txt),
{t, showDiscarded: false, cheapPath: false, cheapSet: new Set()});
/* Knoten-IDs `#name` (SPEC §1, D36): Adresse für Abhängigkeiten und
Beschreibungsblöcke; eindeutig im ganzen Dokument. */
describe('Parser — `#name` als Knoten-ID', () => {
it('extrahiert die ID und nimmt sie aus dem Label', () => {
const [wurzel] = roots(`[ ] Auth-Modul #auth (M)\n - [ ] Login`);
expect([wurzel.label, wurzel.id, wurzel.size]).toEqual(['Auth-Modul', 'auth', 'M']);
expect(wurzel.children[0].id).toBe(null);
});
it('erkennt IDs nur alleinstehend angesetzt — „C#" bleibt Label', () => {
const [wurzel] = roots(`[ ] C# Kurs`);
expect([wurzel.label, wurzel.id]).toEqual(['C# Kurs', null]);
});
it('macht aus `:#a,#b` (Abhängigkeiten, D37) keine Knoten-ID', () => {
const [wurzel] = roots(`[ ] Deploy :#auth,#api`);
expect([wurzel.label, wurzel.id, wurzel.deps])
.toEqual(['Deploy', null, ['auth', 'api']]);
});
it('nimmt nur das ERSTE Token — weitere `#` bleiben im Label (Tickets)', () => {
const [wurzel] = roots(`[ ] Auth #auth siehe #123`);
expect([wurzel.label, wurzel.id]).toEqual(['Auth siehe #123', 'auth']);
});
it('lässt `#` in URLs unberührt (URL wird vorher extrahiert)', () => {
const [wurzel] = roots(`[ ] Doku https://example.org/seite#abschnitt`);
expect([wurzel.id, wurzel.url]).toEqual([null, 'https://example.org/seite#abschnitt']);
});
it('erlaubt numerische und Unicode-IDs', () => {
const nodes = roots(`[ ] Ticket #123\n[ ] Umlaut #größe-1`);
expect(nodes.map(n => n.id)).toEqual(['123', 'größe-1']);
});
/* Bis D60 war so eine Zeile keine: Sie wurde ignoriert und die ID blieb
frei. Jetzt vertritt die ID den fehlenden Titel — gedacht für den Fall,
dass die Kennung schon der Name ist (Ticket-Referenzen). */
it('macht aus einer Zeile mit NUR einer ID einen Knoten mit `#id` als Titel', () => {
const {roots: r, warnings} = parse('- #auth');
expect(r.map(n => [n.label, n.id, n.labelFromId])).toEqual([['#auth', 'auth', true]]);
expect(warnings).toEqual([]);
});
it('nimmt den Doppelpunkt dabei mit weg', () => {
expect(parse('- #US-123:').roots.map(n => n.label)).toEqual(['#US-123']);
});
it('vergibt die ID dabei wirklich — eine zweite meldet sich', () => {
const {roots: r, warnings} = parse('- #auth\n- [ ] Echt #auth');
expect(r.map(n => n.label)).toEqual(['#auth', 'Echt']);
expect(warnings).toEqual([{type: 'duplicateId', line: 2, id: 'auth', firstLine: 1}]);
});
it('setzt `labelFromId` NICHT, wenn ein Titel dasteht', () => {
expect(parse('- #auth: Backend').roots[0].labelFromId).toBe(false);
});
/* Die übrigen Bestandteile werden ganz normal gelesen — übrig bleibt nur
kein Titel. */
it('liest Größe, Status und Tag auch ohne Titel', () => {
const n = parse('- [x] #US-123 (L) @anna').roots[0];
expect([n.label, n.size, n.tags, n.status.key]).toEqual(['#US-123', 'L', ['anna'], 'fertig']);
});
it('lässt eine Zeile ohne ID und ohne Label weiterhin weg', () => {
expect(parse('- (L) @anna').roots).toEqual([]);
});
});
/* Übliche Schreibweise: ID vor dem Titel, abgetrennt durch einen Doppelpunkt.
Der Doppelpunkt ist optional, gehört weder zur ID noch zum Label. */
describe('ID vor dem Titel, `#id: Titel`', () => {
it('schluckt den trennenden Doppelpunkt', () => {
const [n] = roots(`[ ] #auth: Auth-Modul (M)`);
expect([n.id, n.label, n.size]).toEqual(['auth', 'Auth-Modul', 'M']);
});
it('kommt ohne Doppelpunkt zum selben Ergebnis', () => {
const mit = roots(`[ ] #auth: Auth-Modul (M)`)[0];
const ohne = roots(`[ ] #auth Auth-Modul (M)`)[0];
expect([mit.id, mit.label]).toEqual([ohne.id, ohne.label]);
});
it('lässt den Doppelpunkt auch am Zeilenende weg', () => {
const [n] = roots(`[ ] Titel danach #auth:`);
expect([n.id, n.label]).toEqual(['auth', 'Titel danach']);
});
it('vertägt sich mit Abhängigkeiten in derselben Zeile', () => {
const {roots: r, warnings} = parse(`[ ] #api: API\n[ ] #ui: Oberfläche :#api`);
expect(r.map(n => [n.id, n.label, n.deps])).toEqual([
['api', 'API', []], ['ui', 'Oberfläche', ['api']],
]);
expect(warnings).toEqual([]);
});
it('lässt einen Doppelpunkt IM Label unberührt', () => {
const [n] = roots(`[ ] #auth: Regel: nur mit Token`);
expect([n.id, n.label]).toEqual(['auth', 'Regel: nur mit Token']);
});
it('schluckt nur einen Doppelpunkt mit folgendem Leerraum — `#auth:#db` bleibt Abhängigkeit', () => {
const {roots: r} = parse(`[ ] #db: Datenbank\n[ ] #auth:#db Login`);
expect(r[1].id).toBe('auth');
expect(r[1].deps).toEqual(['db']);
expect(r[1].label).toBe('Login');
});
it('nimmt den Doppelpunkt auch im Beschreibungsteil an', () => {
const {roots: r, warnings} = parse(`[ ] #auth: Auth\n---\n#auth:\n Erklärung.`);
expect(r[0].desc).toBe('Erklärung.');
expect(warnings).toEqual([]);
});
});
describe('Doppelte IDs — Warnung an der späteren Zeile', () => {
it('meldet die spätere Zeile und nennt die erste', () => {
const {warnings} = parse(`[ ] A #auth\n[ ] B\n[ ] C #auth`);
expect(warnings).toEqual([{type: 'duplicateId', line: 3, id: 'auth', firstLine: 1}]);
});
it('lässt die spätere ID trotzdem am Knoten stehen (fehlertolerant)', () => {
const nodes = roots(`[ ] A #x\n[ ] B #x`);
expect(nodes.map(n => n.id)).toEqual(['x', 'x']);
});
it('meldet jede weitere Doppelvergabe einzeln', () => {
const {warnings} = parse(`[ ] A #x\n[ ] B #x\n[ ] C #x`);
expect(warnings.map(w => w.line)).toEqual([2, 3]);
});
});
describe('Darstellung — Tooltip und aria, kein Badge', () => {
it('zeigt die ID im Tooltip und im aria-label', () => {
const {html} = render(`[ ] Auth #auth`);
expect(html).toContain('data-tip="#auth · st_geplant · jumpHint"');
expect(html).toContain('aria-label="Auth, a11yStatus, a11yId"');
});
it('ändert Knoten ohne ID nicht', () => {
const {html} = render(`[ ] Ohne`);
expect(html).not.toContain('a11yId');
expect(html).toContain('data-tip="st_geplant · jumpHint"');
});
});