Code changes are here: #57
Fixes panic in tests:
Removes redundant t.Parallel() calls at the top level of test functions that use subtests, ensuring t.Parallel() is only called within subtest closures. This prevents the panic: testing: t.Parallel called multiple times error.
Test structure improvements:
Refactors table-driven tests to use proper parallelization patterns, increasing test isolation and reliability.
General test cleanup:
Ensures consistent error handling and assertion usage across test files.
Fixes race conditions and panics in tests caused by improper use of t.Parallel().
Improves test maintainability and reliability.
In addition to test changes
- Injects logger to all appropriate functions
- main.go is now modular and readable
- Removed deferrun dep which was dealing with signals to cleanup