small refactor to jplspec to handle error messages better and be

symmetric with CDMS

Author: keflavich

astroquery/jplspec/core.py

@@ -104,7 +104,7 @@ def query_lines_async(self, min_frequency, max_frequency,
         if molecule is not None:
             if parse_name_locally:
                 self.lookup_ids = build_lookup()
-                payload['Mol'] = tuple(self.lookup_ids.find(molecule, flags))
+                payload['Mol'] = tuple(self.lookup_ids.find(molecule, flags).values())
                 if len(molecule) == 0:
                     raise ValueError('No matching species found. Please\
                                      refine your search or read the Docs\
@@ -156,6 +156,9 @@ def _parse_result(self, response, verbose=False):
         QN":   Quantum numbers for the lower state.
         """
 
+        if 'Zero lines were found' in response.text:
+            raise ValueError(f"Response was empty; message was '{response.text}'.")
+
         # data starts at 0 since regex was applied
         # Warning for a result with more than 1000 lines:
         # THIS form is currently limited to 1000 lines.

astroquery/jplspec/lookup_table.py

@@ -35,4 +35,4 @@ def find(self, s, flags):
             if match:
                 out[k] = v
 
-        return out.values()
+        return out