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

issue #3051 race condition for waitForFileChooser() fix + fixed wkd test #3114

Merged
merged 3 commits into from Nov 2, 2020

Conversation

limonte
Copy link
Contributor

@limonte limonte commented Nov 2, 2020

Fixes #3051

This PR workarounds the puppeteer issue puppeteer/puppeteer#6040

I subscribed to the issue, to be notified and remove this workaround when the issue is resolved and released.

@limonte limonte changed the title [WIP] Workaround the race condition for waitForFileChooser() Workaround the race condition for waitForFileChooser() Nov 2, 2020
@limonte
Copy link
Contributor Author

limonte commented Nov 2, 2020

Now the can lookup public key from WKD directly test case is failing, but I'm pretty sure this PR has nothing to do with it. Ready for review.

The Attaching file using keyboard test case is good:

image

@limonte limonte marked this pull request as ready for review November 2, 2020 20:52
tomholub
tomholub previously approved these changes Nov 2, 2020
@tomholub tomholub changed the title Workaround the race condition for waitForFileChooser() issue #3051 race condition for waitForFileChooser() fix + fixed wkd test Nov 2, 2020
@tomholub tomholub merged commit 1fc3404 into master Nov 2, 2020
@tomholub
Copy link
Collaborator

tomholub commented Nov 2, 2020

Thanks for debugging this!

@limonte limonte deleted the issue-3051-flaky-test branch November 2, 2020 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

potentially flaky test - attach with keyboard
2 participants