Skip to content

Conversation

@tieneupin
Copy link
Contributor

In our actual production environment, the directories containing data will be interspersed with other directories that we don't have access to, which will lead to PermissionError occurences during the recursive search.

This PR nests the directory scanning logic in a try-except block, logging a warning of the directory that could not be accessed before safely moving on to try the other directories until the maximum depth set is reached.

@tieneupin tieneupin self-assigned this Jan 19, 2026
@tieneupin tieneupin added the bug Something isn't working label Jan 19, 2026
@codecov
Copy link

codecov bot commented Jan 19, 2026

Codecov Report

❌ Patch coverage is 77.77778% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 46.11%. Comparing base (a28e2fb) to head (7c9544c).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #731      +/-   ##
==========================================
+ Coverage   46.03%   46.11%   +0.07%     
==========================================
  Files          91       91              
  Lines        9638     9656      +18     
  Branches     1261     1265       +4     
==========================================
+ Hits         4437     4453      +16     
- Misses       4980     4981       +1     
- Partials      221      222       +1     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tieneupin tieneupin merged commit 467c2d0 into main Jan 19, 2026
17 checks passed
@tieneupin tieneupin deleted the safe-recursive-search branch January 19, 2026 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants