Skip to content

Commit b7f0fb8

Browse files
stephentoubCopilot
stephentoub
and
Copilot
committed
Fix CI test failures: mock session.resume and go mod tidy scenarios
- Node.js/Python: Mock sendRequest for session.resume in overridesBuiltInTool test to avoid 'No authentication info available' error (matching pattern used by existing clientName resume tests) - Go scenarios: Run go mod tidy to add missing google/uuid indirect dependency introduced by new OTel SDK packages Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent bce39e7 commit b7f0fb8

34 files changed

Lines changed: 81 additions & 3 deletions

File tree

‎nodejs/test/client.test.ts‎

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -327,7 +327,9 @@ describe("CopilotClient", () => {
327327
onTestFinished(() => client.forceStop());
328328

329329
const session = await client.createSession({ onPermissionRequest: approveAll });
330-
const spy = vi.spyOn((client as any).connection!, "sendRequest");
330+
const spy = vi
331+
.spyOn((client as any).connection!, "sendRequest")
332+
.mockResolvedValueOnce({ sessionId: session.sessionId });
331333
await client.resumeSession(session.sessionId, {
332334
onPermissionRequest: approveAll,
333335
tools: [

‎python/test_client.py‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -221,6 +221,9 @@ async def test_resume_session_sends_overrides_built_in_tool(self):
221221

222222
async def mock_request(method, params):
223223
captured[method] = params
224+
if method == "session.resume":
225+
# Return a fake response to avoid needing real auth
226+
return {"sessionId": session.session_id}
224227
return await original_request(method, params)
225228

226229
client._client.request = mock_request

‎test/scenarios/auth/byok-anthropic/go/go.mod‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ require (
1111
github.com/go-logr/logr v1.4.3 // indirect
1212
github.com/go-logr/stdr v1.2.2 // indirect
1313
github.com/google/jsonschema-go v0.4.2 // indirect
14+
github.com/google/uuid v1.6.0 // indirect
1415
go.opentelemetry.io/auto/sdk v1.2.1 // indirect
1516
go.opentelemetry.io/otel v1.40.0 // indirect
1617
go.opentelemetry.io/otel/metric v1.40.0 // indirect

‎test/scenarios/auth/byok-azure/go/go.mod‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ require (
1111
github.com/go-logr/logr v1.4.3 // indirect
1212
github.com/go-logr/stdr v1.2.2 // indirect
1313
github.com/google/jsonschema-go v0.4.2 // indirect
14+
github.com/google/uuid v1.6.0 // indirect
1415
go.opentelemetry.io/auto/sdk v1.2.1 // indirect
1516
go.opentelemetry.io/otel v1.40.0 // indirect
1617
go.opentelemetry.io/otel/metric v1.40.0 // indirect

‎test/scenarios/auth/byok-ollama/go/go.mod‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ require (
1111
github.com/go-logr/logr v1.4.3 // indirect
1212
github.com/go-logr/stdr v1.2.2 // indirect
1313
github.com/google/jsonschema-go v0.4.2 // indirect
14+
github.com/google/uuid v1.6.0 // indirect
1415
go.opentelemetry.io/auto/sdk v1.2.1 // indirect
1516
go.opentelemetry.io/otel v1.40.0 // indirect
1617
go.opentelemetry.io/otel/metric v1.40.0 // indirect

‎test/scenarios/auth/byok-openai/go/go.mod‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ require (
1111
github.com/go-logr/logr v1.4.3 // indirect
1212
github.com/go-logr/stdr v1.2.2 // indirect
1313
github.com/google/jsonschema-go v0.4.2 // indirect
14+
github.com/google/uuid v1.6.0 // indirect
1415
go.opentelemetry.io/auto/sdk v1.2.1 // indirect
1516
go.opentelemetry.io/otel v1.40.0 // indirect
1617
go.opentelemetry.io/otel/metric v1.40.0 // indirect

‎test/scenarios/auth/gh-app/go/go.mod‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ require (
1111
github.com/go-logr/logr v1.4.3 // indirect
1212
github.com/go-logr/stdr v1.2.2 // indirect
1313
github.com/google/jsonschema-go v0.4.2 // indirect
14+
github.com/google/uuid v1.6.0 // indirect
1415
go.opentelemetry.io/auto/sdk v1.2.1 // indirect
1516
go.opentelemetry.io/otel v1.40.0 // indirect
1617
go.opentelemetry.io/otel/metric v1.40.0 // indirect

‎test/scenarios/bundling/app-backend-to-server/go/go.mod‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ require (
1111
github.com/go-logr/logr v1.4.3 // indirect
1212
github.com/go-logr/stdr v1.2.2 // indirect
1313
github.com/google/jsonschema-go v0.4.2 // indirect
14+
github.com/google/uuid v1.6.0 // indirect
1415
go.opentelemetry.io/auto/sdk v1.2.1 // indirect
1516
go.opentelemetry.io/otel v1.40.0 // indirect
1617
go.opentelemetry.io/otel/metric v1.40.0 // indirect

‎test/scenarios/bundling/app-direct-server/go/go.mod‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ require (
1111
github.com/go-logr/logr v1.4.3 // indirect
1212
github.com/go-logr/stdr v1.2.2 // indirect
1313
github.com/google/jsonschema-go v0.4.2 // indirect
14+
github.com/google/uuid v1.6.0 // indirect
1415
go.opentelemetry.io/auto/sdk v1.2.1 // indirect
1516
go.opentelemetry.io/otel v1.40.0 // indirect
1617
go.opentelemetry.io/otel/metric v1.40.0 // indirect

‎test/scenarios/bundling/container-proxy/go/go.mod‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ require (
1111
github.com/go-logr/logr v1.4.3 // indirect
1212
github.com/go-logr/stdr v1.2.2 // indirect
1313
github.com/google/jsonschema-go v0.4.2 // indirect
14+
github.com/google/uuid v1.6.0 // indirect
1415
go.opentelemetry.io/auto/sdk v1.2.1 // indirect
1516
go.opentelemetry.io/otel v1.40.0 // indirect
1617
go.opentelemetry.io/otel/metric v1.40.0 // indirect

0 commit comments

Comments
 (0)