waldemarnt/building-testable-apis-with-nodejs-code

Repositório oficial do código utilizado no livro Construindo APIs testáveis com Node.js 📗

📖 Código oficial do livro Construindo APIs testáveis com Node.js

Publicado gratuitamente no Leanpub: Construindo APIs testáveis com Node.js

Pull Requests e contribuições são bem vindas.

Acesso ao livro escrito no GitHub

💥O Livro foi finalizado oficialmente em 2020 🚀🚀🚀

building-testable-apis-with-nodejs-code's Issues

TypeError: this.AuthService is not a constructor

Dada a classe:

class UsersController {
    constructor(User, AuthService) {
        this.User = User;
        this.AuthService = AuthService;
    }

    async authenticate(req, res) {
        const authService = new this.AuthService(this.User);
        const user = await authService.authenticate(req.body);
        if (!user) {
            return res.sendStatus(401);
        }
        const token = this.AuthService.generateToken({
            name: user.name,
            email: user.email,
            password: user.password,
            role: user.role
        });
        return res.send({ token });
    }
}

export default UsersController;

os testes unitários apresentam o erro:

Controller: Users
       login
         should return 401 when the user can not be found:
     TypeError: this.AuthService is not a constructor
      at UsersController.authenticate (src/controllers/users.js:69:29)
      at Context.<anonymous> (test/unit/controllers/users_spec.js:298:35)
      at processImmediate (node:internal/timers:464:21)

para ver o codigo completo atual pode verificar em:
https://github.com/gfkauer/to-do-api

[SUGESTAO] Adicionar alguma logica de negocio

Oi Waldemar, achei muito boa sua iniciativa de disponibilizar o projeto pra galera.

Queria sugerir algo:

• adicionar algumas logicas de negócios diferente para cada item, para mostrar qual a melhor forma de tratar os business exceptions. Por exemplo:
  • se o produto não tem estoque retorna um erro tratado
  • se o produto está faltando alguma informação retorna um erro
• Retornar error codes que o Frontend possa tratar com i18n

{
    "success": false,
    "errors": [
    {
      "code": "those.skus.are.not.found",
      "message": "There are no stock available for those SKUS",
      "data": ["D4324-00192-1923", "D4324-00192-4922", "D4324-00192-9876"]
    },
    {
      "code": "coupon.code.expired",
      "message": "This coupon is not available",
      "data": "JS10"
    }
  ]
}

Meus parabéns pelo projeto é uma ótima ajuda para a comunidade open source.