node jasmine - 如何对涉及redis调用的函数进行单元测试？

Question

我刚开始玩 Jasmine，我还在为间谍/嘲笑的事情苦苦挣扎，例如，我有一个函数

module.exports = (() => {
    ....

    function getUserInfo(id) {
        return new Promise((resolve, reject) => {
            redis.getAsync(id).then(result => {
                resolve(result)
            })
        }) 
    }
    return { getUserInfo: getUserInfo }
})()

然后我开始编写 Jasmine 规范

describe('Test user helper', () => {
    let userInfo

    beforeEach(done => {
        userHelper.getUserInfo('userid123')
            .then(info => {
                userInfo = info
                done() 
            })
    })

    it('return user info if user is found', () => {
        expect(userInfo).toEqual('info of userid 123')
    })
})

它运行良好，但我的问题是如何模拟 redis.getAsync 调用，使其成为真正的独立单元测试？

谢谢。

Answer 1

好问题。您可以模拟 redis 依赖项，但前提是您稍微重写代码以使其更具可测试性。 在这里，这意味着让 redis 成为工厂的参数，该工厂返回包含 getUserInfo 的对象。

当然，这改变了 API，调用者现在需要调用导出来获取对象。为了解决这个问题，我们可以创建一个包装模块，使用标准 redis 对象调用该函数，并返回结果。然后我们将实际的工厂移动到一个内部模块中，它仍然允许对其进行测试。

这可能看起来像

user-helper/factory.js

module.exports = redis => {
  ....

  function getUserInfo(id) {
    return redis.getAsync(id); // note simplified as new Promise was not needed
  }
  return {getUserInfo};
};

user-helper/index.js

// this is the wrapper that preserves existing API
module.exports = require('./factory')(redis);

现在开始测试

const userHelperFactory = require('./user-helper/factory');

function createMockRedis() {
  const users = [
    {userId: 'userid123'},
    // etc.
  ];
  return {
    getAsync: function (id) {
      // Note: I do not know off hand what redis returns, or if it throws,
      // if there is no matching record - adjust this to match.
      return Promise.resolve(users.find(user => user.userId === id));
    }
  };
}

describe('Test user helper', () => {
  const mockRedis = createMockRedis();
  const userHelper = userHelperFactory(mockRedis);

  let userInfo;

  beforeEach(async () => {
    userInfo = await userHelper.getUserInfo('userid123');
  });

  it('must return user info when a matching user exists', () => {
    expect(userInfo).toEqual('info of userid 123');
  });
});

注意：正如在 cmets 中所讨论的，这只是我对手头情况的偶然处理。您可以使用许多其他设置和约定，但主要想法只是基于 IIFE 结果的现有导出，这是一个可靠的模式，我利用 NodeJS /index 约定来保留现有的 API。您也可以使用一个文件并通过module.exports = factory(redis) 和module.exports.factory = factory 导出，但我相信这在NodeJS 中会不那么惯用。更广泛的观点是能够模拟测试，而可测试性通常只是参数化。

参数化非常强大，它的简单性就是为什么使用函数式语言的开发人员有时会嘲笑 OOP 程序员，比如你的真正的，以及我们的秘密咒语，比如“哦，光荣的依赖注入容器，给我一个 instanceofX” :)

并不是 OOP 或 DI 搞错了，而是可测试性、DI、IOC 等只是关于参数化。

有趣的是，如果我们将 redis 作为一个模块加载，并且如果我们使用可配置的模块加载器，例如 SystemJS，我们可以通过简单地在测试级别使用加载器配置来做到这一点。甚至 Webpack 在某种程度上也允许你这样做，但对于 NodeJS，你需要对 Require 函数进行修补，或者创建一堆假包，这不是很好的选择。

对OP的具体回应

谢谢！这是个好主意，但实际上，当我有大量文件要测试时，我需要为每个文件创建一个工厂和 index.js，这似乎很奇怪。

您需要重构 API 表面并简单地导出消费代码必须调用的工厂，而不是应用这些工厂的结果，以减轻负担，但需要权衡取舍，默认实例对消费者有帮助。