Proposal: change default snapshot filename (~breaking)
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 52/100
- Type d'issue
- Fonctionnalité
- Clarté
- Plutôt claire
- Activité
- Calme
- Stack technique
- node.js
- Domaine
- developer-experience, testing
Piste de recherche
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.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- Aucune donnée de langage
- Étoiles
- 3
- Forks
- 4
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de nodejs/test-runner
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 1/100
nodejs/test-runner#46 ·
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 5/100
nodejs/test-runner#45 ·
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 10/100
nodejs/test-runner#44 ·
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 15/100
nodejs/test-runner#43 ·
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 15/100
nodejs/test-runner#42 ·
Toutes les issues de nodejs/test-runner
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
sebastienros/jint#4449 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
sublimelsp/LSP-vue#203 · 1 réaction ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
OCHA-DAP/topo-tools#250 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
yetone/magpie#5248 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour