refactor tests; add remote

Author: keflavich

astroquery/linelists/cdms/tests/test_cdms.py

@@ -138,7 +138,7 @@ def test_hc7n(patch_post):
     CDMS.query_lines(200*u.GHz, 230.755608*u.GHz, molecule='HC7N',parse_name_locally=True)
     """
 
-    tbl = CDMS.query_lines(200*u.GHz, 230.755608*u.GHz, molecule='HC7N', parse_name_locally=True)
+    tbl = CDMS.query_lines(200*u.GHz, 230.755608*u.GHz, molecule='HC7N')
     assert isinstance(tbl, Table)
     assert len(tbl) == 27
     assert set(tbl.keys()) == colname_set

astroquery/linelists/cdms/tests/test_cdms_remote.py

@@ -72,3 +72,32 @@ def test_molecule_with_parens():
             assert tbl[0][col].mask
         else:
             assert tbl[0][col] == val
+
+
+@pytest.mark.remote_data
+def test_complex_molecule_remote():
+    """
+    Part of the regression test for 2409.  See "test_hc7n" in the non-remote
+    tests.  This version covers both the local name parsing and checks whether
+    there are upstream changes.
+    """
+    tbl = CDMS.query_lines(200*u.GHz, 230.755608*u.GHz, molecule='HC7N', parse_name_locally=True)
+    assert isinstance(tbl, Table)
+    assert len(tbl) == 27
+    assert set(tbl.keys()) == colname_set
+
+    assert tbl['FREQ'][0] == 200693.406
+    assert tbl['ERR'][0] == 0.01
+    assert tbl['LGINT'][0] == -2.241
+    assert tbl['MOLWT'][0] == 99
+
+    assert tbl['GUP'][0] == 1071
+    assert tbl['Ju'][0] == 178
+    assert tbl['Jl'][0] == 177
+    assert tbl['vu'][0].mask
+    assert tbl['vl'][0].mask
+    assert tbl['Ku'][0].mask
+    assert tbl['Kl'][0].mask
+    assert tbl['F1u'][0].mask
+    assert tbl['F1l'][0].mask
+    assert tbl['Lab'][0]