fix: skip bash tests on Windows

ThomasK33 · ThomasK33 · commit 31b973da717b · 2025-08-11T10:30:23.000+02:00
Change-Id: I8838e8c2d38766ad142e1f5bb224a2f82ac17d6b
Signed-off-by: Thomas Kosiewski &lt;tk@coder.com&gt;
diff --git a/codersdk/toolsdk/bash_test.go b/codersdk/toolsdk/bash_test.go
@@ -2,6 +2,7 @@ package toolsdk_test
 
 import (
 	"context"
+	"runtime"
 	"testing"
 
 	"github.com/stretchr/testify/require"
@@ -13,6 +14,9 @@ import (
 )
 
 func TestWorkspaceBash(t *testing.T) {
+	if runtime.GOOS == "windows" {
+		t.Skip("Skipping on Windows: Workspace MCP bash tools rely on a Unix-like shell (bash) and POSIX/SSH semantics. Use Linux/macOS or WSL for these tests.")
+	}
 	t.Parallel()
 
 	t.Run("ValidateArgs", func(t *testing.T) {
@@ -96,6 +100,9 @@ func TestWorkspaceBash(t *testing.T) {
 }
 
 func TestNormalizeWorkspaceInput(t *testing.T) {
+	if runtime.GOOS == "windows" {
+		t.Skip("Skipping on Windows: Workspace MCP bash tools rely on a Unix-like shell (bash) and POSIX/SSH semantics. Use Linux/macOS or WSL for these tests.")
+	}
 	t.Parallel()
 
 	testCases := []struct {
@@ -150,6 +157,9 @@ func TestNormalizeWorkspaceInput(t *testing.T) {
 }
 
 func TestAllToolsIncludesBash(t *testing.T) {
+	if runtime.GOOS == "windows" {
+		t.Skip("Skipping on Windows: Workspace MCP bash tools rely on a Unix-like shell (bash) and POSIX/SSH semantics. Use Linux/macOS or WSL for these tests.")
+	}
 	t.Parallel()
 
 	// Verify that WorkspaceBash is included in the All slice
@@ -168,6 +178,9 @@ func TestAllToolsIncludesBash(t *testing.T) {
 // timeout functionality with a real SSH connection and mock clock.
 
 func TestWorkspaceBashTimeout(t *testing.T) {
+	if runtime.GOOS == "windows" {
+		t.Skip("Skipping on Windows: Workspace MCP bash tools rely on a Unix-like shell (bash) and POSIX/SSH semantics. Use Linux/macOS or WSL for these tests.")
+	}
 	t.Parallel()
 
 	t.Run("TimeoutDefaultValue", func(t *testing.T) {
@@ -250,6 +263,9 @@ func TestWorkspaceBashTimeout(t *testing.T) {
 }
 
 func TestWorkspaceBashTimeoutIntegration(t *testing.T) {
+	if runtime.GOOS == "windows" {
+		t.Skip("Skipping on Windows: Workspace MCP bash tools rely on a Unix-like shell (bash) and POSIX/SSH semantics. Use Linux/macOS or WSL for these tests.")
+	}
 	t.Parallel()
 
 	t.Run("ActualTimeoutBehavior", func(t *testing.T) {
@@ -337,6 +353,9 @@ func TestWorkspaceBashTimeoutIntegration(t *testing.T) {
 }
 
 func TestWorkspaceBashBackgroundIntegration(t *testing.T) {
+	if runtime.GOOS == "windows" {
+		t.Skip("Skipping on Windows: Workspace MCP bash tools rely on a Unix-like shell (bash) and POSIX/SSH semantics. Use Linux/macOS or WSL for these tests.")
+	}
 	t.Parallel()
 
 	t.Run("BackgroundCommandCapturesOutput", func(t *testing.T) {
diff --git a/codersdk/toolsdk/toolsdk_test.go b/codersdk/toolsdk/toolsdk_test.go
@@ -4,6 +4,7 @@ import (
 	"context"
 	"encoding/json"
 	"os"
+	"runtime"
 	"sort"
 	"sync"
 	"testing"
@@ -397,6 +398,9 @@ func TestTools(t *testing.T) {
 	})
 
 	t.Run("WorkspaceSSHExec", func(t *testing.T) {
+		if runtime.GOOS == "windows" {
+			t.Skip("WorkspaceSSHExec is not supported on Windows")
+		}
 		// Setup workspace exactly like main SSH tests
 		client, workspace, agentToken := setupWorkspaceForAgent(t)
 
