[tor-commits] [stem/master] Require online target for server descriptor integ test

[atagar at torproject.org]

commit e700b3873be211814c01f8fadc86b76e5912c876
Author: Damian Johnson <atagar at torproject.org>
Date:   Sat Feb 8 13:12:17 2020 -0800

    Require online target for server descriptor integ test
    
    Odd, we have both a test_get_server_descriptor and test_get_server_descriptors
    (plural) integration test but only the later required the online target. Here
    at a coffee shop I got the following so adding this requirement to the other as
    well.
    
      ======================================================================
      ERROR: test_get_server_descriptor
      ----------------------------------------------------------------------
      Traceback (most recent call last):
        File "/home/atagar/Desktop/stem/test/require.py", line 58, in wrapped
          return func(self, *args, **kwargs)
        File "/home/atagar/Desktop/stem/test/integ/control/controller.py", line 1301, in test_get_server_descriptor
          test_relay = self._get_router_status_entry(controller)
        File "/home/atagar/Desktop/stem/test/integ/control/controller.py", line 1489, in _get_router_status_entry
          for desc in controller.get_network_statuses():
        File "/home/atagar/Desktop/stem/stem/control.py", line 482, in wrapped
          for val in func(self, *args, **kwargs):
        File "/home/atagar/Desktop/stem/stem/control.py", line 1979, in get_network_statuses
          raise stem.DescriptorUnavailable('Descriptor information is unavailable, tor might still be downloading it')
      stem.DescriptorUnavailable: Descriptor information is unavailable, tor might still be downloading it
---
 test/integ/control/controller.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/test/integ/control/controller.py b/test/integ/control/controller.py
index 239e0bd7..b6bf9c9a 100644
--- a/test/integ/control/controller.py
+++ b/test/integ/control/controller.py
@@ -1281,6 +1281,7 @@ class TestController(unittest.TestCase):
           break
 
   @test.require.controller
+  @test.require.online
   def test_get_server_descriptor(self):
     """
     Basic checks for get_server_descriptor().