Skip to content

test(rivetkit): import vi for actor-queue waitFor#4825

Draft
NathanFlurry wants to merge 1 commit intodriver-tests-fixes/restore-sleep-wait-before-integrity-checkfrom
driver-tests-fixes/queue-test-vi-import
Draft

test(rivetkit): import vi for actor-queue waitFor#4825
NathanFlurry wants to merge 1 commit intodriver-tests-fixes/restore-sleep-wait-before-integrity-checkfrom
driver-tests-fixes/queue-test-vi-import

Conversation

@NathanFlurry
Copy link
Copy Markdown
Member

@NathanFlurry NathanFlurry commented Apr 28, 2026

Stack Context

Part of the /driver-test-runner fix stack. See parent PR driver-tests-fixes/get-params-failed-retry (#4823).

Why?

tests/driver/actor-queue.test.ts uses vi.waitFor in the test "drains many-queue child actors created from run handlers while connected" but the file's vitest import only pulls in { describe, expect, test }. At runtime this throws ReferenceError: vi is not defined and the test fails.

Add vi to the import.

actor-queue.test.ts uses vi.waitFor in 'drains many-queue child actors
created from run handlers while connected' but the import was missing
vi from the vitest barrel, causing a ReferenceError at runtime. Add vi
to the import.
@railway-app
Copy link
Copy Markdown

railway-app Bot commented Apr 28, 2026

🚅 Deployed to the rivet-pr-4825 environment in rivet-frontend

Service Status Web Updated (UTC)
website 😴 Sleeping (View Logs) Web Apr 28, 2026 at 3:30 am
frontend-cloud 😴 Sleeping (View Logs) Web Apr 28, 2026 at 3:29 am
frontend-inspector ❌ Build Failed (View Logs) Web Apr 28, 2026 at 3:24 am
ladle ✅ Success (View Logs) Web Apr 28, 2026 at 3:22 am
mcp-hub ✅ Success (View Logs) Web Apr 28, 2026 at 3:21 am
kitchen-sink ❌ Build Failed (View Logs) Web Apr 28, 2026 at 3:20 am

Copy link
Copy Markdown
Member Author

NathanFlurry commented Apr 28, 2026

@github-actions
Copy link
Copy Markdown
Contributor

Preview packages published to npm

Install with:

npm install rivetkit@pr-4825

All packages published as 0.0.0-pr.4825.d66dff9 with tag pr-4825.

Engine binary is shipped via @rivetkit/engine-cli on linux-x64-musl, linux-arm64-musl, darwin-x64, and darwin-arm64. Windows users should use the release installer or set RIVET_ENGINE_BINARY.

Docker images:

docker pull rivetdev/engine:slim-d66dff9
docker pull rivetdev/engine:full-d66dff9
Individual packages
npm install rivetkit@pr-4825
npm install @rivetkit/react@pr-4825
npm install @rivetkit/rivetkit-napi@pr-4825
npm install @rivetkit/workflow-engine@pr-4825

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