test.deepEqual may print incorrect property values for subsequently modified properties SERIOUS ISSUE
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript
- Área
- testing-qa
Línea de trabajo
Comienza con test.deepEqual en lib/types.js, usando el stack trace y la reproducción nodeunit-changedobject.js como puntos de entrada. Ejecuta el ejemplo de nodeunit proporcionado y compara la salida del error con su resultado esperado; se considera terminado cuando los valores informados siguen siendo los que estaban presentes al llamar a deepEqual, en lugar de mutaciones posteriores.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
test.deepEqual does not examine failing objects immediately, so incorrect values are printed
Reproduction printout
$ nodeunit ../nodejs3/snippets/modules/nodeunit/nodeunit-changedobject.js
nodeunit-changedobject.js
Expected printout: AssertionError: { property: 'object1' } deepEqual { property: 'object2' }
✖ f
AssertionError: { property: 'overwite' } deepEqual { property: 'object2' }
at Object.assertWrapper [as deepEqual] (/usr/local/lib/node_modules/nodeunit/lib/types.js:83:39)
at Object.module.exports.f (/home/foxyboy/Desktop/c505/node/nodejs3/snippets/modules/nodeunit/nodeunit-changedobject.js:13:7)
at Object.wrapTest (/usr/local/lib/node_modules/nodeunit/lib/core.js:235:16)
at wrapTest (/usr/local/lib/node_modules/nodeunit/lib/core.js:235:16)
at Object.exports.runTest (/usr/local/lib/node_modules/nodeunit/lib/core.js:69:9)
at exports.runSuite (/usr/local/lib/node_modules/nodeunit/lib/core.js:117:25)
at _concat (/usr/local/lib/node_modules/nodeunit/deps/async.js:508:13)
at async.forEachSeries.iterate (/usr/local/lib/node_modules/nodeunit/deps/async.js:118:13)
at async.forEachSeries (/usr/local/lib/node_modules/nodeunit/deps/async.js:134:9)
at _concat (/usr/local/lib/node_modules/nodeunit/deps/async.js:507:9)
Reproduction code
// nodeunit-changedobject.js
module.exports.f = function (test) {
var object1 = {
property: 'object1'
}
var object2 = {
property: 'object2'
}
test.deepEqual(object1, object2)
object1.property = 'overwite'
console.log('Expected printout: AssertionError: { property: \'object1\' } deepEqual { property: \'object2\' }')
test.done()
}
- Lenguaje dominante
- JavaScript
- Estrellas
- 1.9k
- Forks
- 359
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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 caolan/nodeunit
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 42/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
Todos los issues de caolan/nodeunit
Issues similares
-
[APP/MODULE]Abiertoenhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
awesome-android-root/awesome-android-root#195 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
subhranshudash13-dotcom/Contribo#74 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
qq15725/modern-screenshot#174 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 67/100
ThatGuySam/doesitarm#1114 ·