In article <1392492250.26338.83831085.39A5ED08 at webmail.messagingengine.com>, Benjamin Peterson <benjamin at python.org> wrote: > On Sat, Feb 15, 2014, at 10:12 AM, Serhiy Storchaka wrote: > > Although Raymond approved a patch for test_bigmem [2], his expressed the > > insistent recommendation not to do this. So I stop committing new > > reviewed patches. Terry recommended to discuss this in Python-Dev. What > > are your thoughts? > I tend to agree with Raymond. I think such changes are very welcome when > the module or tests are otherwise being changed, but on their on > constitute unnecessary churn. +1 Integrity of the test suite and minimizing code churn top any benefits of more specific messages on failures. The expectation is that most tests will never fail so their changed messages will never be seen. For the rare cases when a test does fail, quite often the test was written in a way that will require examination of the code to understand exactly what the test case was intending to test and why it failed. Having a more specific exception message wouldn't help for many tests without further modifications; the key point is to know that the test failed. -- Ned Deily, nad at acm.org
RetroSearch is an open source project built by @garambo | Open a GitHub Issue
Search and Browse the WWW like it's 1997 | Search results from DuckDuckGo
HTML:
3.2
| Encoding:
UTF-8
| Version:
0.7.4