test(messages): download-test zet URL-functies weer terug (review #122) #123

Merged

janpeter merged 1 commit from fix/m41-g8c-test-url-restore into main

2026-10-03 12:13:30 +02:00

janpeter commented

2026-10-03 12:09:49 +02:00

Owner

Samenvatting

Deze PR verhelpt de MINOR uit de review van scrum4me-workers#122. De download-test zette URL.createObjectURL en URL.revokeObjectURL op het globale object via Object.assign. Daarna herstelde unstubAllGlobals alleen datzelfde, al aangepaste object, dus de mocks bleven bestaan.

Nu worden de oorspronkelijke functies eerst bewaard en in een finally teruggezet.

De volgende test controleert dat er geen mock is achtergebleven.

Test

Rood: de nieuwe controle faalt op de oude opzet (main).
Groen: met deze fix slagen alle 4 tests.
tsc en eslint op het bestand zijn groen.
Er verandert alleen testcode.

🤖 Generated with Claude Code

## Samenvatting Deze PR verhelpt de MINOR uit de review van scrum4me-workers#122. De download-test zette `URL.createObjectURL` en `URL.revokeObjectURL` op het globale object via `Object.assign`. Daarna herstelde `unstubAllGlobals` alleen datzelfde, al aangepaste object, dus de mocks bleven bestaan. Nu worden de oorspronkelijke functies eerst bewaard en in een `finally` teruggezet. De volgende test controleert dat er geen mock is achtergebleven. ## Test - **Rood:** de nieuwe controle faalt op de oude opzet (main). - **Groen:** met deze fix slagen alle 4 tests. - `tsc` en `eslint` op het bestand zijn groen. - Er verandert alleen testcode. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

janpeter added 1 commit

2026-10-03 12:09:49 +02:00

test(messages): download-test zet URL-functies weer terug (review #122 )

CI / Verify (pull_request) Successful in 3m38s

Details

021e82d81a

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

s4m-codex-reviewer reviewed

2026-10-03 12:12:23 +02:00

s4m-codex-reviewer left a comment

COMMENT

geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

Findings:

LOW — tests/components/queue/dispatch-status.test.tsx:69: De cleanup herstelt de waarden van beide URL-functies, maar oorspronkelijk ontbrekende properties blijven bestaan met undefined. Featuredetectie met in of Object.hasOwn ziet daardoor een gewijzigde globale toestand. Bewaar de oorspronkelijke propertydescriptors en verwijder properties die vóór de test ontbraken. De nieuwe isMockFunction-assertions detecteren dit verschil niet.

De wijziging verwijdert de achterblijvende mocks en gebruikt terecht finally voor herstel bij falende assertions. Geen blokkerende findings gevonden. De beperkte testwijziging past bij de productrichtlijn om de kleinste gerichte oplossing te maken; aanvullende productdocumentatie is hiervoor niet nodig.

Verificatie op commit 021e82d81ad44b668ea43861f21b7e45ec23fffc: diff gecontroleerd tegen de bereikbare repository; lint voltooid met nul errors en één bestaande waarschuwing in components/jobs/job-card.tsx; typecheck geslaagd. De gerichte Vitest-run kon niet starten: de native Rollup-module faalde met failed to map segment from shared object. Tests en volledige verify zijn daarom niet als geslaagd bevestigd. Vanwege deze verificatiebeperking en bovenstaande kleine opmerking is het verdict COMMENT.

## COMMENT geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. Findings: - **LOW — __tests__/components/queue/dispatch-status.test.tsx:69**: De cleanup herstelt de waarden van beide URL-functies, maar oorspronkelijk ontbrekende properties blijven bestaan met `undefined`. Featuredetectie met `in` of `Object.hasOwn` ziet daardoor een gewijzigde globale toestand. Bewaar de oorspronkelijke propertydescriptors en verwijder properties die vóór de test ontbraken. De nieuwe `isMockFunction`-assertions detecteren dit verschil niet. De wijziging verwijdert de achterblijvende mocks en gebruikt terecht `finally` voor herstel bij falende assertions. Geen blokkerende findings gevonden. De beperkte testwijziging past bij de productrichtlijn om de kleinste gerichte oplossing te maken; aanvullende productdocumentatie is hiervoor niet nodig. Verificatie op commit `021e82d81ad44b668ea43861f21b7e45ec23fffc`: diff gecontroleerd tegen de bereikbare repository; lint voltooid met nul errors en één bestaande waarschuwing in `components/jobs/job-card.tsx`; typecheck geslaagd. De gerichte Vitest-run kon niet starten: de native Rollup-module faalde met `failed to map segment from shared object`. Tests en volledige verify zijn daarom niet als geslaagd bevestigd. Vanwege deze verificatiebeperking en bovenstaande kleine opmerking is het verdict COMMENT.

janpeter merged commit cc26dc50be into main

2026-10-03 12:13:30 +02:00

janpeter referenced this pull request from a commit

2026-10-03 12:13:31 +02:00

Merge pull request 'test(messages): download-test zet URL-functies weer terug (review #122)' (#123)