Test fixes, add macOS linker warning docs

  - Fix timing issues in performance and goroutine leak tests
  - Add *.out to .gitignore for coverage files
  - Add make test-clean target to run tests without linker warnings
  - Document LC_DYSYMTAB warnings on macOS with CGO enabled

Author: wayneeseguin

.gitignore

@@ -27,3 +27,4 @@ examples/plugins/vault-backend/vault-backend
 examples/plugins/vault-backend/vault:/invalid-token@localhost:8200/logs
 examples/plugins/vault-backend/vault:/test-token@localhost:8200/logs
 examples/plugins/nats-backend/nats-backend
+*.out

Makefile

@@ -20,6 +20,11 @@ test-unit:
 # Run tests (alias for test-unit)
 test: test-unit
 
+# Run tests without linker warnings (CGO disabled)
+test-clean:
+	@echo "Running tests without linker warnings (CGO disabled)..."
+	@CGO_ENABLED=0 go test ./...
+
 # Run tests with verbose output
 test-verbose:
 	@echo "Running tests with verbose output..."
@@ -28,6 +33,7 @@ test-verbose:
 # Run tests with race detector (may show linker warnings on macOS)
 test-race:
 	@echo "Running tests with race detector..."
+	@echo "Note: LC_DYSYMTAB warnings on macOS are a known Go issue and can be safely ignored"
 	@go test -v -race -coverprofile=coverage.out $(shell go list ./... | grep -v /examples/)
 
 # Run integration tests
@@ -190,6 +196,7 @@ help:
 	@echo "  build                    - Build the library"
 	@echo "  test                     - Run unit tests (alias for test-unit)"
 	@echo "  test-unit                - Run unit tests (CGO disabled, no linker warnings)"
+	@echo "  test-clean               - Run tests without linker warnings (CGO disabled)"
 	@echo "  test-verbose             - Run tests with verbose output"
 	@echo "  test-race                - Run tests with race detector (may show warnings)"
 	@echo "  test-integration         - Run integration tests"

docs/macos-linker-warnings.md

@@ -0,0 +1,56 @@
+# macOS Linker Warnings
+
+## Issue
+
+When running Go tests on macOS (especially on Apple Silicon/arm64), you may encounter linker warnings like:
+
+```
+ld: warning: '/private/var/folders/.../000012.o' has malformed LC_DYSYMTAB, 
+expected 98 undefined symbols to start at index 1626, found 95 undefined symbols starting at index 1626
+```
+
+## Cause
+
+These warnings are caused by a known issue with the Go toolchain on macOS when CGO is enabled. The linker (ld) detects a mismatch in the symbol table of the generated object files.
+
+## Solution
+
+### Option 1: Disable CGO (Recommended for tests)
+
+Run tests with CGO disabled to avoid the warnings:
+
+```bash
+CGO_ENABLED=0 go test ./...
+```
+
+Or use the Makefile targets:
+
+```bash
+make test         # Uses CGO_ENABLED=0
+make test-clean   # Explicitly runs without CGO
+```
+
+### Option 2: Ignore the warnings
+
+These warnings are harmless and don't affect the functionality of your tests or binaries. They can be safely ignored.
+
+### Option 3: Use specific linker flags
+
+For production builds where CGO is required, you can suppress debug information:
+
+```bash
+go build -ldflags="-w -s" ./...
+```
+
+## When CGO is Required
+
+Some tests, particularly those using the race detector, require CGO:
+
+```bash
+make test-race   # Will show warnings but is necessary for race detection
+```
+
+## References
+
+- [Go Issue #61229](https://github.com/golang/go/issues/61229)
+- [Apple Developer Forums Discussion](https://developer.apple.com/forums/thread/737577)

examples/performance-optimized/main_test.go

@@ -69,6 +69,7 @@ func TestPerformanceOptimizedExample(t *testing.T) {
 	}
 
 	logger.FlushAll()
+	time.Sleep(50 * time.Millisecond) // Allow time for file to be created
 
 	// Verify log file was created
 	if stat, err := os.Stat("test_performance.log"); err != nil {

pkg/omni/goroutine_leak_test.go

@@ -153,22 +153,32 @@ func TestShutdownTimeout(t *testing.T) {
 		t.Fatalf("Failed to create logger: %v", err)
 	}
 
-	// Fill the message channel
-	for i := 0; i < 1000; i++ {
-		logger.Info("Message %d", i)
-	}
-
-	// Get initial goroutine count
+	// Get initial goroutine count before filling channel
 	initialGoroutines := runtime.NumGoroutine()
 
+	// Fill the message channel to ensure shutdown takes time
+	channelSize := 1000
+	for i := 0; i < channelSize*2; i++ {
+		select {
+		case logger.msgChan <- LogMessage{
+			Level:     LevelInfo,
+			Timestamp: time.Now(),
+			Format:    "Message %d",
+			Args:      []interface{}{i},
+		}:
+		default:
+			// Channel is full, which is what we want
+		}
+	}
+
 	// Shutdown with very short timeout
 	ctx, cancel := context.WithTimeout(context.Background(), 1*time.Millisecond)
 	err = logger.Shutdown(ctx)
 	cancel()
 
-	// Should get timeout error
-	if err == nil {
-		t.Error("Expected timeout error, got nil")
+	// Should get timeout error since channel is full and processing takes time
+	if err == nil || err != context.DeadlineExceeded {
+		t.Errorf("Expected context deadline exceeded error, got %v", err)
 	}
 
 	// Wait for background cleanup