Closed
Bug 636450
Opened 13 years ago
Closed 13 years ago
add a --host-os option for remote testing scripts
Categories
(Testing :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jmaher, Assigned: jmaher)
Details
(Whiteboard: [mobile_unittests])
Attachments
(1 file, 1 obsolete file)
5.91 KB,
patch
|
cmtalbert
:
review+
|
Details | Diff | Splinter Review |
in order to run tests on a remote device, we need to have a host controller (usually osx, but could be linux) act as a webserver with xpcshell+httpd.js. This requires us downloading tests and binaries for the host OS platform, as well as tests for the remote device. This gets confusing quickly, but one problem we have found when running this way is we run scripts from the device tests package and they are configured for the device OS (usually linux). In order to run the webserver, we need to adjust things like the [DY]LD_LIBRARY_PATH. What I have here is a --host-os option that we can pass into the remote test runner scripts. This will tell automation.py and other scripts what the host OS is so we can setup the server(s) properly.
Assignee | ||
Comment 1•13 years ago
|
||
this patch adds the --host-os option to runtestsremote.py (mochitest) and remotereftests.py (reftest). It also fixes some basic devicemanager assignment issues I ran into while testing this on windows
Assignee: nobody → jmaher
Attachment #514793 -
Flags: review?(ctalbert)
Assignee | ||
Updated•13 years ago
|
Whiteboard: [mobile_unittests]
Comment 2•13 years ago
|
||
Any chance you can make "mac" and "macosx" permissible as well? It'd simplify the Buildbot integration.
Assignee | ||
Comment 3•13 years ago
|
||
updated patch to auto detect the host os by using sys.platform. I have tested on win32 and linux successfully.
Attachment #514793 -
Attachment is obsolete: true
Attachment #514793 -
Flags: review?(ctalbert)
Attachment #514814 -
Flags: review?(ctalbert)
Comment on attachment 514814 [details] [diff] [review] detect the host os with sys.platform (1.0) This looks good. I'm not sure when sys.platform returns "mac" or "linux2" but I think this is the correct approach until we can further refactor the webserver bits into their own package/classes as we have discussed. r+
Attachment #514814 -
Flags: review?(ctalbert) → review+
Assignee | ||
Comment 5•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/pushloghtml?changeset=6da15ed37f37
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Component: New Frameworks → General
You need to log in
before you can comment on or make changes to this bug.
Description
•