diff options
author | Robert Speicher <rspeicher@gmail.com> | 2016-04-19 16:00:45 -0400 |
---|---|---|
committer | Robert Speicher <rspeicher@gmail.com> | 2016-04-19 16:00:45 -0400 |
commit | a6ba8647f919cca5f37f663502186d8b6b7642ec (patch) | |
tree | 87cff6fa53a9796b00872db3d8f1bf25f50b4147 /spec/features/users_spec.rb | |
parent | 6a19467c415487ae786df12b04f62647132986ac (diff) | |
download | gitlab-ce-a6ba8647f919cca5f37f663502186d8b6b7642ec.tar.gz |
Improve uniqueness of field names on the signup formrs-unique-signup-fields
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/15075
Diffstat (limited to 'spec/features/users_spec.rb')
-rw-r--r-- | spec/features/users_spec.rb | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/spec/features/users_spec.rb b/spec/features/users_spec.rb index c1248162031..cf116040394 100644 --- a/spec/features/users_spec.rb +++ b/spec/features/users_spec.rb @@ -5,10 +5,10 @@ feature 'Users', feature: true do scenario 'GET /users/sign_in creates a new user account' do visit new_user_session_path - fill_in 'user_name', with: 'Name Surname' - fill_in 'user_username', with: 'Great' - fill_in 'user_email', with: 'name@mail.com' - fill_in 'user_password_sign_up', with: 'password1234' + fill_in 'new_user_name', with: 'Name Surname' + fill_in 'new_user_username', with: 'Great' + fill_in 'new_user_email', with: 'name@mail.com' + fill_in 'new_user_password', with: 'password1234' expect { click_button 'Sign up' }.to change { User.count }.by(1) end @@ -31,10 +31,10 @@ feature 'Users', feature: true do scenario 'Should show one error if email is already taken' do visit new_user_session_path - fill_in 'user_name', with: 'Another user name' - fill_in 'user_username', with: 'anotheruser' - fill_in 'user_email', with: user.email - fill_in 'user_password_sign_up', with: '12341234' + fill_in 'new_user_name', with: 'Another user name' + fill_in 'new_user_username', with: 'anotheruser' + fill_in 'new_user_email', with: user.email + fill_in 'new_user_password', with: '12341234' expect { click_button 'Sign up' }.to change { User.count }.by(0) expect(page).to have_text('Email has already been taken') expect(number_of_errors_on_page(page)).to be(1), 'errors on page:\n #{errors_on_page page}' |