Skip to content

Commit 9e82ab7

Browse files
committed
parameterize tests for various releases
1 parent 95c016d commit 9e82ab7

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

astroquery/sdss/tests/test_sdss.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,7 @@ def data_path(filename):
125125
coords_column = Column(coords_list, name='coordinates')
126126

127127
# List of all data releases.
128-
dr_list = list(range(1, sdss.conf.default_release + 1))
128+
dr_list = list(range(1, sdss.conf.default_release + 3))
129129

130130

131131
# We are not testing queries for DR11 because it is not easily available to

astroquery/sdss/tests/test_sdss_remote.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@
1111
from ... import sdss
1212
from ...exceptions import TimeoutError
1313

14+
dr_list = list(range(8, sdss.conf.default_release + 3))
1415

1516
@pytest.mark.remote_data
1617
class TestSDSSRemote:
@@ -34,8 +35,9 @@ def test_images_timeout(self):
3435
"error with 'No route to host'. We don't know a "
3536
"workaround for this yet.")
3637

37-
def test_sdss_spectrum(self):
38-
xid = sdss.SDSS.query_region(self.coords, spectro=True)
38+
@pytest.mark.parametrize("dr", dr_list)
39+
def test_sdss_spectrum(self, dr):
40+
xid = sdss.SDSS.query_region(self.coords, spectro=True, data_release=dr)
3941
assert isinstance(xid, Table)
4042
sp = sdss.SDSS.get_spectra(matches=xid)
4143

0 commit comments

Comments
 (0)