Skip to content

Commit

Permalink
Don"t call test error handler on skipped test (#116)
Browse files Browse the repository at this point in the history
  • Loading branch information
sh0ked authored Mar 30, 2017
1 parent c40bbf1 commit c8f8c9e
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
5 changes: 3 additions & 2 deletions contesto/basis/test_case.py
Original file line number Diff line number Diff line change
Expand Up @@ -65,8 +65,9 @@ def _run_test_error_handlers(self, func):
def wrapper(*args, **kwargs):
try:
return func(*args, **kwargs)
except:
self._run_handlers('on_test_error')
except Exception as e:
if not isinstance(e, unittest.SkipTest):
self._run_handlers('on_test_error')
raise
return wrapper

Expand Down
2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
'config/*.ini'
],
},
"version": "0.2.11",
"version": "0.2.12",
"install_requires": [
"selenium==2.52.0",
"Appium-Python-Client==0.24",
Expand Down

0 comments on commit c8f8c9e

Please sign in to comment.