Inherit env vars

Author: ejba

pre_commit_maven/utils/maven.py

@@ -35,11 +35,12 @@ def get_maven_path(cwd: str, shell_runner=shell):
     return path
 
 
-def execute(args: list, cwd: str, shell_runner=shell):
+def execute(args: list, cwd: str, shell_runner=shell, env=os.environ.copy()):
     assert args is not None and len(args) > 0, "args not specified"
 
     cmd = [get_maven_path(cwd, shell_runner)] + MAVEN_CLI_OPTS + args
-    env = {"MAVEN_OPTS": " ".join(MAVEN_OPTS)}
+
+    env["MAVEN_OPTS"] = " ".join(MAVEN_OPTS)
     return shell_runner.execute(cmd, cwd=cwd, env=env)
 
 

tests/utils/test_maven.py

@@ -25,7 +25,7 @@ def test_given_goals_then_run_maven(mocker):
     shell_runner.execute.return_value = ExecutionResult(0, "", "")
 
     # when
-    execute(["clean"], ".", shell_runner)
+    execute(["clean"], ".", shell_runner, {})
 
     # then
     shell_runner.exists_file.assert_called_once_with("./mvnw")
@@ -41,7 +41,7 @@ def test_given_mvnw_exists_then_run_mvnw(mocker):
     shell_runner.execute.return_value = ExecutionResult(0, "", "")
 
     # when
-    execute(["clean"], ".", shell_runner)
+    execute(["clean"], ".", shell_runner, {})
 
     # then
     shell_runner.exists_file.assert_called_once_with("./mvnw")