Merge pull request #20012 from eric-wieser/remove-xfail

Fix sympy.testing.pytest.raises to behave like pytest.raises

sympy/multipledispatch/tests/test_core.py

@@ -2,7 +2,7 @@
 
 from sympy.multipledispatch import dispatch
 from sympy.multipledispatch.conflict import AmbiguityWarning
-from sympy.testing.pytest import raises, XFAIL, warns
+from sympy.testing.pytest import raises, warns
 from functools import partial
 
 test_namespace = dict()  # type: Dict[str, Any]
@@ -11,7 +11,6 @@
 dispatch = partial(dispatch, namespace=test_namespace)
 
 
-@XFAIL
 def test_singledispatch():
     @dispatch(int)
     def f(x): # noqa:F811
@@ -66,7 +65,6 @@ def f(x): # noqa:F811
     assert f(C()) == 'a'
 
 
-@XFAIL
 def test_inheritance_and_multiple_dispatch():
     @dispatch(A, A)
     def f(x, y): # noqa:F811

sympy/multipledispatch/tests/test_dispatcher.py

@@ -1,7 +1,7 @@
 from sympy.multipledispatch.dispatcher import (Dispatcher, MDNotImplementedError,
                                          MethodDispatcher, halt_ordering,
                                          restart_ordering)
-from sympy.testing.pytest import raises, XFAIL, warns
+from sympy.testing.pytest import raises, warns
 
 
 def identity(x):
@@ -88,7 +88,6 @@ def on_ambiguity(dispatcher, amb):
     assert ambiguities[0]
 
 
-@XFAIL
 def test_raise_error_on_non_class():
     f = Dispatcher('f')
     assert raises(TypeError, lambda: f.add((1,), inc))
@@ -165,7 +164,6 @@ def two(x, y):
     assert 'x - y' in f._source(1.0, 1.0)
 
 
-@XFAIL
 def test_source_raises_on_missing_function():
     f = Dispatcher('f')
 
@@ -197,13 +195,11 @@ def func(*args):
     assert set(f.ordering) == {(int, object), (object, int)}
 
 
-@XFAIL
 def test_no_implementations():
     f = Dispatcher('f')
     assert raises(NotImplementedError, lambda: f('hello'))
 
 
-@XFAIL
 def test_register_stacking():
     f = Dispatcher('f')
 
@@ -238,7 +234,6 @@ class MyList(list):
     assert f.dispatch(int, int) is add
 
 
-@XFAIL
 def test_not_implemented():
     f = Dispatcher('f')
 
@@ -259,7 +254,6 @@ def _(x):
     assert raises(NotImplementedError, lambda: f(1, 2))
 
 
-@XFAIL
 def test_not_implemented_error():
     f = Dispatcher('f')
 

sympy/testing/pytest.py

@@ -34,6 +34,15 @@
     # Not using pytest so define the things that would have been imported from
     # there.
 
+    # _pytest._code.code.ExceptionInfo
+    class ExceptionInfo:
+        def __init__(self, value):
+            self.value = value
+
+        def __repr__(self):
+            return "<ExceptionInfo {!r}>".format(self.value)
+
+
     def raises(expectedException, code=None):
         """
         Tests that ``code`` raises the exception ``expectedException``.
@@ -54,6 +63,7 @@ def raises(expectedException, code=None):
         >>> from sympy.testing.pytest import raises
 
         >>> raises(ZeroDivisionError, lambda: 1/0)
+        <ExceptionInfo ZeroDivisionError(...)>
         >>> raises(ZeroDivisionError, lambda: 1/2)
         Traceback (most recent call last):
         ...
@@ -92,8 +102,8 @@ def raises(expectedException, code=None):
         elif callable(code):
             try:
                 code()
-            except expectedException:
-                return
+            except expectedException as e:
+                return ExceptionInfo(e)
             raise Failed("DID NOT RAISE")
         elif isinstance(code, str):
             raise TypeError(