Mocha API Testing: ottenere "TypeError: app.address non è una funzione"


102

Il mio problema

Ho codificato un'API CRUD molto semplice e recentemente ho iniziato a codificare anche alcuni test utilizzando chaie chai-httpma ho un problema durante l'esecuzione dei miei test con $ mocha.

Quando eseguo i test ottengo il seguente errore sulla shell:

TypeError: app.address is not a function

Il mio codice

Ecco un esempio di uno dei miei test ( /tests/server-test.js ):

var chai = require('chai');
var mongoose = require('mongoose');
var chaiHttp = require('chai-http');
var server = require('../server/app'); // my express app
var should = chai.should();
var testUtils = require('./test-utils');

chai.use(chaiHttp);

describe('API Tests', function() {
  before(function() {
    mongoose.createConnection('mongodb://localhost/bot-test', myOptionsObj);
  });

  beforeEach(function(done) {
    // I do stuff like populating db
  });

  afterEach(function(done) {
    // I do stuff like deleting populated db
  });

  after(function() {
    mongoose.connection.close();
  });

  describe('Boxes', function() {

    it.only('should list ALL boxes on /boxes GET', function(done) {
      chai.request(server)
        .get('/api/boxes')
        .end(function(err, res){
          res.should.have.status(200);
          done();
        });
    });

    // the rest of the tests would continue here...

  });

});

E i expressfile della mia app ( /server/app.js ):

var mongoose = require('mongoose');
var express = require('express');
var api = require('./routes/api.js');
var app = express();

mongoose.connect('mongodb://localhost/db-dev', myOptionsObj);

// application configuration
require('./config/express')(app);

// routing set up
app.use('/api', api);

var server = app.listen(3000, function () {
  var host = server.address().address;
  var port = server.address().port;

  console.log('App listening at http://%s:%s', host, port);
});

e ( /server/routes/api.js ):

var express = require('express');
var boxController = require('../modules/box/controller');
var thingController = require('../modules/thing/controller');
var router = express.Router();

// API routing
router.get('/boxes', boxController.getAll);
// etc.

module.exports = router;

Note extra

Ho provato a disconnettere la servervariabile nel file /tests/server-test.js prima di eseguire i test:

...
var server = require('../server/app'); // my express app
...

console.log('server: ', server);
...

e il risultato di questo è un oggetto vuoto: server: {}.

Risposte:


228

Non esporti nulla nel modulo dell'app. Prova ad aggiungerlo al tuo file app.js:

module.exports = server

31
l'unico problema che ho con questo è che il codice dell'applicazione non dovrebbe mai essere modificato per adattarsi a un test.
dman

@chovy hai risolto questo problema? Di seguito ho un'alternativa che ha funzionato per me.
Skyguard

1
il potenziale problema è avere a che fare con un server che contiene servizi asincroni, perché chai-http non conosce la logica di questo e verrà eseguito direttamente anche prima che il tuo server sia completamente avviato
atom2ueki

1
@Nabuska in questo caso il server è probabilmente già impostato con app.listen (...)
Garr Godfrey

Per i tuoi test non dovresti usare app.listen () che avvia un server reale, usa invece http.createServer ()
Whyhankee

42

È importante esportare l' http.Serveroggetto restituito da app.listen(3000)invece della sola funzione app, altrimenti otterrai TypeError: app.address is not a function.

Esempio:

index.js

const koa = require('koa');
const app = new koa();
module.exports = app.listen(3000);

index.spec.js

const request = require('supertest');
const app = require('./index.js');

describe('User Registration', () => {
  const agent = request.agent(app);

  it('should ...', () => {

2
Dovresti includere nella tua risposta perché è "importante esportare l' http.Serveroggetto".
GrumpyCrouton

@GrumpyCrouton Ho aggiunto l'errore che otterrai altrimenti
Kim Kern

1
Grazie Kim. Ti ho dato un +1 perché penso che migliori la tua risposta. In futuro, dovresti spiegare perché . Immagina che qualcuno guardi questa domanda e abbia solo una conoscenza di base, una risposta ben ponderata e spiegata gli insegnerà molto di più.
GrumpyCrouton

Per i tuoi test non dovresti usare app.listen () che avvia un server reale, usa invece http.createServer ()
Whyhankee

28

Questo può anche aiutare e soddisfa il punto @dman di modificare il codice dell'applicazione per adattarlo a un test.

fai la tua richiesta al localhost e porta come necessario chai.request('http://localhost:5000')

invece di

chai.request(server)

questo ha risolto lo stesso messaggio di errore che avevo usando Koa JS (v2) e ava js.


3
Dopo aver fatto questo errore non viene visualizzato, ma in get data viene visualizzato null e il codice di stato 200 è ok.
Anita Mehta

4

Le risposte sopra affrontano correttamente il problema: supertestvuole un http.Serverlavoro su cui lavorare. Tuttavia, la chiamata app.listen()per ottenere un server avvierà anche un server in ascolto, questa è una cattiva pratica e non è necessaria.

Puoi aggirare questo usando http.createServer():

import * as http from 'http';
import * as supertest from 'supertest';
import * as test from 'tape';
import * as Koa from 'koa';

const app = new Koa();

# add some routes here

const apptest = supertest(http.createServer(app.callback()));

test('GET /healthcheck', (t) => {
    apptest.get('/healthcheck')
    .expect(200)
    .expect(res => {
      t.equal(res.text, 'Ok');
    })
    .end(t.end.bind(t));
});

1

Abbiamo avuto lo stesso problema quando abbiamo eseguito mocha utilizzando ts-node nel nostro progetto serverless node + typescript.

Il nostro tsconfig.json aveva "sourceMap": true. I file .js e .js.map così generati causano alcuni divertenti problemi di transpiling (simili a questo). Quando eseguiamo mocha runner usando ts-node. Quindi, imposterò il flag sourceMap su false e cancellerò tutti i file .js e .js.map nella nostra directory src. Quindi il problema è sparito.

Se hai già generato file nella tua cartella src, i comandi seguenti sarebbero davvero utili.

trova src -name " .js.map" -exec rm {} \; trova src -name " .js" -exec rm {} \;


0

Per ogni evenienza, se qualcuno usa Hapijs, il problema persiste, perché non usa Express.js, quindi la funzione address () non esiste.

TypeError: app.address is not a function
      at serverAddress (node_modules/chai-http/lib/request.js:282:18)

La soluzione per farlo funzionare

// this makes the server to start up
let server = require('../../server')

// pass this instead of server to avoid error
const API = 'http://localhost:3000'

describe('/GET token ', () => {
    it('JWT token', (done) => {
       chai.request(API)
         .get('/api/token?....')
         .end((err, res) => {
          res.should.have.status(200)
          res.body.should.be.a('object')
          res.body.should.have.property('token')
          done()
      })
    })
  })
Utilizzando il nostro sito, riconosci di aver letto e compreso le nostre Informativa sui cookie e Informativa sulla privacy.
Licensed under cc by-sa 3.0 with attribution required.