Test api Nodejs Rest with mocha and chai. Post getting pending

Asked

Viewed 80 times

0

Hello, I am learning to do api’s Rest tests and I came across a problem using chai and mocha, I was following this example in question: https://medium.com/@rafaelvicio/testando-api-Rest-com-mocha-e-chai-bf3764ac2797 Of course changing things to the api I had already written. The problem is that the POST method is getting pending always, according to something pending documentation does not mean it failed. But I would like to pass this test. My route is returning a json of what was created, so I didn’t quite understand why the test didn’t pass, someone can give me a help on that?

Link to the repository if it helps: https://github.com/ThiagoBussola/api-games

Code of the POST route

router.post("/game", async (req, res) => {
    const title = req.body.title
    const year = req.body.year
    const price = req.body.price
    try {
       const gameCreated = await Game.create({
            title: title,
            year: year,
            price: price
        })
        res.json(gameCreated)
        
    } catch (err) {
        console.log(err)
        res.sendStatus(500)
    }
})

Test code

describe("Teste POST game", () => {
    it("deve criar um novo jogo"), (done) => {

        let game = {
            title: "Jogo criado pelo mocha",
            year: 2020,
            price: 178
        }
        chai.request('localhost:3033')
            .post('/game')
            .send(game)
            .end((err, res) => {
                res.should.have.status(200)
                
                done()
            })
    }
})
node.js rest testing mocha chai

  • Try changing the http status of the test from 200 to 201, as 200 indicates successful request, while 201 is Created.

  • In repostan only your json, Voce should not put res.status(201).json(gameCreated)? 201 to indicate that a resource has been created. Are you using CORS in your application? What are the allowedHeaders allowed? There in your .end((err, res) => {...}, before the res, it would be nice to put a if(err) done(err) see what went wrong.

  • Yes the application is using Cors, as for allowedHeaders, I do not know answer. Anyway, I put the res.status(201), I had done it before, it keeps coming as pending. And also put if(err)... without results, I will put the link to the repository in the question, maybe it helps

1 answer

0


Good that you posted the link to your complete code, it was easy to realize the error.

Simple error, you closed a parenthesis before the time in the following code:

describe("Teste POST game", () => {
   it("deve criar um novo jogo"), (done) => { // veja o `)` antes da vírgula
    ...
   }
})

And look at the code of the previous test:

describe("Teste GET games", () => {
  it("deve receber os jogos da api", (done) => {
    ...
  });
});

Remove this parenthesis and place it at the end, this way:

describe("Teste POST game", () => {
  it("deve criar um novo jogo", (done) => {

    let game = {
      title: "Jogo criado pelo mocha",
      year: 2020,
      price: 178
    };
    chai.request('localhost:3033')
      .post('/game')
      .type('json')
      .send(game)
      .end((err, res) => {
        if (err) {
          console.log(err);
          done(err);
        }
        res.should.have.status(201);

        done();
      });
  }); // <= AQUI NO FINAL
});

When I ran your code, all the tests passed. Then run your tests and see if you can solve ;)

Browser other questions tagged node.js rest testing mocha chai

You are not signed in. Login or sign up in order to post.