Closed Bug 454320 Opened 13 years ago Closed 13 years ago
top level test targets should support specifying a test path
something like |make mochitest-plain TEST_PATH=path/to/test/file.html| would be nice browser tests make things a bit complicated because the path is a bit different from chrome and mochitests though..
Figured out a way to make browser tests OK too.
Attachment #337577 - Flags: review?(ted.mielczarek)
The other option here is to support SOLO_FILE for everything but browser tests. That would keep us inline with check-one and check-interactive. We'd still want to use TEST_PATH for browser tests since you cannot run a single test.
Attachment #337577 - Flags: review?(ted.mielczarek) → review+
A SOLO_FILE addition would be nice, but we'd have to hack that into mochitest first. (I'm sure we could hack it into browser-chrome as well.)
(In reply to comment #3) > A SOLO_FILE addition would be nice, but we'd have to hack that into mochitest > first. (I'm sure we could hack it into browser-chrome as well.) It already works for chrome and mochitests. If you specify one test in TEST_PATH, it works just fine (and only runs that test)
Whiteboard: [has patch][needs review ted] → [has patch][has review]
Ah, I didn't know that. Anyway, I'm sure gavin would support adding that functionality to browserchrome.
(but that doesn't need to block this work)
Cool (since I'm using this as part of my presentation at Seneca since it is way easier to run one test this way than the current method)
Pushed to mozilla-central: http://hg.mozilla.org/mozilla-central/rev/07cb359d26bc
Whiteboard: [has patch][has review]
keep forgetting to mark fixed...
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Component: Testing → Build Config
QA Contact: testing → build-config
Comment on attachment 337577 [details] [diff] [review] v1.0 >diff --git a/browser/build.mk b/browser/build.mk >+# Browser tests live in a slightly different location, so we correct the path >+ifdef TEST_PATH >+BROWSER_TEST_PATH = --test-path=../browser/$(TEST_PATH) This "correction" isn't necessary, --test-paths are sourcedir-relative just like normal mochitests. It only happens to work because it always ends up being "browser/../browser".
You need to log in before you can comment on or make changes to this bug.