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 all webgl1 tests on android #21801

Closed
wants to merge 1 commit into from
Closed

Run all webgl1 tests on android #21801

wants to merge 1 commit into from

Conversation

@jdm
Copy link
Member

jdm commented Sep 24, 2018

This change is Reviewable

@highfive
Copy link

highfive commented Sep 24, 2018

Heads up! This PR modifies the following files:

@jdm
Copy link
Member Author

jdm commented Sep 24, 2018

I am expecting... interesting... results. Also it turns out we haven't been running the existing webgl test at all since #21519. Oops.

@jdm
Copy link
Member Author

jdm commented Sep 24, 2018

@bors-servo try=android

@bors-servo
Copy link
Contributor

bors-servo commented Sep 24, 2018

Trying commit 746ec60 with merge 3c6c7aa...

bors-servo added a commit that referenced this pull request Sep 24, 2018
Run all webgl1 tests on android

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/21801)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Sep 24, 2018

💔 Test failed - android-x86

@SimonSapin
Copy link
Member

SimonSapin commented Sep 25, 2018

Ran 884 tests finished in 3184.0 seconds.
  • 806 ran as expected. 23 tests skipped.
  • 33 tests had errors unexpectedly
  • 19 tests timed out unexpectedly
  • 1 tests unexpectedly okay
  • 44 tests had unexpected subtest results

3.6 seconds per test on average is a lot, but the number of failures is not as bad as I half-expected.

@jdm jdm removed the S-awaiting-review label Mar 11, 2019
@jdm jdm closed this Nov 4, 2019
@jdm jdm deleted the jdm-patch-34 branch Nov 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

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