// Alle migratie-opties op één pagina (Robin, 14-8), en wat daarbij stuk kon. // // Aliassen en verhuizen stonden op de site-bewerkpagina, tussen de kleuren en // de feedinstellingen. Dat is de verkeerde plek: het zijn stap 1 en stap 4 van // een verhuizing, en het scherm eromheen gaat over hoe je site eruitziet. // // De verplaatsing had een stille valkuil: de opslagroute van dat scherm schreef // ap_aliases uit het formulier. Verdwijnt het veld, dan is de waarde leeg, en // dan wist je je claim op je oude account door je accentkleur te wijzigen. De // Move weigert daarna met no_backreference en je snapt niet waarom. import { test } from 'node:test'; import assert from 'node:assert/strict'; import fs from 'fs'; const i18nMod = await import('../src/services/i18n.js'); process.env.DATABASE_PATH = ':memory:'; process.env.PUBLIC_BASE_URL = 'https://ik.example'; const dbMod = await import('../src/config/database.js'); const db = dbMod.default; { const stil = console.log; console.log = () => {}; try { dbMod.initializeDatabase(); } finally { console.log = stil; } } test('het site-bewerkscherm draagt geen alias- of verhuisformulier meer', () => { const ejs = fs.readFileSync('src/views/pages/admin-site-edit.ejs', 'utf8'); assert.ok(!/name="ap_aliases"/.test(ejs), 'het aliasveld hoort bij Migreren'); assert.ok(!/name="move_target"/.test(ejs), 'en de verhuisknop ook'); assert.match(ejs, /\/admin\/migrate/, 'maar er staat wel een wegwijzer, anders zoekt iemand zich rot'); }); test('de migrate-pagina draagt ze allebei wel', () => { const ejs = fs.readFileSync('src/views/pages/admin-migrate.ejs', 'utf8'); assert.match(ejs, /name="ap_aliases"/); assert.match(ejs, /name="move_target"/); // De verhuisknop post naar de BESTAANDE route: een tweede implementatie van // een onomkeerbare actie is precies wat je niet wilt. assert.match(ejs, /action="\/admin\/sites\/<%= site\.slug %>\/move"/); assert.match(ejs, /name="next" value="\/admin\/migrate"/, 'en komt terug waar je vandaan kwam'); }); test('de volgorde op de pagina volgt de verhuizing: claimen eerst, aankondigen laatst', () => { const ejs = fs.readFileSync('src/views/pages/admin-migrate.ejs', 'utf8'); const alias = ejs.indexOf('name="ap_aliases"'); const halen = ejs.indexOf('/admin/migrate/pull'); const move = ejs.indexOf('name="move_target"'); assert.ok(alias > -1 && halen > -1 && move > -1); assert.ok(alias < halen, 'claimen staat boven ophalen: zonder claim geeft de bron niets'); assert.ok(halen < move, 'aankondigen staat onderaan: dat is de onomkeerbare stap'); }); test('een alias overleeft het opslaan van je uiterlijk', async () => { // De valkuil. Geen HTTP nodig: dit gaat om de regel dat een ontbrekend veld // "niet aanraken" betekent en niet "leegmaken". db.prepare("INSERT INTO users (id,username,email,password_hash,role) VALUES ('u1','u','u@t','x','god')").run(); db.prepare("INSERT INTO sites (id,slug,title,owner_id,ap_aliases) VALUES ('s1','ik','Ik','u1',?)") .run(JSON.stringify(['https://oud.example/ap/users/robo'])); const site = db.prepare("SELECT id, ap_aliases FROM sites WHERE slug = 'ik'").get(); const formulierZonderVeld = { title: 'Ik', accent: '#c33' }; // zoals het nu binnenkomt const apAliasesJson = Object.prototype.hasOwnProperty.call(formulierZonderVeld, 'ap_aliases') ? null : (site.ap_aliases || null); db.prepare('UPDATE sites SET ap_aliases = ? WHERE slug = ?').run(apAliasesJson, 'ik'); const na = db.prepare("SELECT ap_aliases FROM sites WHERE slug = 'ik'").get(); assert.deepEqual(JSON.parse(na.ap_aliases), ['https://oud.example/ap/users/robo'], 'je claim op je oude account mag niet sneuvelen op een kleurwijziging'); }); // ── Het stappenplan ─────────────────────────────────────────────── test('het stappenplan zet de volgorde vast die niemand raadt', () => { const ejs = fs.readFileSync('src/views/pages/admin-migrate.ejs', 'utf8'); const plan = ejs.indexOf('mig-steps'); const alias = ejs.indexOf('name="ap_aliases"'); assert.ok(plan > -1, 'er hoort een stappenplan te staan'); assert.ok(plan < alias, 'en bovenaan, voor de knoppen die het beschrijft'); }); test('"hier" draait om als je vanaf de vertrekkende kant kijkt', async () => { // Deze pagina draait op BEIDE instanties. Stap 1, 3 en 4 horen op de nieuwe, // stap 2 en 5 op de oude. Een vaste tekst is dus op de helft van de schermen // onzin. Ik had ze bij het bouwen precies verkeerd om: beide labels werden // "hier", en dat zag ik pas door het te renderen. const ejs = fs.readFileSync('src/views/pages/admin-migrate.ejs', 'utf8'); const m = /var _hier = stap\.isOud \? t\('([a-z.\_]+)'\)\s*:\s*t\('([a-z.\_]+)'\)/.exec(ejs); const d = /var _daar = stap\.isOud \? t\('([a-z.\_]+)'\)\s*:\s*t\('([a-z.\_]+)'\)/.exec(ejs); assert.ok(m && d, 'de perspectief-variabelen horen er te zijn'); assert.equal(m[1], 'mig.plan_new', 'op de oude kant wijst stap 1/3/4 naar de NIEUWE'); assert.equal(m[2], 'mig.plan_here', 'op de nieuwe kant is dat gewoon hier'); assert.equal(d[1], 'mig.plan_here', 'en het aankondigen gebeurt dan HIER'); assert.equal(d[2], 'mig.plan_old', 'terwijl je vanaf de nieuwe kant naar de oude wijst'); assert.notEqual(m[1], d[1], 'de twee mogen nooit hetzelfde label krijgen'); }); test('het aankondigen heet geen "laatste stap" meer', async () => { // Het was er wel een in mijn hoofd, maar niet in de code: de ophaalknop // weigert met not_moved_here zolang de Move niet gedaan is. Aankondigen is // dus stap 2, niet de afsluiter. const i18n = await import('../src/services/i18n.js'); for (const l of ['nl', 'en', 'de']) { const t = i18n.t(l, 'mig.move_title'); assert.doesNotMatch(t, /laatste|last step|letzter/i, `${l}: ${t}`); assert.match(t, /2/, `${l}: het is stap 2, en dat hoort er te staan`); } }); test('stap 5 vertelt wat er moet gebeuren voor je het domein loslaat', async () => { // Er is geen knop voor, en dat staat er ook bij. Een stappenplan dat een stap // beschrijft die niet bestaat is erger dan geen stappenplan. const i18n = await import('../src/services/i18n.js'); // Op BETEKENIS toetsen, niet op een woord. Deze test brak toen "alias" uit de // teksten verdween, en dat is precies de verkeerde reden om rood te worden: // het jargon weghalen was de bedoeling. const nl = i18n.t('nl', 'mig.plan_5_why'); assert.match(nl, /verwijder/i, 'de berichten opruimen'); assert.match(nl, /stap 1/i, 'en de koppeling met je vorige account weghalen'); assert.match(nl, /nog geen knop/i, 'en eerlijk dat het handwerk is'); }); test('de teksten praten geen jargon', () => { // Robin: er zijn miljoenen Klonkt-gebruikers, dus het moet gebruiksvriendelijk. // Woorden als alias, actor, instantie en fediverse zeggen een ontwikkelaar // alles en een gebruiker niets. Deze test bewaakt dat ze niet terugsluipen. const jargon = /\b(alias(sen)?|claimen|actor|instantie|AP-id|origin|slug|URI)\b/i; const sleutels = [ 'lead', 'alias_title', 'alias_hint', 'alias_label', 'alias_note', 'alias_btn', 'import_hint', 'file_label', 'overwrite_label', 'overwrite_hint', 'r_new_ids', 'pull_hint', 'pull_source', 'pull_source_hint', 'move_hint', 'move_warn', 'plan_hint', 'plan_1', 'plan_1_why', 'plan_5_why', 'e_no_backreference', 'e_not_moved_here', 'e_no_source', ]; const fout = []; for (const l of ['nl', 'en', 'de']) { for (const k of sleutels) { const t = i18nMod.t(l, `mig.${k}`); const m = jargon.exec(t); if (m) fout.push(`${l}/${k}: "${m[0]}"`); } } assert.deepEqual(fout, [], `jargon teruggeslopen: ${fout.join(', ')}`); }); test('een gekoppeld account komt terug als handle, niet als URL', async () => { // Je typt @jij@mastodon.social en wij slaan de actor-URL op, want daar draait // het protocol op. Zonder deze vertaling terug kreeg je een adres te zien dat // je nooit hebt ingetypt en niet herkent. const AP = await import('../src/services/ActivityPubService.js'); assert.equal(AP.deriveHandle('https://oud.example/ap/users/robo'), '@robo@oud.example'); // En als er geen fatsoenlijke handle uit te halen valt blijft de URL staan: // een verkeerde handle is erger dan een lelijke URL. const raar = AP.deriveHandle('https://oud.example/'); assert.ok(!/^@[^@\s]+@[^@\s]+$/.test(raar) || raar === '@oud.example@oud.example', `onverwachte handle uit een pad zonder naam: ${raar}`); });