1299441 - Marionette should setup test environment based on mozrunner.utils.test_environment()

Status: NEW

(Testing :: Marionette Client and Harness, defect, P3)

Description

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

Right now Marionette sets up different environment variables on its own. It means that lots others we usually make use of in other test harnesses are missing. One example are variables for ASAN builds.

What we might want to do without having to add all this code again, is to call test_environment() from mozrunner.utils:

https://searchfox.org/mozilla-central/rev/049f3e8d328c4b6bac9bf4b27309541a2e913686/testing/mozbase/mozrunner/mozrunner/utils.py#90

We should check how this works for tests as run in-tree, via test-packages, and for other consumers.

One issue might become the default value for MOZ_DISABLE_NONLOCAL_CONNECTIONS, which we would have to override for now.

Comment 1

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

Eventually we should make use of the environment in the Marionette harness so that we don't have to set the required environment variables in different places like mach and mozharness. Previously we weren't able to because the harness was released on PyPI and in use by folks outside of our tree. But this has been changed since we no longer release to PyPI.

One thing to note is that the firefox-ui-functional remote tests would still need the capability to opt-out or override certain environment variables like the MOZ_DISABLE_NONLOCAL_CONNECTIONS one.

For now this is not a priority.