Port snapshot directory name changes and CI token from upstream

- Change ask-user to ask_user snapshot directory name
- Change mcp-and-agents to mcp_and_agents snapshot directory name
- Add fake token for CI environment to avoid auth issues in E2E tests

Author: brunoborges

src/test/java/com/github/copilot/sdk/AskUserTest.java

@@ -25,7 +25,7 @@
  *
  * <p>
  * These tests use the shared CapiProxy infrastructure for deterministic API
- * response replay. Snapshots are stored in test/snapshots/ask-user/.
+ * response replay. Snapshots are stored in test/snapshots/ask_user/.
  * </p>
  */
 public class AskUserTest {
@@ -46,7 +46,7 @@ static void teardown() throws Exception {
 
     @Test
     void testUserInputHandlerInvokedWhenModelUsesAskUserTool() throws Exception {
-        ctx.configureForTest("ask-user", "should_invoke_user_input_handler_when_model_uses_ask_user_tool");
+        ctx.configureForTest("ask_user", "should_invoke_user_input_handler_when_model_uses_ask_user_tool");
 
         List<UserInputRequest> userInputRequests = new ArrayList<>();
         final String[] sessionIdHolder = new String[1];
@@ -84,7 +84,7 @@ void testUserInputHandlerInvokedWhenModelUsesAskUserTool() throws Exception {
 
     @Test
     void testUserInputRequestWithChoices() throws Exception {
-        ctx.configureForTest("ask-user", "should_receive_choices_in_user_input_request");
+        ctx.configureForTest("ask_user", "should_receive_choices_in_user_input_request");
 
         List<UserInputRequest> userInputRequests = new ArrayList<>();
 
@@ -120,7 +120,7 @@ void testUserInputRequestWithChoices() throws Exception {
      */
     @Test
     void testFreeformUserInputResponse() throws Exception {
-        ctx.configureForTest("ask-user", "should_handle_freeform_user_input_response");
+        ctx.configureForTest("ask_user", "should_handle_freeform_user_input_response");
 
         final List<UserInputRequest> userInputRequests = new ArrayList<>();
         String freeformAnswer = "This is my custom freeform answer that was not in the choices";

src/test/java/com/github/copilot/sdk/E2ETestContext.java

@@ -173,8 +173,16 @@ public Map<String, String> getEnvironment() {
      * @return a new CopilotClient
      */
     public CopilotClient createClient() {
-        return new CopilotClient(new CopilotClientOptions().setCliPath(cliPath).setCwd(workDir.toString())
-                .setEnvironment(getEnvironment()));
+        CopilotClientOptions options = new CopilotClientOptions().setCliPath(cliPath).setCwd(workDir.toString())
+                .setEnvironment(getEnvironment());
+
+        // In CI, use a fake token to avoid auth issues
+        String ci = System.getenv("CI");
+        if (ci != null && !ci.isEmpty()) {
+            options.setGithubToken("fake-token-for-e2e-tests");
+        }
+
+        return new CopilotClient(options);
     }
 
     @Override

src/test/java/com/github/copilot/sdk/McpAndAgentsTest.java

@@ -26,7 +26,7 @@
  *
  * <p>
  * These tests use the shared CapiProxy infrastructure for deterministic API
- * response replay. Snapshots are stored in test/snapshots/mcp-and-agents/.
+ * response replay. Snapshots are stored in test/snapshots/mcp_and_agents/.
  * </p>
  */
 public class McpAndAgentsTest {
@@ -59,7 +59,7 @@ private Map<String, Object> createLocalMcpServer(String command, List<String> ar
 
     @Test
     void testAcceptMcpServerConfigurationOnSessionCreate() throws Exception {
-        ctx.configureForTest("mcp-and-agents", "should_accept_mcp_server_configuration_on_session_create");
+        ctx.configureForTest("mcp_and_agents", "should_accept_mcp_server_configuration_on_session_create");
 
         Map<String, Object> mcpServers = new HashMap<>();
         mcpServers.put("test-server", createLocalMcpServer("echo", List.of("hello")));
@@ -83,7 +83,7 @@ void testAcceptMcpServerConfigurationOnSessionCreate() throws Exception {
 
     @Test
     void testAcceptMcpServerConfigurationOnSessionResume() throws Exception {
-        ctx.configureForTest("mcp-and-agents", "should_accept_mcp_server_configuration_on_session_resume");
+        ctx.configureForTest("mcp_and_agents", "should_accept_mcp_server_configuration_on_session_resume");
 
         try (CopilotClient client = ctx.createClient()) {
             // Create a session first
@@ -115,7 +115,7 @@ void testAcceptMcpServerConfigurationOnSessionResume() throws Exception {
     void testHandleMultipleMcpServers() throws Exception {
         // Use same snapshot as single MCP server test since it doesn't depend on server
         // count
-        ctx.configureForTest("mcp-and-agents", "should_accept_mcp_server_configuration_on_session_create");
+        ctx.configureForTest("mcp_and_agents", "should_accept_mcp_server_configuration_on_session_create");
 
         Map<String, Object> mcpServers = new HashMap<>();
         mcpServers.put("server1", createLocalMcpServer("echo", List.of("server1")));
@@ -133,7 +133,7 @@ void testHandleMultipleMcpServers() throws Exception {
 
     @Test
     void testAcceptCustomAgentConfigurationOnSessionCreate() throws Exception {
-        ctx.configureForTest("mcp-and-agents", "should_accept_custom_agent_configuration_on_session_create");
+        ctx.configureForTest("mcp_and_agents", "should_accept_custom_agent_configuration_on_session_create");
 
         List<CustomAgentConfig> customAgents = List.of(new CustomAgentConfig().setName("test-agent")
                 .setDisplayName("Test Agent").setDescription("A test agent for SDK testing")
@@ -158,7 +158,7 @@ void testAcceptCustomAgentConfigurationOnSessionCreate() throws Exception {
 
     @Test
     void testAcceptCustomAgentConfigurationOnSessionResume() throws Exception {
-        ctx.configureForTest("mcp-and-agents", "should_accept_custom_agent_configuration_on_session_resume");
+        ctx.configureForTest("mcp_and_agents", "should_accept_custom_agent_configuration_on_session_resume");
 
         try (CopilotClient client = ctx.createClient()) {
             // Create a session first
@@ -191,7 +191,7 @@ void testAcceptCustomAgentConfigurationOnSessionResume() throws Exception {
     void testCustomAgentWithToolsConfiguration() throws Exception {
         // Use same snapshot as create test since this just verifies configuration
         // acceptance
-        ctx.configureForTest("mcp-and-agents", "should_accept_custom_agent_configuration_on_session_create");
+        ctx.configureForTest("mcp_and_agents", "should_accept_custom_agent_configuration_on_session_create");
 
         List<CustomAgentConfig> customAgents = List.of(new CustomAgentConfig().setName("tool-agent")
                 .setDisplayName("Tool Agent").setDescription("An agent with specific tools")
@@ -208,7 +208,7 @@ void testCustomAgentWithToolsConfiguration() throws Exception {
     @Test
     void testCustomAgentWithMcpServers() throws Exception {
         // Use combined snapshot since this uses both MCP servers and custom agents
-        ctx.configureForTest("mcp-and-agents", "should_accept_both_mcp_servers_and_custom_agents");
+        ctx.configureForTest("mcp_and_agents", "should_accept_both_mcp_servers_and_custom_agents");
 
         Map<String, Object> agentMcpServers = new HashMap<>();
         agentMcpServers.put("agent-server", createLocalMcpServer("echo", List.of("agent-mcp")));
@@ -228,7 +228,7 @@ void testCustomAgentWithMcpServers() throws Exception {
     @Test
     void testMultipleCustomAgents() throws Exception {
         // Use same snapshot as create test
-        ctx.configureForTest("mcp-and-agents", "should_accept_custom_agent_configuration_on_session_create");
+        ctx.configureForTest("mcp_and_agents", "should_accept_custom_agent_configuration_on_session_create");
 
         List<CustomAgentConfig> customAgents = List.of(
                 new CustomAgentConfig().setName("agent1").setDisplayName("Agent One").setDescription("First agent")
@@ -248,7 +248,7 @@ void testMultipleCustomAgents() throws Exception {
 
     @Test
     void testAcceptBothMcpServersAndCustomAgents() throws Exception {
-        ctx.configureForTest("mcp-and-agents", "should_accept_both_mcp_servers_and_custom_agents");
+        ctx.configureForTest("mcp_and_agents", "should_accept_both_mcp_servers_and_custom_agents");
 
         Map<String, Object> mcpServers = new HashMap<>();
         mcpServers.put("shared-server", createLocalMcpServer("echo", List.of("shared")));