Closed
Bug 672538
Opened 10 years ago
Closed 10 years ago
remotereftest.py and runtestsremote.py reference dm_auto before initialized
Categories
(Testing :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla8
People
(Reporter: gbrown, Assigned: gbrown)
Details
Attachments
(1 file)
2.94 KB,
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
If I execute: MOZ_HOST_BIN="../objdir-firefox/dist/bin/" make reftest-remote I get a failure: Traceback (most recent call last): File "_tests/reftest/remotereftest.py", line 447, in <module> package set: org.mozilla.fennec_unofficial main() File "_tests/reftest/remotereftest.py", line 396, in main dm = dm_auto NameError: global name 'dm_auto' is not defined /bin/sh: @errors=: not found reftest-remote passed Similarly, if I execute: MOZ_HOST_BIN="../objdir-firefox/dist/bin/" make mochitest-remote I get: Traceback (most recent call last): File "_tests/testing/mochitest/runtestsremote.py", line 351, in <module> main() File "_tests/testing/mochitest/runtestsremote.py", line 318, in main dm = dm_auto NameError: global name 'dm_auto' is not defined make: *** [mochitest-remote] Error 1
![]() |
Assignee | |
Updated•10 years ago
|
Summary: remotereftest.py and reference dm_auto before initialized → remotereftest.py and runtestsremote.py reference dm_auto before initialized
![]() |
Assignee | |
Updated•10 years ago
|
Assignee: nobody → gbrown
![]() |
Assignee | |
Comment 1•10 years ago
|
||
I also found that some directories were not being created. For example, if <root>/tests does not exist at the start of a mochitest, the profile directory is not created, and all tests fail. This is caused by a recent change to devicemanagerADB.pushDir (that I put in!); simple fix on the way...
![]() |
Assignee | |
Comment 2•10 years ago
|
||
With these simple changes I can successfully run: cd objdir-droid MOZ_HOST_BIN="../objdir-firefox/dist/bin/" make reftest-remote MOZ_HOST_BIN="../objdir-firefox/dist/bin/" make mochitest-remote (devroot of /mnt/sdcard/tests, with devroot non-existent before test; not all tests pass, but the test suites appear to run to completion).
Attachment #546828 -
Flags: review?(jmaher)
![]() |
Assignee | |
Comment 3•10 years ago
|
||
I just realized/recalled that the dm_auto vs dm_none issue is covered by jmaher's patch at https://bug669549.bugzilla.mozilla.org/attachment.cgi?id=545702, which never landed. Was that an oversight? Do we also want to add the optional run-as package setting now?
Comment 4•10 years ago
|
||
(In reply to comment #3) > Do we also want to add the optional run-as > package setting now? Yes and no. I think we want to change that to an optional "run as root" flag so we can test release builds on rooted phones
Comment 5•10 years ago
|
||
well, we cannot specify org.mozilla.fennec with a nightly build, so that patch is not very useful. I think if we have packageName=org.mozilla.fennec, we shouldn't use it, only use the packageName of org.mozilla.fennec_unofficial.
Comment 6•10 years ago
|
||
Comment on attachment 546828 [details] [diff] [review] patch for review Review of attachment 546828 [details] [diff] [review]: ----------------------------------------------------------------- looks good assuming the "/x" is intended. ::: build/mobile/devicemanagerADB.py @@ +96,5 @@ > # contains symbolic links, the links are pushed, rather than the linked > # files; we push file-by-file to get around this limitation > try: > + if (not self.dirExists(remoteDir)): > + self.mkDirs(remoteDir+"/x") why the +"/x" here?
Attachment #546828 -
Flags: review?(jmaher) → review+
![]() |
Assignee | |
Comment 7•10 years ago
|
||
devicemanager.mkDirs takes a *filename* as a parameter, and creates all the directories in the path to that filename. So if you want to create directory /d1/d2/d3, you need to mkDirs("/d1/d2/d3/something"). (I don't particularly approve of the interface, but that is the way I found it!)
![]() |
Assignee | |
Comment 8•10 years ago
|
||
(In reply to comment #5) Let's defer the packageName-setting/run-as/root-access issues to another bug, if/when necessary, and check in this patch as it is.
Keywords: checkin-needed
Comment 9•10 years ago
|
||
agreed. The more people that can use this the better.
Comment 10•10 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/ff75a2a2e771
Status: NEW → RESOLVED
Closed: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
You need to log in
before you can comment on or make changes to this bug.
Description
•