Changeset 2708282 in Klonkt for test/embed-resolver.test.js

Timestamp:
07/28/2026 10:45:26 AM (6 weeks ago)
Author:
Robin Genis <roboburr@…>
Branches:
main
Children:
d9ad6c5
Parents:
65abc85
Message:

De knop deed niets: AP.deliverTo bestond niet

Op Propose link previews drukken gaf geen zichtbaar resultaat. Oorzaak: de route
riep AP.deliverTo aan, maar die naam bestond alleen als sleutel in het
deps-object dat we aan de guardianship-module doorgeven, niet als export. Dus een
TypeError, een 500 uit Express, en de knop die zichzelf netjes weer inschakelde.
Van buiten af: er gebeurt niks.

Mijn fout zat in de aanname: ik zag deliverTo in een grep-treffer staan en nam
aan dat het een export was. Een grep-treffer is geen export. Er staat nu een test
die alle AP-helpers controleert die de guardian-route aanroept, zodat deze klasse
fout niet nog eens stil kan blijven.

Changed files:
src/services/ActivityPubService.js

  • deliverToActor geexporteerd (stond alleen in de deps)

src/routes/guardian.js

  • gebruikt AP.deliverToActor

test/gated-settings.test.js

  • test dat elke AP-helper die de route aanroept ook echt bestaat
  • test op de Offer-vorm (ward, feature, value, geadresseerd aan de ward)

remarks: 230 tests groen.

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

(No files)

Note: See TracChangeset for help on using the changeset viewer.