Merge pull request #537 from pjenvey/wait_for_times-fix

test: don't hide wait_for_times test failures
mozilla-services · Jul 20, 2016 · 5548eb8 · 5548eb8
2 parents 0c81d69 + fe98d4c
commit 5548eb8
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/autopush/tests/test_websocket.py b/autopush/tests/test_websocket.py
@@ -1261,11 +1261,9 @@ def raise_exception(*args, **kwargs):
 
         def wait_for_times():  # pragma: nocover
             if len(self.proto.log.failure.mock_calls) > 0:
-                try:
-                    eq_(len(self.proto.log.failure.mock_calls), 1)
-                finally:
-                    d.callback(True)
-                    return
+                eq_(len(self.proto.log.failure.mock_calls), 1)
+                d.callback(True)
+                return
             reactor.callLater(0.1, wait_for_times)
 
         reactor.callLater(0.1, wait_for_times)