Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Playwright Firefox Crashes On Page Reload #20791

Closed
eatonjl opened this issue Feb 9, 2023 · 6 comments · Fixed by #20878
Closed

[BUG] Playwright Firefox Crashes On Page Reload #20791

eatonjl opened this issue Feb 9, 2023 · 6 comments · Fixed by #20878
Assignees
Labels

Comments

@eatonjl
Copy link

eatonjl commented Feb 9, 2023

Context:

  • Playwright Version: 1.31.0-alpha-jan-31-2023
  • Operating System: Mac
  • Node.js version: 16.16.0
  • Browser: Firefox
  • Extra:

Code Snippet

Help us help you! Put down a short code snippet that illustrates your bug and
that we can run and debug locally. For example:

import { test } from '@playwright/test';

test('bob', async ({ page }) => {
  await page.goto(
    'https://id2.navexglobal.com/SecureTokenServer/Auth/SamlLogin?SAMLRequest=jZJfa9swFMW%2FitF7LFsLaSbiQNpQFsg2t3b7sJciy9eJmCy5ulL%2FfPvKzko7xkpBD%2BLq8Lv3nKsVil4PfBP80VzDfQD0yVOvDfLpoSDBGW4FKuRG9IDcS15tvu85SzM%2BOOuttJokG0RwXllzYQ2GHlwF7kFJuLneF%2BTo%2FYCc0tZa1wuTGvEATwdtG6FTaXtaHVXTWA3%2BmCJaOuIZLX9WNUm2cR5lxEh%2B46iW%2FcsAGRzU9jeYsTU4OlqiVTSxtwdlSLLbFuSuW3YLaDNYZPOzufwSTytFEy9z1nXZYhlliAF2Br0wviAsY2yWZzO2rFnG48m%2F%2FiJJ%2Bcf3uTKtMoePQ2pOIuTf6rqcnWzdgsPJUhSQ9WqMmk%2BN3bvwP8aK18TJ%2BvP5rui7Vqe%2BA%2F8R2bttabWSz8lGa%2Ft44UB4KEhOkssR6f8%2FSp7mU0W1s26S8mBwAKk6BS1JqnKkXwWhx4IriAje9tM%2BfVwtKg%2BErk9j%2Ff0L1y8%3D&RelayState=cookie%3A1666988419_8131'
  );
  await page.getByPlaceholder('User name').fill('Martin1');
  await page.getByPlaceholder('Password').fill(**[ON REQUEST]**);
  await page.getByRole('button', { name: 'Sign In' }).click();
  await page
    .getByRole('link', { name: 'Policy & Procedure Management' })
    .click();
  await page.locator('id=SideBar_NewDocuments').click();
  const [page1] = await Promise.all([
    page.waitForEvent('popup'),
    page.locator('id=Popup_NewDocument_richtext').click(),
  ]);
  await page1
    .frameLocator('iframe[name="WizardPanelPropertiesEdit_Iframe"]')
    .locator('input[name="ctl00\\$ContentPlaceHolder1\\$TitleField"]')
    .fill('Test');
  await page1
    .frameLocator('iframe[name="WizardPanelPropertiesEdit_Iframe"]')
    .getByRole('button', { name: 'Save' })
    .click();
  await page1.getByRole('cell', { name: 'Reviewers' }).click();
  await page1
    .frameLocator('iframe[name="WizardPanelPropertiesEdit_Iframe"]')
    .frameLocator('#UserFinderFrame')
    .locator('input[type="checkbox"]')
    .check();
  await page1
    .frameLocator('iframe[name="WizardPanelPropertiesEdit_Iframe"]')
    .getByRole('button', { name: 'Save' })
    .click();
  await page1.getByRole('cell', { name: 'Approvers' }).click();
  await page1
    .frameLocator('iframe[name="WizardPanelPropertiesEdit_Iframe"]')
    .frameLocator('#UserFinderFrame')
    .locator('input[type="checkbox"]')
    .check();
  await page1
    .frameLocator('iframe[name="WizardPanelPropertiesEdit_Iframe"]')
    .getByRole('button', { name: 'Save' })
    .click();
  await page.locator('body').click();
  await page1.getByRole('button', { name: 'Submit for Review' }).click();
  await page1
    .frameLocator('iframe[name="Frame_PPMPopupIFrame"]')
    .getByRole('button', { name: 'Submit' })
    .click();
  await page1.reload();
});

This is in response to this issue. I am testing the canary release that fixes an issue with Juggler/Firefox. Upon testing, however, I noted that the reload function causes a timeout or failure of sorts. I tried going to a normal page (like Pizza Hut's website - good pizza) and the reload worked fine. I am not sure what causes it to fail in this scenario, but I thought I would provide the code so it could be looked at. I can provide the username for the login upon request.

@mxschmitt
Copy link
Member

Can you forward us the credentials we need to run this script? Either via email, discord or Slack, thanks.

@eatonjl
Copy link
Author

eatonjl commented Feb 10, 2023

@mxschmitt Sure thing! do you have an email or links to Discord/Slack so I can send that?

@mxschmitt
Copy link
Member

@eatonjl
Copy link
Author

eatonjl commented Feb 10, 2023

@mxschmitt Done!

@mxschmitt
Copy link
Member

I can reproduce it now, will assign it to Andrey so he can take a look.

@eatonjl
Copy link
Author

eatonjl commented Feb 12, 2023

@mxschmitt thanks!

@aslushnikov aslushnikov self-assigned this Feb 13, 2023
aslushnikov added a commit to aslushnikov/playwright that referenced this issue Feb 13, 2023
aslushnikov added a commit to aslushnikov/playwright that referenced this issue Feb 28, 2023
aslushnikov added a commit to aslushnikov/playwright that referenced this issue Mar 2, 2023
While this fixes the regression, this breaks the microsoft#20791.
We'll fix it separately.

Fixes microsoft#21145
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants