web-dev-qa-db-ja.com

モカで非同期関数をテストする

Node.jsで実行され、http apiへの単純なリクエストを行う非同期javascript関数をテストしたいと思います。

const Host = 'localhost';
const PORT = 80;

http = require('http');

var options = {
    Host: Host,
    port: PORT,
    path: '/api/getUser/?userCookieId=26cf7a34c0b91335fbb701f35d118c4c32566bce',
    method: 'GET'
};
doRequest(options, myCallback);

function doRequest(options, callback) {

    var protocol = options.port == 443 ? https : http;
    var req = protocol.request(options, function(res) {

        var output = '';
        res.setEncoding('utf8');

        res.on('data', function(chunk) {
            console.log(chunk);
            output += chunk;
        });

        res.on('error', function(err) {
            throw err;
        });

        res.on('end', function() {
            var dataRes = JSON.parse(output);
            if(res.statusCode != 200) {
                throw new Error('error: ' + res.statusCode);
            } else {
                try {
                    callback(dataRes);                        
                } catch(err) {
                    throw err;
                }
            }
        });

    });

    req.on('error', function(err) {
        throw err;
    });

    req.end();

}

function myCallback(dataRes) {
    console.log(dataRes);
}

このコードを実行すると、期待どおりに応答が表示されます。

これをモカテストで実行すると、リクエストは実行されません。

describe('api', function() {
    it('should load a user', function() {
        assert.doesNotThrow(function() {
            doRequest(options, myCallback, function(err) {
                if (err) throw err;
                done();
            });
        });
        assert.equal(res, '{Object ... }');
    });
});

問題は、次のコードがないことです。

var req = protocol.request(options, function(res) {

単純なconsole.logでも実行されません。

誰か助けてもらえますか？

javascriptnode.jstestingtddmocha
30
MadFool

コールバックdoneを、mochaに提供される関数（この場合はit()関数）の引数として指定する必要があります。そのようです：

_describe('api', function() {
    it('should load a user', function(done) { // added "done" as parameter
        assert.doesNotThrow(function() {
            doRequest(options, function(res) {
                assert.equal(res, '{Object ... }'); // will not fail assert.doesNotThrow
                done(); // call "done()" the parameter
            }, function(err) {
                if (err) throw err; // will fail the assert.doesNotThrow
                done(); // call "done()" the parameter
            });
        });
    });
});
_

また、doRequest(options, callback)のシグネチャは2つの引数を指定しますが、テストで呼び出す場合は3つ提供します。

モカはおそらくメソッドdoRequest(arg1,arg2,arg3)を見つけることができませんでした。

それはいくつかのエラー出力を提供しませんでしたか？多分あなたはより多くの情報を得るためにモカオプションを変更することができます。

編集：

andhoは正しい、2番目のアサートは_assert.doesNotThrow_と並行して呼び出されますが、それは成功コールバックでのみ呼び出されるべきです。

サンプルコードを修正しました。

編集2：

または、エラー処理を簡略化するには（Dan M.のコメントを参照）：

_describe('api', function() {
    it('should load a user', function(done) { // added "done" as parameter
        assert.doesNotThrow(function() {
            doRequest(options, function(res) {
                assert.equal(res, '{Object ... }'); // will not fail assert.doesNotThrow
                done(); // call "done()" the parameter
            }, done);
        });
    });
});
_
46
Risadinha

コールバックをサポートしない非同期関数がある場合、または不要なコールバックを使用する必要がないと考えている場合は、テストを非同期テストに変えることもできます。

の代わりに：

it('should be able to do something', function () {});

単に行う：

it('should be able to do something', async function () {});
                                     ^^^^^

これで非同期関数をawaitできます：

it('should be able to do something', async function () {
  this.timeout(40000);

  var result = await someComplexFunction();

  assert.isBelow(result, 3);
});
3
Thomas W

私のプロジェクトでは、httpクライアントに対して非常によく似たテストを行いました。ここにコードを貼り付けて、役に立てば幸いです。これがクライアントです（私のnodejsサーバーはExpressを使用しており、エラー処理にはPromiseを使用しています）。

var http = require('http');
var querystring = require('querystring');

module.exports = {
  get: function(action, params, res, callback) {
    doPromiseRequest(action, querystring.stringify(params), callback, 'GET', 'application/json')
      .then((response) => callback(response))
      .catch((error) => {
        res.status(500);
        res.render('error', {layout: false, message: error.message, code: 500});
      });
  },
}

function doPromiseRequest(action, params, callback, method, contentType) {
    var options = {
      hostname: 'localhost',
      port: 3000,
      path: '/api/v1/' + action.toString(),
      method: method,
      headers: {
        'Content-Type': contentType,
        'Content-Length': Buffer.byteLength(params)
      }
    };

    return new Promise( (resolve, reject) => {

      var req = http.request(options, 
        function(response) {
          response.setEncoding('utf8');

          var data = '';
          response.on('data', function(chunk) {
            data += chunk;
          });

          response.on('end', function() {
            var parsedResponse;

            try {
              parsedResponse = JSON.parse(data);
            } catch(err) {
              reject({message: `Invalid response from hurricane for ${action}`});
              return;
            }

            if (parsedResponse.error)
              reject(parsedResponse.error);
            else
              resolve(parsedResponse);
          });

          response.on('error', function(err){
            console.log(err.message);
            reject(err);
          });
        });

      req.on('error', function(err) {
        console.log(err);
        reject({message: err.message});
      });

      req.write(params);
      req.end(); 
    });    
}

そしてここにテストがあります：

var http = require('http');
var expect = require('chai').expect;
var sinon = require('sinon');
var PassThrough = require('stream').PassThrough;

describe('Hurricane Client tests', function() {
  before(function() {
    this.request = sinon.stub(http, 'request');
  });

  after(function() {
    http.request.restore();
  });

  it('should convert get result to object', function(done) {
    var expected = { hello: 'world' };
    var response = new PassThrough();
    response.statusCode = 200;
    response.headers = {}
    response.write(JSON.stringify(expected));
    response.end();

    var request = new PassThrough();

    this.request.callsArgWith(1, response).returns(request);

    client.get('any', {}, null, function(result) {
      expect(result).to.eql(expected);
      done();
    });
  });
});
0
Antonio Ganci