Make sure resend email verification redirect to the correct page

openstax · Dec 2, 2015 · 45dfe3e · 45dfe3e
1 parent 52aa1c6
commit 45dfe3e
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/app/controllers/contact_infos_controller.rb b/app/controllers/contact_infos_controller.rb
@@ -38,8 +38,8 @@ def toggle_is_searchable
   def resend_confirmation
     handle_with(ContactInfosResendConfirmation,
                 complete: lambda {
-                  path = current_user.is_email_pending? ? verification_sent_path :
-                                                          profile_path(active_tab: :email)
+                  path = current_user.is_temp? ? verification_sent_path :
+                                                 profile_path(active_tab: :email)
                   redirect_to path,
                     notice: "A verification message has been sent to \"#{
                               @handler_result.outputs[:contact_info].value}\"" })

diff --git a/spec/features/user_manages_emails_spec.rb b/spec/features/user_manages_emails_spec.rb
@@ -18,6 +18,7 @@
     scenario 'success', js: true do
       fill_in 'Email address', with: 'user@mysite.com'
       click_button 'Add Email address'
+      expect(page).to have_content('Change Your Password')
       expect(page).to have_content(
         'A verification message has been sent to "user@mysite.com"')
       expect(page).to have_content('user@mysite.com')

diff --git a/spec/features/user_signs_up_spec.rb b/spec/features/user_signs_up_spec.rb
@@ -169,6 +169,7 @@
 
     click_on 'Resend Verification'
 
+    expect(page).to have_content('Already verified your email address?')
     expect(page).to have_content('A verification message has been sent to "testuser@example.com"')
   end