Add integration test for pre-commit flag (AST-89008)

Author: elchananarb

.github/workflows/ci.yml

@@ -45,6 +45,10 @@ jobs:
         run: go build -o ./bin/cx ./cmd
       - name: Install gocovmerge
         run: go install github.com/wadey/gocovmerge@latest
+      - name: Install pre-commit
+        run: |
+         pip install pre-commit
+         pre-commit install
       - name: Go Integration test
         shell: bash
         env:

.pre-commit-config.yaml

@@ -0,0 +1,3 @@
+repos:
+- repo: local
+  hooks: []

test/integration/pre_commit_test.go

@@ -0,0 +1,77 @@
+//go:build integration
+
+package integration
+
+import (
+	"os"
+	"os/exec"
+	"path/filepath"
+	"strings"
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+)
+
+func TestHooksPreCommitInstallAndUninstallPreCommitHook(t *testing.T) {
+	tmpDir, cleanup := setupTempDir(t)
+	defer cleanup()
+
+	// Initialize Git repository
+	execCmd(t, tmpDir, "git", "init")
+
+	// Install pre-commit hook locally
+	_ = executeCmdNilAssertion(t, "Installing pre-commit hook", "hooks", "pre-commit", "secrets-install-git-hook")
+
+	// Verify hook installation
+	hookPath := filepath.Join(tmpDir, ".git", "hooks", "pre-commit")
+	assert.FileExists(t, hookPath, "Hook should be installed")
+	data, err := os.ReadFile(hookPath)
+	assert.NoError(t, err)
+	assert.Contains(t, string(data), "cx-secret-detection", "Hook should contain cx-secret-detection")
+
+	// Uninstall pre-commit hook
+	_ = executeCmdNilAssertion(t, "Uninstalling cx-secret-detection hook", "hooks", "pre-commit", "secrets-uninstall-git-hook")
+
+	// Verify hook removal
+	hookPath = filepath.Join(tmpDir, ".git", "hooks", "pre-commit")
+	data, err = os.ReadFile(hookPath)
+	assert.NoError(t, err)
+	assert.NotContains(t, string(data), "cx-secret-detection", "Hook content should be cleared after uninstall")
+}
+
+func TestHooksPreCommitUpdatePreCommitHook(t *testing.T) {
+	tmpDir, cleanup := setupTempDir(t)
+	defer cleanup()
+
+	// Initialize Git repository
+	execCmd(t, tmpDir, "git", "init")
+
+	// Install pre-commit hook locally
+	_ = executeCmdNilAssertion(t, "Installing pre-commit hook", "hooks", "pre-commit", "secrets-install-git-hook")
+
+	// Update pre-commit hook
+	err := executeCmdNilAssertion(t, "Updating pre-commit hook", "hooks", "pre-commit", "secrets-update-git-hook")
+	assert.NoError(t, err, "Hook should update successfully")
+}
+
+// Helper functions
+func execCmd(t *testing.T, dir string, name string, args ...string) {
+	cmd := exec.Command(name, args...)
+	cmd.Dir = dir
+	output, err := cmd.CombinedOutput()
+	assert.NoError(t, err, "Failed command %s: %s", strings.Join(cmd.Args, " "), string(output))
+}
+
+func setupTempDir(t *testing.T) (string, func()) {
+	originalWD, err := os.Getwd()
+	assert.NoError(t, err)
+
+	tmpDir := t.TempDir()
+	assert.NoError(t, os.Chdir(tmpDir))
+
+	cleanup := func() {
+		assert.NoError(t, os.Chdir(originalWD))
+	}
+
+	return tmpDir, cleanup
+}