[Zope3-checkins] Re: SVN: Zope3/trunk/src/zope/app/workflow/stateful/browser/ftests/test_processdefinition.py added missing tearDown

Philipp von Weitershausen philipp at weitershausen.de
Wed May 19 06:46:27 EDT 2004


Eckart Hertzler wrote:
> Log message for revision 24819:
> added missing tearDown
> 
> 
> 
> -=-
> Modified: Zope3/trunk/src/zope/app/workflow/stateful/browser/ftests/test_processdefinition.py
> ===================================================================
> --- Zope3/trunk/src/zope/app/workflow/stateful/browser/ftests/test_processdefinition.py	2004-05-19 10:25:41 UTC (rev 24818)
> +++ Zope3/trunk/src/zope/app/workflow/stateful/browser/ftests/test_processdefinition.py	2004-05-19 10:36:42 UTC (rev 24819)
> @@ -46,6 +46,10 @@
>                    'new_value': 'pd' })
>  
>  
> +    def tearDown(self):
> +        BrowserTestCase.tearDown(self)
> +
> +

1. Please use super() where appropriate (setUp, tearDown methods are 
appropriate places usually)

2. Adding this method makes no sense at all, since this testcase 
inherits from BrowserTestCase.

Philipp



More information about the Zope3-Checkins mailing list