[tor-commits] [stem/master] Better output for failing connection tests

atagar at torproject.org atagar at torproject.org
Thu Jul 27 16:30:21 UTC 2017


commit b40029c8e34069e80612439e2716b6b1bd81d6cf
Author: Damian Johnson <atagar at torproject.org>
Date:   Thu Jul 27 09:31:03 2017 -0700

    Better output for failing connection tests
    
    Presently a test for sockstat is failing on our jenkins automated tests. This
    isn't a fix for that, but this does fix the failure handling which is masking
    the issue...
    
      ======================================================================
      ERROR: test_connections_by_sockstat
      ----------------------------------------------------------------------
      Traceback (most recent call last):
        File "/srv/jenkins-workspace/workspace/stem-tor-ci/test/integ/util/connection.py", line 51, in test_connections_by_sockstat
          self.check_resolver(Resolver.SOCKSTAT)
        File "/srv/jenkins-workspace/workspace/stem-tor-ci/test/require.py", line 58, in wrapped
          return func(self, *args, **kwargs)
        File "/srv/jenkins-workspace/workspace/stem-tor-ci/test/integ/util/connection.py", line 33, in check_resolver
          self.fail('Unable to find localhost connection with %s:\n%s' % (resolver, '\n'.join(connections)))
      TypeError: sequence item 0: expected string, Connection found
---
 test/integ/util/connection.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/integ/util/connection.py b/test/integ/util/connection.py
index 9f2a9aeb..563cfbeb 100644
--- a/test/integ/util/connection.py
+++ b/test/integ/util/connection.py
@@ -30,7 +30,7 @@ class TestConnection(unittest.TestCase):
         if conn.local_address == '127.0.0.1' and conn.local_port == test.runner.CONTROL_PORT:
           return
 
-      self.fail('Unable to find localhost connection with %s:\n%s' % (resolver, '\n'.join(connections)))
+      self.fail('Unable to find localhost connection with %s:\n%s' % (resolver, '\n'.join(map(str, connections))))
 
   def test_connections_by_proc(self):
     self.check_resolver(Resolver.PROC)



More information about the tor-commits mailing list