Skip to content
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Add test: cliPath undefined when cliUrl set
Add a unit test to nodejs/test/client.test.ts that verifies CopilotClient does not resolve or set options.cliPath when instantiated with a cliUrl. This ensures providing a remote CLI URL doesn't trigger resolution of a local CLI path.
  • Loading branch information
sergiou87 committed Mar 11, 2026
commit 1a4ffed721bf53e7a0505b2098670341468fe2d0
9 changes: 9 additions & 0 deletions nodejs/test/client.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -210,6 +210,15 @@ describe("CopilotClient", () => {

expect((client as any).isExternalServer).toBe(true);
});

it("should not resolve cliPath when cliUrl is provided", () => {
const client = new CopilotClient({
cliUrl: "localhost:8080",
logLevel: "error",
});

expect(client["options"].cliPath).toBeUndefined();
});
});

describe("Auth options", () => {
Expand Down
Loading