Changeset 0677bac in Klonkt for test/music-uitgave.test.js

Timestamp:

08/09/2026 08:24:16 AM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

d60c498

Parents:

a5d14c7

Message:

De tags kwamen er als onzin uit, en de tekst helemaal niet (shaer-38y)

Live nagekeken op dev, en twee dingen klopten niet.

DE TAGS. posts.tags staat als JSON-ARRAY in de database, niet als kommalijst.
Mijn eigen split leverde #["Doen we Niet" en #"music" op -- geen fout die
omvalt, een die onzin uitlevert. Nu met normalizeTags en tagParts, dezelfde
parser als de rest, dus ook dezelfde CamelCase voor tags van meerdere woorden.

Die twee stonden in ActivityPubService en music/ mag daar niet uit importeren,
dus ze zijn naar ap-core verhuisd -- de plek voor precies dit soort gedeelde
grondstof. Een kopie maken zou hetzelfde uit elkaar lopen opleveren als bij de
afleidingsregel.

DE TEKST. De excerpt bleek in de praktijk vaak leeg, en dan kwam er geen
content mee terwijl de post wel tekst heeft. Nu: excerpt eerst -- die is
geschreven om samen te vatten -- en anders het lijf zelf, zonder shortcodes
(die zijn de muziek, niet het verhaal erover) en zonder opmaak. Blijft er niets
over, dan geen veld: dat is beter dan een leeg veld.

De test gebruikte de kommavorm en testte dus de terugval in plaats van de echte
weg. Nu JSON, zoals het er staat.

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

File:

• test/music-uitgave.test.js (modified) (3 diffs)

test/music-uitgave.test.js

@@ -51 +51 @@
   excerpt: 'Opgenomen op zolder, in een week.',
   cover: '/media/hoes.jpg',
-  tags: 'lofi, zolder',
+  tags: '["lofi","zolder tapes"]',
 });
 
@@ -74 +74 @@
   assert.equal(col.image.type, 'Image');
   assert.equal(col.image.url, `${BASE}/media/hoes.jpg`);
-  assert.deepEqual(col.tag.map((t) => t.name), ['#lofi', '#zolder']);
+  // Het tagveld is JSON in de database, en een tag van twee woorden wordt
+  // CamelCase -- zelfde regel als elders, want het is nu dezelfde parser.
+  assert.deepEqual(col.tag.map((t) => t.name), ['#lofi', '#ZolderTapes']);
+  assert.equal(col.tag[1].href, `${BASE}/tag/zoldertapes`);
   assert.equal(col.url, `${BASE}/drie-nieuwe`);
   assert.equal(col.context, `${BASE}/ap/notes/p-los`);
@@ -153 +156 @@
   assert.equal(M.uitgavePost(SITE, 'tweede'), null);
 });
+
+test('zonder excerpt leent hij de tekst uit het lijf van de post', () => {
+  // Live bleek de excerpt vaak leeg; dan is `content` weglaten slechter dan de
+  // tekst zelf pakken. Shortcodes gaan eruit -- die zijn de muziek, niet het
+  // verhaal erover.
+  const p = maakPost({
+    id: 'p-tekst', slug: 'zonder-excerpt', titel: 'Zonder excerpt',
+    content: '<p>Drie schetsen van vorige week.</p>[[track:t-een]]<div>&nbsp;</div>',
+  });
+  const col = M.buildPostTrackCollection(BASE, site, p);
+  assert.equal(col.content, 'Drie schetsen van vorige week.');
+});
+
+test('en een post die alleen uit shortcodes bestaat leent geen lege tekst', () => {
+  const p = maakPost({
+    id: 'p-kaal', slug: 'alleen-muziek', titel: 'Alleen muziek',
+    content: '[[track:t-twee]]<div> </div>',
+  });
+  const col = M.buildPostTrackCollection(BASE, site, p);
+  assert.equal(col.content, undefined, 'een leeg veld is slechter dan geen veld');
+});