-
-
Notifications
You must be signed in to change notification settings - Fork 251
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FIX] payment_cielo tests #1064
Conversation
a183ca0
to
31aa76a
Compare
/ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
valeu @DiegoParadeda pessoal, esse tipo de problema mostra bem as limitaçoes do VCRPy: testa de forma bem completa mas fica chato parta manter. Nisso minha opiniao é de reservar isso a modulos super comun que todos PSCs dominam no repo e pros outros modulos tentar pegar mais leve com a biblioteca mock. Eu por examplo ja expressei minha opiniao sobre a questao de testar as NFSe com VCRPy, sendo que nem todo PSC vai ter chave para testar e atualizar os dados do VCRPy e eu imagino que esse problema com o payment_cielo é so um gostinho das dores de cabeça que teremos com os modulos de NFSe testados com VCRPy. |
Congratulations, your PR was merged at 0269dfc. Thanks a lot for contributing to OCA. ❤️ |
brincadeira né, a merda do PR #820 que a gente tinha tirado veio junto nesse PR... |
pessoal, esse PR so tinha um commit que parecia certo. Porem a coisa louca foi que o ocabot parece que jogou depois os commits do PR#820 que nao apromavos e tiramos por achar que nao tava no razoavel ainda. |
Se puder tirar eu já faço o rebase aqui e vejo se os commits continuam nas PRs de ajuste de NFSe que tenho em aberto |
sim @marcelsavegnago . Ficamos enrolado num go live com outra regressão que te pergunto quem que sera que fez. Mas vamos tirar de novo daqui pouco. Da ultima vez eh provavel que tiramos os commits da branch sem detonar os commits e que por algum motivo isso voltou, eh o que suspeitamos pelo menos. |
This PR fixes Travis test errors in module payment_cielo