Cleaning up, if there was a value set for PYTEST_VERSION before pytest run by the calling process, we set it back after pytest run
Co-authored-by: Bruno Oliveira <bruno@soliv.dev>
This commit is contained in:
parent
d6d59fed41
commit
2f8e436213
|
@ -153,6 +153,7 @@ def main(
|
||||||
:returns: An exit code.
|
:returns: An exit code.
|
||||||
"""
|
"""
|
||||||
try:
|
try:
|
||||||
|
old_pytest_version = os.environ.get("PYTEST_VERSION")
|
||||||
os.environ["PYTEST_VERSION"] = __version__
|
os.environ["PYTEST_VERSION"] = __version__
|
||||||
try:
|
try:
|
||||||
config = _prepareconfig(args, plugins)
|
config = _prepareconfig(args, plugins)
|
||||||
|
@ -189,7 +190,10 @@ def main(
|
||||||
tw.line(f"ERROR: {msg}\n", red=True)
|
tw.line(f"ERROR: {msg}\n", red=True)
|
||||||
return ExitCode.USAGE_ERROR
|
return ExitCode.USAGE_ERROR
|
||||||
finally:
|
finally:
|
||||||
os.environ.pop("PYTEST_VERSION", None)
|
if old_pytest_version is None:
|
||||||
|
os.environ.pop("PYTEST_VERSION", None)
|
||||||
|
else:
|
||||||
|
os.environ["PYTEST_VERSION"] = old_pytest_version
|
||||||
|
|
||||||
|
|
||||||
def console_main() -> int:
|
def console_main() -> int:
|
||||||
|
|
|
@ -1097,6 +1097,7 @@ def test_outcome_exception_bad_msg() -> None:
|
||||||
|
|
||||||
|
|
||||||
def test_pytest_version_env_var(pytester: Pytester, monkeypatch: MonkeyPatch) -> None:
|
def test_pytest_version_env_var(pytester: Pytester, monkeypatch: MonkeyPatch) -> None:
|
||||||
|
os.environ["PYTEST_VERSION"] = "old version"
|
||||||
pytester.makepyfile(
|
pytester.makepyfile(
|
||||||
"""
|
"""
|
||||||
import pytest
|
import pytest
|
||||||
|
@ -1109,4 +1110,4 @@ def test_pytest_version_env_var(pytester: Pytester, monkeypatch: MonkeyPatch) ->
|
||||||
)
|
)
|
||||||
result = pytester.runpytest_inprocess()
|
result = pytester.runpytest_inprocess()
|
||||||
assert result.ret == ExitCode.OK
|
assert result.ret == ExitCode.OK
|
||||||
assert "PYTEST_VERSION" not in os.environ
|
assert os.environ["PYTEST_VERSION"] == "old version"
|
||||||
|
|
Loading…
Reference in New Issue