3 Commits

Author	SHA1	Message	Date
0x1d	784f0f601f	fix: resolve all golangci-lint issues ... - Add package comments to all packages (pkg/config, pkg/logger, internal/*, cmd/platform) - Fix context key warnings by using custom ContextKey type - Export ContextKey type to avoid unexported-return warnings - Update all context value operations to use ContextKey instead of string - Update RequestIDKey() and UserIDKey() to return ContextKey - Fix error checking issues (errcheck) - Properly handle os.Chdir errors in defer statements - Properly handle os.Setenv/os.Unsetenv errors in tests - Fix security warnings (gosec) - Change directory permissions from 0755 to 0750 in tests - Change file permissions from 0644 to 0600 in tests - Fix unused parameter warnings (revive) - Replace unused parameters with _ in: * RegisterLifecycleHooks lifecycle functions * Mock logger implementations * noOpLogger methods - Fix type assertion issues (staticcheck) - Remove unnecessary type assertions in tests - Use simpler compile-time checks - Fix exported type stuttering warning - Add nolint directive for ConfigProvider (standard interface pattern) - Update golangci-lint configuration - Add version: 2 field (required for newer versions) - Remove unsupported linters (typecheck, gosimple) - Move formatters (gofmt, goimports) to separate formatters section - Simplify linter list to only well-supported linters All linting issues resolved (0 issues reported by golangci-lint). All tests pass and code compiles successfully.	2025-11-05 13:19:54 +01:00
0x1d	82707186a0	fix: remove parallel execution from Gin tests to prevent data races ... - Remove t.Parallel() from tests that use gin.SetMode() - gin.SetMode() modifies global state and is not thread-safe - Tests affected: * TestRequestIDMiddleware_GenerateNewID * TestRequestIDMiddleware_UseExistingID * TestLoggingMiddleware * TestLoggingMiddleware_WithRequestID * TestRequestIDMiddleware_MultipleRequests - Add comments explaining why these tests cannot run in parallel - All tests now pass with race detector enabled (-race flag) This fixes data race warnings that were occurring when running tests with the race detector, specifically when multiple tests tried to set Gin's mode concurrently.	2025-11-05 13:01:27 +01:00
0x1d	0bfdb2c2d7	Add comprehensive test suite for current implementation ... - Add tests for internal/config package (90.9% coverage) - Test all viperConfig getter methods - Test LoadConfig with default and environment-specific configs - Test error handling for missing config files - Add tests for internal/di package (88.1% coverage) - Test Container lifecycle (NewContainer, Start, Stop) - Test providers (ProvideConfig, ProvideLogger, CoreModule) - Test lifecycle hooks registration - Include mock implementations for testing - Add tests for internal/logger package (96.5% coverage) - Test zapLogger with JSON and console formats - Test all logging levels and methods - Test middleware (RequestIDMiddleware, LoggingMiddleware) - Test context helper functions - Include benchmark tests - Update CI workflow to skip tests when no test files exist - Add conditional test execution based on test file presence - Add timeout for test execution - Verify build when no tests are present All tests follow Go best practices with table-driven patterns, parallel execution where safe, and comprehensive coverage.	2025-11-05 12:45:37 +01:00