You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Fix: Windows database cleanup issue in secret tests
The tests were failing on Windows with:
'The process cannot access the file because it is being used by another process'
Root cause:
- Runtime creates a BBolt database that must be explicitly closed
- On Windows, locked files cannot be deleted during test cleanup
- Tests were not calling rt.Close() before finishing
Fixed by:
- Adding 'defer rt.Close()' to both TestNotifySecretsChanged tests
- This ensures the database is properly closed before TempDir cleanup
- Tested locally - all tests pass
Fixes Windows test failures:
- Unit Tests (windows-latest, 1.24.5)
- End-to-End Tests (windows-latest, 1.23.10)
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <[email protected]>