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

Run ./wpt test-jobs --all to see if all tests pass #17217

Closed
wants to merge 1 commit into from

Conversation

foolip
Copy link
Member

@foolip foolip commented Jun 6, 2019

No description provided.

@foolip
Copy link
Member Author

foolip commented Jun 20, 2019

@LukeZielinski I was attempting in 143c349 to mark a failure as expected, but then the failure shifted somewhere else. This is very reminiscent of something you were telling me about debugging these tests, was it state in ChromeDriver leaking between them? Were you able to get to the bottom of that?

This job isn't blocking, but it will be failing on all PRs which trigger the tests. Should we just disable them?

@foolip
Copy link
Member Author

foolip commented Sep 3, 2019

@Ms2ger there are now failures on master in test_tests_affected_null, and my best guess is that it's related to #18787, although it's not obvious how.

Can you check if it was a regression from that change?

@Ms2ger
Copy link
Contributor

Ms2ger commented Sep 3, 2019

Don't you think it might be #18743 and #18688?

@foolip
Copy link
Member Author

foolip commented Sep 3, 2019

Possibly, but they landed longer ago. Will have to bisect I guess...

@gsnedders
Copy link
Member

Yeah, it's the issues @Ms2ger linked to. I'd noticed this last week just before heading off and never got around to filing a bug for it. :/

@foolip
Copy link
Member Author

foolip commented Sep 4, 2019

Fixed in #18819. I'll close this PR and open a new one so that y'all don't have to be notified when I keep rebasing it.

@foolip foolip closed this Sep 4, 2019
@foolip foolip deleted the foolip/all-test-jobs branch September 4, 2019 10:28
@foolip foolip restored the foolip/all-test-jobs branch September 4, 2019 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants