Skip to content

fix(tests): isolate Vercel sandbox SDK fixture - #74586

Closed
joelbrilliant wants to merge 1 commit into
NousResearch:mainfrom
joelbrilliant:fix/vercel-sandbox-test-lazy-deps
Closed

joelbrilliant wants to merge 1 commit into
NousResearch:mainfrom
joelbrilliant:fix/vercel-sandbox-test-lazy-deps

Conversation

@joelbrilliant

Copy link
Copy Markdown
Contributor

Problem

Current main fails all 16 Vercel sandbox environment tests when CI disables lazy installs.

The test fixture injects a fake vercel SDK into sys.modules, but the production lazy dependency loader still checks for the real installed distribution and raises before the fake SDK can be exercised.

Fix

Stub tools.lazy_deps.ensure inside the fake-SDK fixture. Production dependency handling is unchanged.

Verification

  • Reproduced on pristine main: 16 failed
  • Patched Vercel sandbox tests: 16 passed
  • Vercel sandbox and related lazy dependency suites: 83 passed
  • Ruff 0.15.10: clean
  • git diff --check: clean

Signed-off-by: joelbrilliant <joelbrilliant1@gmail.com>
@joelbrilliant

Copy link
Copy Markdown
Contributor Author

Superseded by #74576, which landed the same root-cause fix on main as 8eb06e75b, with the stub scoped directly to _ensure_vercel_sdk.

Closing this duplicate. Its full CI run was green, and I am rebasing the affected feature PRs onto the upstream fix now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant