Closed Bug 755427 Opened 12 years ago Closed 12 years ago

Marionette tests don't find emulator executable in OSX

Categories

(Remote Protocol :: Marionette, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla15

People

(Reporter: marshall, Assigned: marshall)

Details

Attachments

(1 file, 1 obsolete file)

In OSX, Marionette fails when attempting to launch the AOSP emulator from the B2G repo.

It looks like emulator.py is assuming the binaries will be in linux-x86, which is only true in a Linux build environment.

Python trace:

Using /Volumes/Boot2Gecko/B2G/objdir-gecko/marionette_venv/mozbase/mozinfo
Finished processing dependencies for marionette==0.2
running webserver on http://10.250.19.176:55544/
Traceback (most recent call last):
  File "runtests.py", line 409, in <module>
    runner.run_tests(tests, testtype=options.type)
  File "runtests.py", line 249, in run_tests
    self.run_test(test, testtype)
  File "runtests.py", line 266, in run_test
    self.start_marionette()
  File "runtests.py", line 207, in start_marionette
    logcat_dir=self.logcat_dir)
  File "/Volumes/Boot2Gecko/B2G/gecko/testing/marionette/client/marionette/marionette.py", line 129, in __init__
    self.emulator.start()
  File "/Volumes/Boot2Gecko/B2G/gecko/testing/marionette/client/marionette/emulator.py", line 232, in start
    self._check_for_b2g()
  File "/Volumes/Boot2Gecko/B2G/gecko/testing/marionette/client/marionette/emulator.py", line 83, in _check_for_b2g
    self._check_file(self.binary)
  File "/Volumes/Boot2Gecko/B2G/gecko/testing/marionette/client/marionette/emulator.py", line 103, in _check_file
    'B2G_HOME correctly?') % filePath)
Exception: File not found: /Volumes/Boot2Gecko/B2G/out/host/linux-x86/bin/emulator; did you pass the B2G home directory as the homedir parameter, or set B2G_HOME correctly?
Assignee: nobody → marshall
Version: unspecified → Trunk
Attachment #624250 - Attachment is obsolete: true
Comment on attachment 624254 [details] [diff] [review]
Fix for hardcoded emulator linux paths, added --emulator-binary arg to runtests.py

Review of attachment 624254 [details] [diff] [review]:
-----------------------------------------------------------------

Awesome, thanks for the patch! Looks good to me, and I'm adding jgriffin here so he can take a look as well.
Attachment #624254 - Flags: review?(mdas)
Attachment #624254 - Flags: review+
Attachment #624254 - Flags: feedback?(jgriffin)
Comment on attachment 624254 [details] [diff] [review]
Fix for hardcoded emulator linux paths, added --emulator-binary arg to runtests.py

Review of attachment 624254 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me too.
Attachment #624254 - Flags: feedback?(jgriffin) → feedback+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/f81ffb3fba84
Status: NEW → RESOLVED
Closed: 12 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Product: Testing → Remote Protocol
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: