Closed
Bug 1617637
Opened 5 years ago
Closed 5 years ago
Reftest-plain suite_start data still not in correct format: test-ids should be strings
Categories
(Testing :: Reftest, defect, P3)
Tracking
(firefox75 fixed)
RESOLVED
FIXED
mozilla75
Tracking | Status | |
---|---|---|
firefox75 | --- | fixed |
People
(Reporter: gbrown, Assigned: gbrown)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
Instead of:
{"tests": {"layout/reftests/bidi/reftest.list": ["[u'layout/reftests/bidi/bdi-element.html', u'==', u'layout/reftests/bidi/bdi-element-ref.html']", ...
we need:
{"tests": {"layout/reftests/bidi/reftest.list": ["layout/reftests/bidi/bdi-element.html == layout/reftests/bidi/bdi-element-ref.html", ...
Updated•5 years ago
|
Priority: -- → P3
Assignee | ||
Comment 2•5 years ago
|
||
reftest-plain test identifiers may be tuples like ['this', '==', 'that'], which are
undesirable to ActiveData. This patch converts such identifiers to simple strings,
'this == that' in the suite_start message (string format is already used in other
messages, like test_start).
Pushed by gbrown@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/061f9854ee79
Use string rather than tuple for reftest test ids in suite_start; r=ahal
Comment 4•5 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox75:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla75
You need to log in
before you can comment on or make changes to this bug.
Description
•