Merge pull request #11 from amenocal/fix-self-reference

fix: check for local action reference

cmd/workflows.go

@@ -230,6 +230,10 @@ func processAction(action string) (string, error) {
 			actionWithSha = actionWithBranch
 			return actionWithSha, err
 		}
+	} else if strings.Contains(action, "./") {
+		// Action is local
+		logger.Info("Action is local", logger.Args("action:", action))
+		return action, nil
 	}
 	return actionWithSha, nil
 }

cmd/workflows_test.go

@@ -0,0 +1,102 @@
+package cmd
+
+import (
+	"os"
+	"path/filepath"
+	"strings"
+	"testing"
+)
+
+func TestGetWorkflowFiles(t *testing.T) {
+	// Setup: create a temporary directory with test files
+	tempDir := t.TempDir()
+	err := os.MkdirAll(filepath.Join(tempDir, ".github", "workflows"), 0755)
+	if err != nil {
+		t.Fatalf("Unexpected error creating directory: %v", err)
+	}
+	testFiles := []string{
+		"workflow1.yml",
+		"workflow2.yaml",
+		"workflow-pin.yml",
+		"workflow-pin.yaml",
+	}
+	for _, file := range testFiles {
+		err := os.WriteFile(filepath.Join(tempDir, ".github", "workflows", file), []byte{}, 0644)
+		if err != nil {
+			t.Fatalf("Unexpected error writing file: %v", err)
+		}
+	}
+
+	// Override the directory for testing
+	originalDir, _ := os.Getwd()
+	defer func() {
+		err := os.Chdir(originalDir)
+		if err != nil {
+			t.Fatalf("Unexpected error changing directory: %v", err)
+		}
+	}()
+	err = os.Chdir(tempDir)
+	if err != nil {
+		t.Fatalf("Unexpected error changing directory: %v", err)
+	}
+
+	workflowFiles, err := getWorkflowFiles()
+	if err != nil {
+		t.Fatalf("Unexpected error: %v", err)
+	}
+
+	expectedFiles := []string{
+		filepath.Join(".github", "workflows", "workflow1.yml"),
+		filepath.Join(".github", "workflows", "workflow2.yaml"),
+	}
+
+	if len(workflowFiles) != len(expectedFiles) {
+		t.Errorf("Expected %d workflow files, got %d", len(expectedFiles), len(workflowFiles))
+	}
+
+	for _, expected := range expectedFiles {
+		found := false
+		for _, actual := range workflowFiles {
+			if strings.HasSuffix(actual, expected) {
+				found = true
+				break
+			}
+		}
+		if !found {
+			t.Errorf("Expected file %s not found", expected)
+		}
+	}
+}
+
+func TestCreateTempYAMLFile(t *testing.T) {
+	// Setup: create a temporary file
+	tempFile, err := os.CreateTemp("", "tempfile")
+	if err != nil {
+		t.Fatalf("Unexpected error creating temp file: %v", err)
+	}
+	defer tempFile.Close()
+
+	content := []byte("test content")
+	if _, err := tempFile.Write(content); err != nil {
+		t.Fatalf("Unexpected error writing to temp file: %v", err)
+	}
+
+	tempFileName := tempFile.Name()
+	newFileName, err := createTempYAMLFile(tempFileName)
+	if err != nil {
+		t.Fatalf("Unexpected error: %v", err)
+	}
+
+	if !strings.HasSuffix(newFileName, "-pin.yml") {
+		t.Errorf("Expected new file name to end with '-pin.yml', got %s", newFileName)
+	}
+
+	newContent, err := os.ReadFile(newFileName)
+	if err != nil {
+		t.Fatalf("Unexpected error reading new file: %v", err)
+	}
+
+	if string(newContent) != string(content) {
+		t.Errorf("Expected content %s, got %s", string(content), string(newContent))
+	}
+}