Skip to content

test: pin unknown-venue path when registry service absent - #531

Merged
mfw78 merged 2 commits into
dev/m1from
test/510-unknown-venue-no-registry-service
Jul 23, 2026
Merged

test: pin unknown-venue path when registry service absent#531
mfw78 merged 2 commits into
dev/m1from
test/510-unknown-venue-no-registry-service

Conversation

@mfw78

@mfw78 mfw78 commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

What

Add one integration test to crates/videre-host/tests/platform.rs pinning the service-lookup-miss UnknownVenue path in client.rs, where services.get::<VenueRegistry>(NAMESPACE) returns None.

A minimal ClientWithoutRegistry(Videre) Extension wrapper forwards every face boot_single consults (namespace, capabilities, link, manifest_sections, subscriptions, admit_worker) to the real Videre, but leaves service() at its None default, so HostServices::from_extensions seeds no venue registry.

The test client_without_registry_service_resolves_every_venue_to_unknown boots the prebuilt echo-client keeper via Supervisor::boot_single with a [venue]-free manifest, asserts supervisor.services().get::<VenueRegistry>(NAMESPACE).is_none() as the branch precondition, dispatches a chain-1 block, and asserts the keeper stays alive with quote and submit each resolving to unknown-venue.

No production code is touched.

Why

The service-missing branch (registry service absent entirely) is distinct from the already-covered adapter-map miss (registry present, venue id merely unlisted). This test pins the former end to end, exercising the resolve-to-unknown path before any adapter is consulted.

A [venue]-free manifest is required because a keeper declaring [venue] would be refused by admit_worker before it could reach the client face, given no adapters boot under boot_single.

Testing

cargo fmt --all -- --check passes with no diff.

cargo clippy -p videre-host --all-targets --all-features clean after cargo clean -p videre-host.

The test has genuine teeth: seeding a real registry flips the is_none() precondition and fails the test. CI builds the echo-client wasm before nextest, so the test runs rather than skips.

AI Assistance

Implemented, reviewed and verified with Claude Code (Opus).

Closes #510

@mfw78
mfw78 force-pushed the test/510-unknown-venue-no-registry-service branch from 48314f7 to 0989b72 Compare July 23, 2026 14:15
@mfw78
mfw78 changed the base branch from feat/m5-pre-carve-runbook to dev/m1 July 23, 2026 14:15
@mfw78
mfw78 force-pushed the test/510-unknown-venue-no-registry-service branch from 0989b72 to a893011 Compare July 23, 2026 14:52
@mfw78
mfw78 merged commit ee7848e into dev/m1 Jul 23, 2026
7 checks passed
@mfw78
mfw78 deleted the test/510-unknown-venue-no-registry-service branch July 23, 2026 23:19
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