[elbe-devel] [PATCH v2 12/12] tests: set self.stdout before self.skipTest() is called

Bastian Germann bage at linutronix.de
Thu Jul 22 21:47:23 CEST 2021


Am 22.07.21 um 17:10 schrieb Torben Hohn:
> self.skipTest() throws a SkipTest Exception.
> 
> set self.stdout before that.
> 
> Signed-off-by: Torben Hohn <torben.hohn at linutronix.de>

Reviewed-by: Bastian Germann <bage at linutronix.de>

> ---
>   elbepack/tests/test_preproc.py | 2 +-
>   elbepack/tests/test_pylint.py  | 2 +-
>   2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/elbepack/tests/test_preproc.py b/elbepack/tests/test_preproc.py
> index e8f271402..f1695b497 100644
> --- a/elbepack/tests/test_preproc.py
> +++ b/elbepack/tests/test_preproc.py
> @@ -28,8 +28,8 @@ class TestPreproc(ElbeTestCase):
>               system(f'{elbe_exe} preprocess "{self.param}"')
>           except ElbeTestException as e:
>               if self.param in TestPreproc.failure_set:
> -                self.skipTest("Preproc test for %s is expected to fail" % (self.param))
>                   self.stdout = e.out
> +                self.skipTest("Preproc test for %s is expected to fail" % (self.param))
>               else:
>                   raise
>           else:
> diff --git a/elbepack/tests/test_pylint.py b/elbepack/tests/test_pylint.py
> index b05fbd90b..5772927c4 100644
> --- a/elbepack/tests/test_pylint.py
> +++ b/elbepack/tests/test_pylint.py
> @@ -60,8 +60,8 @@ class TestPylint(ElbeTestCase):
>               system("pylint3 %s %s" % (' '.join(self.pylint_opts), self.param))
>           except ElbeTestException as e:
>               if self.param in TestPylint.failure_set:
> -                self.skipTest("Pylint test for %s is expected to fail" % (self.param))
>                   self.stdout = e.out
> +                self.skipTest("Pylint test for %s is expected to fail" % (self.param))
>               else:
>                   raise
>           else:
> 


More information about the elbe-devel mailing list