I renamed registration_email_pattern.feature to user_registration.feature and updated feature and scenario descriptions a bit.

http://drupalcode.org/project/doobie.git/commit/848c9d8

I think this file also needs to test that:
1. Username validation works as in - it can't contain inappropriate characters and it can't be already existing username
2. Email validation works, as in - it should contain @ etc.

Comments

kssundar’s picture

kssundar commented
Status: Active » Needs review
Issue tags: +sprint 16

Updated and committed here - http://drupalcode.org/project/doobie.git/commit/bb5fd88

tvn’s picture

tvn commented

I think for email verification we also need to test that email is not already taken by other user.
Apart from that looks good.

kssundar’s picture

kssundar commented

This is already covered in the feature "Scenario: Create a different user with the same Email Id"

tvn’s picture

tvn commented

Right, looks finished then. Thanks!

kssundar’s picture

kssundar commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
Assigned: Unassigned » kssundar
Status: Needs review » Patch (to be ported)
Issue tags: +sprint 18

Need 7.x update

kssundar’s picture

kssundar commented
Status: Patch (to be ported) » Needs review

Updated 7.x - http://drupalcode.org/project/doobie.git/commit/0de189f

eliza411’s picture

eliza411 commented
Status: Needs review » Reviewed & tested by the community

Looks as ready as possible until D7 Drupal.org is ready for feedback.

eliza411’s picture

eliza411 commented
Status: Reviewed & tested by the community » Closed (duplicate)

Closing in favor of #2045095: Refactor user_registration.feature

eliza411’s picture

eliza411 commented
Issue summary: View changes

.