Skip to content

fix(tests): await async getExecutor() in gemini-web-image-retirement test - #11868

Merged
diegosouzapw merged 2 commits into
release/v3.8.51from
fix/v3851-gemini-retirement-async-getexecutor
Aug 28, 2026
Merged

diegosouzapw merged 2 commits into
release/v3.8.51from
fix/v3851-gemini-retirement-async-getexecutor

Conversation

@diegosouzapw

Copy link
Copy Markdown
Owner

Summary

Test plan

  • node --import tsx/esm --test tests/unit/gemini-web-image-retirement.test.ts — 4/4 passing.

…test

The R0.3 executor-registry refactor (#11220) made getExecutor() async;
this new test (#11708) predates that and called it synchronously,
producing a TypeError (getExecutor(...).getProvider is not a function)
instead of the intended assertion.
@diegosouzapw
diegosouzapw merged commit 700a735 into release/v3.8.51 Aug 28, 2026
9 of 12 checks passed
muhamadgalihsaputra pushed a commit to niyatna/NiyatnaRoute that referenced this pull request Sep 27, 2026
…test (diegosouzapw#11868)

Direct follow-up fixing a bug my own /merge-batch process introduced. Test-only change, focused test passing.
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.

2 participants