Open
Bug 982674
Opened 10 years ago
Updated 2 years ago
Support subtests in manifest parser
Categories
(Testing :: Mozbase, defect)
Tracking
(Not tracked)
NEW
People
(Reporter: jgraham, Unassigned)
Details
No description provided.
Reporter | ||
Comment 1•10 years ago
|
||
(sorry bugzilla UI fail) I have a need for "manifests" that support subtests, so that the expected results of individual subtests can be recorded. At present manifestdestiny doesn't support this use case. An example of my solution is at [1] which uses indentation for hierarchy. It's not super-clear to me how to do this in a way that looks exactly like an ini file, since they are non-hierarchical by design. [1] http://hg.mozilla.org/projects/cedar/file/9af41dfd75f0/testing/web-platform-tests/metadata/dom/historical.html.ini
Comment 2•10 years ago
|
||
This seems hard :/ Do you have an idea how often we would run into cases where we'd want to allow subtests to fail, but still have the overall test pass? There's a pretty big assumption throughout our automation at the moment that TEST-PASS implies all subtests pass.
Reporter | ||
Comment 3•10 years ago
|
||
Well in web-platform-tests it happens all the time. For example [1], [2]. It's considered totally normal in that testsuite to put many tests in a file, and totally normal to have tests that don't pass in specific implementations. Throwing away whole files because one subtest isn't passing would dramatically reduce the usefulness of the suite. [1] http://w3c-test.org/dom/interfaces.html [2] http://w3c-test.org/dom/historical.html
Reporter | ||
Updated•10 years ago
|
Summary: Support subtests in manifest destiny → Support subtests in manifest parser
Comment 4•10 years ago
|
||
Hm, given that subtests are all part of the same file, we could put them all under the section like: [file.js] test1 test2 But here the problem is that you cannot individually disable tests, or specify other keys for them. So why can't we make it similar to include or the upcoming parent section? [file.js:test1] disabled = ... [file.js:test2] skip-if = ...
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•