Fix all tests to make them compatible with unified core setup

deepjyoti30-st · deepjyoti30-st · commit 49cc51af8552 · 2025-04-07T14:39:33.000+05:30
diff --git a/test/end-to-end/generalerror.test.js b/test/end-to-end/generalerror.test.js
@@ -17,8 +17,6 @@
  * Imports
  */
 import assert from "assert";
-import puppeteer from "puppeteer";
-import fetch from "isomorphic-fetch";
 import {
     clearBrowserCookiesWithoutAffectingConsole,
     screenshotOnFailure,
diff --git a/test/end-to-end/mfa.chooserscreen.test.js b/test/end-to-end/mfa.chooserscreen.test.js
@@ -74,9 +74,15 @@ describe("SuperTokens SignIn w/ MFA", function () {
         backendHook("before");
         const coreUrl = await setupCoreApp();
         appConfig.coreUrl = coreUrl;
+        appConfig.accountLinkingConfig = {
+            enabled: true,
+            shouldAutoLink: {
+                shouldAutomaticallyLink: true,
+                shouldRequireVerification: false,
+            },
+        };
         await setupST(appConfig);
 
-        await setAccountLinkingConfig(true, true, false);
         browser = await setupBrowser();
     });
 
diff --git a/test/end-to-end/mfa.default_reqs.test.js b/test/end-to-end/mfa.default_reqs.test.js
@@ -66,7 +66,6 @@ describe("SuperTokens SignIn w/ MFA", function () {
             this.skip();
         }
 
-        await setAccountLinkingConfig(true, true, false);
         await backendHook("before");
         browser = await setupBrowser();
     });
diff --git a/test/end-to-end/multitenancy.dynamic_login_methods.test.js b/test/end-to-end/multitenancy.dynamic_login_methods.test.js
@@ -40,6 +40,7 @@ import {
     setupCoreApp,
     setupST,
     backendHook,
+    setupBrowser,
 } from "../helpers";
 import {
     TEST_CLIENT_BASE_URL,
diff --git a/test/end-to-end/refresherrors.test.js b/test/end-to-end/refresherrors.test.js
@@ -31,10 +31,7 @@ describe("Refresh errors", function () {
         let page;
 
         before(async function () {
-            browser = await puppeteer.launch({
-                args: ["--no-sandbox", "--disable-setuid-sandbox"],
-                headless: true,
-            });
+            browser = await setupBrowser();
         });
 
         after(async function () {
diff --git a/test/end-to-end/routing.test.js b/test/end-to-end/routing.test.js
@@ -29,6 +29,7 @@ import {
     clearBrowserCookiesWithoutAffectingConsole,
     screenshotOnFailure,
     waitForUrl,
+    setupBrowser,
 } from "../helpers";
 /*
  * Tests.
@@ -37,10 +38,7 @@ describe("SuperTokens Routing in Test App", function () {
     let browser, page;
 
     before(async function () {
-        browser = await puppeteer.launch({
-            args: ["--no-sandbox", "--disable-setuid-sandbox"],
-            headless: true,
-        });
+        browser = await setupBrowser();
         page = await browser.newPage();
     });
 
diff --git a/test/end-to-end/signup.test.js b/test/end-to-end/signup.test.js
@@ -61,10 +61,7 @@ describe("SuperTokens SignUp", function () {
         const coreUrl = await setupCoreApp();
         await setupST({ coreUrl });
 
-        browser = await puppeteer.launch({
-            args: ["--no-sandbox", "--disable-setuid-sandbox"],
-            headless: true,
-        });
+        browser = await setupBrowser();
         page = await browser.newPage();
         page.on("console", (consoleObj) => {
             const log = consoleObj.text();
diff --git a/test/end-to-end/thirdparty.test.js b/test/end-to-end/thirdparty.test.js
@@ -32,6 +32,7 @@ import {
     backendHook,
     setupCoreApp,
     setupST,
+    setupBrowser,
 } from "../helpers";
 import { TEST_CLIENT_BASE_URL, SIGN_IN_UP_API, GET_AUTH_URL_API } from "../constants";
 
@@ -56,10 +57,7 @@ export function getThirdPartyTestCases({ authRecipe, rid, signInUpPageLoadLogs,
         const coreUrl = await setupCoreApp();
         await setupST({ coreUrl });
 
-        browser = await puppeteer.launch({
-            args: ["--no-sandbox", "--disable-setuid-sandbox"],
-            headless: true,
-        });
+        browser = await setupBrowser();
         page = await browser.newPage();
         page.on("console", (consoleObj) => {
             const log = consoleObj.text();
diff --git a/test/end-to-end/thirdpartyemailpassword.test.js b/test/end-to-end/thirdpartyemailpassword.test.js
@@ -76,10 +76,7 @@ describe("SuperTokens Third Party Email Password", function () {
         appConfig.coreUrl = coreUrl;
         await setupST(appConfig);
 
-        browser = await puppeteer.launch({
-            args: ["--no-sandbox", "--disable-setuid-sandbox"],
-            headless: true,
-        });
+        browser = await setupBrowser();
         page = await browser.newPage();
         page.on("console", (consoleObj) => {
             const log = consoleObj.text();
diff --git a/test/end-to-end/thirdpartypasswordless.test.js b/test/end-to-end/thirdpartypasswordless.test.js
@@ -72,10 +72,7 @@ describe("SuperTokens Third Party Passwordless", function () {
             appConfig.coreUrl = coreUrl;
             await setupST(appConfig);
 
-            browser = await puppeteer.launch({
-                args: ["--no-sandbox", "--disable-setuid-sandbox"],
-                headless: true,
-            });
+            browser = await setupBrowser();
             page = await browser.newPage();
             page.on("console", (consoleObj) => {
                 const log = consoleObj.text();
diff --git a/test/end-to-end/userContext.test.js b/test/end-to-end/userContext.test.js
@@ -28,6 +28,7 @@ import {
     setupCoreApp,
     setupST,
     screenshotOnFailure,
+    setupBrowser,
 } from "../helpers";
 import { TEST_CLIENT_BASE_URL, RESET_PASSWORD_TOKEN_API, RESET_PASSWORD_API, SIGN_IN_UP_API } from "../constants";
 import assert from "assert";
@@ -42,10 +43,7 @@ describe("SuperTokens userContext with UI components test", function () {
         const coreUrl = await setupCoreApp();
         await setupST({ coreUrl });
 
-        browser = await puppeteer.launch({
-            args: ["--no-sandbox", "--disable-setuid-sandbox"],
-            headless: true,
-        });
+        browser = await setupBrowser();
         page = await browser.newPage();
         page.on("console", (consoleObj) => {
             const log = consoleObj.text();
diff --git a/test/end-to-end/userroles.test.js b/test/end-to-end/userroles.test.js
@@ -52,10 +52,7 @@ describe("User Roles in the frontend", function () {
             const coreUrl = await setupCoreApp();
             await setupST({ coreUrl });
 
-            browser = await puppeteer.launch({
-                args: ["--no-sandbox", "--disable-setuid-sandbox"],
-                headless: true,
-            });
+            browser = await setupBrowser();
             page = await browser.newPage();
             await Promise.all([
                 page.goto(`${TEST_CLIENT_BASE_URL}/auth?authRecipe=emailpassword`),
diff --git a/test/helpers.js b/test/helpers.js
@@ -28,6 +28,7 @@ import {
 import path from "path";
 import assert from "assert";
 import mkdirp from "mkdirp";
+import Puppeteer from "puppeteer";
 import addContext from "mochawesome/addContext";
 
 const SESSION_STORAGE_STATE_KEY = "supertokens-oauth-state";
@@ -831,6 +832,108 @@ export async function screenshotOnFailure(ctx, browser) {
     }
 }
 
+export async function setupBrowser() {
+    const browser = await Puppeteer.launch({
+        args: [
+            "--no-sandbox",
+            "--disable-setuid-sandbox",
+            "--disable-web-security",
+            "--host-resolver-rules=MAP example.com 127.0.0.1, MAP *.example.com 127.0.0.1",
+            // Open DevTools automatically if in non-headless mode and DEV_TOOLS is enabled
+            ...(process.env.HEADLESS === "false" && process.env.DEV_TOOLS === "true"
+                ? ["--auto-open-devtools-for-tabs"]
+                : []),
+        ],
+        headless: process.env.HEADLESS !== "false",
+        devtools: process.env.HEADLESS === "false" && process.env.DEV_TOOLS === "true",
+    });
+    browser.logs = [];
+    function addLog(str) {
+        if (process.env.BROWSER_LOGS !== undefined) {
+            console.log(str);
+        }
+        browser.logs.push(str);
+    }
+    const origNewPage = browser.newPage.bind(browser);
+    browser.newPage = async () => {
+        const page = await origNewPage();
+        page.on("console", (msg) => {
+            if (msg.text().startsWith("com.supertokens")) {
+                addLog(msg.text());
+            } else {
+                addLog(
+                    `browserlog.console ${JSON.stringify({
+                        t: new Date().toISOString(),
+                        message: msg.text(),
+                        pageurl: page.url(),
+                    })}`
+                );
+            }
+        });
+
+        page.on("request", (req) => {
+            addLog(
+                `browserlog.network ${JSON.stringify({
+                    t: new Date().toISOString(),
+                    message: `Requested: ${req.method()} ${req.url()} (${req.postData()})`,
+                    pageurl: page.url(),
+                })}`
+            );
+        });
+        page.on("requestfinished", async (req) => {
+            if (req.method() === "OPTION") {
+                return;
+            }
+            const resp = await req.response();
+            let respText;
+            try {
+                respText = req.url().startsWith(TEST_APPLICATION_SERVER_BASE_URL)
+                    ? await resp.text()
+                    : "response omitted";
+            } catch (e) {
+                respText = "response loading failed " + e.message;
+            }
+            addLog(
+                `browserlog.network ${JSON.stringify({
+                    t: new Date().toISOString(),
+                    message: `Request done: ${req.method()} ${req.url()}: ${resp.status()} ${respText}`,
+                    pageurl: page.url(),
+                })}`
+            );
+        });
+        page.on("requestfailed", async (req) => {
+            if (req.method() === "OPTION") {
+                return;
+            }
+            const resp = await req.response();
+            let respText;
+            try {
+                respText = req.url().startsWith(TEST_APPLICATION_SERVER_BASE_URL)
+                    ? await resp.text()
+                    : "response omitted";
+            } catch (e) {
+                respText = "response loading failed " + e.message;
+            }
+            addLog(
+                `browserlog.network ${JSON.stringify({
+                    t: new Date().toISOString(),
+                    message: `Request failed: ${req.method()} ${req.url()}: ${resp.status()} ${respText}`,
+                    pageurl: page.url(),
+                })}`
+            );
+        });
+        return page;
+    };
+    browser.on("disconnected", async () => {
+        if (process.env.MOCHA_FILE !== undefined) {
+            const reportFile = path.parse(process.env.MOCHA_FILE);
+            const logFile = path.join(reportFile.dir, "logs", "browser.log");
+            await appendFile(logFile, browser.logs.join("\n"));
+        }
+    });
+    return browser;
+}
+
 export async function getPasswordlessDevice(loginAttemptInfo) {
     const deviceResp = await fetch(
         `${TEST_APPLICATION_SERVER_BASE_URL}/test/getDevice?preAuthSessionId=${encodeURIComponent(
diff --git a/test/server/index.js b/test/server/index.js
@@ -577,7 +577,6 @@ function convertToRecipeUserIdIfAvailable(id) {
     return id;
 }
 
-
 let urlencodedParser = bodyParser.urlencoded({ limit: "20mb", extended: true, parameterLimit: 20000 });
 let jsonParser = bodyParser.json({ limit: "20mb" });
 

Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,6 @@ describe("SuperTokens SignIn w/ MFA", function () {`
`66`	`66`	`this.skip();`
`67`	`67`	`}`
`68`	`68`
`69`		`- await setAccountLinkingConfig(true, true, false);`
`70`	`69`	`await backendHook("before");`
`71`	`70`	`browser = await setupBrowser();`
`72`	`71`	`});`