Question

Backgoround

我正在尝试测试一个名为multiQuery的函数，并确保它使用正确的参数和正确的次数调用另一个函数query。

代码

以下是我正在使用的代码的简化。这是我用来创建对象的对象：

const createDb = () => {

    const query = (sql, values, type = "all") => {
        console.log(`original query called: ${sql}, ${values}, ${type}`);
        return Promise.resolve();
    };

    const multiQuery = requests => Promise.all(
        requests.map( req => query(req.sql, req.values, req.mode || "all") )
    );

    return {
        query,
        multiQuery
    };
};

这是测试：

const sinon = require("sinon");
const chai = require("chai");
const expect = chai.expect;
const sinonChai = require("sinon-chai");
chai.use(sinonChai);

describe("multiQuery", () => {

        it("should call query multiple times with the correct parameters", async() => {
            const requests = [
                { sql: "hello", values: [1, 2, 3],  mode: "first"   },
                { sql: "world", values: [4, 5, 6],  mode: "all"     },
                { sql: "moon",  values: [7, 8]  }
            ];

            const db = createDb();
            const spy = sinon.spy( db, "query" );

            await db.multiQuery( requests );

            expect( spy ).to.have.been.calledThrice();
            expect( spy ).to.have.been.calledWith( "hello", [1, 2, 3], "first" );
        });
});

问题

问题在于，无论我做什么，我总是得到消息：

AssertionError：预期的查询被称为三次，但是它被称为0次

我无法修复它。

这是因为multiQuery绑定到原始函数而不是间谍。

激荡

我在考虑在query中注入multiQuery依赖项，但是我不希望每次调用它时都传递依赖项，也不想污染对象的API使用不需要的工厂方法。

我真的不知道如何解决这个问题.... 我该如何解决这个问题？

Answer 1

你对测试失败的原因是正确的。

潜在的解决方案：

const createDb = () => {
   const db = { //the name here isn't important
        query,
        multiQuery
    };

    const query = (sql, values, type = "all") => {
        console.log(`original query called: ${sql}, ${values}, ${type}`);
        return Promise.resolve();
    };

    // now multiQuery references the same 'query' as what is returned
    const multiQuery = requests => Promise.all(
        requests.map( req => db.query(req.sql, req.values, req.mode || "all") )
    );

    return db;
};

Answer 2

在询问并进行更多测试后，我发现如果我将createDb函数更改为以下内容：

const createDb = () => {
    const db = {}
    db.query = (sql, values, type = "all") => {
        console.log(`original query called: ${sql}, ${values}, ${type}`);
        return Promise.resolve();
    };
    db.multiQuery = requests => Promise.all(
        requests.map( req => db.query(req.sql, req.values, req.mode || "all") )
    );

    return db;
}

sinon中的间谍方法将按预期工作。

Sinon.spy没有处理简单对象

Backgoround

代码

问题

激荡

2 个答案: