Changeset 9a585bf in Klonkt for test

Timestamp:

08/16/2026 01:29:11 PM (3 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

21257e4

Parents:

38d5e40

Message:

Track: published en artist_credit erbij

Twee van de vijf gaten naar Funkwhale's bibliotheek-ingest.

published was een omissie in de kleine Emissary-vorm, niet een keuze:
MusicEntitySerializer (regel 1278) eist het en we hebben created_at.

artist_credit leek de onmogelijke. Hun keten is Track -> ArtistCredit ->
Artist, en elke schakel wil id, name en published -- terwijl een artiest
bij ons een tekstkolom is. De MusicBrainz-koppeling van vorige week liet
zien dat dat niet klopt: de site-ACTOR is de artiest. Een echt,
opvraagbaar adres, met de sitetitel als naam en een musicbrainzId zodra
hij gekoppeld is. Er valt niets te verzinnen, en het is niet nieuw --
open.audio leidde op 13-8 al zelf een artist_credit af uit onze
attributedTo. We maken expliciet wat daar toch al gebeurde.

De artiestnaam van de track gaat naar credit en niet naar de entiteit.
Dat is waar hun model de credittekst verwacht, en er een id per
artiestnaam van maken zou identiteit uit een string zijn -- dezelfde fout
die we bij het album vermijden (shaer-756s).

@container: @list op artist_credit is geen opsmuk: ze lezen het veld met
first_attr(FW.artist_credit, "@list"), en zonder die declaratie
expandeert onze array er niet naar. Dan staat er iets dat er goed uitziet
en dat hun lezer niet vindt -- precies het soort stil gat waar deze week
al twee keer een dag in ging zitten.

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

File:

• test/ap-fw-track.test.js (modified) (1 diff)

test/ap-fw-track.test.js

@@ -84 +84 @@
   for (const a of col.orderedItems) assert.equal(a.track.type, 'Track', `${a.name} zonder track`);
 });
+
+// ── artist_credit (shaer-3f8a, het gat naar hun ingest) ──────────────
+//
+// TrackSerializer (regel 1569) eist `artist_credit` met min_length=1, en elke
+// ArtistCredit eist een Artist die zelf id, name en published nodig heeft
+// (MusicEntitySerializer, regel 1278). Dat leek onmogelijk zolang een artiest
+// bij ons tekst was -- tot de MusicBrainz-koppeling liet zien dat de ACTOR de
+// artiest is.
+
+test('de track draagt een artist_credit met een echte Artist', () => {
+  const a = audio('t1');
+  assert.ok(Array.isArray(a.track.artist_credit), 'geen lijst');
+  assert.equal(a.track.artist_credit.length, 1);
+  const ac = a.track.artist_credit[0];
+  assert.equal(ac.type, 'ArtistCredit');
+  assert.ok(ac.id && ac.published, 'ArtistCredit mist id of published');
+  assert.equal(ac.artist.type, 'Artist');
+  assert.equal(ac.artist.id, `${BASE}/ap/users/band`, 'de actor IS de artiest');
+  assert.equal(ac.artist.name, 'De Band');
+  assert.ok(ac.artist.published, 'Artist mist published');
+});
+
+test('de artiestnaam van de track gaat naar credit, niet naar de entiteit', () => {
+  // De kolom is een credittekst, geen identiteit. Er een id van maken zou
+  // dezelfde fout zijn als bij het album (shaer-756s).
+  assert.equal(audio('t1').track.artist_credit[0].credit, 'De Band');
+  assert.equal(audio('t2').track.artist_credit[0].credit, undefined, 'geen artiest, geen credit');
+});
+
+test('de track heeft published -- MusicEntitySerializer eist het', () => {
+  assert.ok(audio('t1').track.published);
+});
+
+test('artist_credit is een @list in de context, anders vindt hun lezer niets', async () => {
+  // Ze lezen dit met first_attr(FW.artist_credit, "@list"). Zonder de
+  // container-declaratie expandeert onze array niet naar een @list en staat er
+  // iets dat er goed uitziet en niet gevonden wordt.
+  const core = await import('../src/services/ap-core.js');
+  const term = core.AP_CONTEXT.find((x) => x && typeof x === 'object' && x.artist_credit);
+  assert.deepEqual(term.artist_credit, { '@id': 'fw:artist_credit', '@type': '@id', '@container': '@list' });
+  assert.equal(term.ArtistCredit, 'fw:ArtistCredit');
+  assert.equal(term.Artist, 'fw:Artist');
+});
+
+test('musicbrainzId komt mee zodra de site gekoppeld is', () => {
+  assert.equal(audio('t1').track.artist_credit[0].artist.musicbrainzId, undefined);
+  db.prepare("UPDATE sites SET mb_artist_id = '5441c29d-3602-4898-b1a1-b77fa23b8e50' WHERE id = 's1'").run();
+  const s2 = db.prepare("SELECT * FROM sites WHERE id = 's1'").get();
+  const a = AP.buildTrackAudio(BASE, s2, AP.openTrack('s1', 't1'), { hostPosts: null });
+  assert.equal(a.track.artist_credit[0].artist.musicbrainzId, '5441c29d-3602-4898-b1a1-b77fa23b8e50');
+  db.prepare("UPDATE sites SET mb_artist_id = NULL WHERE id = 's1'").run();
+});