Add tests to validate custom config dir is used

Author: devm33

dotnet/test/SessionTests.cs

@@ -299,5 +299,16 @@ public async Task Should_Create_Session_With_Custom_Config_Dir()
         var assistantMessage = await TestHelper.GetFinalAssistantMessageAsync(session);
         Assert.NotNull(assistantMessage);
         Assert.Contains("2", assistantMessage!.Data.Content);
+
+        // Verify that the custom config directory was created and contains files
+        Assert.True(Directory.Exists(customConfigDir), $"Custom config dir {customConfigDir} should exist");
+
+        // Check that session state was written to the custom config dir
+        var sessionStateDir = Path.Join(customConfigDir, "session-state");
+        Assert.True(Directory.Exists(sessionStateDir), $"Session state dir {sessionStateDir} should exist");
+
+        // Verify session files exist in the custom config dir
+        var sessionDir = Path.Join(sessionStateDir, session.SessionId);
+        Assert.True(Directory.Exists(sessionDir), $"Session dir {sessionDir} should exist");
     }
 }

go/e2e/session_test.go

@@ -1,6 +1,7 @@
 package e2e
 
 import (
+	"os"
 	"regexp"
 	"strings"
 	"testing"
@@ -645,6 +646,23 @@ func TestSession(t *testing.T) {
 		if assistantMessage.Data.Content == nil || !strings.Contains(*assistantMessage.Data.Content, "2") {
 			t.Errorf("Expected assistant message to contain '2', got %v", assistantMessage.Data.Content)
 		}
+
+		// Verify that the custom config directory was created and contains files
+		if _, err := os.Stat(customConfigDir); os.IsNotExist(err) {
+			t.Errorf("Expected custom config dir %q to exist", customConfigDir)
+		}
+
+		// Check that session state was written to the custom config dir
+		sessionStateDir := customConfigDir + "/session-state"
+		if _, err := os.Stat(sessionStateDir); os.IsNotExist(err) {
+			t.Errorf("Expected session state dir %q to exist", sessionStateDir)
+		}
+
+		// Verify session files exist in the custom config dir
+		sessionDir := sessionStateDir + "/" + session.SessionID
+		if _, err := os.Stat(sessionDir); os.IsNotExist(err) {
+			t.Errorf("Expected session dir %q to exist", sessionDir)
+		}
 	})
 }
 

nodejs/test/e2e/session.test.ts

@@ -1,3 +1,4 @@
+import fs from "fs/promises";
 import { describe, expect, it, onTestFinished } from "vitest";
 import { ParsedHttpExchange } from "../../../test/harness/replayingCapiProxy.js";
 import { CopilotClient } from "../../src/index.js";
@@ -312,6 +313,29 @@ describe("Sessions", async () => {
         await session.send({ prompt: "What is 1+1?" });
         const assistantMessage = await getFinalAssistantMessage(session);
         expect(assistantMessage.data.content).toContain("2");
+
+        // Verify that the custom config directory was created and contains files
+        const customConfigDirExists = await fs
+            .access(customConfigDir)
+            .then(() => true)
+            .catch(() => false);
+        expect(customConfigDirExists).toBe(true);
+
+        // Check that session state was written to the custom config dir
+        const sessionStateDir = `${customConfigDir}/session-state`;
+        const sessionStateDirExists = await fs
+            .access(sessionStateDir)
+            .then(() => true)
+            .catch(() => false);
+        expect(sessionStateDirExists).toBe(true);
+
+        // Verify session files exist in the custom config dir
+        const sessionDir = `${sessionStateDir}/${session.sessionId}`;
+        const sessionDirExists = await fs
+            .access(sessionDir)
+            .then(() => true)
+            .catch(() => false);
+        expect(sessionDirExists).toBe(true);
     });
 });
 

python/e2e/test_session.py

@@ -339,6 +339,21 @@ async def test_should_create_session_with_custom_config_dir(self, ctx: E2ETestCo
         assistant_message = await get_final_assistant_message(session)
         assert "2" in assistant_message.data.content
 
+        # Verify that the custom config directory was created and contains files
+        assert os.path.isdir(custom_config_dir), (
+            f"Custom config dir {custom_config_dir} should exist"
+        )
+
+        # Check that session state was written to the custom config dir
+        session_state_dir = os.path.join(custom_config_dir, "session-state")
+        assert os.path.isdir(session_state_dir), (
+            f"Session state dir {session_state_dir} should exist"
+        )
+
+        # Verify session files exist in the custom config dir
+        session_dir = os.path.join(session_state_dir, session.session_id)
+        assert os.path.isdir(session_dir), f"Session dir {session_dir} should exist"
+
 
 def _get_system_message(exchange: dict) -> str:
     messages = exchange.get("request", {}).get("messages", [])