Changeset e215090 in Klonkt for test

Timestamp:

08/10/2026 05:38:55 PM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

383e897

Parents:

21ef238

Message:

Het verschil draagt ook de rechten

Zonder dit valt de client bij elke verschil-lezing terug op zijn standaard
Caps, en die standaard is 'alles mag': een gesloten poort zou zichzelf stil
openzetten. Dezelfde reden waarom een 304 de caps met rust laat -- dat stond
er al, met de reden erbij, en dat gold hier net zo goed.

Het rechtenblok staat nu in capabilitiesOf() zodat beide lezingen hetzelfde
beschrijven. De pariteitstest leest de bron en wees naar de oude plek; die
wijst nu naar de functie.

862/862 groen.

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

Location:

test

Files:

• feed-changes.test.js (modified) (1 diff)
• gates-capabilities-parity.test.js (modified) (1 diff)

test/feed-changes.test.js

@@ -87 +87 @@
 });
 
+test('het verschil draagt de rechten mee', async () => {
+  // Zonder dit valt de client terug op zijn standaard, en die standaard is
+  // 'alles mag': een gesloten poort zou zichzelf stil openzetten bij elke
+  // verschil-lezing.
+  const vol = await get('');
+  db.prepare("UPDATE sites SET gate_images = 0 WHERE slug = 'kind'").run();
+  db.prepare("INSERT INTO ap_guardianships (slug, other_uri, role, status) VALUES ('kind', 'https://elders/u/oma', 'guardian', 'accepted')").run();
+  post('https://elders/n/9', '<p>na de poort</p>');
+  const delta = await get(`?changes=1&since=${encodeURIComponent(vol.body['shaer:cursor'])}`);
+  assert.equal(delta.body['shaer:capabilities']['shaer:images'], false, 'de dichte poort reist mee');
+});
+
 test('niets veranderd is een leeg verschil, niet een leeg antwoord', async () => {
   const vol = await get('');

test/gates-capabilities-parity.test.js

@@ -29 +29 @@
 const SOURCE = 'src/routes/activitypub.js';
 
-/** De sleutels binnen het shaer:capabilities-blok, gelezen uit de bron. */
+/** De sleutels van het rechtenblok, gelezen uit de bron.
+ *
+ *  Het blok woont sinds 10-8 in capabilitiesOf(), omdat de verschil-lezing
+ *  (?changes=1) dezelfde rechten moet dragen: een antwoord zonder rechten laat
+ *  de client terugvallen op zijn standaard, en die standaard is 'alles mag'.
+ *  Deze toets kijkt daarom naar die functie en niet meer naar de plek in de
+ *  route. */
 function capabilityKeys() {
   const src = fs.readFileSync(SOURCE, 'utf8');
-  const start = src.indexOf("'shaer:capabilities': {");
-  assert.notEqual(start, -1, `${SOURCE} heeft nog een shaer:capabilities-blok`);
+  const start = src.indexOf('function capabilitiesOf(');
+  assert.notEqual(start, -1, `${SOURCE} heeft geen capabilitiesOf() meer`);
   const open = src.indexOf('{', start);
   let depth = 0;