Changeset f50a84b in Klonkt for test

Timestamp:

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

Author:

Robin <roboburr@…>

Branches:

main

Children:

c3e736c

Parents:

013e16c

Message:

Jouw teksten, en de twee dingen die ze beloofden (shaer-mbz)

De titel is nu "MusicBrainz Koppeling" met jouw omschrijving eronder, in drie
talen. Maar die omschrijving belooft twee dingen die de code nog niet deed, en
een label dat liegt is erger dan een lelijk label. Dus allebei gebouwd.

"OF JE MUSICBRAINZ ID ALS JE DIE KENT". Plak je een UUID, dan wordt hij
rechtstreeks opgezocht in plaats van gezocht. Dat is geen luxe: een
zoekopdracht OP een UUID levert bij MusicBrainz niets op, dus zonder deze tak
gaf plakken juist het slechtste resultaat.

"MET TERUG-WEG VALIDATIE". Een koppeling van onze kant is een bewering --
iedereen kan een id in een veld typen. Pas als de MusicBrainz-pagina TERUGWIJST
naar dit domein is het een paar, en dan weet een lezer dat dezelfde persoon aan
allebei de kanten stond. Dezelfde gedachte als rel="me". Het paneel zegt
"wederzijds" of "nog eenzijdig", en bij dat laatste staat erbij WAT je moet doen:
het domein op je MusicBrainz-pagina zetten onder "social networking".

Wij zetten die terugwijzing niet zelf. Kan niet via hun API, en hoort ook niet.

ECHT GEMETEN, en dat gaf de test zijn vorm: robo-burr staat in MusicBrainz met
score 100 (24abe2be-c0bc-4c63-9642-d6f89ec6a00a, "AKA roboburr") en heeft
precies EEN url-relatie -- type "social network", naar sound-fabrics.com. Voor
dev.klonkt.com is de terug-weg dus false, en dat is precies de waarde van de
controle. Die vorm staat nu in een test.

Bij een storing komt er false EN een lege lijst uit: niet kunnen kijken is niet
hetzelfde als niet gevonden, en nooit stilletjes "wederzijds".

Een controle erbij die eist dat alle drie de talen dezelfde mb_-sleutels
hebben. Bij het herschrijven waren er zeven weggevallen die het scherm nog
gebruikt -- een gat in een vertaling geeft een leeg label en dat valt nooit
vanzelf op.

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

File:

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

test/musicbrainz.test.js

@@ -162 +162 @@
   assert.equal(term.sameAs['@type'], '@id', 'het is een URI en geen tekst');
 });
+
+// ── De terug-weg (Robins "social networking"-validatie) ───────────────────
+
+test('een MBID in het zoekveld wordt rechtstreeks opgezocht', async () => {
+  // Zoeken op een UUID levert bij MusicBrainz niets op, dus zonder deze tak
+  // geeft plakken juist het slechtste resultaat.
+  globalThis.fetch = async (url) => {
+    verzoeken.push({ url: String(url) });
+    return { ok: true, json: async () => ({ id: '8be31978-1884-4773-beae-f73df35b92aa', name: 'robo-burr', disambiguation: 'AKA roboburr' }) };
+  };
+  verzoeken.length = 0;
+  const een = await MB.haalArtiest('8be31978-1884-4773-beae-f73df35b92aa');
+  assert.equal(een.naam, 'robo-burr');
+  assert.match(verzoeken[0].url, /\/artist\/8be31978-1884-4773-beae-f73df35b92aa\?/, 'de lookup, niet de zoekopdracht');
+  assert.equal(await MB.haalArtiest('nirvana'), null, 'geen MBID, geen lookup');
+});
+
+test('de terug-weg is pas waar als de pagina ONS domein noemt', async () => {
+  // Echt gemeten op 13-8: robo-burr heeft precies een url-relatie, type
+  // "social network", naar sound-fabrics.com. Dus voor dev.klonkt.com hoort
+  // hier false uit te komen -- en dat is de waarde van de controle.
+  globalThis.fetch = async () => ({
+    ok: true,
+    json: async () => ({ relations: [{ type: 'social network', url: { resource: 'https://sound-fabrics.com/' } }] }),
+  });
+  const mbid = '24abe2be-c0bc-4c63-9642-d6f89ec6a00a';
+  const mis = await MB.controleerTerugweg(mbid, 'https://dev.klonkt.com');
+  assert.equal(mis.verified, false, 'een andere site is geen terugweg');
+  assert.deepEqual(mis.urls, ['https://sound-fabrics.com/']);
+
+  const raak = await MB.controleerTerugweg(mbid, 'https://sound-fabrics.com');
+  assert.equal(raak.verified, true, 'dezelfde host telt, ongeacht pad of slash');
+});
+
+test('niet kunnen kijken is niet hetzelfde als niet gevonden', async () => {
+  // Bij een storing false EN een lege lijst -- nooit stilletjes "wederzijds".
+  globalThis.fetch = async () => { throw new Error('weg'); };
+  const uit = await MB.controleerTerugweg('24abe2be-c0bc-4c63-9642-d6f89ec6a00a', 'https://dev.klonkt.com');
+  assert.deepEqual(uit, { verified: false, urls: [] });
+});