From review of #443 (comment). Confirmed still valid at HEAD. Target: L2 videre-host.
Supervisor::boot_single builds services via HostServices::from_extensions alone and never seeds the venue-registry service. A manifest importing videre:venue/client under that path hits the service-missing case: services.get::<VenueRegistry>(VenueRegistry::NAMESPACE) returns None at client.rs:24, so every quote/submit returns VenueError::UnknownVenue. Real and intentional, but unpinned.
The existing UnknownVenue tests cover a different path — the adapter-map miss, where the registry service is present and the venue id just is not installed (unknown_venue_is_rejected_without_touching_an_adapter, and the platform.rs "unlisted venue id" assertion, both landing on registry.rs:374). Neither drives the service-lookup miss at client.rs:24.
Change
Add a boot_single-based test that boots a module importing videre:venue/client with no registry service registered, and asserts quote/submit return UnknownVenue.
Acceptance criteria
- A test exercises the
services.get::<VenueRegistry> = None branch end to end and asserts UnknownVenue.
Supervisor::boot_singlebuildsservicesviaHostServices::from_extensionsalone and never seeds the venue-registry service. A manifest importingvidere:venue/clientunder that path hits the service-missing case:services.get::<VenueRegistry>(VenueRegistry::NAMESPACE)returnsNoneatclient.rs:24, so everyquote/submitreturnsVenueError::UnknownVenue. Real and intentional, but unpinned.The existing
UnknownVenuetests cover a different path — the adapter-map miss, where the registry service is present and the venue id just is not installed (unknown_venue_is_rejected_without_touching_an_adapter, and the platform.rs "unlisted venue id" assertion, both landing onregistry.rs:374). Neither drives the service-lookup miss atclient.rs:24.Change
Add a
boot_single-based test that boots a module importingvidere:venue/clientwith no registry service registered, and assertsquote/submitreturnUnknownVenue.Acceptance criteria
services.get::<VenueRegistry>=Nonebranch end to end and assertsUnknownVenue.