Hyrum K Wright <hyrum <at> hyrumwright.org> writes: > > [[[ > > Index: subversion/tests/cmdline/svntest/testcase.py > =================================================================== > --- subversion/tests/cmdline/svntest/testcase.py (revision 1067239) > +++ subversion/tests/cmdline/svntest/testcase.py (working copy) > @@ -108,6 +108,13 @@ class TestCase: > """ > return self._delegate.get_sandbox_name() > > + def set_issues(self, issues): > + """Set the issues associated with this test.""" > + if type(issues) == type(0):
You should use isinstance(issues, int). Or even better see below. > +def issue_deco(issues): How about using *issues. Then you won't have to put issues in a list. Also you can simply set_issues().