diff options
author | Jacopo <beschi.jacopo@gmail.com> | 2018-08-07 18:53:33 +0300 |
---|---|---|
committer | Jacopo <beschi.jacopo@gmail.com> | 2018-08-07 18:53:33 +0300 |
commit | ae5480783abce7f4e340a749bded666c2ef7ff29 (patch) | |
tree | 5eb80680032d3a6e81aa3110af4749fb5369a4a6 /spec/features/admin | |
parent | d0f03ed9b34e626141c5a55073d282cce6ea3d10 (diff) |
Backport of EE changes in https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/6804
Diffstat (limited to 'spec/features/admin')
-rw-r--r-- | spec/features/admin/admin_users_spec.rb | 15 |
1 files changed, 14 insertions, 1 deletions
diff --git a/spec/features/admin/admin_users_spec.rb b/spec/features/admin/admin_users_spec.rb index 6c194c9a646..1db6c75b85b 100644 --- a/spec/features/admin/admin_users_spec.rb +++ b/spec/features/admin/admin_users_spec.rb @@ -68,10 +68,12 @@ describe "Admin::Users" do end describe "GET /admin/users/new" do + let(:user_username) { 'bang' } + before do visit new_admin_user_path fill_in "user_name", with: "Big Bang" - fill_in "user_username", with: "bang" + fill_in "user_username", with: user_username fill_in "user_email", with: "bigbang@mail.com" end @@ -112,6 +114,17 @@ describe "Admin::Users" do expect(email.text_part.body).to have_content(user.email) expect(email.text_part.body).to have_content('password') end + + context 'username contains spaces' do + let(:user_username) { 'Bing bang' } + + it "doesn't create the user and shows an error message" do + expect { click_button "Create user" }.to change {User.count}.by(0) + + expect(page).to have_content('The form contains the following error') + expect(page).to have_content('Username can contain only letters, digits') + end + end end describe "GET /admin/users/:id" do |