Changeset 41de7cb in Klonkt for test/conversations.test.js

Timestamp:

08/10/2026 04:13:24 PM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

21ef238

Parents:

f0a33e1

Message:

Een kopje draagt ook het nieuwste bericht VAN de ander

Op het toestel gezien: de hemel noemde mensen opeens 'boiert' en
'hotelbreakfast' in plaats van hun naam, en tekende letters in plaats van
gezichten. Oorzaak: is het nieuwste bericht in een gesprek van MIJ, dan draagt
het kopje mijn byline, en de hemel zoekt naam en gezicht van de ander in een
bericht VAN de ander. Vond hij dat niet, dan viel hij terug op het staartje van
de actor-uri.

conversationHeads geeft nu twee rijen per persoon: het nieuwste bericht (dat
bepaalt waar iemand hangt) en het nieuwste bericht van hem. Vallen ze samen,
dan blijft het een rij. Geen clientwijziging nodig -- conversationsFrom maakt er
dezelfde Conversation van.

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

File:

• test/conversations.test.js (modified) (2 diffs)

test/conversations.test.js

@@ -50 +50 @@
 test('een rij per tegenpartij, hoe druk de drukste ook is', () => {
   const heads = AP.conversationHeads('kind');
-  assert.deepEqual(heads.map((k) => k.other), [OMA, TANTE], 'nieuwste gesprek eerst, allebei aanwezig');
+  assert.deepEqual([...new Set(heads.map((k) => k.other))], [OMA, TANTE], 'nieuwste gesprek eerst, allebei aanwezig');
+});
+
+test('is het nieuwste bericht van MIJ, dan gaat dat van hem ook mee', () => {
+  // Anders draagt het kopje mijn byline en zoekt de hemel de naam en het
+  // gezicht van de ander tevergeefs -- die viel dan terug op het staartje van
+  // de actor-uri. Bij tante is mijn antwoord het nieuwste.
+  const vanTante = AP.conversationHeads('kind').filter((k) => k.other === TANTE);
+  assert.equal(vanTante.length, 2);
+  assert.deepEqual(vanTante.map((k) => k.direction).sort(), ['in', 'out']);
+  // Bij oma is het nieuwste al van haar: dan is een rij genoeg.
+  assert.equal(AP.conversationHeads('kind').filter((k) => k.other === OMA).length, 1);
 });
 
@@ -110 +121 @@
   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');
+  // Drie items voor twee mensen: bij tante gaat haar eigen bericht mee omdat
+  // het nieuwste van mij is (anders mist de hemel haar naam en gezicht).
+  assert.equal(list.body.totalItems, 3);
+  assert.ok(!JSON.stringify(list.body).includes(VREEMDE), 'de publieke vreemde is geen gesprek');
+  const vanTante = list.body.orderedItems.filter((i) => i.actor === TANTE);
+  assert.equal(vanTante.length, 1, 'en dat bericht van tante draagt haar byline');
+  assert.ok(vanTante[0].object.attributedTo.name);
   // De byline zit in de ingesloten actor, net als overal sinds shaer-nmw.
   assert.ok(list.body.orderedItems.every((i) => i.object.attributedTo));