shankar0123 01607f8614 fix: scheduler race — track loop goroutines in WaitGroup ...

Root cause: WaitForCompletion only waited for work goroutines (wg),
but the 5-6 loop goroutines (renewalCheckLoop, jobProcessorLoop, etc.)
were not tracked. After cancel() + WaitForCompletion(), loop goroutines
could still be alive accessing scheduler/mock fields when the next test
started, triggering the race detector.

Fix:
- Start() now adds loop goroutines to wg, so WaitForCompletion blocks
  until both work items AND loops have fully exited
- Removed untracked 100ms timer goroutine for startedChan — now closed
  immediately after launching loops
- Timeout test updated: uses blockCh (ignores context) instead of
  slowDelay (respects context) so it reliably triggers the timeout path

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

2026-03-27 23:31:52 -04:00

..

api

fix(ci): resolve 9 remaining staticcheck issues

2026-03-27 23:20:28 -04:00

cli

feat: M17 OpenSSL/Custom CA issuer connector + M16b CLI tool with bulk import

2026-03-23 18:12:40 -04:00

config

docs: synchronize project documentation with codebase

2026-03-27 22:28:54 -04:00

connector

fix(ci): resolve 185 golangci-lint v2 issues — fix unused, tune config

2026-03-27 23:18:04 -04:00

domain

test(security): TICKET-018 add fuzz tests for command validation and domain parsing

2026-03-27 21:40:49 -04:00

integration

fix: resolve test compilation and runtime failures across codebase

2026-03-27 22:53:46 -04:00

mcp

fix: Go 1.25 upgrade, codebase audit fixes, MCP server tests

2026-03-23 17:36:25 -04:00

repository

fix(ci): restore certs variable declaration in discovery repo test

2026-03-27 23:22:00 -04:00

scheduler

fix: scheduler race — track loop goroutines in WaitGroup

2026-03-27 23:31:52 -04:00

service

fix(ci): resolve 9 remaining staticcheck issues

2026-03-27 23:20:28 -04:00

validation

fix: resolve test compilation and runtime failures across codebase

2026-03-27 22:53:46 -04:00