Changeset 407353e in Klonkt for test

Timestamp:

09/23/2026 03:13:00 PM (4 hours ago)

Author:

Bart <bart@…>

Branches:

main

Parents:

0ad37a0

Message:

Een foto zonder woorden is een bericht (502 direct_failed)

deliverDirectNote begon met !String(text).trim() -> return null, en de inname
maakte van dat null een 502 direct_failed. Sinds de inname een bijlage-only
note doorlaat waren die twee lagen het oneens over wat een bericht is: wie een
foto op een gesprek liet vallen en niets typte kwam precies daar vast te
zitten. Het viel niet op in de tests, want dit was de enige plek waar de regel
stond en hij stond midden in een functie die netwerk doet.

Nu is het een eigen, pure regel -- text, html of bijlage -- met tests eromheen,
zoals de rest van deze laag het doet. Leeg blijft geweigerd: geen tekst, geen
opmaak en geen bijlage is niets.

En String(text) werd String(text || ''): zonder tekst stond er anders
letterlijk "undefined" in de body zodra een aanroeper het veld wegliet.

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

File:

• test/c2s-direct.test.js (modified) (1 diff)

test/c2s-direct.test.js

@@ -77 +77 @@
   assert.equal(like.status, 403);
 });
+
+// Een bijlage KAN het hele bericht zijn (shaer-6lcc).
+//
+// Twee lagen waren het oneens over wat een bericht is: de inname liet een
+// note met alleen een bijlage door, en deliverDirectNote weigerde hem een
+// laag lager op `!text.trim()`. Wie een foto op een gesprek liet vallen en
+// niets typte kreeg daardoor "502 direct_failed".
+test('een direct bericht met alleen een bijlage telt als inhoud', async () => {
+  const { directNoteHasContent } = await import('../src/services/guardianship/delivery.js');
+
+  assert.equal(directNoteHasContent({ text: 'hallo' }), true);
+  assert.equal(directNoteHasContent({ html: '<p>hallo</p>' }), true);
+  assert.equal(directNoteHasContent({
+    text: '',
+    attachments: [{ url: '/media/reply-media/x.png', mediaType: 'image/png' }],
+  }), true, 'een foto zonder woorden is een bericht');
+  assert.equal(directNoteHasContent({
+    text: '   \n  ',
+    attachments: [{ url: '/media/reply-media/x.png', mediaType: 'image/png' }],
+  }), true, 'witruimte is geen tekst, maar de bijlage draagt hem');
+
+  // En leeg blijft leeg: dit mag de deur niet uit.
+  assert.equal(directNoteHasContent({}), false);
+  assert.equal(directNoteHasContent({ text: '   ', html: '', attachments: [] }), false);
+  assert.equal(directNoteHasContent(), false);
+});
+
+// De inname en de bezorging horen dezelfde grens te trekken. Loopt er eentje
+// weg, dan is dat precies de 502 hierboven, en die valt buiten de tests om.
+test('inname en bezorging zijn het eens over een leeg bericht', async () => {
+  const { directNoteHasContent } = await import('../src/services/guardianship/delivery.js');
+  const src = (await import('fs')).readFileSync('src/services/ap-c2s.js', 'utf8');
+  assert.ok(src.includes("error: 'empty_note'"), 'de inname heeft nog een lege-bericht-controle');
+  // De inname laat een bijlage-only note door; de bezorging hoort dat ook te doen.
+  assert.equal(directNoteHasContent({ text: '', attachments: [{ url: '/media/x.png' }] }), true);
+});