summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGiampaolo Rodola <g.rodola@gmail.com>2016-09-30 03:03:11 +0200
committerGiampaolo Rodola <g.rodola@gmail.com>2016-09-30 03:03:11 +0200
commited4fdb5826c7de868a28ee537e116c41ece32b41 (patch)
treedf56846342c31fc2548d9ae79a630e4b8d75eea7
parent72cc351e2e92b40d3c0b03f78764e9061bd5a208 (diff)
downloadpsutil-ed4fdb5826c7de868a28ee537e116c41ece32b41.tar.gz
refactoring
-rw-r--r--psutil/tests/__init__.py22
1 files changed, 3 insertions, 19 deletions
diff --git a/psutil/tests/__init__.py b/psutil/tests/__init__.py
index 0275c2d9..fabe7855 100644
--- a/psutil/tests/__init__.py
+++ b/psutil/tests/__init__.py
@@ -420,7 +420,7 @@ def wait_for_pid(pid, timeout=GLOBAL_TIMEOUT):
time.sleep(0.01)
-@retry(exception=(AssertionError, EnvironmentError), logfun=None)
+@retry(exception=(EnvironmentError, AssertionError), logfun=None)
def wait_for_file(fname, timeout=GLOBAL_TIMEOUT, delete_file=True,
empty=False):
"""Wait for a file to be written on disk with some content."""
@@ -485,27 +485,11 @@ def chdir(dirname):
# ===================================================================
-def retry_before_failing(ntimes=None):
+def retry_before_failing(retries=NO_RETRIES):
"""Decorator which runs a test function and retries N times before
actually failing.
"""
- def decorator(fun):
- @functools.wraps(fun)
- def wrapper(*args, **kwargs):
- times = ntimes or NO_RETRIES
- assert times, times
- for x in range(times):
- try:
- return fun(*args, **kwargs)
- except AssertionError as _:
- err = _
- print("retry (%s)" % err, file=sys.stderr)
- if PY3:
- raise err
- else:
- raise
- return wrapper
- return decorator
+ return retry(exception=AssertionError, timeout=None, retries=retries)
def run_test_module_by_name(name):