Priority: Low
Description
Several important code paths lack test coverage:
- No test for
deps with details=true — the most complex and risky code path in deps.go
- No test for concurrent
RunSCC calls — would validate the mutex correctness
- No negative-path test for
HandleStats with a non-existent directory
compare and checklist tests are thin — only empty-input and basic smoke tests, no validation of output content/structure beyond non-emptiness
Suggested additions
- Test
HandleDeps with details=true on a temp dir with a requirements.txt
- Test concurrent
RunSCC calls with t.Parallel() or sync.WaitGroup
- Test
HandleStats with a path that doesn't exist
- Test
HandleChecklist output contains expected categories
- Test
HandleCompare guidance contains key instructions
Priority: Low
Description
Several important code paths lack test coverage:
depswithdetails=true— the most complex and risky code path in deps.goRunSCCcalls — would validate the mutex correctnessHandleStatswith a non-existent directorycompareandchecklisttests are thin — only empty-input and basic smoke tests, no validation of output content/structure beyond non-emptinessSuggested additions
HandleDepswithdetails=trueon a temp dir with arequirements.txtRunSCCcalls witht.Parallel()orsync.WaitGroupHandleStatswith a path that doesn't existHandleChecklistoutput contains expected categoriesHandleCompareguidance contains key instructions