Proposal: change default snapshot filename (~breaking)
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 52/100
- Issue-Typ
- Feature
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Ruhig
- Tech-Stack
- node.js
- Bereich
- developer-experience, testing
Rechercherichtung
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.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Vorherrschende Sprache
- Keine Sprachdaten
- Sterne
- 3
- Forks
- 4
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus nodejs/test-runner
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 1/100
nodejs/test-runner#46 ·
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 5/100
nodejs/test-runner#45 ·
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 10/100
nodejs/test-runner#44 ·
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 15/100
nodejs/test-runner#43 ·
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 15/100
nodejs/test-runner#42 ·
Alle Issues in nodejs/test-runner
Ähnliche Issues
-
good first issue
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 85/100
r-afael/better-yt-recommendations#36 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Run the dependency drift checkEvtl. vergeben @spaceninja hat das heute übernommen. Offen
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 72/100
cloudfour/lighthouse-parade#449 ·
Maintainer antworten meist innerhalb von 1 Tag
-
[Bug]: Four evaluation scripts unpack init_memmachine_params's five return values into three namesOffen
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 88/100
MemMachine/MemMachine#1911 ·
Maintainer antworten meist innerhalb von 3 Tagen
-
friction
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
kentcdodds/kody#4753 ·
-
Re-saving a posting from a URL with "#app" or tracking parameters creates a duplicate saved jobOffengood first issue
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
ritsth/job-autofill-extension#447 ·
Maintainer antworten meist innerhalb von 2 Tagen