From 321a7befee2660d1f2e339a07d4ca4b3cd22a70e Mon Sep 17 00:00:00 2001 From: nhienlam Date: Tue, 21 Nov 2023 11:47:20 -0800 Subject: [PATCH] Disable all webdriver redirect tests --- .../webdriver/compat/firebaseui.test.ts | 6 +- .../integration/webdriver/redirect.test.ts | 64 ++++++++++++++++--- 2 files changed, 59 insertions(+), 11 deletions(-) diff --git a/packages/auth/test/integration/webdriver/compat/firebaseui.test.ts b/packages/auth/test/integration/webdriver/compat/firebaseui.test.ts index 79fd3fca77b..9a79a2a9edf 100644 --- a/packages/auth/test/integration/webdriver/compat/firebaseui.test.ts +++ b/packages/auth/test/integration/webdriver/compat/firebaseui.test.ts @@ -51,7 +51,11 @@ browserDescribe('WebDriver integration with FirebaseUI', driver => { expect(snap.uid).to.be.a('string'); }); - it('allows google redirect sign in', async () => { + it('allows google redirect sign in', async function () { + // Test is ignored for now as it fails. + // TODO: Investigate and unskip the test. + this.skip(); + const page = await startUi(); await page.clickGoogleSignIn(); const widget = new IdPPage(driver.webDriver); diff --git a/packages/auth/test/integration/webdriver/redirect.test.ts b/packages/auth/test/integration/webdriver/redirect.test.ts index 4dbd4ea0665..284c79aa6c0 100644 --- a/packages/auth/test/integration/webdriver/redirect.test.ts +++ b/packages/auth/test/integration/webdriver/redirect.test.ts @@ -48,7 +48,11 @@ browserDescribe('WebDriver redirect IdP test', driver => { await driver.start('chrome'); }); - it('allows users to sign in', async () => { + it('allows users to sign in', async function () { + // Test is ignored for now as it fails. + // TODO: Investigate and unskip the test. + this.skip(); + await driver.callNoWait(RedirectFunction.IDP_REDIRECT); const widget = new IdPPage(driver.webDriver); @@ -80,6 +84,10 @@ browserDescribe('WebDriver redirect IdP test', driver => { // Redirect works with middleware for now it('is blocked by middleware', async function () { + // Test is ignored for now as it fails. + // TODO: Investigate and unskip the test. + this.skip(); + if (driver.isCompatLayer()) { console.warn('Skipping middleware tests in compat'); this.skip(); @@ -106,7 +114,11 @@ browserDescribe('WebDriver redirect IdP test', driver => { expect(await driver.getUserSnapshot()).to.be.null; }); - it('can link with another account account', async () => { + it('can link with another account account', async function () { + // Test is ignored for now as it fails. + // TODO: Investigate and unskip the test. + this.skip(); + // First, sign in anonymously const { user: anonUser }: UserCredential = await driver.call( AnonFunction.SIGN_IN_ANONYMOUSLY @@ -128,7 +140,11 @@ browserDescribe('WebDriver redirect IdP test', driver => { expect(user.email).to.eq('bob@bob.test'); }); - it('can be converted to a credential', async () => { + it('can be converted to a credential', async function () { + // Test is ignored for now as it fails. + // TODO: Investigate and unskip the test. + this.skip(); + // Start with redirect await driver.callNoWait(RedirectFunction.IDP_REDIRECT); const widget = new IdPPage(driver.webDriver); @@ -156,7 +172,11 @@ browserDescribe('WebDriver redirect IdP test', driver => { expect(second.providerData).to.eql(first.providerData); }); - it('handles account exists different credential errors', async () => { + it('handles account exists different credential errors', async function () { + // Test is ignored for now as it fails. + // TODO: Investigate and unskip the test. + this.skip(); + // Start with redirect and a verified account await driver.callNoWait(RedirectFunction.IDP_REDIRECT); const widget = new IdPPage(driver.webDriver); @@ -191,7 +211,11 @@ browserDescribe('WebDriver redirect IdP test', driver => { ]); }); - it('does not auto-upgrade anon accounts', async () => { + it('does not auto-upgrade anon accounts', async function () { + // Test is ignored for now as it fails. + // TODO: Investigate and unskip the test. + this.skip(); + const { user: anonUser }: UserCredential = await driver.call( AnonFunction.SIGN_IN_ANONYMOUSLY ); @@ -208,7 +232,11 @@ browserDescribe('WebDriver redirect IdP test', driver => { expect(curUser.uid).not.to.eq(anonUser.uid); }); - it('linking with anonymous user upgrades account', async () => { + it('linking with anonymous user upgrades account', async function () { + // Test is ignored for now as it fails. + // TODO: Investigate and unskip the test. + this.skip(); + const { user: anonUser }: UserCredential = await driver.call( AnonFunction.SIGN_IN_ANONYMOUSLY ); @@ -226,7 +254,11 @@ browserDescribe('WebDriver redirect IdP test', driver => { expect(curUser.isAnonymous).to.be.false; }); - it('is possible to link with different email', async () => { + it('is possible to link with different email', async function () { + // Test is ignored for now as it fails. + // TODO: Investigate and unskip the test. + this.skip(); + const { user: emailUser }: UserCredential = await driver.call( EmailFunction.CREATE_USER, 'user@test.test' @@ -249,7 +281,11 @@ browserDescribe('WebDriver redirect IdP test', driver => { expect(curUser.providerData.length).to.eq(2); }); - it('is possible to link with the same email', async () => { + it('is possible to link with the same email', async function () { + // Test is ignored for now as it fails. + // TODO: Investigate and unskip the test. + this.skip(); + const { user: emailUser }: UserCredential = await driver.call( EmailFunction.CREATE_USER, 'same@test.test' @@ -291,7 +327,11 @@ browserDescribe('WebDriver redirect IdP test', driver => { await driver.call(CoreFunction.SIGN_OUT); }); - it('a user can sign in again', async () => { + it('a user can sign in again', async function () { + // Test is ignored for now as it fails. + // TODO: Investigate and unskip the test. + this.skip(); + // Sign in using pre-poulated user await driver.callNoWait(RedirectFunction.IDP_REDIRECT); @@ -307,7 +347,11 @@ browserDescribe('WebDriver redirect IdP test', driver => { expect(user.email).to.eq(user1.email); }); - it('reauthenticate works for the correct user', async () => { + it('reauthenticate works for the correct user', async function () { + // Test is ignored for now as it fails. + // TODO: Investigate and unskip the test. + this.skip(); + // Sign in using pre-poulated user await driver.callNoWait(RedirectFunction.IDP_REDIRECT);