Skip to content

Commit dfb58bf

Browse files
committed
Activate user on email verification
1 parent a2226b0 commit dfb58bf

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

tests/unit/accounts/test_views.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -699,7 +699,7 @@ def test_reset_password_password_date_changed(self, pyramid_request):
699699
class TestVerifyEmail:
700700

701701
def test_verify_email(self, db_request, user_service, token_service):
702-
user = UserFactory()
702+
user = UserFactory(is_active=False)
703703
email = EmailFactory(user=user, verified=False)
704704
db_request.user = user
705705
db_request.GET.update({"token": "RANDOM_KEY"})
@@ -721,6 +721,7 @@ def test_verify_email(self, db_request, user_service, token_service):
721721

722722
db_request.db.flush()
723723
assert email.verified
724+
assert user.is_active
724725
assert isinstance(result, HTTPSeeOther)
725726
assert result.headers["Location"] == "/"
726727
assert db_request.route_path.calls == [pretend.call('manage.profile')]

warehouse/accounts/views.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -378,6 +378,7 @@ def _error(message):
378378
return _error("Email already verified")
379379

380380
email.verified = True
381+
request.user.is_active = True
381382

382383
request.session.flash(
383384
f'Email address {email.email} verified.', queue='success'

0 commit comments

Comments
 (0)