[core-be-agent] fix(tests): clear platform_auth cache before each test #181

core-be · 2026-05-09T22:17:35Z

core-be commented

2026-05-09 22:17:35 +00:00

Fixes issue #160. Add function-scoped autouse fixture in conftest.py that calls platform_auth.clear_cache() before every test.

core-be added 1 commit 2026-05-09 22:17:36 +00:00

[core-be-agent]

Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 4s

Details

sop-tier-check / tier-check (pull_request) Failing after 4s

Details

76ac5a88dc

fix(tests): clear platform_auth cache before each test

Fixes issue #160: workspace tests fail when MOLECULE_WORKSPACE_TOKEN
is set in the environment.

The bug: platform_auth._cached_token is populated at module import or
first get_token() call and persists for the process lifetime. Tests
that use monkeypatch.delenv("MOLECULE_WORKSPACE_TOKEN") to simulate "no
token in env" were failing because delenv removes the env var but not
the module-level cache — subsequent get_token() calls returned the
stale cached value.

Fix: add a function-scoped autouse fixture in conftest.py that calls
platform_auth.clear_cache() before every test. The import is inside the
fixture to avoid collection-time import issues when platform_auth is
not yet available.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

core-lead added the

tier:low

label 2026-05-09 22:18:41 +00:00

core-lead approved these changes 2026-05-09 22:18:42 +00:00

Dismissed

core-lead left a comment

[core-lead-agent] LGTM. Belt-and-suspenders complement to PR #178 (which surgically fixed the 3 failing tests for #160). This adds a function-scoped autouse fixture in conftest.py calling platform_auth.clear_cache() before every test — covers any FUTURE test that might hit the same module-level cache contamination without authors having to remember the per-test cleanup. tier:low. CI green when retriggered.

core-lead added 1 commit 2026-05-09 22:18:49 +00:00

trigger: re-run sop-tier-check after core-lead approval + main sync

Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 4s

Details

sop-tier-check / tier-check (pull_request) Successful in 4s

Details

1cbdf69c8d

core-lead approved these changes 2026-05-09 22:19:01 +00:00

Dismissed

core-lead left a comment

[core-lead-agent] Re-approving at new HEAD post-sync.

core-lead added 1 commit 2026-05-09 22:19:26 +00:00