Eliminate dead code by using pytests.raises instead of try/except/else

diff --git a/tests/test_api.py b/tests/test_api.py
index 343d1de6e2de672a5a390921c9e63bde4d61bf69..e37d5cfcaccfbb567837c7302320c6be4c8f3c4e 100644 (file)
--- a/tests/test_api.py
+++ b/tests/test_api.py
@@ -223,12 +223,8 @@ class TestUndefined(object):
         pytest.raises(UndefinedError, t.render, test=test)
 
     def test_undefined_and_special_attributes(self):
-        try:
+        with pytest.raises(AttributeError):
             Undefined('Foo').__dict__
-        except AttributeError:
-            pass
-        else:
-            assert False, "Expected actual attribute error"
 
     def test_logging_undefined(self):
         _messages = []

diff --git a/tests/test_security.py b/tests/test_security.py
index 5c8639c4efac5d940ae6fb3b74a52a2be4a161e5..6b2743fc2a968b92f40b96ad0dd915877b233f69 100644 (file)
--- a/tests/test_security.py
+++ b/tests/test_security.py
@@ -140,12 +140,8 @@ class TestSandbox(object):
             assert t.render(ctx) == rv
             env.intercepted_binops = frozenset(['+'])
             t = env.from_string('{{ %s }}' % expr)
-            try:
+            with pytest.raises(TemplateRuntimeError):
                 t.render(ctx)
-            except TemplateRuntimeError as e:
-                pass
-            else:
-                assert False, 'expected runtime error'
 
     def test_unary_operator_intercepting(self, env):
         def disable_op(arg):
@@ -157,12 +153,8 @@ class TestSandbox(object):
             assert t.render(ctx) == rv
             env.intercepted_unops = frozenset(['-'])
             t = env.from_string('{{ %s }}' % expr)
-            try:
+            with pytest.raises(TemplateRuntimeError):
                 t.render(ctx)
-            except TemplateRuntimeError as e:
-                pass
-            else:
-                assert False, 'expected runtime error'
 
 
 @pytest.mark.sandbox