In che modo un test di unità viene instradato con Express?

Sto imparando a utilizzare Node.js e sto giocando con Express . Mi piace molto il framework; tuttavia, ho problemi a capire come scrivere un test di unità / integrazione per un percorso.

Essere in grado di testare unità semplici moduli è facile e l'ho fatto con Mocha ; tuttavia, i miei unit test con Express hanno esito negativo poiché l'oggetto risposta che sto passando non conserva i valori.

Route-Function Under Test (rotte / index.js):

exports.index = function(req, res){
  res.render('index', { title: 'Express' })
};

Modulo di test unitario:

var should = require("should")
    , routes = require("../routes");

var request = {};
var response = {
    viewName: ""
    , data : {}
    , render: function(view, viewData) {
        viewName = view;
        data = viewData;
    }
};

describe("Routing", function(){
    describe("Default Route", function(){
        it("should provide the a title and the index view name", function(){
        routes.index(request, response);
        response.viewName.should.equal("index");
        });

    });
});

Quando lo eseguo, non riesce per "Errore: rilevate perdite globali: viewName, dati".

1. Dove sto sbagliando in modo che possa farlo funzionare?

2. C'è un modo migliore per me di testare il mio codice a questo livello?

Aggiornamento 1. Corretto frammento di codice poiché inizialmente avevo dimenticato "it ()".

Cambia il tuo oggetto risposta:

var response = {
    viewName: ""
    , data : {}
    , render: function(view, viewData) {
        this.viewName = view;
        this.data = viewData;
    }
};

E funzionerà.

Come altri hanno consigliato nei commenti, sembra che il modo canonico per testare i controller Express sia tramite supertest .

Un test di esempio potrebbe essere simile a questo:

describe('GET /users', function(){
  it('respond with json', function(done){
    request(app)
      .get('/users')
      .set('Accept', 'application/json')
      .expect(200)
      .end(function(err, res){
        if (err) return done(err);
        done()
      });
  })
});

Upside: puoi testare l'intero stack in una volta.

Lato negativo: si sente e si comporta un po 'come un test di integrazione.

Sono giunto alla conclusione che l'unico modo per testare veramente le applicazioni express è mantenere molta separazione tra i gestori delle richieste e la logica di base.

Pertanto, la logica dell'applicazione dovrebbe trovarsi in moduli separati che possono essere requiretestati su unità e avere una dipendenza minima dalle classi di richiesta espressa e risposta in quanto tali.

Quindi nei gestori delle richieste è necessario chiamare i metodi appropriati delle classi logiche principali.

Farò un esempio una volta terminata la ristrutturazione della mia attuale app!

Immagino che qualcosa del genere ? (Sentiti libero di sborsare il succo o commentare, sto ancora esplorando questo).

modificare

Ecco un piccolo esempio, in linea. Vedi l'essenza per un esempio più dettagliato.

/// usercontroller.js
var UserController = {
   _database: null,
   setDatabase: function(db) { this._database = db; },

   findUserByEmail: function(email, callback) {
       this._database.collection('usercollection').findOne({ email: email }, callback);
   }
};

module.exports = UserController;

/// routes.js

/* GET user by email */
router.get('/:email', function(req, res) {
    var UserController = require('./usercontroller');
    UserController.setDB(databaseHandleFromSomewhere);
    UserController.findUserByEmail(req.params.email, function(err, result) {
        if (err) throw err;
        res.json(result);
    });
});

Il modo più semplice per testare HTTP con express è rubare l'helper http di TJ

Io personalmente uso il suo aiutante

it("should do something", function (done) {
    request(app())
    .get('/session/new')
    .expect('GET', done)
})

Se vuoi testare specificamente il tuo oggetto rotte, passa i mock corretti

describe("Default Route", function(){
    it("should provide the a title and the index view name", function(done){
        routes.index({}, {
            render: function (viewName) {
                viewName.should.equal("index")
                done()
            }
        })
    })
})

se unit test con express 4 nota questo esempio di gjohnson :

var express = require('express');
var request = require('supertest');
var app = express();
var router = express.Router();
router.get('/user', function(req, res){
  res.send(200, { name: 'tobi' });
});
app.use(router);
request(app)
  .get('/user')
  .expect('Content-Type', /json/)
  .expect('Content-Length', '15')
  .expect(200)
  .end(function(err, res){
    if (err) throw err;
  });

Mi chiedevo anche questo, ma specificamente per i test unitari e non per i test di integrazione. Questo è quello che sto facendo adesso

test('/api base path', function onTest(t) {
  t.plan(1);

  var path = routerObj.path;

  t.equals(path, '/api');
});


test('Subrouters loaded', function onTest(t) {
  t.plan(1);

  var router = routerObj.router;

  t.equals(router.stack.length, 5);
});

Dove il routerObj è solo {router: expressRouter, path: '/api'}. Quindi carico i subrouter con var loginRouterInfo = require('./login')(express.Router({mergeParams: true}));e quindi l'app express chiama una funzione di inizializzazione che prende il router express come parametro. L'initRouter chiama quindi router.use(loginRouterInfo.path, loginRouterInfo.router);per montare il subrouter.

Il subrouter può essere testato con:

var test = require('tape');
var routerInit = require('../login');
var express = require('express');
var routerObj = routerInit(express.Router());

test('/login base path', function onTest(t) {
  t.plan(1);

  var path = routerObj.path;

  t.equals(path, '/login');
});


test('GET /', function onTest(t) {
  t.plan(2);

  var route = routerObj.router.stack[0].route;

  var routeGetMethod = route.methods.get;
  t.equals(routeGetMethod, true);

  var routePath = route.path;
  t.equals(routePath, '/');
});

Per ottenere test unitari anziché test di integrazione, ho deriso l'oggetto risposta del gestore delle richieste.

/* app.js */
import endpointHandler from './endpointHandler';
// ...
app.post('/endpoint', endpointHandler);
// ...

/* endpointHandler.js */
const endpointHandler = (req, res) => {
  try {
    const { username, location } = req.body;

    if (!(username && location)) {
      throw ({ status: 400, message: 'Missing parameters' });
    }

    res.status(200).json({
      location,
      user,
      message: 'Thanks for sharing your location with me.',
    });
  } catch (error) {
    console.error(error);
    res.status(error.status).send(error.message);
  }
};

export default endpointHandler;

/* response.mock.js */
import { EventEmitter } from 'events';

class Response extends EventEmitter {
  private resStatus;

  json(response, status) {
    this.send(response, status);
  }

  send(response, status) {
    this.emit('response', {
      response,
      status: this.resStatus || status,
    });
  }

  status(status) {
    this.resStatus = status;
    return this;
  }
}

export default Response;

/* endpointHandler.test.js */
import Response from './response.mock';
import endpointHandler from './endpointHander';

describe('endpoint handler test suite', () => {
  it('should fail on empty body', (done) => {
    const res = new Response();

    res.on('response', (response) => {
      expect(response.status).toBe(400);
      done();
    });

    endpointHandler({ body: {} }, res);
  });
});

Quindi, per ottenere il test di integrazione, puoi deridere il tuo endpointHandler e chiamare l'endpoint con supertest .

Nel mio caso l'unico che volevo verificare è se è stato chiamato il gestore destro. Volevo utilizzare supertest per sfruttare la semplicità di effettuare le richieste al middleware di routing. Sto usando Typescript a e questa è la soluzione che ha funzionato per me

// ProductController.ts

import { Request, Response } from "express";

class ProductController {
  getAll(req: Request, res: Response): void {
    console.log("this has not been implemented yet");
  }
}
export default ProductController

I percorsi

// routes.ts
import ProductController  from "./ProductController"

const app = express();
const productController = new ProductController();
app.get("/product", productController.getAll);

I test

// routes.test.ts

import request from "supertest";
import { Request, Response } from "express";

const mockGetAll = jest
  .fn()
  .mockImplementation((req: Request, res: Response) => {
    res.send({ value: "Hello visitor from the future" });
  });

jest.doMock("./ProductController", () => {
  return jest.fn().mockImplementation(() => {
    return {
      getAll: mockGetAll,

    };
  });
});

import app from "./routes";

describe("Routes", () => {
  beforeEach(() => {
    mockGetAll.mockImplementation((req: Request, res: Response) => {
      res.send({ value: "You can also change the implementation" });
    });
  });

  it("GET /product integration test", async () => {
    const result = await request(app).get("/product");

    expect(mockGetAll).toHaveBeenCalledTimes(1);

  });



  it("GET an undefined route should return status 404", async () => {
    const response = await request(app).get("/random");
    expect(response.status).toBe(404);
  });
});

Ho avuto dei problemi per far funzionare la presa in giro. ma usare jest.doMock e l'ordine specifico che vedi nell'esempio lo fa funzionare.