Re-enable zip test with proper temp directory setup

- Removed t.Skip() and rewrote TestZipFolder to use a temporary directory
- Added debug logging to trace execution in CI
- Temporarily limited CI to run only the zip test for debugging

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Author: andy-tier1app

.github/workflows/ci.yml

@@ -118,13 +118,15 @@ jobs:
 
       - name: Run tests with coverage (excluding CGO-dependent packages)
         run: |
-          go test -v -race -coverprofile=coverage.out -covermode=atomic $(go list ./... | grep -v 'internal/cli' | grep -v 'internal/capture/procps/linux')
-
-      - name: Display coverage summary
-        run: |
-          go tool cover -func=coverage.out | tail -20
-          echo ""
-          echo "=== Total Coverage ==="
-          go tool cover -func=coverage.out | grep total
+          # TEMP: Run only zip test for debugging
+          go test -v -race ./internal/agent/ondemand/... -run TestZipFolder
+
+      # TEMP: Skipping coverage summary while debugging zip test
+      # - name: Display coverage summary
+      #   run: |
+      #     go tool cover -func=coverage.out | tail -20
+      #     echo ""
+      #     echo "=== Total Coverage ==="
+      #     go tool cover -func=coverage.out | grep total
 
 

internal/agent/ondemand/zip_test.go

@@ -1,15 +1,93 @@
 package ondemand
 
-import "testing"
+import (
+	"archive/zip"
+	"os"
+	"path/filepath"
+	"testing"
+)
 
 func TestZipFolder(t *testing.T) {
-	// TODO: Revisit this test - currently failing in CI
-	// Test may fail if "zip" folder doesn't exist or is not accessible in CI environment.
-	// Needs proper test data setup or mocking of filesystem.
-	t.Skip("Skipping until zip folder test can be properly set up")
+	// Create a temporary directory for the test
+	tempDir, err := os.MkdirTemp("", "ziptest")
+	if err != nil {
+		t.Fatalf("Failed to create temp dir: %v", err)
+	}
+	defer os.RemoveAll(tempDir)
+
+	t.Logf("DEBUG: Created temp dir: %s", tempDir)
+
+	// Create a test folder inside temp dir
+	testFolder := filepath.Join(tempDir, "testfolder")
+	err = os.Mkdir(testFolder, 0755)
+	if err != nil {
+		t.Fatalf("Failed to create test folder: %v", err)
+	}
+	t.Logf("DEBUG: Created test folder: %s", testFolder)
+
+	// Create some test files
+	testFiles := []struct {
+		name    string
+		content string
+	}{
+		{"file1.txt", "Hello, World!"},
+		{"file2.txt", "Test content"},
+		{"file3.log", "Log data here"},
+	}
+
+	for _, tf := range testFiles {
+		filePath := filepath.Join(testFolder, tf.name)
+		err = os.WriteFile(filePath, []byte(tf.content), 0644)
+		if err != nil {
+			t.Fatalf("Failed to create test file %s: %v", tf.name, err)
+		}
+		t.Logf("DEBUG: Created test file: %s", filePath)
+	}
+
+	// Change to temp dir so the zip is created there
+	originalDir, err := os.Getwd()
+	if err != nil {
+		t.Fatalf("Failed to get current dir: %v", err)
+	}
+	defer os.Chdir(originalDir)
 
-	_, err := ZipFolder("zip")
+	err = os.Chdir(tempDir)
 	if err != nil {
-		t.Fatal(err)
+		t.Fatalf("Failed to change to temp dir: %v", err)
 	}
+	t.Logf("DEBUG: Changed to dir: %s", tempDir)
+
+	// Call ZipFolder with the relative path
+	zipName, err := ZipFolder("testfolder")
+	if err != nil {
+		t.Fatalf("ZipFolder failed: %v", err)
+	}
+	t.Logf("DEBUG: Created zip file: %s", zipName)
+
+	// Verify the zip file was created
+	zipPath := filepath.Join(tempDir, zipName)
+	if _, err := os.Stat(zipPath); os.IsNotExist(err) {
+		t.Fatalf("Zip file was not created at %s", zipPath)
+	}
+	t.Logf("DEBUG: Verified zip exists at: %s", zipPath)
+
+	// Open and verify the zip contents
+	reader, err := zip.OpenReader(zipPath)
+	if err != nil {
+		t.Fatalf("Failed to open zip file: %v", err)
+	}
+	defer reader.Close()
+
+	t.Logf("DEBUG: Zip contains %d files", len(reader.File))
+	for _, f := range reader.File {
+		t.Logf("DEBUG: Zip entry: %s (size: %d)", f.Name, f.UncompressedSize64)
+	}
+
+	// Verify all test files are in the zip
+	if len(reader.File) != len(testFiles) {
+		t.Errorf("Expected %d files in zip, got %d", len(testFiles), len(reader.File))
+	}
+
+	// Cleanup the zip file
+	os.Remove(zipPath)
 }