Skip to content

Bring back chromium tests#20651

Closed
stacey-gammon wants to merge 3 commits intoelastic:masterfrom
stacey-gammon:2018-07-10-pngbitbitbug
Closed

Bring back chromium tests#20651
stacey-gammon wants to merge 3 commits intoelastic:masterfrom
stacey-gammon:2018-07-10-pngbitbitbug

Conversation

@stacey-gammon
Copy link

No description provided.

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@stacey-gammon
Copy link
Author

Tests all passed when the original code was kept the same except for the introduction of a verification phase directly after Page.captureScreenshot. Next step is to remove that code and double check the error will still be hit.

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@stacey-gammon
Copy link
Author

stacey-gammon commented Jul 11, 2018

Everything is working now??? Maybe a package was upgraded since the failures? Or more likely, some random timing change is hiding it. :(

@stacey-gammon
Copy link
Author

retest

@elasticmachine
Copy link
Contributor

💔 Build Failed

@stacey-gammon
Copy link
Author

Okay, hit the bug that time. Will revert to moving the check earlier again and run a few times.

@elasticmachine
Copy link
Contributor

💔 Build Failed

@stacey-gammon
Copy link
Author

Failed on selenium tests (#20387 (comment), but xpack passed.

jenkins, test this

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@stacey-gammon
Copy link
Author

Closing in favor of #20770

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.

2 participants