[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[tor-commits] [stem/master] Compare exit policy strings in unit tests



commit 45e88d45da2e5693108d56ac4a44884b45045e0a
Author: Damian Johnson <atagar@xxxxxxxxxxxxxx>
Date:   Tue Sep 25 15:56:55 2018 -0700

    Compare exit policy strings in unit tests
    
    Oops, my prior commit made jenkins sad...
    
            ======================================================================
            FAIL: test_get_exit_policy_if_not_relaying
            ----------------------------------------------------------------------
            Traceback (most recent call last):
              File "/usr/lib/python2.7/dist-packages/mock/mock.py", line 1305, in patched
              return func(*args, **keywargs)
              File "/srv/jenkins-workspace/workspace/stem-tor-ci/test/unit/control/controller.py", line 247, in test_get_exit_policy_if_not_relaying
              self.assertEqual(expected, self.controller.get_exit_policy())
            AssertionError: <stem.exit_policy.ExitPolicy object at 0x7f680846e550> != <stem.exit_policy.ExitPolicy object at 0x7f6808399650>
    
            ----------------------------------------------------------------------
    
    No doubt I'm missing a mock. This isn't gonna fix that, but this should make
    the assertion failure more helpful.
---
 test/unit/control/controller.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/test/unit/control/controller.py b/test/unit/control/controller.py
index 0ae9fafd..29be956f 100644
--- a/test/unit/control/controller.py
+++ b/test/unit/control/controller.py
@@ -196,7 +196,7 @@ class TestControl(unittest.TestCase):
       'accept *:*',
     )
 
-    self.assertEqual(expected, self.controller.get_exit_policy())
+    self.assertEqual(str(expected), str(self.controller.get_exit_policy()))
 
   @patch('stem.control.Controller.get_info')
   @patch('stem.control.Controller.get_conf')
@@ -244,10 +244,10 @@ class TestControl(unittest.TestCase):
     get_info_mock.side_effect = getinfo_response
 
     exit_policy_exception = stem.OperationFailed('552', 'Not running in server mode')
-    self.assertEqual(expected, self.controller.get_exit_policy())
+    self.assertEqual(str(expected), str(self.controller.get_exit_policy()))
 
     exit_policy_exception = stem.OperationFailed('551', 'Descriptor still rebuilding - not ready yet')
-    self.assertEqual(expected, self.controller.get_exit_policy())
+    self.assertEqual(str(expected), str(self.controller.get_exit_policy()))
 
   @patch('stem.control.Controller.get_info')
   @patch('stem.control.Controller.get_conf')

_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits