Skip to content

RSDK-14642: Composite E2E tests (4/5) - #6542

Open
Daniel Botros (danielbotros) wants to merge 3 commits into
composable-apis-p3c-clientfrom
composable-apis-p3d-web
Open

Daniel Botros (danielbotros) wants to merge 3 commits into
composable-apis-p3c-clientfrom
composable-apis-p3d-web

Conversation

@danielbotros

@danielbotros Daniel Botros (danielbotros) commented Sep 24, 2026 •

Copy link
Copy Markdown
Member

RSDK-14642 — PR 4/5 of the Composable APIs server-side work (epic RSDK-14329). Stacked on #6541.

Over-the-wire end-to-end tests for composites through the web/gRPC layer. No production changes are needed: an API-qualified lookup already routes through FindBySimpleNameAndAPI, which unwraps a composite to the sub-resource serving that API, so every one of a composite's APIs is reachable over its gRPC service — including the foreign/custom-API path, whose resolution unwraps upstream in ResourceFromProtoMessage.

Tests (multiapi_composite_web_test.go): api-less resolution over a client, colliding Properties routed per-API across the wire (builtin and modular), and a module's custom/foreign gizmo API via foreignServiceHandler on a separate-process server whose binary doesn't import the gizmo package — run both with the composite declared under the custom gizmo API and under a builtin API (the regression case for the ResourceRPCAPIs composite-expansion fix).

🤖 Generated with Claude Code

@danielbotros
Daniel Botros (danielbotros) added this pull request to stack #6544 September 24, 2026 21:18
@danielbotros
Daniel Botros (danielbotros) removed this pull request from stack #6544 September 24, 2026 21:32
@danielbotros
Daniel Botros (danielbotros) added this pull request to stack #6545 September 24, 2026 21:32
@viambot viambot added the safe to test This pull request is marked safe to test from a trusted zone label Sep 24, 2026
@viambot viambot added safe to test This pull request is marked safe to test from a trusted zone and removed safe to test This pull request is marked safe to test from a trusted zone labels Sep 24, 2026
@viambot viambot added safe to test This pull request is marked safe to test from a trusted zone and removed safe to test This pull request is marked safe to test from a trusted zone labels Sep 24, 2026
@viambot viambot added safe to test This pull request is marked safe to test from a trusted zone and removed safe to test This pull request is marked safe to test from a trusted zone labels Sep 24, 2026
@viambot viambot added safe to test This pull request is marked safe to test from a trusted zone and removed safe to test This pull request is marked safe to test from a trusted zone labels Sep 24, 2026
@viambot viambot added safe to test This pull request is marked safe to test from a trusted zone and removed safe to test This pull request is marked safe to test from a trusted zone labels Sep 25, 2026
@viambot viambot added safe to test This pull request is marked safe to test from a trusted zone and removed safe to test This pull request is marked safe to test from a trusted zone labels Sep 25, 2026
@viambot viambot added safe to test This pull request is marked safe to test from a trusted zone and removed safe to test This pull request is marked safe to test from a trusted zone labels Sep 25, 2026
@viambot viambot added safe to test This pull request is marked safe to test from a trusted zone and removed safe to test This pull request is marked safe to test from a trusted zone labels Sep 25, 2026
@viambot viambot added safe to test This pull request is marked safe to test from a trusted zone and removed safe to test This pull request is marked safe to test from a trusted zone labels Sep 25, 2026
@viambot viambot added safe to test This pull request is marked safe to test from a trusted zone and removed safe to test This pull request is marked safe to test from a trusted zone labels Sep 25, 2026
@viambot viambot removed the safe to test This pull request is marked safe to test from a trusted zone label Sep 25, 2026
Adds end-to-end tests exercising composites over the web/gRPC layer (custom/foreign
APIs, colliding methods, over-client). No production changes are needed: an
API-qualified lookup already routes through FindBySimpleNameAndAPI, which unwraps a
composite to the sub-resource serving that API.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
(cherry picked from commit 2dd9310)
Follows the Compose signature change (name now taken from the subs).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
(cherry picked from commit e860b40)
…omplexcombomodule path

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

safe to test This pull request is marked safe to test from a trusted zone

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants