测试同步代码

Question

我有一段代码使用node-sync，如下所示：

function funcA() {
  return new Promise(function(resolve, reject) {
    Sync(function () {
      return funcB.sync();
    }, function (err, result) {
      if(err) {
        reject(err);
      } else {
        resolve(result);
      }
    });
}

这段代码是用mocha+chai测试的：

it("should return array", function() {
  return funcA().then(function(result) {
    expect(result).to.be.an.instanceof(Array);
  });
});

几个月前它工作得很好，但现在这个测试总是超时：

错误：超过 2000 毫秒的超时。确保在此测试中调用了 done() 回调。

到目前为止我已经尝试过：

• 使用 done() 而不是返回承诺
• 将node-sync 替换为synchronize.js
• 增加超时

我发现，这个测试的 expect(... 部分实际上是被调用的，但只是在 mocha 杀死测试之后。无论当前设置什么超时间隔，在我收到 Error: timeout 消息后，expect(.. 总是会被调用约 20 毫秒。

我通过在测试文件顶部添加setInterval(function(){}, 10) 解决了这个问题。 我想知道为什么会这样，是否有更好的方法来解决这个问题？

[编辑] 看起来这是一个特定于节点版本的问题。测试在 0.12.4 上失败，但在 0.10.38 上正常运行。

[编辑] 实际代码可用here。

Answer 1

根据您的code，我假设您的 funcB 函数正在以同步方式运行代码。

所以当我以这种方式创建 funcB 时：

function funcB() {
  return [1, 2, 3];
}

并运行测试，Mocha 报错：

错误：超过 2000 毫秒的超时。确保在此测试中调用了 done() 回调。

但如果我将 异步函数 中的 funcB 转换为：

function funcB(cb) {
  process.nextTick(function () {
    cb(null, [1, 2, 3]);
  });
}

Mocha 运行测试没有任何问题：

✓ 应该返回一个数组

---

所以我运行正常的完整代码（注释的 funcB 是会导致错误的代码）是这样的：

// install dependencies
// npm install promise
// npm install sync
var Promise = require('promise');
var assert = require('assert');
var Sync = require('sync');

function funcA() {
  return new Promise(function (resolve, reject) {
    Sync(function () {
      return funcB.sync();
    }, function (err, result) {
      if (err) {
        reject(err);
      } else {
        resolve(result);
      }
    });
  });
}

// function funcB() {
//   return [1, 2, 3];
// }

function funcB(cb) {
  process.nextTick(function () {
    cb(null, [1, 2, 3]);
  });
}

it("should return an array", function(done) {
  return funcA().then(
    function (result) {
      console.log(result);
      assert.equal(Array.isArray(result), true);
      done();
    }
  );
});

所以我认为我认为误用由同步库创建的sync method（在同步函数上使用它）是导致此问题的原因。

Answer 2

您可能只是错过了done() 回调：

it("should return array", function(done) {
    funcA().then(function(result) {
        expect(result).to.be.an.instanceof(Array);
        done();
    });
});

http://mochajs.org/#asynchronous-code

Answer 3

您可以对 mocha 可执行文件使用超时参数。

例如，如果您想要 500 毫秒的超时，只需将您的 package.json 更改为：

"scripts": {
    "test": "mocha specs --timeout 500 --require specs/helpers/chai.js"
},

也许你的承诺被拒绝了，所以你必须使用catch方法调用done。

it("should return array", function(done) {
    return funcA().then(function(result) {
        expect(result).to.be.an.instanceof(Array);
        done();
    }).catch(done);
});

这也将帮助您调试承诺代码中可能发生的最终错误。