Closed Bug 799537 Opened 8 years ago Closed 8 years ago
move xpconnect chrome mochitests in tests/mochitest to tests/chrome
js/xpconnect/tests/mochitest/Makefile.in has a MOCHITEST_CHROME_FILES section, whereas chrome mochitests should be in js/xpconnect/tests/chrome/.
test_bug760131.html, previously referenced as a chrome test, is obviously a plain mochitest.
Attachment #669594 - Flags: review?(bobbyholley+bmo)
while at it, here is my attempt to sort MOCHITEST_FILES. I'll merge both patches appropriately.
Maybe we should just merge those folders.
Comment on attachment 669594 [details] [diff] [review] patch v0 test_bug760131.html is actually a chrome test, even though it's not a .xul file. So it shouldn't be moved to MOCHITEST_FILES.
Attachment #669594 - Flags: review?(bobbyholley+bmo) → review-
got it ! Here is a new patch with light sorting of MOCHITEST_FILES in tests/mochitest/Makefile.in included.
Hm, why do file_bug795275.html and such move up in the list? Can we keep everything in sorted order according to bug number? (that is to say, test_123 and file_123 together).
not easy to keep everything sorted according to bug number since file naming is not consistent, but here is another try.
Comment on attachment 673357 [details] [diff] [review] patch v2 -- another try test_bug361111.xul never gets removed from the other list here as far as I can tell. Maybe first do a patch that just moves the mochitest-chrome files, and then a second patch that does the sorting?
Attachment #673357 - Flags: review?(bobbyholley+bmo) → review-
Comment on attachment 673412 [details] [diff] [review] patch v3 -- part 1 - Move r=bholley
Attachment #673412 - Flags: review?(bobbyholley+bmo) → review+
Comment on attachment 673416 [details] [diff] [review] patch v3 -- part 2 - Sorting Erg, hrm - doesn't appear to be sorted? Stuff like file_mozMatchesSelector.html is mixed in with the other stuff. Basically, the stuff with the bug numbers in it should all be together, sorted by bug number, and everything else should be alphabetical. Sorry if that was unclear, and that this is turning out to be so finicky. :-(
Attachment #673416 - Flags: review?(bobbyholley+bmo) → review-
Oh I see. I previously put file_mozMatchesSelector.html btw. file_bug505915.html and test_bug553407.html because it was related to bug 533596. But a visual sorting makes sense.
Comment on attachment 673431 [details] [diff] [review] patch v4 -- part 2 - Sorting parfait ;-) r=bholley
Attachment #673431 - Flags: review?(bobbyholley+bmo) → review+
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
You need to log in before you can comment on or make changes to this bug.