pagarme / superbowleto

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

[BPROC-531] Removes async boleto on caixa provider #378

Closed JeanCigoli closed 2 years ago

JeanCigoli commented 2 years ago

Descrição

Verificamos que em alguns casos de exceção na resposta da BoletoAPI para a Superbowleto o fluxo async é acionado, mesmo com a loja tendo o registro async desabilitado. Com isso acabamos mandando o boleto para o fluxo de registro em background e exibindo uma reposta inconsistente para o cliente.

O intuito dessa alteração é remover todo fluxo async do provedor caixa, então ao gerar uma exceção será retornado o status refused.

A alteração consistiu em retirar de dentro da função de registro do boleto da caixa a geração de uma exceção no código, e retornar uma mensagem padrão com o status correto, evitando assim que o boleto caia no fluxo de pending_registration.

image

Alterações existentes no PR:

Impacto de deploy

Nenhum impacto esperado na aplicação.

Testes

Foram criados testes de integração que cobrem as alterações realizadas nesse PR, validando os retornos da função de registro do boleto. Além disso todos os demais testes continuaram passando corretamente:

image

Caso necessário, como realizar o rollback deste PR?

Revert do commit do PR.

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: