Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
I noticed that some of our
test_*.py
files were not being linted viamake test
ormake lint
.Due to a known issue,
pylint
requires__init__.py
files in directories in order to automatically discover them and test the modules in them. See e.g. pylint-dev/pylint#352. A--recursive
flag was added to a recent version (see pylint-dev/pylint#3528) that was supposed to lift this requirement, but I cannot seem to make it work.So, for now, I've added the missing
__init__.py
files underuwtools/tests
and fixed the couple of linter errors that were then detected.Type of change
How Has This Been Tested?
Passes
make test
.Checklist