source: Klonkt/test/migrate-page.test.js@ 7e474f7

// 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';

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');
  const nl = i18n.t('nl', 'mig.plan_5_why');
  assert.match(nl, /verwijder/i, 'de Delete-ronde');
  assert.match(nl, /alias/i, 'en de alias weghalen');
  assert.match(nl, /nog geen knop/i, 'en eerlijk dat het handwerk is');
});