Changeset 3bbf73d in Klonkt for test

Timestamp:

08/10/2026 02:43:16 PM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

f0a33e1

Parents:

f549cf5

Message:

Alleen namen: het Nederlands van vandaag terug naar Engels

shaer-8ms zegt het al: Engelse namen, Nederlands commentaar waar dat helpt. Ik
ben er vandaag opnieuw van afgedreven en zet dat recht voordat er meer bovenop
komt -- gatesFor, messageItem, sentItem, conversationItems, ownHandle,
CONVERSATION_UNION, MESSAGE_COLUMNS, cursorOf, en de locals eromheen
(rows/more/oldest/direction/incoming/heads/self/size).

Geen gedrag, alleen namen, en daarom een eigen commit: die is na te lopen.
851/851 groen voor en na. Het commentaar blijft Nederlands.

Co-Authored-By: Claude Opus 5 <noreply@…>

Location:

test

Files:

• conversations.test.js (modified) (7 diffs)
• standard-shapes.test.js (modified) (1 diff)
• thread.test.js (modified) (5 diffs)

test/conversations.test.js

@@ -26 +26 @@
 const VREEMDE = 'https://elders/u/vreemde';
 
-const binnen = (uri, van, stamp) =>
+const arrived = (uri, van, stamp) =>
   db.prepare(`INSERT INTO ap_mentions (slug, object_uri, actor_uri, actor_name, content, published)
               VALUES ('kind', ?, ?, 'iemand', '<p>hoi</p>', ?)`).run(uri, van, stamp);
-const verstuurd = (id, naar, stamp, visibility = 'direct') =>
+const sent = (id, naar, stamp, visibility = 'direct') =>
   db.prepare(`INSERT INTO ap_outbox (id, site_slug, post_id, to_actor, to_actors, content, visibility, created_at)
               VALUES (?, 'kind', 'p1', ?, ?, '<p>terug</p>', ?, ?)`)
@@ -35 +35 @@
 
 // Een DRUK gesprek met oma: genoeg om in de oude lezing de rest weg te drukken.
-for (let i = 0; i < 80; i++) binnen(`https://elders/n/oma-${i}`, OMA, `2026-08-09T10:${String(i % 60).padStart(2, '0')}:00Z`);
+for (let i = 0; i < 80; i++) arrived(`https://elders/n/oma-${i}`, OMA, `2026-08-09T10:${String(i % 60).padStart(2, '0')}:00Z`);
 // En een stil gesprek met tante, ouder dan al die drukte.
-binnen('https://elders/n/tante-1', TANTE, '2026-08-01T09:00:00Z');
-verstuurd('mijn-1', TANTE, '2026-08-01T09:30:00Z');
+arrived('https://elders/n/tante-1', TANTE, '2026-08-01T09:00:00Z');
+sent('mijn-1', TANTE, '2026-08-01T09:30:00Z');
 // Een PUBLIEK antwoord aan een vreemde is geen gesprek.
-verstuurd('mijn-publiek', VREEMDE, '2026-08-09T12:00:00Z', 'public');
+sent('mijn-publiek', VREEMDE, '2026-08-09T12:00:00Z', 'public');
 
 test('de oude lezing verliest tante achter een druk gesprek -- dat is de bug', () => {
@@ -49 +49 @@
 
 test('een rij per tegenpartij, hoe druk de drukste ook is', () => {
-  const koppen = AP.conversationHeads('kind');
-  assert.deepEqual(koppen.map((k) => k.other), [OMA, TANTE], 'nieuwste gesprek eerst, allebei aanwezig');
+  const heads = AP.conversationHeads('kind');
+  assert.deepEqual(heads.map((k) => k.other), [OMA, TANTE], 'nieuwste gesprek eerst, allebei aanwezig');
 });
 
@@ -58 +58 @@
 
 test('het gesprek draagt beide kanten onder EEN limiet', () => {
-  const g = AP.conversationHistory('kind', TANTE, { limit: 60 });
-  assert.equal(g.rijen.length, 2);
-  assert.deepEqual(g.rijen.map((r) => r.richting), ['uit', 'in'], 'nieuwste eerst, mijn antwoord bovenaan');
-  assert.equal(g.meer, false);
+  const talk = AP.conversationHistory('kind', TANTE, { limit: 60 });
+  assert.equal(talk.rows.length, 2);
+  assert.deepEqual(talk.rows.map((r) => r.direction), ['out', 'in'], 'nieuwste eerst, mijn antwoord bovenaan');
+  assert.equal(talk.more, false);
 });
 
@@ -67 +67 @@
   // De 80 berichten van oma delen stempels (i % 60), en dat is met opzet:
   // twee berichten in dezelfde seconde is bij DM's een gesprek, geen randgeval.
-  const eerste = AP.conversationHistory('kind', OMA, { limit: 30 });
-  assert.equal(eerste.rijen.length, 30);
-  assert.equal(eerste.meer, true, 'er is meer, en dat mag de client weten');
+  const first = AP.conversationHistory('kind', OMA, { limit: 30 });
+  assert.equal(first.rows.length, 30);
+  assert.equal(first.more, true, 'er is meer, en dat mag de client weten');
 
-  const tweede = AP.conversationHistory('kind', OMA, { limit: 30, before: eerste.oudste });
-  assert.equal(tweede.rijen.length, 30);
-  // Geen overlap en geen gat: de tweede pagina begint waar de eerste ophield.
-  const ids = new Set(eerste.rijen.map((r) => r.ref));
-  assert.ok(tweede.rijen.every((r) => !ids.has(r.ref)), 'geen dubbele');
+  const second = AP.conversationHistory('kind', OMA, { limit: 30, before: first.oldest });
+  assert.equal(second.rows.length, 30);
+  // Geen overlap en geen gat: de second pagina begint waar de first ophield.
+  const ids = new Set(first.rows.map((r) => r.ref));
+  assert.ok(second.rows.every((r) => !ids.has(r.ref)), 'geen dubbele');
 
-  const derde = AP.conversationHistory('kind', OMA, { limit: 30, before: tweede.oudste });
-  assert.equal(derde.rijen.length, 20, 'de staart');
-  assert.equal(derde.meer, false, 'en dan is het op');
+  const third = AP.conversationHistory('kind', OMA, { limit: 30, before: second.oldest });
+  assert.equal(third.rows.length, 20, 'de staart');
+  assert.equal(third.more, false, 'en dan is het op');
   // Alles bij elkaar: precies 80, geen dubbele en niets overgeslagen. Met een
   // cursor op alleen de stempel zou de grensseconde hier stil wegvallen.
-  const alles = new Set([...eerste.rijen, ...tweede.rijen, ...derde.rijen].map((r) => r.ref));
-  assert.equal(alles.size, 80, 'elke berichtje precies een keer');
+  const allRefs = new Set([...first.rows, ...second.rows, ...third.rows].map((r) => r.ref));
+  assert.equal(allRefs.size, 80, 'elke berichtje precies een keer');
 });
 
@@ -101 +101 @@
   t.after(() => server.close());
   await new Promise((r) => server.once('listening', r));
-  const haal = async (pad) => {
+  const get = async (pad) => {
     const r = await fetch(`http://127.0.0.1:${server.address().port}${pad}`,
       { headers: { Authorization: `Bearer ${bearer}` } });
@@ -108 +108 @@
   const slug = '/ap/users/kind';
 
-  const g = await haal(`${slug}/conversations`);
-  assert.equal(g.status, 200);
-  assert.equal(g.body.totalItems, 2, 'oma en tante, en niet de publieke vreemde');
+  const list = await get(`${slug}/conversations`);
+  assert.equal(list.status, 200);
+  assert.equal(list.body.totalItems, 2, 'oma en tante, en niet de publieke vreemde');
   // De byline zit in de ingesloten actor, net als overal sinds shaer-nmw.
-  assert.ok(g.body.orderedItems.every((i) => i.object.attributedTo));
+  assert.ok(list.body.orderedItems.every((i) => i.object.attributedTo));
 
-  const eerste = await haal(`${slug}/messages?with=${encodeURIComponent(OMA)}&limit=30`);
-  assert.equal(eerste.body.orderedItems.length, 30);
-  assert.ok(eerste.body.next, 'er is meer, en de standaardvorm zegt het');
-  assert.ok(eerste.body.next.includes('limit=30'), 'de paginagrootte reist mee');
+  const first = await get(`${slug}/messages?with=${encodeURIComponent(OMA)}&limit=30`);
+  assert.equal(first.body.orderedItems.length, 30);
+  assert.ok(first.body.next, 'er is meer, en de standaardvorm zegt het');
+  assert.ok(first.body.next.includes('limit=30'), 'de paginagrootte reist mee');
 
   // De next-link volgen doet wat hij belooft: geen dubbele, en uiteindelijk op.
-  const pad2 = eerste.body.next.replace(/^https?:\/\/[^/]+/, '');
-  const tweede = await haal(pad2);
-  const eersteIds = new Set(eerste.body.orderedItems.map((i) => i.object.id));
-  assert.ok(tweede.body.orderedItems.every((i) => !eersteIds.has(i.object.id)), 'geen overlap');
-  const derde = await haal(tweede.body.next.replace(/^https?:\/\/[^/]+/, ''));
-  assert.equal(derde.body.orderedItems.length, 20);
-  assert.equal(derde.body.next, undefined, 'op is op, en dat staat er ook');
+  const path2 = first.body.next.replace(/^https?:\/\/[^/]+/, '');
+  const second = await get(path2);
+  const firstIds = new Set(first.body.orderedItems.map((i) => i.object.id));
+  assert.ok(second.body.orderedItems.every((i) => !firstIds.has(i.object.id)), 'geen overlap');
+  const third = await get(second.body.next.replace(/^https?:\/\/[^/]+/, ''));
+  assert.equal(third.body.orderedItems.length, 20);
+  assert.equal(third.body.next, undefined, 'op is op, en dat staat er ook');
 
   // Een gesprek draagt beide kanten; het eigen antwoord aan tante hoort erbij.
-  const tante = await haal(`${slug}/messages?with=${encodeURIComponent(TANTE)}`);
-  assert.equal(tante.body.orderedItems.length, 2);
-  assert.ok(tante.body.orderedItems.some((i) => i.actor === 'https://klonkt.test/ap/users/kind'), 'mijn eigen kant zit erin');
+  const withAunt = await get(`${slug}/messages?with=${encodeURIComponent(TANTE)}`);
+  assert.equal(withAunt.body.orderedItems.length, 2);
+  assert.ok(withAunt.body.orderedItems.some((i) => i.actor === 'https://klonkt.test/ap/users/kind'), 'mijn eigen kant zit erin');
 
-  const stuk = await haal(`${slug}/messages?with=nonsens`);
-  assert.equal(stuk.status, 400, 'geen actor-uri, geen gesprek');
+  const broken = await get(`${slug}/messages?with=nonsens`);
+  assert.equal(broken.status, 400, 'geen actor-uri, geen gesprek');
 });

test/standard-shapes.test.js

@@ -117 +117 @@
     { headers: { Authorization: `Bearer ${bearer}` } })).json();
 
-  const tekst = JSON.stringify(doc);
-  // Op de SLEUTEL, niet op een stuk tekst: 'shaer:quote' zit ook in
+  const text = JSON.stringify(doc);
+  // Op de SLEUTEL, niet op een stuk text: 'shaer:quote' zit ook in
   // shaer:quoteCards, en dat is een poortnaam die blijft. Een te grove zeef
   // zou hier rood staan om iets dat klopt -- of erger, groen om iets dat niet
   // klopt.
-  for (const weg of ['shaer:author', 'shaer:booster', 'shaer:quote', 'shaer:embed']) {
-    assert.ok(!tekst.includes(`"${weg}":`), `${weg} hoort weg te zijn`);
+  for (const gone of ['shaer:author', 'shaer:booster', 'shaer:quote', 'shaer:embed']) {
+    assert.ok(!text.includes(`"${gone}":`), `${gone} hoort weg te zijn`);
   }
   // Wat WEL mag blijven, en waarom: per-lezer-interactiestatus heeft in AS2
   // geen tegenhanger (Mastodon lost het in zijn eigen REST-API op), en de
   // FEP-633c-termen zijn bewust van ons en in standaardisatie.
-  assert.ok(tekst.includes('shaer:liked'), 'interactiestatus blijft, die heeft geen standaard');
+  assert.ok(text.includes('shaer:liked'), 'interactiestatus blijft, die heeft geen standaard');
 
   const item = doc.orderedItems.find((i) => i.object.id === 'https://elders/n1');

test/thread.test.js

@@ -55 +55 @@
 // een ECHT verzoek aan een testserver op localhost, en dat mag de stub niet
 // beantwoorden met zijn 404.
-const echteFetch = globalThis.fetch;
+const realFetch = globalThis.fetch;
 
 globalThis.fetch = async (url) => {
@@ -171 +171 @@
   t.after(() => server.close());
   await new Promise((r) => server.once('listening', r));
-  const haal = async () => (await (await echteFetch(
+  const get = async () => (await (await realFetch(
     `http://127.0.0.1:${server.address().port}/ap/users/kind/thread?object=${encodeURIComponent(BRON)}`,
     { headers: { Authorization: `Bearer ${bearer}` } })).json());
 
-  const doc = await haal();
+  const doc = await get();
   assert.ok(!JSON.stringify(doc).includes('"shaer:author":'), 'geen dialect meer op deze leg');
-  const vanTante = doc.orderedItems.find((n) => n.id.endsWith('/replies/2'));
-  assert.equal(vanTante.attributedTo.name, 'tante', 'de byline zit in attributedTo');
-  assert.equal(vanTante.attributedTo.preferredUsername, 'tante');
-  assert.ok((vanTante.tag || []).some((x) => x.type === 'Emoji'), 'open poort: de emoji van het antwoord blijft');
+  const fromAunt = doc.orderedItems.find((n) => n.id.endsWith('/replies/2'));
+  assert.equal(fromAunt.attributedTo.name, 'tante', 'de byline zit in attributedTo');
+  assert.equal(fromAunt.attributedTo.preferredUsername, 'tante');
+  assert.ok((fromAunt.tag || []).some((x) => x.type === 'Emoji'), 'open poort: de emoji van het antwoord blijft');
 
   // Poort dicht: FEP-9098 zit in de tag van de ingesloten actor, dus daar
@@ -188 +188 @@
   db.prepare("INSERT INTO ap_guardianships (slug, other_uri, role, status) VALUES ('kind', ?, 'guardian', 'accepted')")
     .run('https://203.0.113.20/users/tante');
-  const dicht = await haal();
-  const na = dicht.orderedItems.find((n) => n.id.endsWith('/replies/2'));
-  assert.ok(!(na.tag || []).some((x) => x.type === 'Emoji'), 'dichte poort: geen emoji meer');
-  assert.ok(!na.attributedTo.tag, 'ook niet in de byline');
+  const closed = await get();
+  const after = closed.orderedItems.find((n) => n.id.endsWith('/replies/2'));
+  assert.ok(!(after.tag || []).some((x) => x.type === 'Emoji'), 'dichte poort: geen emoji meer');
+  assert.ok(!after.attributedTo.tag, 'ook niet in de byline');
 });
 
@@ -217 +217 @@
   const vorige = globalThis.fetch;
   t.after(() => { globalThis.fetch = vorige; });
-  const vraag = async (uri) => {
-    const r = await echteFetch(
+  const ask = async (uri) => {
+    const r = await realFetch(
       `http://127.0.0.1:${server.address().port}/ap/users/kind/thread?object=${encodeURIComponent(uri)}`,
       { headers: { Authorization: `Bearer ${bearer}` } });
@@ -226 +226 @@
   // De bron zegt nee -- precies wat boiert.eu doet met een friends-only post
   // voor iemand die de auteur niet (meer) volgt.
-  const DICHT = 'https://203.0.113.50/notes/dicht';
+  const SHUT = 'https://203.0.113.50/notes/dicht';
   globalThis.fetch = async () => new Response('nope', { status: 404 });
-  const nee = await vraag(DICHT);
-  assert.equal(nee.status, 404);
-  assert.equal(nee.body.error, 'not shared by source');
-  assert.equal(nee.body.sourceStatus, 404);
+  const refused = await ask(SHUT);
+  assert.equal(refused.status, 404);
+  assert.equal(refused.body.error, 'not shared by source');
+  assert.equal(refused.body.sourceStatus, 404);
 
   // De bron ligt eruit. Dezelfde lege uitkomst, een andere waarheid.
-  const STUK = 'https://203.0.113.51/notes/stuk';
+  const BROKEN = 'https://203.0.113.51/notes/stuk';
   globalThis.fetch = async () => new Response('boem', { status: 500 });
-  const stuk = await vraag(STUK);
-  assert.equal(stuk.status, 502);
-  assert.equal(stuk.body.error, 'source unreachable');
-});
+  const broken = await ask(BROKEN);
+  assert.equal(broken.status, 502);
+  assert.equal(broken.body.error, 'source unreachable');
+});