twisted.test.test_failure.FailureTestCase.testBrokenStr breaks trial in debug mode

[exarkun]

exarkun@kunai:/tmp$ trial -b twisted.test.test_failure
Running 9 tests.
> /home/exarkun/Projects/Twisted/trunk/twisted/trial/runner.py(284)run()
-> try:
(Pdb) c
twisted.test.test_failure
  FailureTestCase
    testBrokenStr ... Traceback (most recent call last):
  File "/home/exarkun/Projects/Twisted/trunk/bin/trial", line 24, in ?
    run()
  File "/home/exarkun/Projects/Twisted/trunk/twisted/scripts/trial.py", line 345, in run
    test_result = trialRunner.run(suite)
  File "/home/exarkun/Projects/Twisted/trunk/twisted/trial/runner.py", line 787, in run
    debugger.runcall(suite.run, result)
  File "/usr/lib/python2.4/bdb.py", line 404, in runcall
    res = func(*args, **kwds)
  File "/home/exarkun/Projects/Twisted/trunk/twisted/trial/runner.py", line 285, in run
    TestSuite.run(self, result)
  File "/home/exarkun/Projects/Twisted/trunk/twisted/trial/runner.py", line 149, in run
    test(result)
  File "/home/exarkun/Projects/Twisted/trunk/twisted/trial/runner.py", line 138, in __call__
    return self.run(result)
  File "/home/exarkun/Projects/Twisted/trunk/twisted/trial/runner.py", line 176, in run
    super(LoggedSuite, self).run(result)
  File "/home/exarkun/Projects/Twisted/trunk/twisted/trial/runner.py", line 149, in run
    test(result)
  File "/home/exarkun/Projects/Twisted/trunk/twisted/trial/runner.py", line 138, in __call__
    return self.run(result)
  File "/home/exarkun/Projects/Twisted/trunk/twisted/trial/runner.py", line 149, in run
    test(result)
  File "/home/exarkun/Projects/Twisted/trunk/twisted/trial/runner.py", line 138, in __call__
    return self.run(result)
  File "/home/exarkun/Projects/Twisted/trunk/twisted/trial/runner.py", line 149, in run
    test(result)
  File "/home/exarkun/Projects/Twisted/trunk/twisted/trial/runner.py", line 138, in __call__
    return self.run(result)
  File "/home/exarkun/Projects/Twisted/trunk/twisted/trial/runner.py", line 149, in run
    test(result)
  File "/home/exarkun/Projects/Twisted/trunk/twisted/trial/unittest.py", line 537, in __call__
    return self.run(*args, **kwargs)
  File "/home/exarkun/Projects/Twisted/trunk/twisted/trial/unittest.py", line 761, in run
    d = self.deferSetUp(None, result)
  File "/home/exarkun/Projects/Twisted/trunk/twisted/trial/unittest.py", line 566, in deferSetUp
    errbackArgs=(result,))
  File "/home/exarkun/Projects/Twisted/trunk/twisted/internet/defer.py", line 182, in addCallbacks
    self._runCallbacks()
  File "/home/exarkun/Projects/Twisted/trunk/twisted/internet/defer.py", line 331, in _runCallbacks
    self.result = failure.Failure()
  File "/home/exarkun/Projects/Twisted/trunk/twisted/python/failure.py", line 382, in _debuginit
    print "Jumping into debugger for post-mortem of exception '%s':" % exc[1]
  File "/home/exarkun/Projects/Twisted/trunk/twisted/test/test_failure.py", line 23, in __str__
    raise self
twisted.test.test_failure.BrokenStrexarkun@kunai:/tmp$ 

[jesstess]

Thanks for the branch, fijal. A few comments:

• twisted/python/failure.py needs a copyright bump
• needs a NEWS file
• even though _debuginit is private, there's a lot going on and it'd benefit from a docstring.

Relatedly, it looks like twisted.test.test_failure.FailureTestCase.testConstructionFails fails in debug mode with exceptions.ValueError: A valid traceback must be passed if no exception is being handled, which possibly needs its own ticket.

[fijal]

Closing this, created 4302