particle-os/debian-forge
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

test: use del os.environ[] instead of os.unsetenv (thanks Simon)

Browse source 
Simon discovered that `os.unsetenv()` will not remove the env
from os.environ so this commit fixes this.

Thanks for figuring this out!
This commit is contained in:
Michael Vogt 2024-03-08 17:50:58 +01:00 • committed by Ondřej Budai
parent 2efe2de09c
commit 945209a80c
1 changed files with 6 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

8
test/run/test_exports.py
View file

@ -78,12 +78,16 @@ def test_exports_normal(osb, tmp_path, jsondata, testing_libdir): # pylint: dis
@pytest.mark.skipif(os.getuid() != 0, reason="root-only")
def test_exports_with_force_no_preserve_owner(osb, tmp_path, jsondata, testing_libdir):
k = "OSBUILD_EXPORT_FORCE_NO_PRESERVE_OWNER"
with contextlib.ExitStack() as cm:
k = "OSBUILD_EXPORT_FORCE_NO_PRESERVE_OWNER"
os.environ[k] = "1"
cm.callback(os.unsetenv, k)
def _delenv():
del os.environ[k]
cm.callback(_delenv)
osb.compile(jsondata, output_dir=tmp_path, exports=["image"], libdir=testing_libdir)
expected_export = tmp_path / "image/foo.img"
assert expected_export.exists()
assert expected_export.stat().st_uid == 0
assert k not in os.environ