caolan / caolan/nodeunit

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

未关闭
#176 1 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
主要语言
JavaScript
星标
1.9k
派生
359
PR 合并指标
30 天内没有已合并 PR

描述

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

``` js
// 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()
}
```

贡献指南

这个仓库没有索引到贡献指南

调研方向

从 lib/types.js 中的 test.deepEqual 开始,以堆栈跟踪和 nodeunit-changedobject.js 重现作为入口。运行提供的 nodeunit 示例,并将失败输出与预期打印结果进行比较;当报告的值仍然是调用 deepEqual 时存在的值,而不是之后发生突变后的值时,即表示完成。

由索引模型根据 Issue 内容生成。

评估

技术栈
javascript
领域
testing-qa
Issue 类型
缺陷
难度
3/5
预计耗时
1-2 天
活跃度
停滞
描述清晰度
基本清楚
新手友好度
45/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。