Skip to content

Commit ab67e59

Browse files
Fix issues with failing test that was skipped
1 parent a186233 commit ab67e59

File tree

1 file changed

+15
-7
lines changed

1 file changed

+15
-7
lines changed

test/end-to-end/emailverification.test.js

Lines changed: 15 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -627,19 +627,27 @@ describe("Email Verification", () => {
627627
});
628628
});
629629
describe("Email Verified", function () {
630-
// TODO: Does it make sense to test this since this is already tested in the "Verify Email with token screen" section?
631-
it.skip("Should redirect to onSuccessfulRedirect when email is already verified", async function () {
630+
it("Should redirect to onSuccessfulRedirect when email is already verified", async function () {
632631
await Promise.all([
633632
page.goto(`${TEST_CLIENT_BASE_URL}/auth?mode=REQUIRED`),
634633
page.waitForNavigation({ waitUntil: "networkidle0" }),
635634
]);
636-
const { fieldValues } = getDefaultSignUpFieldValues();
637-
await setInputValues(
638-
page,
639-
fieldValues.filter((fv) => ["email", "password"].includes(fv.name))
640-
);
635+
636+
await toggleSignInSignUp(page);
637+
const { fieldValues, postValues } = getDefaultSignUpFieldValues();
638+
await signUp(page, fieldValues, postValues, "emailpassword");
639+
640+
const latestURLWithToken_ = await getLatestURLWithToken();
641+
await Promise.all([
642+
page.waitForNavigation({ waitUntil: "networkidle0" }),
643+
page.goto(latestURLWithToken_),
644+
]);
645+
const title_ = await getTextByDataSupertokens(page, "headerTitle");
646+
assert.deepStrictEqual(title_, "Email verification successful!");
647+
641648
await Promise.all([submitForm(page), page.waitForNavigation({ waitUntil: "networkidle0" })]);
642649
await page.waitForSelector(".sessionInfo-user-id");
650+
643651
await Promise.all([
644652
page.goto(`${TEST_CLIENT_BASE_URL}/auth/verify-email`),
645653
page.waitForNavigation({ waitUntil: "networkidle0" }),

0 commit comments

Comments
 (0)