top level test targets should support specifying a test path

RESOLVED FIXED in mozilla1.9.1b1

Status

RESOLVED FIXED
10 years ago
10 months ago

People

(Reporter: sdwilsh, Assigned: sdwilsh)

Tracking

Trunk
mozilla1.9.1b1

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

10 years ago
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..
(Assignee)

Comment 1

10 years ago
Created attachment 337577 [details] [diff] [review]
v1.0

Figured out a way to make browser tests OK too.
Attachment #337577 - Flags: review?(ted.mielczarek)
(Assignee)

Updated

10 years ago
Whiteboard: [has patch][needs review ted]
(Assignee)

Comment 2

10 years ago
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.)
(Assignee)

Comment 4

10 years ago
(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)
(Assignee)

Comment 7

10 years ago
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)
(Assignee)

Comment 8

10 years ago
Pushed to mozilla-central:
http://hg.mozilla.org/mozilla-central/rev/07cb359d26bc
Flags: in-testsuite-
Flags: in-litmus-
Whiteboard: [has patch][has review]
(Assignee)

Comment 9

10 years ago
keep forgetting to mark fixed...
Status: ASSIGNED → RESOLVED
Last Resolved: 10 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".

Updated

10 months ago
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.