Closed
Bug 987746
Opened 10 years ago
Closed 9 years ago
Create a mach target for running gaia-ui tests
Categories
(Firefox OS Graveyard :: Gaia::UI Tests, defect, P3)
Firefox OS Graveyard
Gaia::UI Tests
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: ahal, Unassigned)
References
Details
(Keywords: ateam-b2g-task)
Attachments
(1 file)
6.26 KB,
patch
|
zcampbell
:
feedback-
|
Details | Diff | Splinter Review |
I needed to run gaia-ui tests to test our a marionette refactor I'm working on, and figured it could benefit from a mach target. Seeing as the b2g-desktop mochitest/reftests also require a gaia profile, it didn't seem like a stretch to create a gaiatest command.
Reporter | ||
Comment 1•10 years ago
|
||
This target should work both in m-c with b2g_desktop and in the B2G repo with a device. There is one problem with it, and that's that 'virtualenv_manager.call_setup()' uses the system python executable even after a call to 'virtualenv_manager.activate()'. :gps, is this a bug? Or is there another way I can install a module via setup.py with virtualenvmanager? Zac, could you take a look at this and let me know if you would change anything? I especially want to know if you would add anything to the default 'testvars.json' or if there are any other command line options you think would be useful to pass on (currently this just does --profile and --testvars).
Assignee: nobody → ahalberstadt
Status: NEW → ASSIGNED
Attachment #8396430 -
Flags: feedback?
Flags: needinfo?(gps)
Reporter | ||
Updated•10 years ago
|
Attachment #8396430 -
Flags: feedback? → feedback?(zcampbell)
Comment 2•10 years ago
|
||
I think call_setup() using sys.executable() is a bug. If the virtualenv is activated, it should be using self.python_path. Arguably, self.activate() should monkeypatch sys.path to the virtualenv's Python binary. But I think I ran into unexpected problems with this when I tried it before. Hopefully bug 985141 can address these problems. If you want to work on that, power to you :)
Flags: needinfo?(gps)
Reporter | ||
Comment 3•10 years ago
|
||
I think bug 985141 is a bit of a bigger scope than I have time for, maybe in the future :). I'll file a separate bug to get call_setup() using the proper executable.
Updated•10 years ago
|
Keywords: ateam-b2g-task
Updated•10 years ago
|
Priority: -- → P3
Comment 4•10 years ago
|
||
Comment on attachment 8396430 [details] [diff] [review] Create gaiatest mach command Review of attachment 8396430 [details] [diff] [review]: ----------------------------------------------------------------- I'm not the right person to review this because I am not familiar with mach commands.
Attachment #8396430 -
Flags: feedback?(zcampbell)
Reporter | ||
Comment 5•10 years ago
|
||
I was mostly just looking for feedback around the cli options I was passing in and the dummy testvars.json I made.
Comment 6•10 years ago
|
||
Comment on attachment 8396430 [details] [diff] [review] Create gaiatest mach command f? me again
Attachment #8396430 -
Flags: feedback?(zcampbell)
Comment 7•10 years ago
|
||
Comment on attachment 8396430 [details] [diff] [review] Create gaiatest mach command I didn't run it, I only reviewed the config. It looks ok to me except for the timezone settings in testvars; on desktopb2g it can pick that up from the host computer. We really only that on device where we wipe the profile and need to sync it back to the host computer.
Attachment #8396430 -
Flags: feedback?(zcampbell) → feedback-
QA Whiteboard: [fxosqa-auto-backlog-]
Reporter | ||
Updated•9 years ago
|
Assignee: ahalberstadt → nobody
Status: ASSIGNED → NEW
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•