Merge pull request #332 from msdemlei/jobs_warn

Fixing jobs UnknownElement warning

pyvo/io/uws/__init__.py

@@ -1,5 +1,5 @@
 # Licensed under a 3-clause BSD style license - see LICENSE.rst
-__all__ = ['parse_job', 'parse_job_list', 'JobFile', 'JobList']
+__all__ = ['parse_job', 'parse_job_list', 'JobFile']
 
 from .endpoint import *
 from .tree import *

pyvo/io/uws/endpoint.py

@@ -8,7 +8,7 @@
 from astropy.io.votable.util import convert_to_writable_filelike
 
 from ...utils.xml.elements import xmlattribute, parse_for_object
-from .tree import JobSummary, JobList
+from .tree import JobSummary, Jobs
 
 __all__ = ["parse_job", "parse_job_list", "JobFile"]
 
@@ -18,7 +18,7 @@ def parse_job_list(
 ):
     """
     Parses a job xml file (or file-like object), and returns a
-    `~pyvo.io.uws.tree.JobList` object.
+    `~pyvo.io.uws.tree.Jobs` object.
 
     Parameters
     ----------
@@ -39,14 +39,14 @@ def parse_job_list(
 
     Returns
     -------
-    `~pyvo.io.uws.tree.JobList` object
+    `~pyvo.io.uws.tree.Jobs` object
 
     See also
     --------
     pyvo.io.vosi.exceptions : The exceptions this function may raise.
     """
-    return parse_for_object(source, JobList, pedantic, filename,
-                            _debug_python_based_parser)
+    return parse_for_object(source, Jobs, pedantic, filename,
+                            _debug_python_based_parser).joblist
 
 
 def parse_job(

pyvo/io/uws/tree.py

@@ -278,17 +278,27 @@ def message(self, message):
         self._message = message
 
 
-class JobList(UWSElement, HomogeneousList):
+class Jobs(HomogeneousList, UWSElement):
+    """A parsed representation of the joblist endpoint.
+    """
     def __init__(self, config=None, pos=None, _name='jobs', **kwargs):
         HomogeneousList.__init__(self, JobSummary)
         UWSElement.__init__(self, config, pos, _name, **kwargs)
 
-    @uwselement(name='jobref')
+    @uwselement
     def jobs(self):
         return self
 
     @jobs.adder
     def jobs(self, iterator, tag, data, config, pos):
+        return
+
+    @uwselement(name='jobref')
+    def joblist(self):
+        return self
+
+    @joblist.adder
+    def joblist(self, iterator, tag, data, config, pos):
         job = JobSummary(config, pos, 'jobref', **data)
         job.parse(iterator, config)
         self.append(job)