Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: move reset password email #2281

Merged
merged 58 commits into from
Apr 11, 2024
Merged
Changes from 1 commit
Commits
Show all changes
58 commits
Select commit Hold shift + click to select a range
6226ef9
fix: move verification emails to Dotdigital
evemartin Mar 28, 2024
d3b94b6
Merge branch 'master' into move-verification-emails-to-dotdigital
evemartin Mar 28, 2024
48cb482
fix: fix dotdigital email tests
evemartin Apr 2, 2024
dc6b581
fix: fix parent email test
evemartin Apr 2, 2024
84a5001
fix: add github secret binding
evemartin Apr 2, 2024
3eaaecb
fix: add chrome to dev container
evemartin Apr 2, 2024
55d6029
fix: attempt to fix under 13 test
evemartin Apr 2, 2024
33d57f4
fix: attempt to fix patching
evemartin Apr 2, 2024
f81868a
fix: continue work on updating tests
evemartin Apr 4, 2024
261cf33
fix: continue to update tests
evemartin Apr 4, 2024
0aa6d29
fix: fix package names
evemartin Apr 4, 2024
d4e95fe
fix: fix test assert
evemartin Apr 4, 2024
3ba97c2
fix: fix assert
evemartin Apr 4, 2024
9651a2a
fix: update tests
evemartin Apr 4, 2024
fcdf86d
fix: rearrange decorators
evemartin Apr 4, 2024
537f333
fix: continue work on tests
evemartin Apr 4, 2024
8add1ec
fix: fix typo
evemartin Apr 4, 2024
ea6239a
fix: testing something out
evemartin Apr 4, 2024
688013b
fix: testing something out
evemartin Apr 4, 2024
3ef54f4
fix: undo chrome changes
evemartin Apr 4, 2024
6dbe672
fix: adding additional mocks
evemartin Apr 4, 2024
f3a1cae
fix: debug tests
evemartin Apr 4, 2024
9ee8e3e
fix: debug tests
evemartin Apr 4, 2024
3305851
fix: debug tests
evemartin Apr 4, 2024
6502383
fix: debug tests
evemartin Apr 4, 2024
bd4942c
fix: debug tests
evemartin Apr 4, 2024
296d1cf
fix: debug tests
evemartin Apr 4, 2024
4327fee
fix: debug tests
evemartin Apr 4, 2024
dd8b086
fix: debug tests
evemartin Apr 4, 2024
a1f0674
fix: debug tests
evemartin Apr 4, 2024
1dc615a
fix: add forgotten import
evemartin Apr 4, 2024
643dfd2
fix: update dotdigital auth
evemartin Apr 9, 2024
5ab9fe3
fix: fix environmental variable
evemartin Apr 9, 2024
1425477
fix: address PR comments
evemartin Apr 9, 2024
566f528
fix: address PR comments
evemartin Apr 9, 2024
6abe884
fix: debug tests
evemartin Apr 9, 2024
e829610
fix: debug tests
evemartin Apr 9, 2024
1bc639f
fix: tidy code, replace verification reminder emails
evemartin Apr 9, 2024
4d3ac93
fix: fix verification reminder email tests
evemartin Apr 9, 2024
cc1e7c1
fix: debug tests
evemartin Apr 9, 2024
4e9cc43
fix: debug tests
evemartin Apr 9, 2024
dac0b40
fix: fix patch mock
evemartin Apr 10, 2024
3e472f3
fix: move reset password email
evemartin Apr 10, 2024
b8947e9
fix: tidy code
evemartin Apr 10, 2024
7d4f4fd
fix: remove old code
evemartin Apr 10, 2024
7d5a464
fix: debug tests
evemartin Apr 10, 2024
f9a1f52
fix: debug tests
evemartin Apr 10, 2024
4f4181c
fix: debug tests
evemartin Apr 10, 2024
23a53b8
Update ci.yml
evemartin Apr 10, 2024
68b4fa0
Update ci.yml
evemartin Apr 10, 2024
e68775d
Update ci.yml
evemartin Apr 10, 2024
1571ed1
Update ci.yml
evemartin Apr 10, 2024
3dc98b3
Merge branch 'master' into move-reset-password-email
evemartin Apr 10, 2024
7c7bfa1
fix: tidy code
evemartin Apr 11, 2024
c33ca26
fix: tidy code
evemartin Apr 11, 2024
6407c50
Merge branch 'master' into move-reset-password-email
faucomte97 Apr 11, 2024
b92837d
fix: address PR comments
evemartin Apr 11, 2024
8ab1243
Merge branch 'master' into move-reset-password-email
faucomte97 Apr 11, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: adding additional mocks
evemartin committed Apr 4, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
commit 6dbe672d32f08d08094dd2cba48abed92dad0568
3 changes: 2 additions & 1 deletion cfl_common/common/tests/utils/email.py
Original file line number Diff line number Diff line change
@@ -15,7 +15,8 @@ def follow_verify_email_link_to_teacher_dashboard(page, email):


def follow_verify_email_link_to_login(page, email, user_type):
_follow_verify_email_link(page, email)
# _follow_verify_email_link(page, email)
page.browser.get(email)

if user_type == "teacher":
return go_to_teacher_login_page(page.browser)
9 changes: 6 additions & 3 deletions cfl_common/common/tests/utils/student.py
Original file line number Diff line number Diff line change
@@ -5,6 +5,7 @@
from common.helpers.generators import generate_login_id
from common.models import Class, Student
from django.core import mail
from unittest.mock import Mock, patch

from . import email

@@ -118,16 +119,18 @@ def signup_duplicate_independent_student_fail(page, duplicate_email=None):
return page, name, username, email_address, password


def create_independent_student(page):
@patch("common.helpers.emails.send_dotdigital_email")
def create_independent_student(page, mock_send_dotdigital_email):
page = page.go_to_signup_page()

name, username, email_address, password = generate_independent_student_details()
page = page.independent_student_signup(name, email_address, password=password, confirm_password=password)

page = page.return_to_home_page()

page = email.follow_verify_email_link_to_login(page, mail.outbox[0], "independent")
mail.outbox = []
verification_url = mock_send_dotdigital_email.call_args.kwargs["personalization_values"]["VERIFICATION_LINK"]

page = email.follow_verify_email_link_to_login(page, verification_url, "independent")

return page, name, username, email_address, password

2 changes: 0 additions & 2 deletions portal/tests/test_independent_student.py
Original file line number Diff line number Diff line change
@@ -266,8 +266,6 @@ def test_login_not_verified(self):

assert page.has_login_failed("independent_student_login_form", INVALID_LOGIN_MESSAGE)

verify_email(page)

assert is_email_verified_message_showing(self.selenium)

page = page.independent_student_login(username, password)
1 change: 0 additions & 1 deletion portal/tests/test_teacher.py
Original file line number Diff line number Diff line change
@@ -398,7 +398,6 @@ def test_signup_email_verification(self, mock_send_dotdigital_email: Mock):

# Get verification link from function call
verification_url = mock_send_dotdigital_email.call_args.kwargs["personalization_values"]["VERIFICATION_LINK"]
print(verification_url)

# Verify the email properly
verification_response = c.get(verification_url)