Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

test.deepEqual may print incorrect property values for subsequently modified properties SERIOUS ISSUE

Aperta
  • Qualcosa non va?
#176 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
45/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Ferma
Stack tecnologico
javascript
Ambito
testing-qa

Direzione di ricerca

Inizia con test.deepEqual in lib/types.js, usando lo stack trace e la riproduzione nodeunit-changedobject.js come punti di ingresso. Esegui l'esempio nodeunit fornito e confronta l'output dell'errore con il relativo output atteso; il lavoro è completato quando i valori riportati rimangono quelli presenti al momento della chiamata a deepEqual, anziché quelli risultanti da mutazioni successive.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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()
}
Lingua principale
JavaScript
Stelle
1.9k
Fork
359
Nessuna PR unita negli ultimi 30g

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di caolan/nodeunit

Tutte le issue di caolan/nodeunit

Issue simili

Altre issue su JavaScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.