Unit tests for filehelpfunctions.collect_python_source_files()

[jurgenwigg]

I've noticed strange behaviour in the collect_python_source_files() function:

dirs are completely unused. Is that expected?

EXCLUDE_DIRS could be a global constant for this module.

[nocomplexity]

Hi there! Thanks again so much for the PR—this is a really helpful addition!

I’m excited to get this merged. Adding more tests to these functions is a win; It will help us catch any tricky side effects down the road and keep the codebase stable. More tests on some functions will improve unwanted side effects in future!

The variable dirs is used in this function. dirs is used as a control mechanism for traversal. That’s how the function now avoids unwanted directories efficiently. The use of os.walk() combined with the slice assignment dirs[:] is a way to controls which subdirectories the loop will enter next. EXCLUDE_DIRS is only used in this function. Maybe in future configurable for users without touching code in a visual UI.

Great work, and thanks again for your contribution!

Regards!

[nocomplexity]

Updated file header: (C) 2026 @jurgenwigg
I do not want credit for your work!
Regards!