Closed wilbert closed 10 years ago
felipero
commented
10 years ago Olá Wilbert...
Os testes não estão passando. O build deu erro. Inicialmente acho bom deixar sem a gem de debbuger também. Cada dev debuga de um jeito diferente e pode resultar em conflitos.
Se resolver isso eu faço o merge. :)
wilbert
commented
10 years ago Opa, tudo bem Felipe,
Cara, não sei porque os testes não passaram, mas removi a gem debugger, alterei os testes para serem compatíveis com Ruby 1.8.7 (adicionei => no lugar do : ) e aparentemente rodou:
wilbert
commented
10 years ago Por outro lado, alguns outros builds não funcionam, sem nenhum alteração, como se fosse uma inconstância na API da Cielo.
Assim vira bagunça!
felipero
commented
10 years ago Fiz o merge e vejo como ficam os testes. Valeu!
felipero
commented
10 years ago Fiz o merge. Agora o que precisa mesmo é mockar o retorno da api da cielo pra que os testes não fiquem dependentes da API e para que fiquem mais rápidos. Tratar o erro da api da Cielo quando retornar algum erro na chamada.
Eu estou sem tempo nesses dias, se você puder fazer, é uma boa experiência, senão, quando eu puder, faço isso.
wilbert
commented
10 years ago Deixe comigo, vou fazer o mockup e outra melhorias.
felipero
commented
10 years ago Massa.
wilbert
commented
10 years ago Ei cara, adicionei o mock com o Webmock, no lugar do Fakeweb, mas tá dando erros somente com o ruby 1.8.7, o que acha?
https://travis-ci.org/wilbert/cielo/builds/14722064
Bom, resolvi alterar pra o webmock porque ele dá mais possibilidade e possibilita reaproveitar mais o código de teste, mas teria que ficar claro que os testes não funcionariam no ruby 1.8.7, se é que isso é importante.
Se você não se importar, com a falta de compatibilidade eu faço outro pull request.
O uso de token, além de permitir maior segurança dos dados, ainda permite a cobrança recorrente aproveitando essa nova implementação da API 1.2.0 da Cielo.