Skip to content

test(runtime-host): migrate queue tests to typed requests - #3792

Closed
liugddx wants to merge 1 commit into
apache:mainfrom
liugddx:fix/runtime-host-typed-queue-tests
Closed

test(runtime-host): migrate queue tests to typed requests#3792
liugddx wants to merge 1 commit into
apache:mainfrom
liugddx:fix/runtime-host-typed-queue-tests

Conversation

@liugddx

@liugddx liugddx commented Aug 25, 2026

Copy link
Copy Markdown
Member

Summary

Migrate the three stale RuntimeHostConnection alias calls left in execution-host-queue.test.ts after #3784 made typed request<K>() the sole operation API.

  • queryTurn -> request('turn.query', ...)
  • stopTurn -> request('turn.stop', ...)
  • startTurn -> request('turn.start', ...)

Inputs and assertions are unchanged. This restores the Runtime Host TypeScript build on current main and allows native CI lanes to reach their actual tests.

Verification

  • Core, Storage, MCP, Runtime, and Runtime Host builds passed locally.
  • lint, format, and git diff --check passed.
  • The complete queue suite currently cannot start on current main because fix(storage): converge duplicate session metadata migration 30 #3788 leaves message_admissions absent. That failure reproduces without this patch and is independent of the typed request migration.

Fixes #3790.

AI use

Codex located and migrated the three stale test calls and ran the listed checks.

@liugddx

liugddx commented Aug 25, 2026

Copy link
Copy Markdown
Member Author

Superseded by #3796. The typed-request test migration cannot pass required CI independently because current main also needs the duplicate migration-30 convergence from #3788; #3796 keeps the two root-cause fixes as separate commits in one mergeable repair.

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.

fix(runtime-host): migrate queue tests off removed connection aliases

1 participant