Commit 57c0d9ec authored by Robert Schilling's avatar Robert Schilling

Merge pull request #7870 from mr-vinn/fix-notify-spec

Update expected value in notify_spec
parents 5b5912f9 97e2f8a0
...@@ -44,7 +44,9 @@ describe Notify do ...@@ -44,7 +44,9 @@ describe Notify do
let(:example_site_path) { root_path } let(:example_site_path) { root_path }
let(:new_user) { create(:user, email: 'newguy@example.com', created_by_id: 1) } let(:new_user) { create(:user, email: 'newguy@example.com', created_by_id: 1) }
subject { Notify.new_user_email(new_user.id, new_user.password, 'kETLwRaayvigPq_x3SNM') } token = 'kETLwRaayvigPq_x3SNM'
subject { Notify.new_user_email(new_user.id, new_user.password, token) }
it_behaves_like 'an email sent from GitLab' it_behaves_like 'an email sent from GitLab'
...@@ -65,7 +67,10 @@ describe Notify do ...@@ -65,7 +67,10 @@ describe Notify do
end end
it 'includes a link for user to set password' do it 'includes a link for user to set password' do
should have_body_text 'http://localhost/users/password/edit?reset_password_token=kETLwRaayvigPq_x3SNM' params = "reset_password_token=#{token}"
should have_body_text(
%r{http://localhost(:\d+)?/users/password/edit\?#{params}}
)
end end
it 'includes a link to the site' do it 'includes a link to the site' do
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment