Changeset ed5e7ab in Klonkt for test/archive-export.test.js


Ignore:
Timestamp:
08/07/2026 05:41:20 PM (5 weeks ago)
Author:
Robin <roboburr@…>
Branches:
main
Children:
c25eff7
Parents:
ba76bf5 (diff), a4fea5e (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.
Message:

Merge de drie nieuwste van GitHub, met hun exporter-fix boven de mijne

De parallelle sessie pushte drie commits terwijl deze kant aan het werk was. Een
ervan, 6248497, repareert dezelfde exporter-tijdzonebug die hier een uur eerder in
ba76bf5 was gerepareerd. Twee keer hetzelfde werk, langs twee routes gevonden: daar
via een vraag van Bart, hier via de test die in Europe/Amsterdam omviel.

Het enige conflict zat dan ook in ArchiveExportService.js, op precies die functie.
Hun versie is aangehouden, en dat is geen hoffelijkheid maar een meting: hun regex
accepteert ook een stempel zonder seconden ("2026-07-01 12:56"), waar de mijne
seconden eist. Ze pakten bovendien export-archive.mjs, import-archive.mjs en een
nieuwe instance-env.mjs mee, plus een test in archive-export.test.js. Mijn variant is
daarmee vervallen en verdwijnt uit de boom; de bug blijft gedekt.

Nagemeten dat die keuze de dekking niet kost: archive-import 21 groen en
archive-export 16 groen, in UTC en in Europe/Amsterdam. Zonder een fix valt de
import-test alleen buiten UTC om, dus de tijdzone-afhankelijkheid is echt weg.

Changed files:
src/services/ArchiveExportService.js

  • conflict opgelost ten gunste van 6248497; de eigen toISO is eruit

remarks: dit is de derde keer vandaag dat de GitHub-mirror faalde omdat GitHub
verder stond dan prutfolio.git. De VPS is volgens de deploy-afspraak canoniek,
maar de andere sessie pusht rechtstreeks naar GitHub, dus het divergeert elke paar
uur opnieuw en dat kostte nu dubbel werk. Eén bron aanwijzen is een besluit voor
Robin, geen technische keuze.

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

File:
1 edited

Legend:

Unmodified
Added
Removed
  • test/archive-export.test.js

    rba76bf5 red5e7ab  
    175175});
    176176
     177test('een SQL-tijdstempel wordt als UTC gelezen, niet als lokale tijd', () => {
     178  // SQLite schrijft CURRENT_TIMESTAMP in UTC zonder zone erbij. Date.parse leest
     179  // die vorm als LOKALE tijd, en dan schuift elk tijdstempel in het archief mee
     180  // met de machine die de export draait -- in Amsterdam twee uur.
     181  //
     182  // Deze test is alleen zinvol onder een NIET-UTC tijdzone; draai hem daarom ook
     183  // eens als:  TZ=Europe/Amsterdam node --test test/archive-export.test.js
     184  post('tijdstip', { published_at: '2026-07-01 12:56:10', created_at: '2026-07-01 12:56:10' });
     185  const o = lees(AX.buildArchive('me'), 'posts/tijdstip.json');
     186  assert.equal(o.published, '2026-07-01T12:56:10.000Z',
     187    'het moment uit de database is UTC en hoort dat te blijven');
     188});
     189
    177190test('een concept gaat gewoon mee', () => {
    178191  post('concept', { status: 'draft', published_at: null });
Note: See TracChangeset for help on using the changeset viewer.