使用Mocha.js测试预定义的回调函数

Question

我正在尝试为以下功能编写测试。

function services(api){
    request(`${api}?action=services`, function(err, res, body) {
        if (!err && res.statusCode === 200){
            var resJson = JSON.parse(body);
            var numberOfServices = resJson.length;
            console.log("Service called: services");
            console.log("-----------");
            for (i = 0; i < numberOfServices; i++){
                console.log("Service ID: " + resJson[i].service);
                console.log("Service Name: " + resJson[i].name);
                console.log("-----------");
            }
            return resJson;
        }
    }); 
}

该测试正在检查该函数是否返回对象。 resJson是要返回并测试的对象。

下面是使用Mocha.js和Chai.js断言库编写的测试用例。

var chai = require('chai');
var assert = chai.assert;
var sendRequest = require('../request');

describe('Test 1', function() {

    var api = 'http://instant-fans.com/api/v2';

    it('services() should return an object of services', function(done) {
        var object = sendRequest.services(api);
        assert.isObject(object);
    });

});

但是，当我运行测试时，它失败并显示以下控制台输出。 说resJson是不确定的。 我猜想Mocha试图在函数services()返回对象之前断言resJson是一个对象，但是我不确定如何解决这个问题。

 Test 1
    1) services() should return an object of services

 0 passing (27ms)
 1 failing

  1) Test 1 services() should return an object of services:
     AssertionError: expected undefined to be an object
      at Function.assert.isObject (node_modules/chai/lib/chai/interface/assert.js:555:35)
      at Context.<anonymous> (test/requestTest.js:11:16)

我尝试过在线搜索，我看到人们使用done()方法解决了这个问题。 但是，由于我在我的services()函数中使用了回调，因此无法正常工作。

Answer 1

我建议您阅读有关异步函数用法的问题

---

您不能像在var object = sendRequest.services(api)那样将异步函数调用分配给变量。 您需要使用callback ，应将它作为参数添加到services函数中

function services(api, callback){
    // your logic here
}

代替执行return resJson您可以在services函数中调用callback(resJson) 。 然后，在测试中，您需要使用第二个参数作为function(result)来调用services function(result)

sendRequest.services(api, function(result){
    assert.isObject(result);
});

编辑

这就是您的services功能的外观

function services(api, callback){
    request(`${api}?action=services`, function(err, res, body) {
        if (!err && res.statusCode === 200){
            var resJson = JSON.parse(body);
            var numberOfServices = resJson.length;
            console.log("Service called: services");
            console.log("-----------");
            for (i = 0; i < numberOfServices; i++){
                console.log("Service ID: " + resJson[i].service);
                console.log("Service Name: " + resJson[i].name);
                console.log("-----------");
            }
            callback(resJson);
        }
    }); 
}

Answer 2

您的代码似乎正在执行异步请求sendRequest.services(api);

您需要做的是处理在Mocha测试中可用的done()回调： it('services() should return an object of services', function(done) {})

您需要做的是重新编写services功能，因为它不正确。 您不能从异步函数返回，异步函数通常基于I / O，并且不会阻塞Nodejs主线程。 为了使您的主线程访问该值，您必须传递一个回调函数，该回调函数将在该值可用后执行。

function services(api, cb){
    request(`${api}?action=services`, function(err, res, body) {
        var resJson;
        if (!err && res.statusCode === 200){
            resJson = JSON.parse(body);
            var numberOfServices = resJson.length;
            console.log("Service called: services");
            console.log("-----------");
            for (i = 0; i < numberOfServices; i++){
                console.log("Service ID: " + resJson[i].service);
                console.log("Service Name: " + resJson[i].name);
                console.log("-----------");
            }
        }
        if (cb) {
            cb(err, resJson);
        }
    }); 
}

并这样重写测试：

describe('Test 1', function() {
    var api = 'http://instant-fans.com/api/v2';

    it('services() should return an object of services', function(done) {
        sendRequest.services(api, function(resJson) {
            assert.isObject(resJson);
            done();
        });
    });

});

我将走得更远，并将错误传递给回调函数（这是nodejs中的常见做法）：

cb(err, resJson);

您的测试可以打印错误以进行更好的调试：

sendRequest.services(api, function(err, resJson) {
    if (err) {
        console.error(err);
    }
    assert.isObject(resJson);
    done();
});