fix failing cucumber tests (issue #106)
- Scenario: Distribute to commits not linked to a GitHub account
User#recipient_label method was prefering to show identifier instead of email when nick was undefined but the test was expecting an email address - not sure which was the intended behavior but showing email to the user rather than an obtuse ID seems more reasonable
is it possible this could be a private email address ? then perhaps to modify the test instead to expexct an ID - the original implementation would still show the email address however in cases where no nick nor ID is defined if such a state is possible
several tests that filled fields on the Distribution edit form were were passing and failing intermittently - this was most likely due to the 'auto-complete' feature
notably scenarios:
- Send distribution to multiple users
- Remove a distribution line
- Create distribution line without an amount
adding a one second delay to these steps has made all tests passing
- Given /^I add the GitHub user "(.*?)" to the recipients$/
- Given /^I add the user with email "(.*?)" through his identifier to the recipients$/
- When /^I add the commit "(.*?)" to the recipients$/
the additions to the last commit were intended to be included in the previous commit
these steps were not the source of any test failures but if the autocomplete feature was the cause of the failures addressed by this pull request, then the potential exists for these to be problematic - as these failures were intermittent and therefore insideous its best to guard against them
the sibling 'email' field does not implement the autocomplete feature so i assume no wait is needed for its related step