Changeset f59a46c in Klonkt for test/musicbrainz.test.js

Timestamp:

08/13/2026 01:09:24 PM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

013e16c

Parents:

6401681

git-author:

Robin <roboburr@…> (08/13/2026 12:52:57 PM)

git-committer:

Robin <roboburr@…> (08/13/2026 01:09:24 PM)

Message:

Stap 3: de koppeling gaat de draad op als schema:sameAs (shaer-mbz)

Alleen als de artiest ZELF gekozen heeft. Staat er niets, dan staat er niets --
een lege of geraden verwijzing is erger dan geen.

WAAROM sameAs EN NIET alsoKnownAs, en dat is geen smaakkwestie. alsoKnownAs is
in AS2 gereserveerd voor vroegere IDENTITEITEN van dezelfde actor, en FEP-7628
leunt erop bij een verhuizing: het oude adres controleert of het nieuwe hem daar
noemt voordat het volgers meegeeft. Een MBID daartussen zetten zou een
verhuizing kunnen laten mislukken. schema:sameAs zegt precies het goede: dit
ding is ook bekend onder die URI.

Geen nieuwe namespace: schema stond al in onze context, naast license, position
en bitrate. Dit is er een termregel bij.

EEN KRINGLOOP VERMEDEN. MusicBrainzService leent safeFetch uit
ActivityPubService, dus die er andersom uit laten importeren zou een cirkel
maken. isMbid en artiestUrl zijn pure vormcontroles zonder afhankelijkheden en
wonen nu in ap-core -- dezelfde plek en dezelfde reden als de tagparsers van
gisteren. MusicBrainzService exporteert ze her, zodat een aanroeper er niet over
hoeft na te denken.

DE ACTOR IS DE LAATSTE DEUR. Het scherm zeeft al op een echte MBID, maar
artiestUrl doet het nog een keer: wat hier langskomt gaat naar iedereen, en een
half adres is erger dan geen. Een test zet met opzet rommel in de kolom en eist
dat er niets naar buiten gaat.

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

File:

• test/musicbrainz.test.js (modified) (1 diff)

test/musicbrainz.test.js

@@ -124 +124 @@
   assert.equal(s.mb_artist_name, null, 'anders blijft er een naam staan zonder koppeling');
 });
+
+// ── Stap 3: op de draad ───────────────────────────────────────────────────
+
+test('de actor draagt sameAs zodra er gekoppeld is -- en anders niets', async () => {
+  const db = dbMod.default;
+  const AP = await import('../src/services/ActivityPubService.js');
+  const BASE = 'https://ons.test';
+  const kaal = db.prepare("SELECT * FROM sites WHERE id = 's1'").get();
+  assert.equal(AP.buildActor(BASE, kaal).sameAs, undefined,
+    'zonder koppeling staat er niets -- een lege verwijzing is erger dan geen');
+
+  db.prepare('UPDATE sites SET mb_artist_id = ? WHERE id = ?')
+    .run('8be31978-1884-4773-beae-f73df35b92aa', 's1');
+  const gekoppeld = db.prepare("SELECT * FROM sites WHERE id = 's1'").get();
+  const actor = AP.buildActor(BASE, gekoppeld);
+  assert.equal(actor.sameAs, 'https://musicbrainz.org/artist/8be31978-1884-4773-beae-f73df35b92aa');
+});
+
+test('rommel in de kolom komt NIET op de draad', () => {
+  // Het scherm zeeft al, maar de actor is de laatste deur. Wat hier langskomt
+  // gaat naar iedereen, en een half adres is erger dan geen.
+  const db = dbMod.default;
+  db.prepare('UPDATE sites SET mb_artist_id = ? WHERE id = ?').run('nirvana', 's1');
+  const s = db.prepare("SELECT * FROM sites WHERE id = 's1'").get();
+  assert.equal(MB.artiestUrl(s.mb_artist_id), null);
+});
+
+test('sameAs is een eigen term en NIET alsoKnownAs', async () => {
+  // alsoKnownAs is in AS2 voor vroegere IDENTITEITEN van dezelfde actor, en
+  // FEP-7628 leunt erop bij een verhuizing: het oude adres controleert of het
+  // nieuwe hem daar noemt. Een MBID daar neerzetten zou een verhuizing kunnen
+  // laten mislukken.
+  const core = await import('../src/services/ap-core.js');
+  const term = core.AP_CONTEXT.find((x) => x && typeof x === 'object' && x.sameAs);
+  assert.ok(term, 'de term is gedeclareerd, anders laat een strikte lezer hem vallen');
+  assert.equal(term.sameAs['@id'], 'schema:sameAs');
+  assert.equal(term.sameAs['@type'], '@id', 'het is een URI en geen tekst');
+});