Changeset 3bbf73d in Klonkt for test/conversations.test.js

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@…>

File:

• test/conversations.test.js (modified) (7 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');
 });