Proposal: change default snapshot filename (~breaking)
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 52/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- node.js
- Área
- developer-experience, testing
Línea de trabajo
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.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- Sin datos de lenguaje
- Estrellas
- 3
- Forks
- 4
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de nodejs/test-runner
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 1/100
nodejs/test-runner#46 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 5/100
nodejs/test-runner#45 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 10/100
nodejs/test-runner#44 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 15/100
nodejs/test-runner#43 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 15/100
nodejs/test-runner#42 ·
Todos los issues de nodejs/test-runner
Issues similares
-
good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
r-afael/better-yt-recommendations#36 ·
Los mantenedores suelen responder en 1 día
-
Run the dependency drift checkPosiblemente ocupada @spaceninja la tomó hoy. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
cloudfour/lighthouse-parade#449 ·
Los mantenedores suelen responder en 1 día
-
[Bug]: Four evaluation scripts unpack init_memmachine_params's five return values into three namesAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
MemMachine/MemMachine#1911 ·
Los mantenedores suelen responder en 3 días
-
friction
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
kentcdodds/kody#4753 ·
-
Re-saving a posting from a URL with "#app" or tracking parameters creates a duplicate saved jobAbiertogood first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
ritsth/job-autofill-extension#447 ·
Los mantenedores suelen responder en 2 días