test(codersdk/toolsdk): skip coder_workspace_bash tool test on windows (#19351)

ethanndickson · web-flow · commit b2b3edf0f1b0 · 2025-08-14T15:49:48.000+10:00
Fixes flakes on the nightly-gauntlet like: https://github.com/coder/coder/actions/runs/16955654896 since there's no `bash` on windows... ``` === Failed === FAIL: codersdk/toolsdk (0.00s) PASS The following tools were not tested: - coder_workspace_bash Please ensure that all tools are tested using testTool(). If you just added a new tool, please add a test for it. NOTE: if you just ran an individual test, this is expected. FAIL github.com/coder/coder/v2/codersdk/toolsdk 4.185s ```
diff --git a/codersdk/toolsdk/toolsdk_test.go b/codersdk/toolsdk/toolsdk_test.go
@@ -665,6 +665,10 @@ func TestMain(m *testing.M) {
 	var untested []string
 	for _, tool := range toolsdk.All {
 		if tested, ok := testedTools.Load(tool.Name); !ok || !tested.(bool) {
+			// Test is skipped on Windows
+			if runtime.GOOS == "windows" && tool.Name == "coder_workspace_bash" {
+				continue
+			}
 			untested = append(untested, tool.Name)
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -665,6 +665,10 @@ func TestMain(m *testing.M) {`
`665`	`665`	`var untested []string`
`666`	`666`	`for _, tool := range toolsdk.All {`
`667`	`667`	`if tested, ok := testedTools.Load(tool.Name); !ok \|\| !tested.(bool) {`
	`668`	`+ // Test is skipped on Windows`
	`669`	`+ if runtime.GOOS == "windows" && tool.Name == "coder_workspace_bash" {`
	`670`	`+ continue`
	`671`	`+ }`
`668`	`672`	`untested = append(untested, tool.Name)`
`669`	`673`	`}`
`670`	`674`	`}`