fix

strr-host-pm-web/README.md

@@ -87,4 +87,4 @@ To use this auth state in other tests:
 - Tests using BCEID login that rely on a OTP (test/prod environments) should use the saved auth user state. Generating a new OTP for each test on login can cause the tests to fail.
 - A maximum of 4 workers seems to be the sweet spot for tests to pass without colliding with each other.
 - Setting to 1 worker will fully disable running tests in parallel.
-- Using the Playwright extension/testing tab do not execute the global setup and save the auth state. You must run `pnpm:e2e` to at least create the auth files before running tests with the extension.
+- Using the Playwright extension/testing tab does not execute the global setup and save the auth state. You must run `pnpm:e2e` to at least create the auth files before running tests with the extension.

strr-host-pm-web/tests/e2e/test-utils/auth-setup.ts

@@ -52,23 +52,4 @@ export async function authSetup (
   await page.waitForURL(baseUrl + '**')
   await page.context().storageState({ path: `tests/e2e/.auth/${storagePath}.json` })
   await browser.close()
-
-  // await page.goto(baseUrl + 'en-CA/auth/login')
-
-  // if (loginMethod === LoginSource.BCSC) {
-  //   await page.getByRole('button', { name: 'Continue with BC Services Card' }).click()
-  //   await page.getByLabel('Log in with Test with').click()
-  //   await page.getByLabel('Email or username').fill(username)
-  //   await page.getByLabel('Password').fill(password)
-  //   await page.getByRole('button', { name: 'Continue' }).click()
-  // } else if (loginMethod === LoginSource.BCEID) {
-  //   await page.getByRole('button', { name: 'Continue with BCeID' }).click()
-  //   await page.locator('#user').fill(username)
-  //   await page.getByLabel('Password').fill(password)
-  //   await page.getByRole('button', { name: 'Continue' }).click()
-  // }
-
-  // await page.waitForURL(baseUrl + '**')
-  // await page.context().storageState({ path: `tests/e2e/.auth/${storagePath}.json` })
-  // await browser.close()
 }