RadioAstronomySoftwareGroup / RadioAstronomySoftwareGroup/pyuvdata
Break up long tests
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 102
- Forks
- 35
- Avg merge
- 1d 20h
- Merged PRs (30d)
- 14
Description
There are a number of really long tests in test_uvdata.py that really should be broken up, probably using the pytest's parametrize feature.
A non-exhaustive list identified in #967 includes:
test_reorder_freqs
test_sum_vis_errors
test_flex_spw_add_concat
test_break_add
test_smart_slicing
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start in test_uvdata.py and review the listed tests: test_reorder_freqs, test_sum_vis_errors, test_flex_spw_add_concat, test_break_add, and test_smart_slicing. Read the pytest parametrization guidance and identify which repeated cases can be expressed as parameters. Done means the long tests are broken into clearer parametrized tests while preserving their existing coverage.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- testing-qa
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100