错误:<tohavebeencalled>:期待间谍,但得到了功能

时间:2018-02-20 11:03:44

标签: javascript jasmine

这是测试代码

var list = new List([1, 2, 3, 4]);
var list2 = new List([5, 1]);

beforeAll(function () {
  spyOn(list.values, 'map').and.callThrough();

  list.map(plusOne);
});

it('Array.prototype.map()', function () {
  expect(list.values.map).not.toHaveBeenCalled();
});

This results in the following error 1) List must not call native Array function Array.prototype.map()   Message:
    Error: <toHaveBeenCalled> : Expected a spy, but got Function.
    Usage: expect(<spyObj>).toHaveBeenCalled()

  class List {
    constructor(clist = []) {
        this.values = clist;
    }
    map(f) {
        var temp = [];
        this.values.forEach((item, index) => {
                   temp.push(f(item));
                });
        this.values = temp;
        return this;
    }
}
module.exports = { List };

我认为这不是单元测试失败因为我得到相同的消息,无论我是否调用not.tohaveBeenCalled()或toHaveBeenCalled()。

我正在使用节点8.9.4和jasmine 2.8.0。

我相信语法是正确的,因为当我针对这些测试运行其他代码时,它们会通过。但是我的代码导致了这个错误。

我的问题是上述错误是什么意思? 的问候,

1 个答案:

答案 0 :(得分:2)

我只是运行以下测试,它适用于jasmine@3.0.0

fit('Spy on map works', () => {
        let someArray = [1, 3, 5];
        spyOn(someArray, 'map').and.callThrough();
        someArray.map(function(r){ console.log(r); });
        expect(someArray.map).toHaveBeenCalled();
 });

您可能希望运行此示例以了解它是否适​​用于您的测试。

正如我在评论中所说,您的列表映射方法会使用新数组覆盖list.values。因此,间谍不再存在了。尝试类似的事情:

someArray.forEach((item, index) => {
                   someArray[index] = f(item);
});

解释发生的事情:

//WORKS
 fit('Spy on map works', () => {
    let someArray = [1, 3, 5];
    spyOn(someArray, 'map').and.callThrough();
    someArray.forEach((item, index) => {
        someArray[index] = (item + '_t');
    });
    someArray.map(function(r){ console.log(r); });
    expect(someArray.map).toHaveBeenCalled();
});
//FAILS because array is another object.
fit('Spy on map fails', () => {
    let someArray = [1, 3, 5];
    spyOn(someArray, 'map').and.callThrough();

    let tempArray = [];
    someArray.forEach((item, index) => {
        tempArray.push(item + '_t');
    });
    someArray = tempArray;

    someArray.map(function(r){ console.log(r); });
    expect(someArray.map).toHaveBeenCalled();
});

但是,你可以只监视原型。这样的事情:

 spyOn(Array.prototype, 'map').and.callThrough();

然后,你的测试应该有效。