Sinon: Deep Equal 不应该完全等于错误

创建于 2017-03-09  ·  3评论  ·  资料来源: sinonjs/sinon

嗨诗农! 感谢您提供很棒的库,其中包含添加到预发布版本的酷炫新功能。 我一直在为新引入的 API 使用 pre(我打算自己编写扩展,我在挖掘源代码时发现了代码)但是 .6 版本破坏了我们的大型测试套件。

  • 诗浓版本:2.0.0-pre.6
  • 环境 : macOS with Node6+
  • 示例网址:
  • 您正在使用的其他库:
    摩卡、巴别塔

你期望会发生什么?
当传递不同但“相等”的错误实例时,我没想到.deep.equals断言会失败
实际发生了什么
对于Error任何子类,深equals确实精确equalshttps://github.com/sinonjs/sinon/blob/master/lib/sinon/util/core/deep-equal.js#L54 -L56

Medium Help wanted stale

最有用的评论

嗨,大家好!

这确实很容易解决,我愿意为此做一个公关。 但是,我想建议一种不同的方法,而不仅仅是修复if子句。
Chai我们有一个名为deep-eql的模块,它进行深度相等性检查。 它已经过广泛的测试和性能微调,并且在 Chai 的核心上运行良好。

通过采用相同的库,我们将来也会在相同的代码库上工作,以改进它并使每个使用它的项目变得更好。

此外,由于此库接受comparator参数,它仍然允许我们使用use方法

如果你不想采用它也没关系,但我只是想在解决这个问题之前提出建议。

所有3条评论

嗨,大家好!

这确实很容易解决,我愿意为此做一个公关。 但是,我想建议一种不同的方法,而不仅仅是修复if子句。
Chai我们有一个名为deep-eql的模块,它进行深度相等性检查。 它已经过广泛的测试和性能微调,并且在 Chai 的核心上运行良好。

通过采用相同的库,我们将来也会在相同的代码库上工作,以改进它并使每个使用它的项目变得更好。

此外,由于此库接受comparator参数,它仍然允许我们使用use方法

如果你不想采用它也没关系,但我只是想在解决这个问题之前提出建议。

@lucasfcosta我喜欢提议的解决方案。 将match争论

我很想看到一个减少代码库大小的 PR

此问题已自动标记为陈旧,因为它最近没有活动。 如果没有进一步的活动发生,它将被关闭。 感谢你的贡献。

此页面是否有帮助?
0 / 5 - 0 等级