From 2d978bf44a7a8126198cd97b43fe8a8deac1af4a Mon Sep 17 00:00:00 2001 From: kshitijk4poor <82637225+kshitijk4poor@users.noreply.github.com> Date: Sat, 20 Jun 2026 00:22:55 +0530 Subject: [PATCH] test(cron): make env-sanitize probe var deterministic next(iter(frozenset)) picked a different blocklist var each run (PYTHONHASHSEED-dependent), hurting reproducibility. sorted()[0] keeps the invariant-style assertion (any real blocklisted var) while making failures reproducible. Follow-up to salvaged PR #49207. --- tests/cron/test_cron_script.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tests/cron/test_cron_script.py b/tests/cron/test_cron_script.py index 783320728bd..ee02d043017 100644 --- a/tests/cron/test_cron_script.py +++ b/tests/cron/test_cron_script.py @@ -137,7 +137,9 @@ class TestRunJobScript: from tools.environments.local import _HERMES_PROVIDER_ENV_BLOCKLIST from cron.scheduler import _run_job_script - blocked_var = next(iter(_HERMES_PROVIDER_ENV_BLOCKLIST)) + # sorted() so the probed var is deterministic across runs + # (frozenset iteration order varies with PYTHONHASHSEED). + blocked_var = sorted(_HERMES_PROVIDER_ENV_BLOCKLIST)[0] monkeypatch.setenv(blocked_var, "must_not_leak") script = cron_env / "scripts" / "env_probe.py"