On Wed, 2010-11-24 at 12:09 +0100, Josef Skladanka wrote:
Just a minor fix in the templates - we changed the ExceptionCatcher decorator & the tests, but we forgot to change the test template.
Oh good catch, thanks! ACK from me.
Thanks, James
doc/test_class.py.template | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/doc/test_class.py.template b/doc/test_class.py.template index 00aef9d..590bd16 100644 --- a/doc/test_class.py.template +++ b/doc/test_class.py.template @@ -43,7 +43,7 @@ class testclassname(AutoQATest): # <-- UPDATE Classname # This is where you make sure that any required packages are installed, # services are started, your test code is compiled, and so on. This is # an optional method.
- #@ExceptionCatcher("self.setup_failed")
- #@ExceptionCatcher() #def setup(self, some_params, **kwargs): # utils.system('yum -y install yum-utils')
@@ -51,7 +51,7 @@ class testclassname(AutoQATest): # <-- UPDATE Classname # typically use this method to parse the autoqa config data passed from # the server or create some empty result structures. This is an optional # method - if you don't need to initialize anything, delete this block.
- #@ExceptionCatcher("self.initialize_failed")
- #@ExceptionCatcher() #def initialize(self, config, **kwargs): #**kwargs needs to stay # super(testclassname, self).initialize(config) # <-- UPDATE Classname # #your extra initialization code goes here
@@ -63,7 +63,7 @@ class testclassname(AutoQATest): # <-- UPDATE Classname # self.summary: one line test result summary # self.outputs: full test output (string or list of strings) # self.highlights: important lines to notice (string or list of strings)
- @ExceptionCatcher("self.run_once_failed")
- @ExceptionCatcher() def run_once(self, some_params, **kwargs): #**kwargs needs to stay cmd = 'test_binary --param %s' % some_params self.outputs = utils.system_output(cmd, retain_output=True)