Add some fixes for userContext and test server

deepjyoti30-st · deepjyoti30-st · commit 407a60117df9 · 2025-03-25T15:01:07.000+05:30
diff --git a/.mocharc.yml b/.mocharc.yml
@@ -8,4 +8,4 @@ require:
     - test/test.mocha.env
 reporter: mocha-multi-reporters
 reporter-option:
-  - configFile=mocha-multi-reporters.json
+    - configFile=mocha-multi-reporters.json
diff --git a/mocha-multi-reporters.json b/mocha-multi-reporters.json
@@ -6,4 +6,4 @@
     "mochawesomeReporterOptions": {
         "reportDir": "test_report/mochawesome"
     }
-}
+}
diff --git a/test/end-to-end/emailverification.test.js b/test/end-to-end/emailverification.test.js
@@ -618,7 +618,7 @@ describe("Email Verification", () => {
                 const title = await getTextByDataSupertokens(page, "headerTitle");
                 assert.deepStrictEqual(title, "Email verification successful!");
                 await submitForm(page);
-                 // Make sure to wait for status to update. Flakes in headless.
+                // Make sure to wait for status to update. Flakes in headless.
                 await new Promise((r) => setTimeout(r, 500));
                 const logoutButton = await getLogoutButton(page);
                 await Promise.all([logoutButton.click(), page.waitForNavigation({ waitUntil: "networkidle0" })]);
diff --git a/test/end-to-end/mfa.factorscreen.otp.test.js b/test/end-to-end/mfa.factorscreen.otp.test.js
@@ -33,12 +33,7 @@ import {
     backendHook,
     setupST,
 } from "../helpers";
-import {
-    CONSUME_CODE_API,
-    CREATE_CODE_API,
-    MFA_INFO_API,
-    SOMETHING_WENT_WRONG_ERROR,
-} from "../constants";
+import { CONSUME_CODE_API, CREATE_CODE_API, MFA_INFO_API, SOMETHING_WENT_WRONG_ERROR } from "../constants";
 
 import { TEST_CLIENT_BASE_URL } from "../constants";
 import { getTestPhoneNumber } from "../exampleTestHelpers";
diff --git a/test/end-to-end/mfa.factorscreen.totp.test.js b/test/end-to-end/mfa.factorscreen.totp.test.js
@@ -29,7 +29,7 @@ import {
     setupBrowser,
     backendHook,
     setupCoreApp,
-    setupST
+    setupST,
 } from "../helpers";
 import { CREATE_TOTP_DEVICE_API, MFA_INFO_API } from "../constants";
 
@@ -115,7 +115,7 @@ describe("SuperTokens SignIn w/ MFA", function () {
                     ...appConfig,
                     mfaInfo: {
                         allowedToSetup: ["totp"],
-                    }
+                    },
                 });
                 page = await browser.newPage();
 
@@ -146,7 +146,7 @@ describe("SuperTokens SignIn w/ MFA", function () {
                         requirements: [],
                         alreadySetup: [factorId],
                         allowedToSetup: [factorId],
-                    }
+                    },
                 });
 
                 await tryEmailPasswordSignIn(page, email);
@@ -172,7 +172,7 @@ describe("SuperTokens SignIn w/ MFA", function () {
                         requirements: [],
                         alreadySetup: [factorId],
                         allowedToSetup: [],
-                    }
+                    },
                 });
 
                 await tryEmailPasswordSignIn(page, email);
@@ -192,7 +192,7 @@ describe("SuperTokens SignIn w/ MFA", function () {
                         requirements: [factorId],
                         alreadySetup: [],
                         allowedToSetup: [],
-                    }
+                    },
                 });
 
                 await tryEmailPasswordSignIn(page, email);
@@ -211,7 +211,7 @@ describe("SuperTokens SignIn w/ MFA", function () {
                         requirements: [factorId],
                         alreadySetup: [factorId],
                         allowedToSetup: [],
-                    }
+                    },
                 });
 
                 await tryEmailPasswordSignIn(page, email);
@@ -245,7 +245,7 @@ describe("SuperTokens SignIn w/ MFA", function () {
                         requirements: [factorId],
                         alreadySetup: [factorId],
                         allowedToSetup: [],
-                    }
+                    },
                 });
 
                 await tryEmailPasswordSignIn(page, email);
@@ -262,7 +262,7 @@ describe("SuperTokens SignIn w/ MFA", function () {
                         requirements: [factorId],
                         alreadySetup: [factorId],
                         allowedToSetup: [],
-                    }
+                    },
                 });
 
                 await tryEmailPasswordSignIn(page, email);
@@ -304,7 +304,7 @@ describe("SuperTokens SignIn w/ MFA", function () {
                         requirements: [factorId],
                         alreadySetup: [],
                         allowedToSetup: [factorId],
-                    }
+                    },
                 });
 
                 await page.setRequestInterception(true);
@@ -341,7 +341,7 @@ describe("SuperTokens SignIn w/ MFA", function () {
                         requirements: [],
                         alreadySetup: [factorId],
                         allowedToSetup: [],
-                    }
+                    },
                 });
 
                 await tryEmailPasswordSignIn(page, email);
@@ -360,7 +360,7 @@ describe("SuperTokens SignIn w/ MFA", function () {
                         requirements: [],
                         alreadySetup: [factorId],
                         allowedToSetup: [factorId],
-                    }
+                    },
                 });
 
                 await tryEmailPasswordSignIn(page, email);
@@ -381,7 +381,7 @@ describe("SuperTokens SignIn w/ MFA", function () {
                         requirements: [],
                         alreadySetup: [],
                         allowedToSetup: [factorId],
-                    }
+                    },
                 });
 
                 await tryEmailPasswordSignIn(page, email);
@@ -402,7 +402,7 @@ describe("SuperTokens SignIn w/ MFA", function () {
                         requirements: [],
                         alreadySetup: [factorId],
                         allowedToSetup: [],
-                    }
+                    },
                 });
 
                 await tryEmailPasswordSignIn(page, email);
@@ -423,7 +423,7 @@ describe("SuperTokens SignIn w/ MFA", function () {
                         requirements: [{ oneOf: [factorId, "otp-email"] }],
                         alreadySetup: ["otp-email"],
                         allowedToSetup: [factorId],
-                    }
+                    },
                 });
 
                 await tryEmailPasswordSignIn(page, email);
@@ -445,7 +445,7 @@ describe("SuperTokens SignIn w/ MFA", function () {
                         requirements: [{ oneOf: [factorId, "otp-email"] }],
                         alreadySetup: [factorId, "otp-email"],
                         allowedToSetup: [],
-                    }
+                    },
                 });
 
                 await tryEmailPasswordSignIn(page, email);
@@ -466,7 +466,7 @@ describe("SuperTokens SignIn w/ MFA", function () {
                         requirements: [factorId],
                         alreadySetup: [],
                         allowedToSetup: [factorId],
-                    }
+                    },
                 });
 
                 await tryEmailPasswordSignIn(page, email);
@@ -488,7 +488,7 @@ describe("SuperTokens SignIn w/ MFA", function () {
                         requirements: [factorId],
                         alreadySetup: [factorId],
                         allowedToSetup: [],
-                    }
+                    },
                 });
 
                 await tryEmailPasswordSignIn(page, email);
diff --git a/test/end-to-end/mfa.requirement_handling.test.js b/test/end-to-end/mfa.requirement_handling.test.js
@@ -27,7 +27,7 @@ import {
     setupBrowser,
     backendHook,
     setupCoreApp,
-    setupST
+    setupST,
 } from "../helpers";
 import { TEST_CLIENT_BASE_URL } from "../constants";
 import { getTestPhoneNumber } from "../exampleTestHelpers";
@@ -135,7 +135,6 @@ describe("SuperTokens SignIn w/ MFA", function () {
 
         describe("multistep requirement list", () => {
             it("multistep requirements should happen in order (allOfInAnyOrder -> oneOf)", async () => {
-
                 appConfig.mfaInfo = {
                     requirements: [{ allOfInAnyOrder: ["otp-phone", "totp"] }, { oneOf: ["otp-email"] }],
                 };
diff --git a/test/end-to-end/mfa.signin.test.js b/test/end-to-end/mfa.signin.test.js
@@ -89,7 +89,7 @@ describe("SuperTokens SignIn w/ MFA", function () {
                 consoleLogs.push(log);
             }
         });
-        
+
         consoleLogs = await clearBrowserCookiesWithoutAffectingConsole(page, []);
 
         await page.evaluate(() => window.localStorage.removeItem("supertokens-passwordless-loginAttemptInfo"));
@@ -117,7 +117,7 @@ describe("SuperTokens SignIn w/ MFA", function () {
             ...appConfig,
             mfaInfo: {
                 requirements: ["otp-email"],
-            }
+            },
         });
 
         await tryEmailPasswordSignUp(page, email);
diff --git a/test/end-to-end/multitenancy.tenant_interactions.test.js b/test/end-to-end/multitenancy.tenant_interactions.test.js
@@ -78,7 +78,7 @@ describe("SuperTokens Multitenancy tenant interactions", function () {
 
     beforeEach(async function () {
         await backendHook("beforeEach");
-        
+
         const coreUrl = await setupCoreApp();
         appConfig.coreUrl = coreUrl;
         await setupST(appConfig);
diff --git a/test/end-to-end/oauth2provider.test.js b/test/end-to-end/oauth2provider.test.js
@@ -43,11 +43,7 @@ import {
     setupCoreApp,
     setupST,
 } from "../helpers";
-import {
-    TEST_CLIENT_BASE_URL,
-    TEST_APPLICATION_SERVER_BASE_URL,
-    SOMETHING_WENT_WRONG_ERROR,
-} from "../constants";
+import { TEST_CLIENT_BASE_URL, TEST_APPLICATION_SERVER_BASE_URL, SOMETHING_WENT_WRONG_ERROR } from "../constants";
 
 // We do no thave to use a separate domain for the oauth2 client, since the way we are testing
 // the lib doesn't interact with the supertokens session handling.
diff --git a/test/end-to-end/passwordless.test_gen.js b/test/end-to-end/passwordless.test_gen.js
@@ -351,7 +351,6 @@ export function getPasswordlessTestCases({ authRecipe, logId, generalErrorRecipe
             });
         });
 
-
         describe(`UserInputCode`, function () {
             this.timeout(60000);
 
diff --git a/test/end-to-end/resetpasswordusingtoken.test.js b/test/end-to-end/resetpasswordusingtoken.test.js
@@ -18,12 +18,7 @@
  */
 
 import assert from "assert";
-import {
-    EMAIL_EXISTS_API,
-    RESET_PASSWORD_API,
-    RESET_PASSWORD_TOKEN_API,
-    TEST_CLIENT_BASE_URL,
-} from "../constants";
+import { EMAIL_EXISTS_API, RESET_PASSWORD_API, RESET_PASSWORD_TOKEN_API, TEST_CLIENT_BASE_URL } from "../constants";
 import {
     clearBrowserCookiesWithoutAffectingConsole,
     getInputAdornmentsError,
diff --git a/test/end-to-end/signin.test.js b/test/end-to-end/signin.test.js
@@ -76,7 +76,7 @@ describe("SuperTokens SignIn", function () {
         await setupST({ coreUrl });
         browser = await setupBrowser();
     });
-    
+
     beforeEach(async function () {
         backendHook("beforeEach");
         page = await browser.newPage();
diff --git a/test/end-to-end/signup.test.js b/test/end-to-end/signup.test.js
@@ -46,12 +46,7 @@ import {
     setupCoreApp,
     setupST,
 } from "../helpers";
-import {
-    EMAIL_EXISTS_API,
-    SIGN_UP_API,
-    SOMETHING_WENT_WRONG_ERROR,
-    TEST_CLIENT_BASE_URL,
-} from "../constants";
+import { EMAIL_EXISTS_API, SIGN_UP_API, SOMETHING_WENT_WRONG_ERROR, TEST_CLIENT_BASE_URL } from "../constants";
 
 /*
  * Tests.
diff --git a/test/end-to-end/thirdpartypasswordless.test.js b/test/end-to-end/thirdpartypasswordless.test.js
@@ -37,7 +37,7 @@ import {
     backendHook,
     setupCoreApp,
     setupST,
-    screenshotOnFailure
+    screenshotOnFailure,
 } from "../helpers";
 import { TEST_CLIENT_BASE_URL, SIGN_IN_UP_API, GET_AUTH_URL_API } from "../constants";
 
diff --git a/test/end-to-end/thirdpartypasswordless.tp.test.js b/test/end-to-end/thirdpartypasswordless.tp.test.js
@@ -17,9 +17,7 @@
  * Imports
  */
 
-import {
-    getFeatureFlags,
-} from "../helpers";
+import { getFeatureFlags } from "../helpers";
 import { getThirdPartyTestCases } from "./thirdparty.test";
 
 /*
diff --git a/test/end-to-end/userContext.test.js b/test/end-to-end/userContext.test.js
@@ -25,20 +25,12 @@ import {
     waitForUrl,
     setupBrowser,
     loginWithAuth0,
-    loginWithAuth0,
-    waitForUrl,
-    setupBrowser,
     backendHook,
     setupCoreApp,
     setupST,
     screenshotOnFailure,
 } from "../helpers";
-import {
-    TEST_CLIENT_BASE_URL,
-    RESET_PASSWORD_TOKEN_API,
-    RESET_PASSWORD_API,
-    SIGN_IN_UP_API,
-} from "../constants";
+import { TEST_CLIENT_BASE_URL, RESET_PASSWORD_TOKEN_API, RESET_PASSWORD_API, SIGN_IN_UP_API } from "../constants";
 import assert from "assert";
 
 describe("SuperTokens userContext with UI components test", function () {
diff --git a/test/end-to-end/userroles.test.js b/test/end-to-end/userroles.test.js
@@ -86,7 +86,7 @@ describe("User Roles in the frontend", function () {
             await page?.close();
             await backendHook("afterEach");
         });
-    
+
         after(async function () {
             await browser?.close();
             await backendHook("after");
diff --git a/test/helpers.js b/test/helpers.js
@@ -1221,10 +1221,7 @@ export async function backendHook(hookType) {
     );
 }
 
-export async function setupCoreApp({
-    appId,
-    coreConfig,
-} = {}) {
+export async function setupCoreApp({ appId, coreConfig } = {}) {
     const response = await fetch(`${TEST_SERVER_BASE_URL}/test/setup/app`, {
         method: "POST",
         headers: new Headers([["content-type", "application/json"]]),
diff --git a/test/server/index.js b/test/server/index.js
@@ -195,29 +195,11 @@ initST();
     await addLicense();
 })();
 
-app.use(
-    cors({
-        origin: websiteDomain,
-        allowedHeaders: ["content-type", ...SuperTokens.getAllCORSHeaders()],
-        methods: ["GET", "PUT", "POST", "DELETE"],
-        credentials: true,
-    })
-);
-
-app.use(middleware());
-
-app.get("/ping", async (req, res) => {
-    res.send("success");
-});
-
 /**
  * Create a core application and initialize ST with the required config
  * @returns URL for the new core application
  */
-async function setupApp({
-    appId,
-    coreConfig,
-} = {}) {
+async function setupApp({ appId, coreConfig } = {}) {
     const coreAppUrl = await setupCoreApplication({ appId, coreConfig });
     console.log("Connection URI: " + coreAppUrl);
 

Original file line number	Diff line number	Diff line change
`@@ -6,4 +6,4 @@`
`6`	`6`	`"mochawesomeReporterOptions": {`
`7`	`7`	`"reportDir": "test_report/mochawesome"`
`8`	`8`	`}`
`9`		`-}`
	`9`	`+}`