Changeset 19a9a1e in Klonkt for test

Timestamp:

08/15/2026 07:43:02 AM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

bec9001

Parents:

e1da6a1

Message:

Library: audience meesturen en de ongeordende collectievorm gebruiken

De 500 op /ap/users/dev/library kwam van open.audio, niet van ons, en de
oorzaak staat in hun bron (federation/serializers.py op develop):

LibrarySerializer.create() regel 1147

"privacy_level": privacy[validated_dataaudience]

audience staat als required=False in de serializer, dus als wij het
weglaten zit de sleutel niet in validated_data en is dat een KeyError --
een 500 op elke poging onze bibliotheek op te halen. Wij stuurden hem
niet.

Het veld is bovendien gewoon waar: alles in de bak is fedi_open. En bij
hen is het precies het verschil tussen privacy_level 'everyone' en 'me',
oftewel of onze nummers daar afspeelbaar zijn.

Daarnaast de vorm. Hun CollectionPageSerializer (regel 1157) accepteert
alleen CollectionPage met items; wij stuurden OrderedCollectionPage
met orderedItems, en dan komt er van de inhoud niets binnen. Dat is geen
dialect maar de andere helft van AS2: een platenkast heeft geen volgorde
die iets betekent, en hun LibrarySerializer accepteert als type ook alleen
as:Collection of fw:Library. De outbox is wél geordend en blijft zoals hij
was.

De eisen staan nu met bronverwijzing in test/ap-library.test.js, want ze
stonden nergens bij ons opgeschreven -- en het volgende dat gebeurt is
anders dat iemand dit 'opruimt' naar de gewone vorm.

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

Location:

test

Files:

• ap-collection-paging.test.js (modified) (1 diff)
• ap-library.test.js (modified) (2 diffs)

test/ap-collection-paging.test.js

@@ -84 +84 @@
     assert.notEqual(pagina.body.id, wortel.body.id, `${pad}: pagina deelt zijn id met de collectie`);
 
-    // En hij mag niet zelf weer een `first` dragen -- dat is de lus waar
-    // open.audio in liep.
-    assert.equal(pagina.body.first, undefined, `${pad}: de pagina draagt zelf een first`);
+    // Een pagina MAG `first` dragen (AS2: CollectionPage erft van Collection,
+    // en Funkwhale eist het zelfs op library-pagina's). Wat niet mag is een
+    // `next` naar zichzelf: dat is de lus, en die zit in de verwijzing die een
+    // lezer volgt, niet in de aanwezigheid van het veld.
+    assert.notEqual(pagina.body.next, eerste, `${pad}: next wijst naar de pagina zelf`);
+
+    // De items horen bij het type: geordend -> orderedItems, ongeordend ->
+    // items. Een `Collection` met `orderedItems` is voor een strikte lezer leeg.
+    const sleutel = pagina.body.type === 'CollectionPage' ? 'items' : 'orderedItems';
+    assert.ok(Array.isArray(pagina.body[sleutel]), `${pad}: ${pagina.body.type} zonder ${sleutel}`);
   });
 }

test/ap-library.test.js

@@ -53 +53 @@
   // van de bak zijn poort overrulen.
   const lib = AP.buildLibrary(BASE, site, AP.siteOpenTracks('s1'));
-  assert.deepEqual(lib.orderedItems.map((a) => a.name), ['Het open nummer']);
+  assert.deepEqual(lib.items.map((a) => a.name), ['Het open nummer']);
 });
 
@@ -59 +59 @@
   // Dit is het veld waar het om begonnen was: het haakje waar een upload aan komt.
   const lib = AP.buildLibrary(BASE, site, AP.siteOpenTracks('s1'));
-  for (const a of lib.orderedItems) assert.equal(a.library, LIB);
+  for (const a of lib.items) assert.equal(a.library, LIB);
   // Ook als de track ergens anders vandaan komt -- de losse ophaal, de
   // trackcollectie, de playlist. Anders hangt het er maar op een plek.
   const los = AP.buildTrackAudio(BASE, site, AP.openTrack('s1', 't1'), { standalone: true });
   assert.equal(los.library, LIB);
+});
+
+// Wat hun serializers WERKELIJK eisen, uit de bron en niet uit de docs.
+// api/funkwhale_api/federation/serializers.py op develop, gelezen 15-8:
+//
+//   LibrarySerializer(PaginatedCollectionSerializer)   regel 1071
+//     type in [as:Collection, fw:Library] · name · id · first · last · totalItems
+//     audience: ChoiceField(required=False) MAAR create() doet
+//       privacy[validated_data["audience"]]  -- ontbreekt de sleutel, dan een
+//       KeyError en dus een 500. Zo gaf open.audio op 15-8 een 500 op onze URL.
+//     en het is de schakel naar privacy_level 'everyone' == afspeelbaar.
+//
+//   CollectionPageSerializer                            regel 1157
+//     type == as:CollectionPage (NIET OrderedCollectionPage) · items (NIET
+//     orderedItems) · id · first · last · partOf · totalItems
+//
+// Deze test staat er omdat die eisen nergens anders bij ons zijn opgeschreven.
+// Zonder hem is het volgende dat iemand de bibliotheek 'opruimt' naar de
+// gewone OrderedCollection-vorm, en dan is hij daar stil weer leeg.
+test('de bibliotheek voldoet aan Funkwhale LibrarySerializer', () => {
+  const lib = AP.buildLibrary(BASE, site, AP.siteOpenTracks('s1'));
+  assert.equal(lib.audience, 'https://www.w3.org/ns/activitystreams#Public',
+    'zonder audience een KeyError in hun create() -- 500');
+  assert.ok(Array.isArray(lib.items), 'Collection draagt items, geen orderedItems');
+  assert.equal(lib.orderedItems, undefined);
+  for (const veld of ['id', 'name', 'first', 'last', 'totalItems', 'attributedTo']) {
+    assert.ok(lib[veld] !== undefined, `${veld} ontbreekt`);
+  }
+});
+
+test('een bibliotheekpagina voldoet aan Funkwhale CollectionPageSerializer', () => {
+  const p = AP.buildLibrary(BASE, site, AP.siteOpenTracks('s1'), { page: 1 });
+  assert.equal(p.type, 'CollectionPage', 'OrderedCollectionPage wijzen ze af');
+  assert.ok(Array.isArray(p.items), 'items, niet orderedItems');
+  assert.equal(p.orderedItems, undefined);
+  assert.equal(p.partOf, LIB);
+  assert.equal(p.id, `${LIB}?page=1`);
+  for (const veld of ['first', 'last', 'totalItems']) {
+    assert.ok(p[veld] !== undefined, `${veld} ontbreekt op de pagina`);
+  }
 });