update load_local_pkg exclude regex with regtest pattern #8350
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.
Fixes #8348
test_suffix.py
tests that calljwst.stpipe.utilities.find_suffixes
are skipped in unit test runs due toimportorskip
calls in the regtests (like):jwst/jwst/regtest/test_stp_jw00697013.py
Line 30 in bf3b06f
pysiaf
is not installed for CI unit test runs and the call tofind_suffixes
results in an attempt to import every.py
file in thejwst
directory structure that is not in atest
folder (soregtest
is included). This results in an attempt to import for example the above referenced file, the importorskip is called and the test (in this case one intest_suffix
) is skipped.This PR updates the regex to also exclude the
regtest
directory.As a comparison here is the latest
py310-xdist
run log on main:https://github.com/spacetelescope/jwst/actions/runs/8238566482/job/22529809746#step:10:248
showing:
With this PR the
py310-xdist
run log:https://github.com/spacetelescope/jwst/actions/runs/8254717816/job/22579513473?pr=8350
shows 2 tests were unskipped:
The 2 tests that now run in the unit tests CI are:
Checklist for maintainers
CHANGES.rst
within the relevant release sectionHow to run regression tests on a PR