Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mark slow tests as slow #3209

Open
stscijirahubjdat opened this issue Oct 1, 2024 · 0 comments
Open

Mark slow tests as slow #3209

stscijirahubjdat opened this issue Oct 1, 2024 · 0 comments
Labels

Comments

@stscijirahubjdat
Copy link

stscijirahubjdat commented Oct 1, 2024

Issue JDAT-4826 was created on JIRA by Pey-Lian Lim:

At the time of writing, these are the top 10 slowest tests (without the flag --run-slow). I propose we mark any test function longer than 5s with @⁣pytest.mark.slow so they do not run unless --run-slow is explicitly provided. Either that or refactor test or functional code to make the affected tests faster.

Motivation: Make local run less painful with faster turnaround time for debugging.

$ pytest --remote-data --pyargs jdaviz -vvv --durations=10
...
================================================= slowest 10 durations =================================================
331.60s call     jdaviz/core/tests/test_autoconfig.py::test_launcher
60.56s call     jdaviz/configs/cubeviz/plugins/spectral_extraction/tests/test_spectral_extraction.py::test_spectral_extraction_scientific_validation[4.85-aperture1-0.01-mast:jwst/product/jw01050-o003_t005_miri_ch1-shortmediumlong_s3d.fits-https://archive.stsci.edu/hlsps/reference-atlases/cdbs/current_calspec/hd159222_mod_007.fits]
44.01s call     jdaviz/configs/default/plugins/data_quality/tests/test_data_quality.py::test_cubeviz_layer_visibility_bug
42.13s call     jdaviz/configs/default/plugins/data_quality/tests/test_data_quality.py::test_data_quality_plugin
39.51s call     jdaviz/configs/cubeviz/plugins/spectral_extraction/tests/test_spectral_extraction.py::test_spectral_extraction_scientific_validation[4.85-aperture0-1e-05-mast:jwst/product/jw01524-o003_t002_miri_ch1-shortmediumlong_s3d.fits-https://archive.stsci.edu/hlsps/reference-atlases/cdbs/current_calspec/delumi_mod_005.fits]
20.04s call     jdaviz/configs/cubeviz/plugins/moment_maps/tests/test_moment_maps.py::test_momentmap_nirspec_prism
7.61s call     jdaviz/configs/mosviz/tests/test_parsers.py::test_nirspec_level2_parser
2.82s call     jdaviz/configs/default/plugins/data_quality/tests/test_data_quality.py::test_data_quality_plugin_hst_wfc3
2.77s call     jdaviz/configs/specviz2d/plugins/spectral_extraction/tests/test_spectral_extraction.py::test_horne_extract_self_profile
2.68s setup    jdaviz/configs/imviz/tests/test_simple_aper_phot.py::TestAdvancedAperPhot::test_sky_background[Subset 2-12.269274711608887-gauss100_fits_wcs_block_reduced[PRIMARY,1]-4]
@pllim pllim added the testing label Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants