summaryrefslogtreecommitdiff
path: root/spec/features/users_spec.rb
blob: 1124b42721f91f870136e2959242d9bc7abd0030 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
require 'spec_helper'

feature 'Users', js: true do
  let(:user) { create(:user, username: 'user1', name: 'User 1', email: 'user1@gitlab.com') }

  scenario 'GET /users/sign_in creates a new user account' do
    visit new_user_session_path
    click_link 'Register'
    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_email_confirmation',  with: 'name@mail.com'
    fill_in 'new_user_password',            with: 'password1234'
    expect { click_button 'Register' }.to change { User.count }.by(1)
  end

  scenario 'Successful user signin invalidates password reset token' do
    expect(user.reset_password_token).to be_nil

    visit new_user_password_path
    fill_in 'user_email', with: user.email
    click_button 'Reset password'

    user.reload
    expect(user.reset_password_token).not_to be_nil

    gitlab_sign_in(user)
    expect(current_path).to eq root_path

    user.reload
    expect(user.reset_password_token).to be_nil
  end

  scenario 'Should show one error if email is already taken' do
    visit new_user_session_path
    click_link 'Register'
    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_email_confirmation',  with: user.email
    fill_in 'new_user_password',            with: '12341234'
    expect { click_button 'Register' }.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}'
  end

  describe 'redirect alias routes' do
    before do
      expect(user).to be_persisted
    end

    scenario '/u/user1 redirects to user page' do
      visit '/u/user1'

      expect(current_path).to eq user_path(user)
      expect(page).to have_text(user.name)
    end

    scenario '/u/user1/groups redirects to user groups page' do
      visit '/u/user1/groups'

      expect(current_path).to eq user_groups_path(user)
    end

    scenario '/u/user1/projects redirects to user projects page' do
      visit '/u/user1/projects'

      expect(current_path).to eq user_projects_path(user)
    end
  end

  feature 'username validation' do
    let(:loading_icon) { '.fa.fa-spinner' }
    let(:username_input) { 'new_user_username' }

    before(:each) do
      visit new_user_session_path
      click_link 'Register'
    end

    scenario 'doesn\'t show an error border if the username is available' do
      fill_in username_input, with: 'new-user'
      wait_for_requests
      expect(find('.username')).not_to have_css '.gl-field-error-outline'
    end

    scenario 'does not show an error border if the username contains dots (.)' do
      fill_in username_input, with: 'new.user.username'
      wait_for_requests
      expect(find('.username')).not_to have_css '.gl-field-error-outline'
    end

    scenario 'shows an error border if the username already exists' do
      fill_in username_input, with: user.username
      wait_for_requests
      expect(find('.username')).to have_css '.gl-field-error-outline'
    end

    scenario 'shows an  error border if the username contains special characters' do
      fill_in username_input, with: 'new$user!username'
      wait_for_requests
      expect(find('.username')).to have_css '.gl-field-error-outline'
    end
  end

  def errors_on_page(page)
    page.find('#error_explanation').find('ul').all('li').map { |item| item.text }.join("\n")
  end

  def number_of_errors_on_page(page)
    page.find('#error_explanation').find('ul').all('li').count
  end
end