Skip to content

Commit e4dd4b6

Browse files
stephentoubstephentoub
committed
Keep blocked in-process model suites excluded
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4e94289c-90c8-4664-a878-056539754908
1 parent 0330bc6 commit e4dd4b6

2 files changed

Lines changed: 91 additions & 0 deletions

File tree

‎java/sdk/pom.xml‎

Lines changed: 41 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -629,6 +629,36 @@ did not produce the multi-release output. Re-build on JDK 25+ and verify the
629629
<skipTests>false</skipTests>
630630
<forkCount>1</forkCount>
631631
<parallel>none</parallel>
632+
<!--
633+
TODO(cli-1.0.81-2): CLI 1.0.81-5 still stops
634+
completing model-driven turns when hosted in-process
635+
against CAPI. The shared runtime then poisons every
636+
later model-driven test. These classes still run over
637+
stdio; pure-RPC in-process coverage remains enabled.
638+
-->
639+
<excludes>
640+
<exclude>**/AskUserTest.java</exclude>
641+
<exclude>**/CompactionTest.java</exclude>
642+
<exclude>**/CopilotSessionTest.java</exclude>
643+
<exclude>**/ErrorHandlingTest.java</exclude>
644+
<exclude>**/EventFidelityTest.java</exclude>
645+
<exclude>**/ExecutorWiringTest.java</exclude>
646+
<exclude>**/HooksTest.java</exclude>
647+
<exclude>**/McpAndAgentsTest.java</exclude>
648+
<exclude>**/McpOAuthResumeE2ETest.java</exclude>
649+
<exclude>**/PermissionsTest.java</exclude>
650+
<exclude>**/PreMcpToolCallHookTest.java</exclude>
651+
<exclude>**/RpcServerE2ETest.java</exclude>
652+
<exclude>**/RpcSessionStateExtrasE2ETest.java</exclude>
653+
<exclude>**/SessionConfigE2ETest.java</exclude>
654+
<exclude>**/SessionEventsE2ETest.java</exclude>
655+
<exclude>**/SessionTodosChangedTest.java</exclude>
656+
<exclude>**/SkillsTest.java</exclude>
657+
<exclude>**/StreamingFidelityTest.java</exclude>
658+
<exclude>**/SubagentHooksE2ETest.java</exclude>
659+
<exclude>**/ToolResultsTest.java</exclude>
660+
<exclude>**/ToolsTest.java</exclude>
661+
</excludes>
632662
<environmentVariables>
633663
<COPILOT_SDK_DEFAULT_CONNECTION>inprocess</COPILOT_SDK_DEFAULT_CONNECTION>
634664
</environmentVariables>
@@ -640,6 +670,17 @@ did not produce the multi-release output. Re-build on JDK 25+ and verify the
640670
<configuration>
641671
<forkCount>1</forkCount>
642672
<parallel>none</parallel>
673+
<!--
674+
TODO(cli-1.0.81-2): same in-process CAPI turn hang as
675+
the Surefire exclusions above.
676+
-->
677+
<excludes>
678+
<exclude>**/ErgonomicToolDefinitionIT.java</exclude>
679+
<exclude>**/GitHubTelemetryForwardingIT.java</exclude>
680+
<exclude>**/LowLevelToolDefinitionIT.java</exclude>
681+
<exclude>**/RewindIT.java</exclude>
682+
<exclude>**/SystemMessageSectionsIT.java</exclude>
683+
</excludes>
643684
<environmentVariables>
644685
<COPILOT_CLI_PATH>${copilot.inprocess.cli.path}</COPILOT_CLI_PATH>
645686
<COPILOT_SDK_DEFAULT_CONNECTION>inprocess</COPILOT_SDK_DEFAULT_CONNECTION>

‎nodejs/vitest.config.ts‎

Lines changed: 50 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,55 @@ import { defineConfig } from "vitest/config";
22

33
const integrationTestTimeout = process.platform === "win32" ? 60000 : 30000;
44

5+
const isInProcessTransport =
6+
(process.env.COPILOT_SDK_DEFAULT_CONNECTION ?? "").toLowerCase() === "inprocess";
7+
8+
// TODO(cli-1.0.81-2): CLI 1.0.81-5 still stops completing model-driven turns when
9+
// hosted in-process against CAPI. The shared runtime then poisons every later
10+
// model-driven test. These suites still run over stdio on all three OSes, while
11+
// pure-RPC in-process coverage remains enabled.
12+
const inProcessBlockedE2E = [
13+
"**/test/e2e/abort.e2e.test.ts",
14+
"**/test/e2e/agent_and_compact_rpc.e2e.test.ts",
15+
"**/test/e2e/ask_user.e2e.test.ts",
16+
"**/test/e2e/builtin_tools.e2e.test.ts",
17+
"**/test/e2e/client_api.e2e.test.ts",
18+
"**/test/e2e/client_lifecycle.e2e.test.ts",
19+
"**/test/e2e/compaction.e2e.test.ts",
20+
"**/test/e2e/copilot_request_cancel_error.e2e.test.ts",
21+
"**/test/e2e/copilot_request_handler.e2e.test.ts",
22+
"**/test/e2e/copilot_request_session_id.e2e.test.ts",
23+
"**/test/e2e/disabled_mcp_servers.e2e.test.ts",
24+
"**/test/e2e/event_fidelity.e2e.test.ts",
25+
"**/test/e2e/hooks.e2e.test.ts",
26+
"**/test/e2e/hooks_extended.e2e.test.ts",
27+
"**/test/e2e/mcp_and_agents.e2e.test.ts",
28+
"**/test/e2e/mode_empty.e2e.test.ts",
29+
"**/test/e2e/multi_turn.e2e.test.ts",
30+
"**/test/e2e/permissions.e2e.test.ts",
31+
"**/test/e2e/pre_mcp_tool_call_hook.e2e.test.ts",
32+
"**/test/e2e/provider_endpoint.e2e.test.ts",
33+
"**/test/e2e/rewind.e2e.test.ts",
34+
"**/test/e2e/rpc_event_side_effects.e2e.test.ts",
35+
"**/test/e2e/rpc_session_state.e2e.test.ts",
36+
"**/test/e2e/rpc_session_state_extras.e2e.test.ts",
37+
"**/test/e2e/rpc_shell_and_fleet.e2e.test.ts",
38+
"**/test/e2e/session.e2e.test.ts",
39+
"**/test/e2e/session_config.e2e.test.ts",
40+
"**/test/e2e/session_fs.e2e.test.ts",
41+
"**/test/e2e/session_fs_sqlite.e2e.test.ts",
42+
"**/test/e2e/session_lifecycle.e2e.test.ts",
43+
"**/test/e2e/session_todos_changed.e2e.test.ts",
44+
"**/test/e2e/skills.e2e.test.ts",
45+
"**/test/e2e/streaming_fidelity.e2e.test.ts",
46+
"**/test/e2e/subagent_hooks.e2e.test.ts",
47+
"**/test/e2e/suspend.e2e.test.ts",
48+
"**/test/e2e/system_message_sections.e2e.test.ts",
49+
"**/test/e2e/system_message_transform.e2e.test.ts",
50+
"**/test/e2e/tool_results.e2e.test.ts",
51+
"**/test/e2e/tools.e2e.test.ts",
52+
];
53+
554
export default defineConfig({
655
test: {
756
globals: true,
@@ -17,6 +66,7 @@ export default defineConfig({
1766
"**/dist/**",
1867
"**/*.d.ts",
1968
"**/basic-test.ts", // Old manual test
69+
...(isInProcessTransport ? inProcessBlockedE2E : []),
2070
],
2171
},
2272
});

0 commit comments

Comments
 (0)