On Friday 24 May 2013 at 16:50, Harro van der Klauw wrote:
> As long as it throws an error hinting that you might need yielding=True it 
> should be obvious on how to fix 
> the backwards incompatibility issue as soon as you run your tests.
> 
> I don't see a big problem with this, updating of a requirement is something 
> that you should never do automatically.
Good point, the error that is thrown when more than one value is yielded should 
include a note about yielding=True.

Fixtures that only yield one value cannot really be detected and raise an 
explicit error. Such tests would very likely fail since the fixture value will 
be a yielded value rather than the generator itself. I think it will be hard to 
put a helpful error message there since it will just look like an uncaught 
exception/assertion failure, but at least the problem will not pass without 
notice.

Cheers
Andreas 


_______________________________________________
Pytest-dev mailing list
[email protected]
http://mail.python.org/mailman/listinfo/pytest-dev

Reply via email to