sdk: tests: isolating xdg_state_home to address persistence files causing flaky tests

Author: lucaspar

sdk/tests/conftest.py

@@ -45,6 +45,17 @@ def pytest_runtest_setup(item) -> None:
 # ==== fixtures
 
 
+@pytest.fixture(autouse=True)
+def isolate_xdg_state(tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
+    """Redirect XDG_STATE_HOME to a per-test tmpdir.
+
+    Prevents persistence files from accumulating across pytest sessions when
+    pytest recycles its directory numbers (pytest-0, -1, -2, then back to -0),
+    which would cause the same local_root hash to reuse stale upload records.
+    """
+    monkeypatch.setenv("XDG_STATE_HOME", str(tmp_path / "state"))
+
+
 @pytest.fixture
 def anyio_backend():
     return "asyncio"

sdk/tests/test_uploads.py

@@ -1091,6 +1091,9 @@ async def test_persistence_full_workflow(tmp_path: Path, client: Client) -> None
     file1 = root / "file1.txt"
     await file1.write_text("content1", encoding="utf-8")
 
+    file_exists = await file1.exists()
+    assert file_exists, "Test setup failed: file1 was not created successfully"
+
     workload = UploadWorkload(client=client, local_root=Path(root), persist_state=True)
 
     assert workload._persistence_manager is not None, (