fix(e2e): make oidc logout test more robust

Resolves an intermittent failure in the oidc-suites logout test
by using a more robust pattern for waiting for navigation after a
click. This avoids a race condition where the test could time out
before the navigation completes.

Author: ryanbas21

.changeset/config.json

@@ -13,7 +13,6 @@
   "baseBranch": "main",
   "updateInternalDependencies": "patch",
   "ignore": [
-    "ping-javascript-sdk",
     "scratchpad",
     "@forgerock/pingone-scripts",
     "@forgerock/device-client",

e2e/oidc-suites/src/logout.spec.ts

@@ -38,9 +38,9 @@ test.describe('Logout tests', () => {
 
     await page.getByLabel('User Name').fill(pingAmUsername);
     await page.getByRole('textbox', { name: 'Password' }).fill(pingAmPassword);
+    const amNavigation = page.waitForURL('http://localhost:8443/ping-am/**');
     await page.getByRole('button', { name: 'Next' }).click();
-
-    await page.waitForURL('http://localhost:8443/ping-am/**');
+    await amNavigation;
     expect(page.url()).toContain('code');
     expect(page.url()).toContain('state');
     await expect(page.getByRole('button', { name: 'Login (Background)' })).toBeHidden();
@@ -74,9 +74,9 @@ test.describe('Logout tests', () => {
 
     await page.getByLabel('Username').fill(pingOneUsername);
     await page.getByRole('textbox', { name: 'Password' }).fill(pingOnePassword);
+    const p1Navigation = page.waitForURL('http://localhost:8443/ping-one/**');
     await page.getByRole('button', { name: 'Sign On' }).click();
-
-    await page.waitForURL('http://localhost:8443/ping-one/**');
+    await p1Navigation;
     expect(page.url()).toContain('code');
     expect(page.url()).toContain('state');
     await expect(page.getByRole('button', { name: 'Login (Background)' })).toBeHidden();