Correções no cucumber

[gabrielpoca]

Vou usar esta feature como exemplo

Feature: Payments

In order to manage the payments methods
I want to select the payment methods available

Stakeholder: Owner of an online store
Priority: High

Scenario: Modify the payment method
    Given I should be owner of the store and authenticated
    And I am logged as store owner
    And I am in the store
    And I am on page setup payment methods
    When I select the payments methods that i want
    And I Press the submit button
    Then I should see the sucess message

Scenario: Select one or more payments methods
    Given I sould be a new owner of the new store
    And I am in the criation store page
    When I select one or more the methods i want
    And I press submit button
    Then I should see the sucess message

Scenario: Do not select any payment method
    Given I sould be a new owner of the new store
    And I am in the criation store page
    When I not select any payment method
    And I press submit button
    Then I should see the error message

Identificar as pessoas

Acho que não há necessidade de escrever palavras a mais do que necessário. Por exemplo, nestes casos Stakeholder: Owner of an online store, Given I should be owner of the store and authenticated, Given I sould be a new owner of the new store, não há necessidade de dizer store, basta por exemplo dizer Given I am a owner uma vez que owner é o nome da role das pessoas que tem lojas (digo eu que é, mas se não é foi definido um). Ou então pode mudar-se para Given I am a store owner.

Passos repetidos

    Given I should be owner of the store and authenticated
    And I am logged as store owner

Na primeira dizem que está authenticated e na segunda logged. Basicamente a mesma coisa. E outra coisa é coerencia com o resto das features que tem authenticated acho e o primeiro está em dois passos.

Outra coisa sobre estes dois é que está um should no primeiro e um am no segundo. Basicamente o inglês da primeira está errado.

Inglês

Then I should see the error message devia ser Then I should see a error message. And I am in the criation store page, criation não existe.

Outros

Estes são menos importantes e se forem coerentes ao longo do resto das features não estão mal.

And I press submit button. Isto pode ficar só And I press submit.

[naps62]

Then I should see a error message

Grammar nazi:

Then I should see an error message

[azevedo]

porque optaram por retirar o "As a ..." logo no inicio da feature? pelos vistos substituiram essa linha por isto "Stakeholder: Owner of an online store Priority: High"

[azevedo]

nao é suposto as pessoas que trabalham nisto responderem? os teus minions estão a trabalhar bem gabriel :)

[gabrielpoca]

As correções foram feitas ;)

[azevedo]

ok, mas era uma pergunta que gostava de ter visto respondida :)

[gabrielpoca]

Por acaso não te sei responder, não me lembra de ter falado de fazer assim.