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

Ignore unexpected MessageEvents #21150

merged 1 commit into from Jan 14, 2020


Copy link

danyao commented Jan 13, 2020

When running these tests in the web runner, extra MessageEvents sometimes appear before the ones generated by the test iframe and causes the test to fail. This patch modifies the test to ignore any messages that are not from the test iframe to improve the consistency of the test.

Danyao Wang
@rsolomakhin rsolomakhin merged commit 2aad066 into web-platform-tests:master Jan 14, 2020
10 checks passed
10 checks passed
Azure Pipelines Build #20200113.55 succeeded
Azure Pipelines (./wpt test-jobs) ./wpt test-jobs succeeded
Azure Pipelines (affected tests without changes: Safari Technology Preview) affected tests without changes: Safari Technology Preview succeeded
Azure Pipelines (affected tests: Safari Technology Preview) affected tests: Safari Technology Preview succeeded
Azure Pipelines ( hook: safari-preview-affected-tests) hook: safari-preview-affected-tests succeeded
Azure Pipelines ( hook: safari-preview-affected-tests-without-changes) hook: safari-preview-affected-tests-without-changes succeeded
Community-TC (pull_request) TaskGroup: success
Details - chrome[experimental] Chrome results
Details - firefox[experimental] Firefox results
Details - safari[experimental] Safari results
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.