nose-devs/nose — Issue #185: Minor maintenance issue in suite.py with __tests property
Author: jpellerin
Created: 2011-12-14 16:03:09+00:00
State: open

Please provide any additional information below.
This is a very minor point.

I am new to python, so please bear with me. I had never seen prpopety call
before, so I had to research.

I belive in suite.py the line in LazySuite :

# lazy property so subclasses can override loadtests()
_tests = property(lambda self: self.loadtests(),
None, None,
'Tests in this suite (iter)')

may be a potential problem.

In my experience, to use property the class/base class must be derived from
object for setx and delx methods to work correctly.

In this case, since only the getx method is used, this should work ok, but
if the latter methods are ever used, the property will not work corectly.
Maybe, at a minimum deserves a comment?

Thanks,
William Meadows

Google Code Info:
Issue #: 33
Author: william....@gmail.com
Created On: 2007-02-14T15:49:28.000Z
Closed On: 2007-03-24T15:56:52.000Z