Changeset f704237 in Klonkt for test

Timestamp:

08/14/2026 02:35:06 AM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

47f1183

Parents:

fc664ce

Message:

Een track zonder bestand is niet hetzelfde als een kapotte track

Robin, over het ene nummer dat niet meekwam: "ah, dat was een no audio 'audio'
voor alleen spotify en youtube links". Dat is een bestaand Klonkt-type, geen
ongelukje: geen gehost bestand, wel externe links, en buildNote maakt er een
embed-kaart van (trackEmbedLinks). Mijn regel "geen bestand, geen track" was te
grof en gooide hem weg.

"Geen bestand" en "niets om te tonen" zijn niet hetzelfde. De regel blijft
staan waar hij hoort: een track zonder bestand EN zonder links wordt nog steeds
geweigerd, want die kan niets.

Het archiefformaat heeft nu drie staten in plaats van twee: included, missing,
en linkOnly. Een link-only track telt dus ook niet meer als "ontbrekend", want
er ontbreekt niets.

EN EEN TWEEDE FOUT DIE HIERONDER LAG. De pull viste de audio-URL uit de
url-lijst met element NUL. buildTrackAudio zet daar een text/html-link naar de
post voor het bestand: wie blind de eerste pakt downloadt een HTML-pagina en
schrijft die weg als mp3. Nu wordt de link met mediaType audio/* gezocht, met
/audio/stream/ als terugval. Dat het bij Robin niet misging is geluk: zijn
tracks hadden geen host-post in die lijst.

Changed files:
src/services/ArchiveExportService.js

• shaer:availability kent linkOnly; die telt niet als audioMissing

src/services/ArchiveImportService.js

• link-only tracks komen mee, met media_id NULL zoals op de bron
• de schrijftak kan een track zonder bestand aan

src/services/MigrationService.js

• de pull vist de audio-link uit de lijst in plaats van element nul
• een track met alleen externe links wordt gewoon aangemaakt

test/archive-audio.test.js

• link-only reist mee; zonder bestand en zonder links blijft geweigerd

test/fep1580-migration.test.js

• hetzelfde voor de ophaalknop, plus: de audio-link wordt gevist

remarks: Robins Youngstown-nummer komt hiermee alsnog over. Suite 1009 groen;
tegenproef op beide, en de eerste poging daarvan kwam niet aan (te slordige
sabotage), dus opnieuw gedaan met twee rode tests als bewijs.

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

Location:

test

Files:

• archive-audio.test.js (modified) (1 diff)
• fep1580-migration.test.js (modified) (1 diff)

test/archive-audio.test.js

@@ -258 +258 @@
 });
 
+test('een track met alleen Spotify/YouTube-links reist gewoon mee', () => {
+  // Robins Youngstown-nummer. Klonkt kent link-only tracks: geen gehost
+  // bestand, wel externe links, en buildNote maakt daar een embed-kaart van.
+  // Mijn regel "geen bestand, geen track" gooide die weg, en dat was te grof:
+  // "geen bestand" en "niets om te tonen" zijn niet hetzelfde.
+  leeg();
+  db.prepare(`INSERT INTO audio_tracks (id, site_id, title, artist, media_id, link_spotify, link_youtube)
+              VALUES ('lo','s1','Alleen links','Youngstown',NULL,
+                      'https://open.spotify.com/track/abc','https://www.youtube.com/watch?v=xyz')`).run();
+
+  const uit = AX.buildArchive('me');
+  const rij = JSON.parse(uit.files.get('tracks.json').toString('utf8')).orderedItems.find((x) => x.id === 'lo');
+  assert.equal(rij['shaer:availability'], 'linkOnly', 'een eigen staat, niet "missing"');
+  assert.equal(uit.counts.audioMissing, 0, 'en hij telt niet als ontbrekend: er mist niets');
+  assert.deepEqual(rij.url, ['https://open.spotify.com/track/abc', 'https://www.youtube.com/watch?v=xyz']);
+
+  leeg();
+  const r = AI.importArchive(uit.files, { slug: 'me', origin: 'https://nieuw.test' });
+  assert.equal(r.tracks, 1);
+  assert.equal(r.tracksMissing, 0);
+  const t = db.prepare("SELECT title, artist, media_id, link_spotify, link_youtube FROM audio_tracks WHERE id = 'lo'").get();
+  assert.ok(t, 'de track hoort er te staan');
+  assert.equal(t.media_id, null, 'zonder mediarij, precies zoals op de bron');
+  assert.equal(t.title, 'Alleen links');
+  assert.equal(t.link_spotify, 'https://open.spotify.com/track/abc');
+  assert.equal(t.link_youtube, 'https://www.youtube.com/watch?v=xyz');
+});
+
+test('een track zonder bestand EN zonder links blijft wel geweigerd', () => {
+  // De oorspronkelijke regel blijft staan waar hij hoort: dit is een track die
+  // niets kan tonen en niets kan afspelen.
+  leeg();
+  track('kapot', 'Kapot', { padInDb: '/weg/kapot.mp3', schrijf: false });
+  const uit = AX.buildArchive('me');
+  assert.equal(uit.counts.audioMissing, 1);
+  leeg();
+  const r = AI.importArchive(uit.files, { slug: 'me', origin: 'https://nieuw.test' });
+  assert.equal(r.tracks, 0);
+  assert.equal(r.tracksMissing, 1);
+  assert.equal(db.prepare('SELECT COUNT(*) n FROM audio_tracks').get().n, 0);
+});
+
 test.after(() => { try { fs.rmSync(TMP, { recursive: true, force: true }); } catch { /* niets */ } });

test/fep1580-migration.test.js

@@ -369 +369 @@
 });
 
+test('een link-only nummer komt ook via de ophaalknop mee', async () => {
+  // Dezelfde regel als bij de zip. Over AP heeft zo'n track geen audio-link in
+  // zijn url-lijst, alleen de externe. De vorige versie viste blind de eerste
+  // url en strandde daarop.
+  const s = site({ aliases: [BRON] });
+  const kaart = new Map([
+    [BRON, { id: BRON, type: 'Person', movedTo: IK, outbox: `${BRON}/outbox`, streams: [`${BRON}/tracks`] }],
+    [`${BRON}/outbox`, { type: 'OrderedCollection', totalItems: 0, first: `${BRON}/outbox?page=1` }],
+    [`${BRON}/outbox?page=1`, { type: 'OrderedCollectionPage', orderedItems: [] }],
+    [`${BRON}/tracks`, { type: 'OrderedCollection', orderedItems: [{
+      id: `${BRON}/tracks/lo`, type: 'Audio', name: 'Alleen links', summary: 'Youngstown',
+      url: [
+        { type: 'Link', href: 'https://oud.example/eenpost', mediaType: 'text/html' },
+        { type: 'Link', href: 'https://open.spotify.com/track/abc' },
+        { type: 'Link', href: 'https://www.youtube.com/watch?v=xyz' },
+      ],
+    }] }],
+  ]);
+  const gehaald = [];
+  const r = await stil(() => Mig.ingestFromSource(s, { deps: {
+    getJson: async (_s, url) => kaart.get(url) || null,
+    noteId: (b, id) => `${b}/ap/notes/${id}`,
+    noteVisibility: AP.noteVisibility,
+    audioRoot: '/nep/audio', mediaRoot: '/nep/media',
+    signHeaders: () => ({ Signature: 'nep' }),
+    safeFetch: async (u) => { gehaald.push(u); return { ok: true, arrayBuffer: async () => Buffer.from('x'), headers: { get: () => 'audio/mpeg' } }; },
+    fs: { mkdirSync() {}, writeFileSync() {}, statSync() { throw new Error('ENOENT'); } },
+    path,
+  } }));
+  assert.equal(r.tracksBinnen, 1);
+  assert.equal(r.tracksMislukt, 0);
+  assert.equal(r.tracksLinks, 1);
+  assert.deepEqual(gehaald, [], 'er valt niets te downloaden, en dat hoort ook niet geprobeerd');
+  const t = db.prepare("SELECT title, artist, media_id, link_spotify, link_youtube FROM audio_tracks WHERE id = 'lo'").get();
+  assert.ok(t, 'de track staat er');
+  assert.equal(t.media_id, null);
+  assert.equal(t.artist, 'Youngstown', 'de artiest komt uit summary');
+  assert.equal(t.link_spotify, 'https://open.spotify.com/track/abc');
+  assert.equal(t.link_youtube, 'https://www.youtube.com/watch?v=xyz');
+});
+
+test('de audio-link wordt uit de lijst GEVIST, niet blind de eerste gepakt', async () => {
+  // buildTrackAudio zet een text/html-link naar de post VOOR het bestand. Wie
+  // element nul pakt downloadt een HTML-pagina en noemt dat een mp3.
+  const s = site({ aliases: [BRON] });
+  const MP3 = 'https://oud.example/audio/stream/x.mp3';
+  const kaart = new Map([
+    [BRON, { id: BRON, type: 'Person', movedTo: IK, outbox: `${BRON}/outbox`, streams: [`${BRON}/tracks`] }],
+    [`${BRON}/outbox`, { type: 'OrderedCollection', totalItems: 0, first: `${BRON}/outbox?page=1` }],
+    [`${BRON}/outbox?page=1`, { type: 'OrderedCollectionPage', orderedItems: [] }],
+    [`${BRON}/tracks`, { type: 'OrderedCollection', orderedItems: [{
+      id: `${BRON}/tracks/m1`, type: 'Audio', name: 'Met bestand',
+      url: [
+        { type: 'Link', href: 'https://oud.example/depost', mediaType: 'text/html' },
+        { type: 'Link', href: MP3, mediaType: 'audio/mpeg' },
+      ],
+    }] }],
+  ]);
+  const gehaald = [];
+  await stil(() => Mig.ingestFromSource(s, { deps: {
+    getJson: async (_s, url) => kaart.get(url) || null,
+    noteId: (b, id) => `${b}/ap/notes/${id}`,
+    noteVisibility: AP.noteVisibility,
+    audioRoot: '/nep/audio', mediaRoot: '/nep/media',
+    signHeaders: () => ({ Signature: 'nep' }),
+    safeFetch: async (u) => { gehaald.push(u); return { ok: true, arrayBuffer: async () => Buffer.from('x'), headers: { get: () => 'audio/mpeg' } }; },
+    fs: { mkdirSync() {}, writeFileSync() {}, statSync() { throw new Error('ENOENT'); } },
+    path,
+  } }));
+  assert.deepEqual(gehaald, [MP3], 'het bestand, niet de postpagina');
+});
+
 test('een nummer waarvan de bytes niet komen levert GEEN track op', async () => {
   // Dezelfde regel als bij de zip. Een nummer dat in de lijst staat en 404't is