fix: use default option for goto() method (networkidle -> load) (#36)

Author: vOrigins

src/packages/shared/pages/BasePage.ts

@@ -13,7 +13,7 @@ export class BasePage {
   async goto(url: string, options?: GotoOptions): Promise<void> {
     const _url = new URL(url, BASE_ORIGIN).toString()
     await report.step(`Go to "${url}"`, async () => {
-      await this.page.goto(_url, { waitUntil: 'networkidle', ...options })
+      await this.page.goto(_url, options)
     })
   }
 

src/packages/shared/pages/LoginPage.ts

@@ -29,6 +29,6 @@ export class LoginPage extends BasePage {
     await this.loginTxtFld.fill(credentials.email)
     await this.passwordTxtFld.fill(credentials.password)
     await this.signInBtn.click()
-    await this.page.waitForLoadState('networkidle')
+    await this.page.waitForLoadState()
   }
 }

src/services/storage-state/ss.helpers.ts

@@ -30,7 +30,7 @@ export async function saveSsFileByLocalViaBrowser(credentials: Credentials): Pro
   await page.goto(`${BASE_ORIGIN}/login`)
   const loginPage = new LoginPage(page)
   await loginPage.signIn(credentials)
-  await page.waitForLoadState('networkidle')
+  await page.waitForLoadState()
   await page.context().storageState({ path: `${SS_PATH_PREFIX}${BASE_URL.hostname}-${credentials.email}.json` })
   await browser.close()
 
@@ -46,7 +46,7 @@ export async function saveSsFileBySsoViaBrowser(credentials: Credentials): Promi
     headless: !!process.env.CI,
   })
   const page = await newPage(browser, credentials)
-  await page.goto(BASE_ORIGIN, { waitUntil: 'networkidle', timeout: PAGE_LOADING })
+  await page.goto(BASE_ORIGIN, { timeout: PAGE_LOADING })
   for (let i = 0; i < 10; i++) {
     const ss = await page.context().storageState()
     if (ss.cookies.length !== 0 && ss.origins.length !== 0) {