source: Klonkt/test/note-body-media-shape.test.js@ 54b027a

main
Last change on this file since 54b027a was 54b027a, checked in by Robin <roboburr@…>, 4 weeks ago

Een vreemde note mag de Krant niet meenemen

Robin meldde dat /news op beta niet meer werkte. Oorzaak:

_m.filter is not a function note-body.ejs:20

Drie posts van europeanpirates.eu dragen media_json = "[]": een JSON-STRING met
daarin [], niet een array. De try/catch eromheen vangt KAPOTTE json, maar niet
geldige json van het VERKEERDE TYPE. JSON.parse('"[]"') geeft netjes de string
"[]" terug, en een string heeft geen .filter.

Gevolg: één vreemde note nam de hele pagina mee. Niet de post werd overgeslagen,
de Krant lag plat. Wat er van een andere server binnenkomt is niet van ons, dus de
vorm moet op het punt van gebruik afgedwongen worden en niet aangenomen.

Zelfde wacht gezet in guardian.js, waar dezelfde parse stond. Dat is nog
vervelender: het guardian-paneel is precies het scherm dat het moet doen op het
moment dat er iets aan de hand is.

Changed files:
src/views/partials/note-body.ejs

  • Array.isArray-controle na de parse

src/routes/guardian.js

  • dezelfde controle rond media_json in het wardpaneel

New file:
test/note-body-media-shape.test.js

  • een string, een object, een getal en kapotte json slopen de pagina niet
  • en een normale lijst wordt nog steeds getoond, want een fix die alle media wegpoetst is geen fix

remarks: tegenproef gedaan. Zonder de fix vallen de string, het object en het getal
om; kapotte json en de normale lijst blijven groen. De test meet dus het gat en niet
zichzelf. Suite 937 in UTC en Europe/Amsterdam.

Onderweg twee keer mijn eigen testopzet moeten repareren (ontbrekende emojiHtml- en
thumb-helper); dat was de test, niet de code.

Nog te doen: de drie rijen op beta zelf opruimen, en uitzoeken welk pad ooit dubbel
gestringificeerd heeft. In de huidige code staat die dubbele stringify niet meer.

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

  • Property mode set to 100644
File size: 2.4 KB
Line 
1// Een vreemde note mag nooit de hele pagina meenemen.
2//
3// Drie posts van een remote server droegen media_json = "[]": een JSON-STRING met
4// daarin [], niet een array. De try/catch eromheen vangt kapotte json, maar niet
5// geldige json van het verkeerde type: JSON.parse('"[]"') geeft netjes de string
6// "[]" terug, en een string heeft geen .filter. Daarmee lag /news plat op beta.
7//
8// Wat er van een andere server binnenkomt is niet van ons. De vorm moet dus
9// afgedwongen worden op het moment van gebruik, niet aangenomen.
10import { test } from 'node:test';
11import assert from 'node:assert/strict';
12import ejs from 'ejs';
13import path from 'path';
14
15const VIEWS = path.resolve('src/views');
16const PARTIAL = path.join(VIEWS, 'partials/note-body.ejs');
17
18// Alles wat note-body van een rij verwacht, met lege waarden. Per test overschrijven
19// we alleen media_json, want daar gaat het hier om.
20function rij(extra) {
21 return {
22 content: '<p>hoi</p>', media_json: null, emoji_json: null, quote_json: null,
23 embed_json: null, nsfw: 0, cw: null, poll_json: null, link_json: null,
24 ...extra,
25 };
26}
27
28async function render(nb) {
29 return ejs.renderFile(PARTIAL, {
30 nb,
31 noteQuote: () => null,
32 emojiHtml: (html) => String(html || ''),
33 thumb: (u) => String(u || ''),
34 t: (k) => k,
35 formatDate: (d) => String(d),
36 formatDateTime: (d) => String(d),
37 }, { async: true, views: [VIEWS] });
38}
39
40test('een media_json die een STRING is sloopt de pagina niet', async () => {
41 // Precies wat er op beta stond.
42 const html = await render(rij({ media_json: '"[]"' }));
43 assert.match(html, /hoi/, 'de tekst van de post hoort er gewoon te staan');
44});
45
46test('een media_json die een OBJECT is sloopt de pagina niet', async () => {
47 const html = await render(rij({ media_json: '{"url":"https://a.example/x.png"}' }));
48 assert.match(html, /hoi/);
49});
50
51test('een media_json die een GETAL is sloopt de pagina niet', async () => {
52 const html = await render(rij({ media_json: '42' }));
53 assert.match(html, /hoi/);
54});
55
56test('kapotte json blijft ook gewoon werken', async () => {
57 const html = await render(rij({ media_json: '{niet eens json' }));
58 assert.match(html, /hoi/);
59});
60
61test('en een NORMALE lijst wordt nog steeds getoond', async () => {
62 const html = await render(rij({
63 media_json: JSON.stringify([{ url: 'https://a.example/plaatje.png', type: 'image/png' }]),
64 }));
65 assert.match(html, /plaatje\.png/, 'de fix mag geen media wegpoetsen die wel klopt');
66});
Note: See TracBrowser for help on using the repository browser.