Clean up zip test and restore full CI test suite

- Removed debug logging from TestZipFolder
- Restored full test suite in CI workflow

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

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

Author: andy-tier1app

.github/workflows/ci.yml

@@ -118,15 +118,13 @@ jobs:
 
       - name: Run tests with coverage (excluding CGO-dependent packages)
         run: |
-          # 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
+          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
 
 

internal/agent/ondemand/zip_test.go

@@ -15,17 +15,13 @@ func TestZipFolder(t *testing.T) {
 	}
 	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 {
+	if err := os.Mkdir(testFolder, 0755); err != nil {
 		t.Fatalf("Failed to create test folder: %v", err)
 	}
-	t.Logf("DEBUG: Created test folder: %s", testFolder)
 
-	// Create some test files
+	// Create test files
 	testFiles := []struct {
 		name    string
 		content string
@@ -37,11 +33,9 @@ func TestZipFolder(t *testing.T) {
 
 	for _, tf := range testFiles {
 		filePath := filepath.Join(testFolder, tf.name)
-		err = os.WriteFile(filePath, []byte(tf.content), 0644)
-		if err != nil {
+		if err := os.WriteFile(filePath, []byte(tf.content), 0644); 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
@@ -51,25 +45,21 @@ func TestZipFolder(t *testing.T) {
 	}
 	defer os.Chdir(originalDir)
 
-	err = os.Chdir(tempDir)
-	if err != nil {
+	if err := os.Chdir(tempDir); err != nil {
 		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)
@@ -78,16 +68,7 @@ func TestZipFolder(t *testing.T) {
 	}
 	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)
 }