source: Klonkt/test/help-c2s.test.js@ 3ccca13

main
Last change on this file since 3ccca13 was 26b1848, checked in by Robin <roboburr@…>, 4 weeks ago

Drie spellingen van dezelfde naam, en "door wie" was de hele vraag (shaer-lgo)

Zichtbaar geworden in de echte data van vandaag: naast een markering van
boiert.eu met handle @boiert@… stond die van dev met handle NULL. Op het
scherm valt dat terug op de kale URI.

Onze eigen markering werd op twee plekken geboekt, en allebei anders:

routes/guardian.js (het paneel) null
ActivityPubService (de app, C2S) @${site.slug} -- zonder host
inkomend van een ander de echte handle

Nu alle drie deriveHandle(me). Die stond module-privé in ActivityPubService en
is geexporteerd in plaats van in de route nagebouwd -- een tweede afleiding van
dezelfde naam is precies de naad die vandaag al twee keer beet.

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

  • Property mode set to 100644
File size: 6.5 KB
Line 
1// Een hulpvraag oppikken en afsluiten vanuit de app (shaer-lgo, Barts melding 8-8).
2//
3// De apps lazen hulpvragen uit de FEED en kregen de staat niet mee. Ze wisten dus
4// niet of er al iemand op af was, en een afgehandeld verzoek bleef gewoon staan.
5//
6// Wat hier bewaakt wordt: de app loopt door dezelfde markering en dezelfde staat
7// als het paneel. Twee berekeningen zouden twee guardians een ander beeld geven
8// van hetzelfde kind -- en bij een reddingsboei is dat het gevaarlijkste dat er
9// mis kan gaan.
10//
11// In-memory SQLite. Run: npm test
12import { test } from 'node:test';
13import assert from 'node:assert/strict';
14
15process.env.DATABASE_PATH = ':memory:';
16process.env.PUBLIC_BASE_URL = 'https://oma.test';
17
18const dbMod = await import('../src/config/database.js');
19const db = dbMod.default;
20dbMod.initializeDatabase();
21const AP = await import('../src/services/ActivityPubService.js');
22const queues = await import('../src/services/guardianship/queues.js');
23const rel = await import('../src/services/guardianship/relations.js');
24
25db.prepare('INSERT INTO users (id, username, email, password_hash, role) VALUES (?,?,?,?,?)')
26 .run('u1', 'u1', 'u1@t', 'x', 'god');
27db.prepare('INSERT INTO sites (id, slug, title, owner_id) VALUES (?,?,?,?)').run('s1', 'oma', 'Oma', 'u1');
28const site = () => db.prepare('SELECT * FROM sites WHERE id = ?').get('s1');
29const user = { id: 'u1', username: 'u1' };
30const KIND = 'https://elders.test/ap/users/kind';
31const BOEI = 'https://elders.test/ap/notes/boei-1';
32rel.commitWardForGuardian('oma', KIND, { handle: '@kind' });
33db.prepare(`INSERT INTO ap_mentions (slug, object_uri, actor_uri, actor_handle, content, help_request)
34 VALUES (?,?,?,?,?,1)`).run('oma', BOEI, KIND, '@kind', '<p>help</p>');
35
36const marker = (kind) => ({
37 type: 'Create',
38 object: {
39 type: 'Note',
40 content: '<p>.</p>',
41 to: [KIND],
42 [kind === 'handled' ? 'shaer:helpHandled' : 'shaer:helpPickup']: BOEI,
43 },
44});
45
46test('een verse hulpvraag staat OPEN', () => {
47 const h = queues.helpItemsFor('oma').find((x) => x.object_uri === BOEI);
48 assert.equal(h.state.open, true);
49 assert.equal(h.state.handled, null);
50});
51
52test('oppikken vanuit de app landt in dezelfde staat als het paneel', async () => {
53 await AP.ingestOutboxActivity(site(), user, marker('pickup'));
54 const h = queues.helpItemsFor('oma').find((x) => x.object_uri === BOEI);
55 assert.equal(h.state.pickedUpBy.length, 1);
56 // Nog steeds open: oppikken is geen afhandelen. De faalstand hier is
57 // "iedereen denkt dat het geregeld is", en die is gevaarlijker dan geen
58 // markering.
59 assert.equal(h.state.open, true);
60});
61
62test('onze EIGEN markering draagt onze eigen handle', () => {
63 // "Door wie" was de hele vraag van deze bead. Een binnengekomen markering
64 // draagt de handle van de afzender wel; onze eigen rij stond op null en viel
65 // op het scherm terug op de kale URI. Gezien op 11-8 in de echte data naast
66 // een rij van boiert.eu, die zijn handle wel had.
67 const coll = queues.helpCollection('https://x/queues/help', 'oma');
68 const item = coll.orderedItems.find((i) => i.id === BOEI);
69 assert.deepEqual(item['shaer:pickedUpBy'], ['@oma@oma.test'],
70 'een handle en geen URI');
71});
72
73test('en de apps krijgen te horen HOE OUD dat oppakken is', () => {
74 // Barts besluit: opgepikt vervalt niet maar veroudert zichtbaar -- het
75 // verschil tussen "er is iemand mee bezig" en "er was ooit iemand mee bezig".
76 // Het paneel toonde dat al uit dezelfde staat; de apps kregen het veld niet,
77 // dus daar zag een oppak van vijf minuten eruit als een van vijf dagen.
78 const coll = queues.helpCollection('https://x/queues/help', 'oma');
79 const item = coll.orderedItems.find((i) => i.id === BOEI);
80 assert.ok(item['shaer:oldestPickupAt'], 'het veld staat er');
81 assert.ok(!Number.isNaN(Date.parse(item['shaer:oldestPickupAt'])), 'en is een leesbaar tijdstip');
82 // Een TIJDSTIP en geen leeftijd: een leeftijd maakt elk antwoord anders en
83 // dan kan de ETag nooit gelijk zijn. De client rekent zelf terug.
84 assert.equal(typeof item['shaer:oldestPickupAt'], 'string');
85});
86
87test('afsluiten vanuit de app sluit hem ook echt', async () => {
88 await AP.ingestOutboxActivity(site(), user, marker('handled'));
89 const h = queues.helpItemsFor('oma').find((x) => x.object_uri === BOEI);
90 assert.equal(h.state.open, false);
91 assert.ok(h.state.handled, 'met wie het deed erbij');
92});
93
94test('de collectie voor de apps draagt de staat plat mee', () => {
95 // Een app hoeft hem niet af te leiden, en kan hem dus ook niet anders
96 // afleiden dan het paneel.
97 const coll = queues.helpCollection('https://x/queues/help', 'oma');
98 const item = coll.orderedItems.find((i) => i.id === BOEI);
99 assert.equal(item['shaer:open'], false);
100 assert.ok(item['shaer:handledBy']);
101 assert.equal(item['shaer:helpRequest'], true);
102 // En de ouderdom is WEG zodra hij is afgehandeld: hoe lang er iemand naar
103 // keek is dan geen informatie meer, en zou als "er wacht nog iets" lezen.
104 assert.equal(item['shaer:oldestPickupAt'], undefined);
105});
106
107test('een note zonder markering blijft een gewoon bericht', async () => {
108 const uit = await AP.ingestOutboxActivity(site(), user, {
109 type: 'Create', object: { type: 'Note', content: '<p>hoi</p>', to: [KIND] },
110 });
111 assert.notEqual(uit.status, 400);
112});
113
114test('OPEN vragen worden nooit afgekapt, hoeveel het er ook zijn', () => {
115 // Barts punt (8-8): ik noemde tientallen hulpvragen bij een guardian een
116 // randgeval, maar voor een jeugdzorgmedewerker is dat een caseload en dus een
117 // gewone dinsdag. De gebruiker die dit het hardst nodig heeft was precies
118 // degene voor wie het brak.
119 //
120 // Zonder deze regel viel een oudere vraag buiten de queue, vond de app geen
121 // staat, en toonde hem -- terecht, want bij twijfel OPEN -- alsof er nog
122 // iemand op moest.
123 for (let i = 0; i < 120; i++) {
124 db.prepare(`INSERT INTO ap_mentions (slug, object_uri, actor_uri, actor_handle, content, help_request)
125 VALUES (?,?,?,?,?,1)`).run('oma', `https://elders.test/ap/notes/veel-${i}`, KIND, '@kind', '<p>help</p>');
126 }
127 const open = queues.helpItemsFor('oma').filter((h) => h.state.open);
128 assert.equal(open.length, 120, 'alle 120 open vragen komen mee');
129});
130
131test('de collectie zegt dat je uit afwezigheid mag concluderen', () => {
132 // Zonder die vlag mag een app niets afleiden uit een ontbrekende staat, en
133 // valt hij terug op bij-twijfel-open. Dat is de veilige kant, maar dan komt
134 // een afgehandelde vraag terug in het zicht.
135 const coll = queues.helpCollection('https://x/queues/help', 'oma');
136 assert.equal(coll['shaer:openComplete'], true);
137});
Note: See TracBrowser for help on using the repository browser.