tox plugin: Pass through lower-case proxy environment variables

launchpad-buildd sets `http_proxy` and `https_proxy`, but tox 3.24.5 only passes through the upper-case versions. This is fixed in development versions of tox (tox-dev/tox#2372), but work around it for now.
jugmac00 · Apr 8, 2022 · f46a8dc · f46a8dc
1 parent 043c932
commit f46a8dc
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 9 deletions.
diff --git a/NEWS.rst b/NEWS.rst
@@ -5,7 +5,9 @@ Version history
 0.0.7   (unreleased)
 ====================
 
-- Nothing yet.
+- tox plugin: Work around https://github.com/tox-dev/tox/issues/2372 by
+  telling ``tox`` to pass through lower-case ``http_proxy`` and
+  ``https_proxy`` environment variables.
 
 0.0.6   (2022-04-05)
 ====================

diff --git a/lpcraft/plugin/tests/test_plugins.py b/lpcraft/plugin/tests/test_plugins.py
@@ -60,7 +60,7 @@ def test_tox_plugin(self, mock_get_host_architecture, mock_get_provider):
                         "apache2",
                     ],
                     cwd=PosixPath("/root/project"),
-                    env={"PLUGIN": "tox"},
+                    env={"TOX_TESTENV_PASSENV": "http_proxy https_proxy"},
                     stdout=ANY,
                     stderr=ANY,
                 ),
@@ -73,7 +73,7 @@ def test_tox_plugin(self, mock_get_host_architecture, mock_get_provider):
                         "python3 -m pip install tox==3.24.5; tox",
                     ],
                     cwd=PosixPath("/root/project"),
-                    env={"PLUGIN": "tox"},
+                    env={"TOX_TESTENV_PASSENV": "http_proxy https_proxy"},
                     stdout=ANY,
                     stderr=ANY,
                 ),
@@ -152,14 +152,14 @@ def test_run_command_from_configuration_takes_precedence(
                         "apache2",
                     ],
                     cwd=PosixPath("/root/project"),
-                    env={"PLUGIN": "tox"},
+                    env={"TOX_TESTENV_PASSENV": "http_proxy https_proxy"},
                     stdout=ANY,
                     stderr=ANY,
                 ),
                 call(
                     ["bash", "--noprofile", "--norc", "-ec", "ls"],
                     cwd=PosixPath("/root/project"),
-                    env={"PLUGIN": "tox"},
+                    env={"TOX_TESTENV_PASSENV": "http_proxy https_proxy"},
                     stdout=ANY,
                     stderr=ANY,
                 ),

diff --git a/lpcraft/plugins/plugins.py b/lpcraft/plugins/plugins.py
@@ -34,10 +34,10 @@ def lpcraft_execute_run(self) -> str:
 
     @hookimpl  # type: ignore
     def lpcraft_set_environment(self) -> dict[str, str | None]:
-        # XXX jugmac00 2021-12-17: this was added to raise coverage and is not
-        # necessary. Let's remove this once we have a plugin which actually
-        # needs to set environment variables.
-        return {"PLUGIN": "tox"}
+        # Work around https://github.com/tox-dev/tox/issues/2372: without
+        # this, tox won't pass through the lower-case proxy environment
+        # variables set by launchpad-buildd.
+        return {"TOX_TESTENV_PASSENV": "http_proxy https_proxy"}
 
 
 @register(name="pyproject-build")