0

我正在使用 AVA 和 Sinon 测试一个小功能。函数看起来基本上是这样的（为简洁而编辑）：

mergeDefaults: function (opts) {
  **console.log('log 1 ->', opts);**
  opts = _.defaultsDeep(opts, defaultOptions);
  return opts;
}

我编写了一个测试来确保将正确的参数传递到_.defaultsDeep。

test.before(t => {
  sandbox = sinon.sandbox.create();
  defaultsDeepSpy = sandbox.spy(_, 'defaultsDeep');
  mergeDefaults(Object.assign({}, testOptions));
});

test('mergeDefaults runs _.defaultsDeep with the correct parameters', t => {
  **console.log('log 2 ->', defaultsDeepSpy.getCall(0).args[0]);**
  t.is(defaultsDeepSpy.getCall(0).args[0], testOptions);
  t.is(defaultsDeepSpy.getCall(0).args[1], defaultOptions);
});

test.after.always(t => {
  sandbox.restore();
});

我遇到的问题是日志 1 和日志 2 不一样。据我了解，

spy.getCall(n).args[m]

返回从第 n 次调用该函数时传递给被监视函数的第 m 个参数。但是，在这种情况下，日志 2 实际上是返回函数的结果，而不是第一个参数。

选项对象看起来像这样：

testOptions = {
  key1: 11,
  key2: 22,
  key3: 33
}
defaultOptions = {
  key1: 1,
  key2: 2,
  key3: 3,
  key4: 4,
  key5: 5
}

但是 console.logs 看起来像这样：

log 1 -> testOptions = {
  key1: 11,
  key2: 22,
  key3: 33
}
log 2 -> testOptions = {
  key1: 11,
  key2: 22,
  key3: 33,
  key4: 4,
  key5: 5
}

所以 spy.getCall(0).args[0] 又名“log 2”实际上返回了 _.defaultsDeep 运行后“opts”变成了什么。这感觉就像是 Sinon 库中的一个错误，但更有可能是我做错了什么。非常感谢您的帮助，在此先感谢！

4

1 回答 1

0

您遇到的根本问题是对参数的引用（在这种情况下opts）与您使用值操作的对象相同（因为所有对象在 JavaScript 中都是通过引用传递的）。实际上，您正在更改opts指向的物理位置。

由于 Sinon 不获取您传入的对象的“快照”或“副本”（它只返回您传入的相同对象引用），因此您也影响了 Sinon 返回的内容。

要解决此问题，代码应类似于

mergeDefaults: function (opts) {
    return _.defaultsDeep(Object.assign({}, opts), defaultOptions);
}

这Object.assign将创建一个新对象（以及内存中的相关位置）。因此，它将opts保持不变，因此您可以检查传入的内容。

于 2017-08-04T15:11:25.023 回答