pagarme / superbowleto

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

Configuration: add logic to find boleto config #340

Closed alinemarjorie closed 4 years ago

alinemarjorie commented 4 years ago

Este PR visa tirar algumas responsabilidades do boleto-processor do gateway e inserir uma nova lógica ao superbowleto.

Segue as etapas que foram adicionadas ao chegar uma requisição:

Pontos de atenção:

peace :peace_symbol:

relates to https://mundipagg.atlassian.net/browse/PGHOST-25

jeanaugustoos commented 4 years ago

Atualmente, bradesco. Depois, shopfacil

vlopes commented 4 years ago

Queria fazer uma ressalva sobre a delecao de itens nos testes (eu to sem embasamento no momento entao e so a minha opniao :smile: ). Eu nao acho legal a gente deletar as coisas apos cada teste porque isso pode acabar causando uma falsa sensacao de que os testes passaram quando na realidade eles podem quebrar. Pensando em um exemplo rapido:

Eu vi que alguns testes ja faziam isso antes das alteracoes desse PR, mas acho que e um ponto importante pra gente ficar de olho :eyes: .

alinemarjorie commented 4 years ago

Queria fazer uma ressalva sobre a delecao de itens nos testes (eu to sem embasamento no momento entao e so a minha opniao ).

Boa, Vini! Eu optei pela limpeza do dados por ser uma boa prática, pois um teste nao deveria impactar em outro, dado isso é interessante resetarmos o banco para garantirmos que nao teremos side effects. (minhas palavras aqui, mentira são as do Jean kkkk) Mas além disso, eu estou apagando depois de cada teste, por companyId, para não deletar a seed, pq qdo estava só apagando tudo depois do arquivo, deletava a seed com o default e quebrava os outros testes.