Closed
Bug 1321197
(stylo-style-mochitest)
Opened 8 years ago
Closed 6 years ago
[meta] stylo: Have stylo pass all style system mochitests
Categories
(Testing :: Mochitest, defect, P5)
Testing
Mochitest
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: xidorn, Assigned: xidorn)
References
(Depends on 1 open bug, Blocks 1 open bug, )
Details
(Keywords: meta)
I'm going to use this bug to track issues blocking running style system tests with stylo.
I think we should do the following:
1. fix crash bugs found from running the tests
2. create a new test task which runs a subset of style system tests on ci
3. fix other bugs identified by the tests
Assignee | ||
Updated•8 years ago
|
Depends on: 1302945, stylo-css-animations
Updated•8 years ago
|
Assignee | ||
Updated•8 years ago
|
Alias: stylo-style-mochitest
Assignee | ||
Comment 2•8 years ago
|
||
Filed a Servo side meta bug for this.
See Also: → https://github.com/servo/servo/issues/15404
Assignee | ||
Updated•8 years ago
|
Summary: [meta] stylo: Run style system mochitests on stylo → [meta] stylo: Have style system mochitests green with stylo
Assignee | ||
Updated•8 years ago
|
Summary: [meta] stylo: Have style system mochitests green with stylo → [meta] stylo: Have stylo pass all style system mochitests
Assignee | ||
Updated•8 years ago
|
Comment 3•8 years ago
|
||
Now that this job is running on autoland / mc, what do you think about temporarily disabling test_align_justify_computed_values.html and test_acid3_test46.html so that we can keep it green?
Flags: needinfo?(xidorn+moz)
Assignee | ||
Comment 4•8 years ago
|
||
test_align_justify_computed_values.html and test_acid3_test46.html are not the only mochitests currently fail. There are tons of tests are still failing (see https://gist.github.com/upsuper/082268f59e6f267d5a466c2cbe698e58). Only them showing up in the panel is just because they are the first two tests run in the task and failures from them have already run out of space of that panel.
In addition to failures, there are also many non-fatal assertions (see bug 1337615).
So making that task green isn't that trivial. We may need to disable a wide range of mochitests to make it green (not sure how many, but we would lose the biggest tests including test_{inherit,initial,value}_{storage,computation}.html).
I've filed bug 1337674 to figure out the best path to green the task with the known failure list. But I'm unsure about what is the most helpful way.
Flags: needinfo?(xidorn+moz)
Updated•8 years ago
|
Priority: P1 → P5
Depends on: 1353191
Depends on: 1364746
Assignee | ||
Comment 5•6 years ago
|
||
Mark fixed as a tracking bug.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•