Skip to content

/ servo

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

Update web-platform-tests #18756

Merged
merged 9 commits into from Oct 5, 2017
Merged

Update web-platform-tests #18756

merged 9 commits into from Oct 5, 2017

Conversation

@nox
Copy link
Member

nox commented Oct 5, 2017 •
edited by SimonSapin

This updates web-platform-tests to a46616a5b18e. It also removes our local copy of wptrunner in favour of the upstream one that we can modify and sync more easily. Carried from #17416.

This change is Reviewable
jdm added 9 commits Jun 19, 2017
@jdm @Ms2ger
Update web-platform-tests to revision a46616a5b18e83587ddbbed756c7b96…
578498b 
…cbb4b015d
@jdm @Ms2ger
Use git's path stripping when upstreaming patches.
d9baadd
@jdm @Ms2ger
Use upstream wptrunner from web-platform-tests.
ca340ec
@jdm @Ms2ger
Allow running wpt directories by path on the command line.
933b42e
@jdm @Ms2ger
Avoid testharness.js errors for assert_throws.
2fb74cb
@jdm @Ms2ger
Minimal changes to get WPT tests running on Windows.
c1c60be
@jdm @Ms2ger
Avoid endless testrunner restart loops.
da86772 
If the first test selected by a testrunner thread had a CRASH expectation,
the testrunner believed that it needed to restart and would never actually
get around to running the test.
@jdm @Ms2ger
Fix up a test metadata classification.
bceaca9
@jdm @nox
Update WPT results.
Loading status checks…
3347094
@highfive
Copy link

highfive commented Oct 5, 2017

Heads up! This PR modifies the following files:

  • @wafflespeanut: python/servo/devenv_commands.py, python/mach_bootstrap.py
  • @jgraham: tests/wpt/harness/test/testdata/reftest/green-ref.html, tests/wpt/harness/tox.ini, tests/wpt/harness/wptrunner/testharnessreport.js, tests/wpt/harness/wptrunner/browsers/init.py, tests/wpt/harness/wptrunner/products.py and 151 more
@nox nox mentioned this pull request Oct 5, 2017
Closed
@nox
Copy link
Member Author

nox commented Oct 5, 2017

I updated the expectations as per #17416 (comment).

@bors-servo r=jgraham p=10
@bors-servo
Copy link
Contributor

bors-servo commented Oct 5, 2017

📌 Commit 3347094 has been approved by jgraham
@highfive highfive assigned jgraham and unassigned cbrewster Oct 5, 2017
@jdm
Copy link
Member

jdm commented Oct 5, 2017

The test is intermittent. File an issue.
@nox
Copy link
Member Author

nox commented Oct 5, 2017

The test is intermittent. File an issue.

It never fails locally though.
@bors-servo
Copy link
Contributor

bors-servo commented Oct 5, 2017

⌛ Testing commit 3347094 with merge fdb15a6...
bors-servo added a commit that referenced this pull request Oct 5, 2017
@bors-servo
Auto merge of #18756 - servo:wptup, r=jgraham
Loading status checks…
fdb15a6 
Update web-platform-tests

This updates web-platform-tests to a46616a5b18e. It also removes our local copy of wptrunner in favour of the upstream one that we can modify and sync more easily. Carried from #17416.

<!-- 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/18756)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Oct 5, 2017

💔 Test failed - linux-rel-wpt
@nox
Copy link
Member Author

nox commented Oct 5, 2017

SORRY FOR NOT TRUSTING YOU JOSH

@bors-servo retry #18762
@bors-servo
Copy link
Contributor

bors-servo commented Oct 5, 2017

⌛ Testing commit 3347094 with merge 139055f...
bors-servo added a commit that referenced this pull request Oct 5, 2017
@bors-servo
Auto merge of #18756 - servo:wptup, r=jgraham
Loading status checks…
139055f 
Update web-platform-tests

This updates web-platform-tests to a46616a5b18e. It also removes our local copy of wptrunner in favour of the upstream one that we can modify and sync more easily. Carried from #17416.

<!-- 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/18756)
<!-- Reviewable:end -->
@highfive highfive removed the S-tests-failed label Oct 5, 2017
@bors-servo
Copy link
Contributor

bors-servo commented Oct 5, 2017

☀️ Test successful - android, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css1, mac-rel-css2, mac-rel-wpt1, mac-rel-wpt2, mac-rel-wpt3, mac-rel-wpt4, windows-msvc-dev
Approved by: jgraham
Pushing 139055f to master...
@bors-servo bors-servo merged commit 3347094 into master Oct 5, 2017
3 of 4 checks passed
3 of 4 checks passed
dependency-ci An error occured
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@bors-servo bors-servo mentioned this pull request Oct 5, 2017
Merged
4 of 4 tasks complete
@Ms2ger Ms2ger deleted the wptup branch Oct 5, 2017
jdm added a commit to web-platform-tests/wpt that referenced this pull request Oct 12, 2017
@jdm
Minimal changes to get WPT tests running on Windows.
12316f4 
Upstreamed from servo/servo#18756 [ci skip]
@servo-wpt-sync servo-wpt-sync mentioned this pull request Oct 12, 2017
Merged
jakearchibald added a commit to jakearchibald/web-platform-tests that referenced this pull request Nov 16, 2017
@jdm @jakearchibald
Minimal changes to get WPT tests running on Windows.
ef399c8 
Upstreamed from servo/servo#18756 [ci skip]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jgraham jgraham

Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

6 participants
@nox @highfive @bors-servo @jdm @jgraham @cbrewster
You can’t perform that action at this time.