ci: skip two flacky tests depending on os (#1606)

Andrey Nering created 1 week ago

Change summary

internal/agent/agent_test.go      | 4 ++++
internal/shell/background_test.go | 5 +++++
2 files changed, 9 insertions(+)

Detailed changes

internal/agent/agent_test.go

@@ -453,6 +453,10 @@ func TestCoderAgent(t *testing.T) {
 				require.Contains(t, string(content), "Hello, Crush!", "Expected file to contain 'Hello, Crush!'")
 			})
 			t.Run("sourcegraph tool", func(t *testing.T) {
+				if runtime.GOOS == "darwin" {
+					t.Skip("skipping flacky test on macos for now")
+				}
+
 				agent, env := setupAgent(t, pair)
 
 				session, err := env.sessions.Create(t.Context(), "New Session")

internal/shell/background_test.go

@@ -2,6 +2,7 @@ package shell
 
 import (
 	"context"
+	"runtime"
 	"strings"
 	"testing"
 	"time"
@@ -171,6 +172,10 @@ func TestBackgroundShell_WithBlockFuncs(t *testing.T) {
 }
 
 func TestBackgroundShellManager_List(t *testing.T) {
+	if runtime.GOOS == "windows" {
+		t.Skip("skipping flacky test on windows")
+	}
+
 	t.Parallel()
 
 	ctx := context.Background()