Merge pull request #665 from bsipocz/TST_vizier_updates

TST: updating tests for vizier changes

Author: ManonMarchand

docs/registry/index.rst

@@ -254,24 +254,23 @@ constraint on the description ``get_service(service_type='conesearch', keyword='
 
   >>> for interface in voresource.list_interfaces():
   ...     print(interface)
-  Interface(type='tap#aux', description='', url='http://tapvizier.cds.unistra.fr/TAPVizieR/tap')
-  Interface(type='vr:webbrowser', description='', url='http://vizier.cds.unistra.fr/viz-bin/VizieR-2?-source=J/ApJ/727/14')
+  Interface(type='tap#aux', description='', url='https://tapvizier.cds.unistra.fr/TAPVizieR/tap')
   Interface(type='conesearch', description='Cone search capability for table J/ApJ/727/14/table2 (AKARI IRC 3-24{mu}m, and Spitzer MIPS 24/70{mu}m photometry of Abell 2255 member galaxies)', url='https://vizier.cds.unistra.fr/viz-bin/conesearch/J/ApJ/727/14/table2?')
 
 Or construct the service object directly from the list of interfaces with:
 
 .. doctest-remote-data::
 
   >>> voresource.list_interfaces()[0].to_service()
-  TAPService(baseurl : 'http://tapvizier.cds.unistra.fr/TAPVizieR/tap', description : '')
+  TAPService(baseurl : 'https://tapvizier.cds.unistra.fr/TAPVizieR/tap', description : '')
 
 The list of interfaces can also be filtered to interfaces corresponding to services of a
 specific service type:
 
 .. doctest-remote-data::
 
   >>> voresource.list_interfaces("tap")
-  [Interface(type='tap#aux', description='', url='http://tapvizier.cds.unistra.fr/TAPVizieR/tap')]
+  [Interface(type='tap#aux', description='', url='https://tapvizier.cds.unistra.fr/TAPVizieR/tap')]
 
 To operate TAP services, you need to know what tables make up a
 resource; you could construct a TAP service and access its ``tables``

pyvo/registry/tests/test_regtap.py

@@ -314,6 +314,7 @@ class _NS:
 
     We need this here to let us conveniently construct _FakeResults.
     """
+
     def __init__(self, **kwargs):
         for k, v in kwargs.items():
             setattr(self, k, v)
@@ -326,6 +327,7 @@ class _FakeResults:
     It is constructed with a dictionary that should eventually be
     used as the mapping in the Record.
     """
+
     def __init__(self, valdict):
         self.fieldnames = list(valdict.keys())
         self.resultstable = _NS(array=_NS(data=[list(valdict.values())]))
@@ -727,7 +729,7 @@ def test_get_alt_identifier():
     rsc = _makeRegistryRecord(ivoid="ivo://cds.vizier/i/337")
     assert set(rsc.get_alt_identifiers()) == {
         'doi:10.26093/cds/vizier.1337',
-        'bibcode:2016yCat.1337....0G'}
+    }
 
 
 @pytest.mark.remote_data