3

这是一个关于在 Sails JS 中使用 Mocha 进行异步测试的问题。

我正在使用超测试库在 Sails JS 中编写控制器测试。我想检查一个方法是否在 HTTP POST 上被调用到我们的控制器。为此，我正在对该方法进行存根，并期望它在以下方式中被调用end()：

request(sails.hooks.http.app)
    .post('heartbeat/create')
    .send('device: 1')
    .end(function(err, res) {
        expect(publishCreateStub.called).to.be.true;
        done();
    });

当我运行它时，期望失败，因为断言时未调用该方法。但是，当我将期望放在 asetTimeout中时，它会起作用：

request(sails.hooks.http.app)
    .post('heartbeat/create')
    .send('device: 1')
    .end(function(err, res) {
        setTimeOut(function() {
            expect(publishCreateStub.called).to.be.true;
            done();
        }, 1000);
    });

有没有办法让测试通过而没有setTimeout？

这是我正在测试的代码部分：HeartbeatController#create

您还可以通过发送拉取请求帮助我们解决问题：https ://github.com/multunus/one-mdm/issues/1

4

2 回答 2

3

实际问题是您的控制器不等待publishCreate被调用。因此，您在201 Created没有检查是否已创建任何内容的情况下做出响应。

要么失败，要么Heartbeat.findOne失败Heartbeat.publishCreate，但你会更清楚地知道这一点。

要解决此问题，您应该修改控制器，将响应部分移动到Heartbeat承诺回调中：

create: function (req, res, next) {
  if(req.isSocket) {
    Heartbeat.watch(req);
  }

  Heartbeat.create(req.body)
    .exec(function(error, heartbeat) {
      if(error) {
        res.status(422);
        return res.send('Invalid heartbeat data');
      }

      Heartbeat.findOne(heartbeat.id).populate('device').then(function(newHeartbeat) {
        Heartbeat.publishCreate(newHeartbeat.device);
      }).then(function() {
        // success
        res.status(201);
        res.json({
          device: heartbeat.device
        });
      }, function(err) {
        // something bad happened
        next(err);
       });

    });
}

在我的示例中，我将实际错误处理委托给下一个快速错误处理中间件：

next(err);

但是您可以决定自己处理错误，例如：

res.status(400);
res.send('Can not publish Heartbeat create');
于 2014-11-24T09:57:48.737 回答
1

您的问题不在您的测试中，而是在您的应用程序中：

Heartbeat.create(req.body)
  .exec(function(error, heartbeat) {
    if(error) {
      res.status(422);
      return res.send('Invalid heartbeat data');
    }        

    Heartbeat.findOne(heartbeat.id).populate('device').then(function(newHeartbeat) {
      Heartbeat.publishCreate(newHeartbeat.device);
    });

    res.status(201);
    return res.json({
      device: heartbeat.device
    });
  });
}

https://github.com/multunus/one-mdm/blob/master/api/controllers/HeartbeatController.js

在这里，您正在创建一个承诺，但没有返回该承诺。因此，在继续之前，您无法等待承诺完成（或错误）。

在不更详细地了解您的应用程序的情况下，我不确定修复。我会考虑将中间的位抽象为服务*，然后您的HeartbeatController测试可以检查服务是否被调用（同步）并且您对服务的测试可以执行使用承诺的异步部分。

* 更新：通过“服务”，我的意思是把它从 HTTP 的任何问题中抽象出来，所以它不知道请求或响应对象。最终结果是您已将 HTTP 部分（控制器）与管理 Heartbeat 对象的部分分开。这使得这两个部分更容易进行单元测试。

于 2014-11-24T09:38:13.523 回答