search

pagarme / superbowleto

:football: A microservice to issue, register and manage boletos
MIT License
5 stars 0 forks source link

errors: fix prototype of InternalServerError #241

Closed evaporei closed 7 years ago

evaporei commented 7 years ago

InternalServerError was being set with the wrong prototype. Before it was DatabaseError, now it is InternalServerError.

Description

Write a brief and explicative description of your pull request.

Your checklist for this pull request

:rotating_light: Please review this items for a good pull request. :four_leaf_clover:

  1. I've read the project's Contributing Guidelines
  2. My commits are well written and follow pagarme/git-style-guide
  3. My changes are well covered by tests and logs
  4. I've updated the project docs (if needed)
  5. I fell safe about this implementation
  6. I feel comfortable with the code I wrote, and I'm not ashamed to show it to my friends

In a good pull request, everything above is true :relaxed:

wilkmaia commented 7 years ago

Você só esqueceu de adicionar aqueles testes aqui :)

evaporei commented 7 years ago

@wilkmaia Sim, vou subir os testes dos erros depois. Desta vez decidi que iria fazer uma mudança por vez. Achei que poderia separar em uma correção (branch fix-internal-server-error) e uma adição de testes unitários (branch unit-tests-errors).

wilkmaia commented 7 years ago

@otaviopace você pode ter mais de uma mudança lógica num PR. A questão de uma mudança lógica é por commit :)

Pode mandar os dois commits no mesmo PR sem problema.

Para isso, basta acrescentar o commit a esta branch e dar push que ele vem pro PR.

evaporei commented 7 years ago

@wilkmaia Beleza, vou subir mais tarde os testes nessa mesma branch. Havia separado pois achei que tinha sentido. :)

evaporei commented 7 years ago

Preciso realizar mais algum procedimento ou correção para que o merge possa ser feito? Desculpe se estou fazendo algo errado no pull request.

grvcoelho commented 7 years ago

Oi @otaviopace, está :100:

Obrigado!