jason/mempalace
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

test(cli): prime monkeypatch undo so palace env doesn't leak

Browse Source 
monkeypatch.delenv(name, raising=False) on a missing key registers no
undo entry, so the env var cmd_init writes leaked into test_config_from_file
on Python 3.13 / Windows / macOS.

Prime the slot with setenv before delenv so teardown rolls back the write.
This commit is contained in:
Igor Lins e Silva
2026-05-03 06:27:37 -03:00
parent 01b3183e5d
commit a91b7ee5c2
1 changed files with 9 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/test_cli.py
+9 -3
View File
@@ -187,9 +187,15 @@ def test_cmd_init_honors_palace_flag(tmp_path, monkeypatch):
palace = tmp_path / "custom_palace" palace = tmp_path / "custom_palace"
# Make sure no leftover env var from another test leaks in — we want to # Make sure no leftover env var from another test leaks in — we want to
# verify that --palace ALONE drives the resolution. # verify that --palace ALONE drives the resolution. Prime monkeypatch's
monkeypatch.delenv("MEMPALACE_PALACE_PATH", raising=False) # undo list with setenv first so that the env var ``cmd_init`` writes
monkeypatch.delenv("MEMPAL_PALACE_PATH", raising=False) # below is rolled back at teardown (``delenv(raising=False)`` on a
# missing key registers no undo entry, which would leak into the next
# test).
monkeypatch.setenv("MEMPALACE_PALACE_PATH", "")
monkeypatch.setenv("MEMPAL_PALACE_PATH", "")
monkeypatch.delenv("MEMPALACE_PALACE_PATH")
monkeypatch.delenv("MEMPAL_PALACE_PATH")
args = argparse.Namespace( args = argparse.Namespace(
dir=str(project), dir=str(project),