nowindows

mafredri · mafredri · commit 5d3681c5be3c · 2025-06-24T18:37:21.000Z
diff --git a/agent/agentcontainers/api_test.go b/agent/agentcontainers/api_test.go
@@ -2074,6 +2074,10 @@ func TestAPI(t *testing.T) {
 	t.Run("IgnoreCustomization", func(t *testing.T) {
 		t.Parallel()
 
+		if runtime.GOOS == "windows" {
+			t.Skip("Dev Container tests are not supported on Windows (this test uses mocks but fails due to Windows paths)")
+		}
+
 		ctx := testutil.Context(t, testutil.WaitShort)
 
 		startTime := time.Date(2025, 1, 1, 12, 0, 0, 0, time.UTC)
@@ -2196,13 +2200,7 @@ func TestAPI(t *testing.T) {
 		err = json.NewDecoder(rec.Body).Decode(&response)
 		require.NoError(t, err)
 
-		assert.Len(t, response.Devcontainers, 1, "devcontainer should be in response when ignore=false")
-		assert.Len(t, response.Containers, 1, "regular container should still be listed")
-		assert.Equal(t, "/workspace/project", response.Devcontainers[0].WorkspaceFolder)
-		assert.Len(t, fakeSAC.created, 1, "sub agent should be created when ignore=false")
-		createdAgentID := fakeSAC.created[0].ID
-
-		t.Log("Phase 2: Done, waiting for sub agent to exit")
+		t.Log("Phase 2: Cont, waiting for sub agent to exit")
 		exitSubAgentOnce.Do(func() {
 			close(exitSubAgent)
 		})
@@ -2212,6 +2210,12 @@ func TestAPI(t *testing.T) {
 			t.Fatal("timeout waiting for sub agent to exit")
 		}
 
+		assert.Len(t, response.Devcontainers, 1, "devcontainer should be in response when ignore=false")
+		assert.Len(t, response.Containers, 1, "regular container should still be listed")
+		assert.Equal(t, "/workspace/project", response.Devcontainers[0].WorkspaceFolder)
+		require.Len(t, fakeSAC.created, 1, "sub agent should be created when ignore=false")
+		createdAgentID := fakeSAC.created[0].ID
+
 		t.Log("Phase 3: Change back to ignore=true and test sub agent deletion")
 		fDCCLI.readConfig.Configuration.Customizations.Coder.Ignore = true
 		testutil.RequireSend(ctx, t, fakeSAC.deleteErrC, nil)
