Remove todo tests.

debugmm · Mar 9, 2021 · 083745c · 083745c
1 parent 097364e
commit 083745c
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 49 deletions.
diff --git a/src/twisted/newsfragments/10122.misc b/src/twisted/newsfragments/10122.misc
diff --git a/src/twisted/trial/test/test_assertions.py b/src/twisted/trial/test/test_assertions.py
@@ -1224,21 +1224,6 @@ def test_successResultOfWithException(self):
             self.failureException, self.successResultOf, self.raisesException()
         )
 
-    def test_successResultOfWithFailureHasTraceback(self):
-        """
-        L{SynchronousTestCase.successResultOf} raises a
-        L{SynchronousTestCase.failureException} that has the original failure
-        traceback when called with a coroutine with a failure result.
-        """
-        try:
-            self.successResultOf(self.raisesException())
-        except self.failureException as e:
-            self.assertIn(self.failure.getTraceback(), str(e))
-
-    test_successResultOfWithFailureHasTraceback.todo = (  # type: ignore[attr-defined]
-        "Tracebacks aren't preserved by exceptions later wrapped in Failures"
-    )
-
     def test_failureResultOfWithoutResult(self):
         """
         L{SynchronousTestCase.failureResultOf} raises
@@ -1290,23 +1275,6 @@ def test_failureResultOfWithWrongExceptionOneExpectedException(self):
                 str(e),
             )
 
-    def test_failureResultOfWithWrongExceptionOneExpectedExceptionHasTB(self):
-        """
-        L{SynchronousTestCase.failureResultOf} raises
-        L{SynchronousTestCase.failureException} when called with a coroutine
-        that raises an exception with a failure type that was not expected, and
-        the L{SynchronousTestCase.failureException} message contains the
-        original exception traceback.
-        """
-        try:
-            self.failureResultOf(self.raisesException(), KeyError)
-        except self.failureException as e:
-            self.assertIn(self.failure.getTraceback(), str(e))
-
-    test_failureResultOfWithWrongExceptionOneExpectedExceptionHasTB.todo = (  # type: ignore[attr-defined]
-        "Tracebacks aren't preserved by exceptions later wrapped in Failures"
-    )
-
     def test_failureResultOfWithWrongExceptionMultiExpectedExceptions(self):
         """
         L{SynchronousTestCase.failureResultOf} raises
@@ -1328,23 +1296,6 @@ def test_failureResultOfWithWrongExceptionMultiExpectedExceptions(self):
                 str(e),
             )
 
-    def test_failureResultOfWithWrongExceptionMultiExpectedExceptionsHasTB(self):
-        """
-        L{SynchronousTestCase.failureResultOf} raises
-        L{SynchronousTestCase.failureException} when called with a coroutine
-        that raises an exception of a type that was not expected, and the
-        L{SynchronousTestCase.failureException} message contains the original
-        exception traceback in the error message.
-        """
-        try:
-            self.failureResultOf(self.raisesException(), KeyError, IOError)
-        except self.failureException as e:
-            self.assertIn(self.failure.getTraceback(), str(e))
-
-    test_failureResultOfWithWrongExceptionMultiExpectedExceptionsHasTB.todo = (  # type: ignore[attr-defined]
-        "Tracebacks aren't preserved by exceptions later wrapped in Failures"
-    )
-
     def test_successResultOfWithSuccessResult(self):
         """
         When passed a coroutine which currently has a result (ie, if converted