nose-devs/nose — Issue #45: .addError() not called on SkipTest, etc.
Author: jpellerin
Created: 2011-12-14 15:56:51+00:00
State: open

What steps will reproduce the problem?
1. Write a plugin, P, that uses .addError
2. Raise SkipTest in a test

What is the expected output? What do you see instead?

Expected: .addError() is called on P
Got: .addError() is not called on P (assuming
nose.plugins.skip.Skip.addError is called before P.addError by the plugin
manager)

See attached patch for a proper test.

Please use labels and text to provide additional information.

As well as having .addError called, we also still need an interface for
plugins to find out whether an error is ignored. For example, plugin debug
needs this, so that it doesn't try to drop into the debugger on errors that
won't cause an unsuccessful test run, like skips. ATM, this is done by
returning False, which is what stops other plugins' .addError() methods
from getting called.

Discussion:
http://groups.google.com/group/nose-dev/browse_thread/thread/b1d726998dc79bd9

Google Code Info:
Issue #: 169
Author: j...@pobox.com
Created On: 2008-03-10T23:47:10.000Z
Closed On: