Closed Bug 956862 Opened 6 years ago Closed 6 years ago

Move various mochitests to manifests

Categories

(Firefox Build System :: General, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
mozilla29

People

(Reporter: Ms2ger, Assigned: Ms2ger)

References

Details

Attachments

(1 file)

Attached patch Patch v1Splinter Review
44 files changed, 82 insertions(+), 273 deletions(-)
Attachment #8356238 - Flags: review?(mshal)
Comment on attachment 8356238 [details] [diff] [review]
Patch v1

>diff --git a/content/html/content/test/moz.build b/content/html/content/test/moz.build
>--- a/content/html/content/test/moz.build
>+++ b/content/html/content/test/moz.build
>-TEST_DIRS += ['forms']

I think you want to rm content/html/content/test/forms/moz.build now.

>--- a/dom/browser-element/mochitest/priority/Makefile.in
>+++ /dev/null
>@@ -1,39 +0,0 @@
>-# Note: ../browserElementTestHelpers.js makes all tests in this directory OOP,
>-# because testing the process-priority manager without OOP frames doesn't make
>-# much sense.

Is the above comment worth moving over to mochitest.ini?

Everything else looks good to me!
Attachment #8356238 - Flags: review?(mshal) → review+
https://hg.mozilla.org/mozilla-central/rev/f2d8bb102e05
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.