darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-09-08 03:48:25 -04:00
Code Issues Packages Projects Releases Wiki Activity

Better flash message for users with active accounts who click on

Browse Source 
verification link again.
This commit is contained in:
Adam Shimali
2016-03-29 13:21:51 +01:00
parent 352f169fb1
commit db24a633c1
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/main/views/verify.py
View File

@@ -61,7 +61,7 @@ def verify_email(token):
abort(404) abort(404)
if user.is_active(): if user.is_active():
flash("You have already verified your email address.") flash("That verification link has expired.")
return redirect(url_for('main.sign_in')) return redirect(url_for('main.sign_in'))
session['user_details'] = {"email": user.email_address, "id": user.id} session['user_details'] = {"email": user.email_address, "id": user.id}

2
tests/app/main/views/test_verify.py
View File

@@ -146,4 +146,4 @@ def test_verify_email_redirects_to_sign_in_if_user_active(app_,
page = BeautifulSoup(response.data.decode('utf-8'), 'html.parser') page = BeautifulSoup(response.data.decode('utf-8'), 'html.parser')
assert page.h1.text == 'Sign in' assert page.h1.text == 'Sign in'
flash_banner = page.find('div', class_='banner-dangerous').string.strip() flash_banner = page.find('div', class_='banner-dangerous').string.strip()
assert flash_banner == "You have already verified your email address." assert flash_banner == "That verification link has expired."