Closed Bug 859252 Opened 7 years ago Closed 7 years ago

Import reftests in dom/imptests

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla23

People

(Reporter: Ms2ger, Assigned: Ms2ger)

References

Details

Attachments

(3 files, 1 obsolete file)

No description provided.
Depends on: 866424
smontagu for the bidi-related ones, mounir for the others. Note that most of these are moves, as they already lived somewhere else in m-c.
Attachment #742824 - Flags: review?(mounir)
Attachment #742824 - Flags: review?(smontagu)
Attached patch Part c: Run themSplinter Review
Haven't automated this for now, given the small number of dirs.
Attachment #742825 - Flags: review?(jhammel)
Comment on attachment 742824 [details] [diff] [review]
Part b: Import HTML's reftests

Review of attachment 742824 [details] [diff] [review]:
-----------------------------------------------------------------

r=me for the bidi tests. As far as I remember, none of the changes in the version that I checked in to layout/reftests/bidi/dirAuto are vital, and some of them may even be deleterious (e.g. adding dir to the <bdi>s in dir_auto-contained-bdi-L.html).
Attachment #742824 - Flags: review?(smontagu) → review+
Attachment #742824 - Flags: review?(mounir) → review+
Comment on attachment 742823 [details] [diff] [review]
Part a: Update importTestsuite.py to import reftests as well

Review of attachment 742823 [details] [diff] [review]:
-----------------------------------------------------------------

lgtm

::: dom/imptests/importTestsuite.py
@@ +110,4 @@
>          for mochitest in d["mochitests"]:
>              shutil.copy("%s/%s" % (sourcedir, mochitest), "%s/test_%s" % (destdir, mochitest))
> +        for reftest in sorted(reftestfiles):
> +            shutil.copy("%s/%s" % (sourcedir, reftest), "%s/%s" % (destdir, reftest))

it'd be nice to combine these loops into a two layer nested loop (but not impportant)
Attachment #742823 - Flags: review?(jhammel) → review+
Attachment #742825 - Flags: review?(jhammel) → review+
Attached patch Part d: A few more (obsolete) — Splinter Review
Attachment #747401 - Flags: review?(mounir)
https://hg.mozilla.org/mozilla-central/rev/abe0099bab0e
https://hg.mozilla.org/mozilla-central/rev/92e51876e67d
https://hg.mozilla.org/mozilla-central/rev/02f7faabdbd9
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
Comment on attachment 747401 [details] [diff] [review]
Part d: A few more

Will do this in another bug.
Attachment #747401 - Attachment is obsolete: true
Attachment #747401 - Flags: review?(mounir)
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.