search

vindi / vindi-magento

Módulo de cobrança recorrente para Magento
https://www.vindi.com.br
GNU General Public License v3.0
12 stars 14 forks source link

Ajusta tratativa para número de parcelas em cartões de crédito salvos #122

Closed st3llaris closed 5 years ago

st3llaris commented 5 years ago

Motivação

O PR #119 tratava o parcelamento para cartões de crédito, na primeira compra. Esse PR trata agora o parcelamento para cartões salvos no checkout.

Solução Proposta

Tratativa para setar como "1" quando o número de parcelas não for informado no checkout com o cartão de crédito salvo.

augustohub commented 5 years ago

Perguntei pro Vitor se o termo "Cest" no final do nome da classe dos testes era algum typo pra "Test"(não é), e ele me passou essa documentação: https://codeception.com/docs/07-AdvancedUsage

Achei legal compartilhar aqui para quem quiser entender melhor como esses testes estão funcionando.

ghost commented 5 years ago

Dúvida: 1 - Eu ví que existe um nível de abstração de algumas interações de fluxo, como:

public function addSubscriptionToCart($I)
    {
        $I->amOnPage('/vindi-subscription.html');
        $I->click('Add to Cart');
    }

Está em nível de Helpers essas funções, será que não seria interessante pensarmos em utilizar um design pattern como Page Objects para organizar e abstrair ?

wagner commented 5 years ago

Me removi pois só quis deixar um comentário. Não vou revisar.

laerte-guimaraes commented 5 years ago

será que não seria interessante pensarmos em utilizar um design pattern como Page Objects para organizar e abstrair ?

Acredito que para o momento atual isso tornaria o fluxo mais complexo :thinking: Tentei utilizar o padrão do codeception, e abstrair alguns métodos que são utilizados constantemente. Sei que temos mais coisas que podemos abstrair, porém podemos pensar nisso com mais calma quando formos implementar cenários maiores (ainda não cobertos)