Proposal: change default snapshot filename (~breaking)
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 52/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Tranquilla
- Stack tecnologico
- node.js
- Ambito
- developer-experience, testing
Direzione di ricerca
Read the snapshot implementation linked as nodejs-loaders/…/test/snapshots.mjs and trace how the default filename is generated and existing files are read. The change is complete when new snapshots use the .snap.cjs name and the proposed handling of old .snapshot files is covered without breaking existing tests.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Problem(s) to solve
The current default value of a snapshot filename is to append .snapshot to the name of generating file (aka a "test file"). For instance foo.test.js → foo.test.js.snapshot.
This breaks any IDE trying to help with the file.
A snapshot file is actually a CJS file.
Proposed solution(s)
Adjust the default filename to ${TEST_FILE_BASENAME}.snap.cjs (see nodejs-loaders/…/test/snapshots.ts).
Ex foo.test.js → foo.test.snap.cjs
IDEs recognise this and render the snapshot file as the CJS that it actually is.
We can mitigate a breaking change by detecting the old on read and renaming the file on disk whilst also outputting a message to the console.
- Lingua principale
- Nessun dato sulla lingua
- Stelle
- 3
- Fork
- 4
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di nodejs/test-runner
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 1/100
nodejs/test-runner#46 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 5/100
nodejs/test-runner#45 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 10/100
nodejs/test-runner#44 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 15/100
nodejs/test-runner#43 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 15/100
nodejs/test-runner#42 ·
Tutte le issue di nodejs/test-runner
Issue simili
-
Issue: ready for confirmation Reported on 2.4.9 Triage: Dev.Experience
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
magento/magento2#41773 · 1 commento ·
I maintainer di solito rispondono entro 5 giorni
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
NousResearch/hermes-agent#148799 ·
I maintainer di solito rispondono entro 1 giorno
-
normalizePath removes the Windows device namespace prefixForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
I maintainer di solito rispondono entro 1 giorno
-
bug mcp small
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
ai-driven-dev/framework#1147 ·
I maintainer di solito rispondono entro 1 giorno