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 #17416

Closed
wants to merge 9 commits into from
Closed

Update web-platform-tests #17416

wants to merge 9 commits into from

Conversation

@jdm
Copy link
Member

jdm commented Jun 19, 2017 •
edited by larsbergstrom

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.

This change is Reviewable
@highfive
Copy link

highfive commented Jun 19, 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
@jdm
Copy link
Member Author

jdm commented Jun 19, 2017

@bors-servo: try
@bors-servo
Copy link
Contributor

bors-servo commented Jun 19, 2017

⌛ Trying commit f999759 with merge 10b245e...
bors-servo added a commit that referenced this pull request Jun 19, 2017
@bors-servo
Auto merge of #17416 - jdm:updatewpt2, r=<try>
Loading status checks…
10b245e 
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.

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

bors-servo commented Jun 19, 2017

💔 Test failed - mac-rel-wpt1
@jdm
Copy link
Member Author

jdm commented Jun 19, 2017

@bors-servo: try
@bors-servo
Copy link
Contributor

bors-servo commented Jun 19, 2017

⌛ Trying commit d39b787 with merge f26cc84...
bors-servo added a commit that referenced this pull request Jun 19, 2017
@bors-servo
Auto merge of #17416 - jdm:updatewpt2, r=<try>
Loading status checks…
f26cc84 
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.

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

bors-servo commented Jun 20, 2017

💔 Test failed - mac-rel-wpt1
@jdm jdm force-pushed the jdm:updatewpt2 branch from d39b787 to 526db98 Jun 20, 2017
@jdm
Copy link
Member Author

jdm commented Jun 20, 2017

@bors-servo: try
@bors-servo
Copy link
Contributor

bors-servo commented Jun 20, 2017

⌛ Trying commit 526db98 with merge 6618260...
bors-servo added a commit that referenced this pull request Jun 20, 2017
@bors-servo
Auto merge of #17416 - jdm:updatewpt2, r=<try>
Loading status checks…
6618260 
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.

<!-- 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/17416)
<!-- Reviewable:end -->
@bors-servo bors-servo mentioned this pull request Jun 20, 2017
Merged
4 of 5 tasks complete
@bors-servo
Copy link
Contributor

bors-servo commented Jun 20, 2017

💥 Test timed out
@jdm
Copy link
Member Author

jdm commented Jun 20, 2017

@bors-servo: try
@jdm
Copy link
Member Author

jdm commented Jun 20, 2017

@bors-servo: retry
@bors-servo
Copy link
Contributor

bors-servo commented Jun 20, 2017

⌛ Trying commit 526db98 with merge 9e36f4f...
bors-servo added a commit that referenced this pull request Jun 20, 2017
@bors-servo
Auto merge of #17416 - jdm:updatewpt2, r=<try>
Loading status checks…
9e36f4f 
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.

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

bors-servo commented Jun 20, 2017

💔 Test failed - arm32
@jdm jdm force-pushed the jdm:updatewpt2 branch from 526db98 to 5df06c7 Jun 20, 2017
@jdm
Copy link
Member Author

jdm commented Jun 20, 2017

@bors-servo: try
@bors-servo
Copy link
Contributor

bors-servo commented Oct 4, 2017

☔ The latest upstream changes (presumably #18724) made this pull request unmergeable. Please resolve the merge conflicts.
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 @Ms2ger
Update WPT results.
Loading status checks…
2825030
@Ms2ger Ms2ger force-pushed the jdm:updatewpt2 branch from 93ecb3b to 2825030 Oct 4, 2017
@Ms2ger
Copy link
Contributor

Ms2ger commented Oct 4, 2017

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

bors-servo commented Oct 4, 2017

📌 Commit 2825030 has been approved by jgraham
@bors-servo
Copy link
Contributor

bors-servo commented Oct 4, 2017

⌛ Testing commit 2825030 with merge c950e9d...
bors-servo added a commit that referenced this pull request Oct 4, 2017
@bors-servo
Auto merge of #17416 - jdm:updatewpt2, r=jgraham
Loading status checks…
c950e9d 
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.

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

bors-servo commented Oct 5, 2017

💔 Test failed - mac-rel-wpt1
@mbrubeck
Copy link
Contributor

mbrubeck commented Oct 5, 2017 

  ▶ OK [expected TIMEOUT] /html/semantics/tabular-data/processing-model-1/span-limits.html

  ▶ Unexpected subtest result in /html/semantics/tabular-data/processing-model-1/span-limits.html:
  │ FAIL [expected PASS] colspan of 1001 must be treated as 1000
  │   → assert_equals: expected 2009 but got 2022.1
  │ 
  │ @http://web-platform.test:8000/html/semantics/tabular-data/processing-model-1/span-limits.html:40:5
  │ Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:1412:20
  │ test@http://web-platform.test:8000/resources/testharness.js:501:9
  └ @http://web-platform.test:8000/html/semantics/tabular-data/processing-model-1/span-limits.html:39:1

  ▶ Unexpected subtest result in /html/semantics/tabular-data/processing-model-1/span-limits.html:
  │ FAIL [expected PASS] rowspan of 65534 must work
  │   → assert_equals: expected 393335 but got 137
  │ 
  │ @http://web-platform.test:8000/html/semantics/tabular-data/processing-model-1/span-limits.html:51:5
  │ Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:1412:20
  │ test@http://web-platform.test:8000/resources/testharness.js:501:9
  └ @http://web-platform.test:8000/html/semantics/tabular-data/processing-model-1/span-limits.html:44:1

  ▶ Unexpected subtest result in /html/semantics/tabular-data/processing-model-1/span-limits.html:
  │ FAIL [expected PASS] rowspan of 65535 must be treated as 65534
  │   → assert_equals: expected 393335 but got 137
  │ 
  │ @http://web-platform.test:8000/html/semantics/tabular-data/processing-model-1/span-limits.html:62:5
  │ Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:1412:20
  │ test@http://web-platform.test:8000/resources/testharness.js:501:9
  └ @http://web-platform.test:8000/html/semantics/tabular-data/processing-model-1/span-limits.html:55:1
@nox nox mentioned this pull request Oct 5, 2017
Merged
@nox
Copy link
Member

nox commented Oct 5, 2017

See #18756.
@nox nox closed this Oct 5, 2017
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 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 -->
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
S-tests-failed
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

8 participants
@jdm @highfive @bors-servo @nox @Ms2ger @mbrubeck @jgraham @glennw
You can’t perform that action at this time.