Closed Bug 1617637 Opened 4 years ago Closed 4 years ago

Reftest-plain suite_start data still not in correct format: test-ids should be strings

Categories

(Testing :: Reftest, defect, P3)

Version 3
defect

Tracking

(firefox75 fixed)

RESOLVED FIXED
mozilla75
Tracking Status
firefox75 --- fixed

People

(Reporter: gbrown, Assigned: gbrown)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

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", ...

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
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla75
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: