805297 - Marionette doesn't recognize relative test path from current working directory

Status: RESOLVED FIXED

(Remote Protocol :: Marionette, defect)

Description

[Henrik Skupin [:whimboo][⌚️UTC+2]]

When I was running the first B2G Marionette tests via ./test.sh (B2G repository) I have noticed that Marionette does not work when you specify a relative path to the test or the manifest. Reason is that we do a couple of cwd changes in the venv_test.sh script to get everything setup and installed, and the relative path of the test gets added to the marionette folder. But it should be from the cwd of the user.

The patch I have works fine and leaves the tree green:
https://tbpl.mozilla.org/?tree=Try&rev=cc7dfeeaea34

But sadly we have a strange delay in getting the development version of Marionette installed. I will investigate that before uploading a patch.

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Patch v1

With this patch we will ensure to keep the CWD of the user when the script is called via B2G/test.sh. As noted in the patch there is one exception when we have to change the CWD to install Marionette in develop mode. Not doing that will cause a hang.

Comment 2

[Jonathan Griffin (:jgriffin)]

Comment on attachment 675265 [details] [diff] [review]
Patch v1

Looks good, thanks!

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+2]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/48d71f461152

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/48d71f461152

Comment 5

[Jonathan Griffin (:jgriffin)]

https://hg.mozilla.org/projects/ash/rev/6ae05c782555

Comment 6

[Jonathan Griffin (:jgriffin)]

http://hg.mozilla.org/releases/mozilla-aurora/rev/dda05e842e90