Adds testing for PR creation

Author: kalverra

tools/flakeguard/golang/golang.go

@@ -213,10 +213,11 @@ type SkipTest struct {
 // SkipTests finds all package/test pairs provided and skips them
 func SkipTests(repoPath string, testsToSkip []SkipTest) error {
 	for _, testToSkip := range testsToSkip {
-		cmd := exec.Command("go", "list", "-json", testToSkip.Package)
+		cmd := exec.Command("go", "list", "-f", "{{.Dir}}", testToSkip.Package)
+		cmd.Dir = repoPath
 		out, err := cmd.CombinedOutput()
 		if err != nil {
-			return fmt.Errorf("error getting package path for %s: %w", testToSkip.Package, err)
+			return fmt.Errorf("error getting package path for %s: %w\nOutput:\n%s", testToSkip.Package, err, string(out))
 		}
 		packagePath := strings.TrimSpace(string(out))
 		log.Debug().Str("package", testToSkip.Package).Str("test", testToSkip.Name).Str("packagePath", packagePath).Msg("Skipping test")

tools/flakeguard/golang/golang_test.go

@@ -2,9 +2,11 @@ package golang
 
 import (
 	"errors"
+	"os"
 	"testing"
 
 	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
 )
 
 // Mock version of hasTests function to simulate various scenarios
@@ -54,3 +56,27 @@ func TestFilterPackagesWithTests(t *testing.T) {
 		assert.Equal(t, expected, result, "Expected empty slice for packages with errors")
 	})
 }
+
+func TestSkipTests(t *testing.T) {
+	// Create a temp dir copying over the testdata directory
+	tempDir, err := os.MkdirTemp("./", "testdata-*")
+	require.NoError(t, err, "Failed to create temp dir")
+	t.Cleanup(func() {
+		if !t.Failed() {
+			if err := os.RemoveAll(tempDir); err != nil {
+				t.Fatalf("Failed to remove temp dir: %v", err)
+			}
+			return
+		}
+		t.Logf("Skipping cleanup of temp dir %s because test failed, leaving it for debugging", tempDir)
+	})
+
+	// Copy the testdata directory to the temp dir for running the test on
+	err = os.CopyFS(tempDir, os.DirFS("testdata"))
+	testsToSkip := []SkipTest{
+		{Package: "package_a", Name: "TestPackAFail"},
+		{Package: "package_b", Name: "TestPackBFail"},
+	}
+	err = SkipTests(tempDir, testsToSkip)
+	assert.NoError(t, err)
+}

tools/flakeguard/golang/testdata/go.mod

@@ -0,0 +1,3 @@
+module github.com/owner/repo/testdata
+
+go 1.24.3

tools/flakeguard/golang/testdata/package_a/pack_a_test.go

@@ -0,0 +1,14 @@
+package package_a
+
+import "testing"
+
+// This test should not get skipped by flakeguard
+func TestPackAPass(t *testing.T) {
+	t.Log("This is a passing test")
+}
+
+// This test should get skipped by flakeguard
+func TestPackAFail(t *testing.T) {
+	t.Log("This is a failing test")
+	t.FailNow()
+}

tools/flakeguard/golang/testdata/package_b/pack_b_test.go

@@ -0,0 +1,14 @@
+package package_b
+
+import "testing"
+
+// This test should get skipped by flakeguard
+func TestPackBFail(t *testing.T) {
+	t.Log("This is a failing test")
+	t.FailNow()
+}
+
+// This test should not get skipped by flakeguard
+func TestPackBPass(t *testing.T) {
+	t.Log("This is a passing test")
+}