Question

我尝试使用Mocha，Chai和Supertest编写单元测试，但不幸的是，现在没有成功。
我有下一条路线：

var express = require('express');
var GoogleUrl = require('google-url');
var _ = require('lodash');
var token = require('../middlewares/token');
var Url = require('../models/url');
var config = require('../config');
var router = express();

router.post('/urls/create', token.required, createShortUrl);

module.exports = router;

function createShortUrl(req, res) {

    _.trim(req.body.list_tags);
    var tags = _.split(req.body.list_tags, ',');
    tags.splice(tags.length - 1, 1);

    var date = returnDate();
    var time = returnTime();

    var googleUrl = new GoogleUrl({
        'key': config.get('google_key')
    });

    googleUrl.shorten(req.body.full_url, function (err, shortUrl) {

        if (err) {
            res.status(500).json(err);
        }

        var url = new Url({
            'author': req.payload.username,
            'description': req.body.description,
            'full_url': req.body.full_url,
            'short_url': shortUrl,
            'list_tags': tags,
            'date': date,
            'time': time
        });

        url.save(function (err) {

            if (err) {
                res.status(500).json(err);
            } else {
                res.status(200).json('Shortener url is created!');
            }

        });

    });

}

   function returnDate() {
    var d = new Date();
    var yyyy = d.getFullYear();
    var mm = d.getMonth() < 9 ? '0' + (d.getMonth() + 1) : (d.getMonth() + 1);
    var dd = d.getDate() < 10 ? '0' + d.getDate() : d.getDate();
    return dd + '-' + mm + '-' + yyyy;
}

function returnTime() {
    var d = new Date();
    var hh = d.getHours() < 10 ? '0' + d.getHours() : d.getHours();
    var min = d.getMinutes() < 10 ? '0' + d.getMinutes() : d.getMinutes();
    var ss = d.getSeconds() < 10 ? '0' + d.getSeconds() : d.getSeconds();
    return hh + ':' + min + ':' + ss;
}

我使用Mongoose将数据保存在MongoDB中。对于路线，我写了单元测试：

var app = require('../server');
var chai = require('chai');
var request = require('supertest');
var jwt = require('express-jwt');
var config = require('../config');

var expect = chai.expect;

describe('Urls Tests', function () {

  var url = {
    author : 'Alexey',
    description : 'grrggr',
    full_url : 'https://github.com',
    short_url : 'short_url_by_google',
    date : '30-06-2017',
    time : '18:21:27',
    count_click : 0,
    list_tags : [
      'Sport',
      'Football'
    ]
  };

  var token = {
    required: jwt({
      secret: config.get('jwtSecret'),
      userProperty: 'payload',
      getToken: 'token',
    })
  };

describe('## Create url ', function() {
    it('should create a url', token.required, function(done) {
      request(app)
        .post('/urls/create')
        .send(url)
        .end(function(err, res) {
        expect(res.statusCode).to.equal(200);
        expect(res.body.author).to.equal('Alexey');
        url = res.body;
        done();
      });
    });
  });

});

当我运行单元测试时，我收到错误：

TypeError: next is not a function
      at Context.middleware (node_modules\express-jwt\lib\index.js:76:16)

我无法理解为什么会得到它。请帮帮我！我认为我创造的错误很少，因此可以找到。

更新

middlewares/token.js：

var jwt = require('express-jwt');
var config = require('../config');

function getTokenFromHeader(req){
    if (req.headers.authorization && req.headers.authorization.split(' ')[0] === 'Token') {
        return req.headers.authorization.split(' ')[1];
    }

    return null;
}

var token = {
    required: jwt({
        secret: config.get('jwtSecret'),
        userProperty: 'payload',
        getToken: getTokenFromHeader
    })
};

module.exports = token;

Answer 1

错误发生在it('should create a url', token.required, function(done)行。由于令牌应该是中间件，因此需要参数req，res和next。

在测试中，您没有向服务器提供令牌，因此您可以从测试中删除token.required。

但是，如果您确实对令牌身份验证进行了测试，则可以使用自己的token.required函数在测试中手动调用next来解决此问题。您还需要在请求中为服务器提供jwt。

Answer 2

it('should create a url', token.required, function(done) {

Mocha测试不允许使用Connect样式的中间件。如果没有该支持，将token.required传递给it()将会调用token.required。 jwt()内的token.required正在尝试调用回调next，它是Connect中间件函数args的一部分

const express = require('express')
const server = express()

server.use((req, res, next) => {
  console.log('this is middleware')
  return next() // Call next middleware 
})

考虑到你想要测试路线，你无论如何都不需要这样做。如果路线工作正常，则需要钥匙并且可以正常工作。您可以通过在没有JWT的情况下发出请求来测试该场景，并断言它通过状态码失败。

NodeJs：next不是函数

2 个答案: