pyuvdata
pyuvdata copied to clipboard
Break up long tests
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
FWIW, I've gone through and cleaned up all but test_smart_slicing in #979, either because I was cleaning them up anyways (per a request from @plaplant to break up some monolithic tests), or because I was the one who originally wrote the ultra-long test (opps). But there are a few more that are fairly long -- choosing a fairly arbitrary cutoff of 150 lines of non-blank lines (excluding @pytest calls), where there's a bit of a break in the distribution of test lengths:
test_add(339 lines; UVData version)test_select_with_ant_str(338 lines)test_fast_concat(334 lines)test_add(310 lines; UVBeam version)test_parse_ants(268 lines)test_add_drift(212 lines)test_miriad_location_handling(209 lines)test_conjugate_bls(205 lines)test_redundancy_contract_expand(200 lines)test_smart_slicing(193 lines)test_no_deg_units(175 lines)test_select_bls(162 lines)test_healpix_interpolation(153 lines)