Merge pull request #17 from robertbaldyga/fs_utils-timeout
Introduce option to define timeout for time-consuming file operations
This commit is contained in:
commit
073793c83a
@ -164,9 +164,8 @@ def create_file(path):
|
||||
return TestRun.executor.run_expect_success(cmd)
|
||||
|
||||
|
||||
def compare(file, other_file):
|
||||
output = TestRun.executor.run(
|
||||
f"cmp --silent \"{file}\" \"{other_file}\"")
|
||||
def compare(file, other_file, timeout: timedelta = timedelta(minutes=30)):
|
||||
output = TestRun.executor.run(f"cmp --silent \"{file}\" \"{other_file}\"", timeout)
|
||||
if output.exit_code == 0:
|
||||
return True
|
||||
elif output.exit_code > 1:
|
||||
@ -175,9 +174,8 @@ def compare(file, other_file):
|
||||
return False
|
||||
|
||||
|
||||
def diff(file, other_file):
|
||||
output = TestRun.executor.run(
|
||||
f"diff \"{file}\" \"{other_file}\"")
|
||||
def diff(file, other_file, timeout: timedelta = timedelta(minutes=30)):
|
||||
output = TestRun.executor.run(f"diff \"{file}\" \"{other_file}\"", timeout)
|
||||
if output.exit_code == 0:
|
||||
return None
|
||||
elif output.exit_code > 1:
|
||||
@ -186,6 +184,13 @@ def diff(file, other_file):
|
||||
return output.stderr
|
||||
|
||||
|
||||
def md5sum(file, binary=True, timeout: timedelta = timedelta(minutes=30)):
|
||||
output = TestRun.executor.run(f"md5sum {'-b' if binary else ''} {file}", timeout)
|
||||
if output.exit_code != 0:
|
||||
raise Exception(f"Md5sum command execution failed! {output.stdout}\n{output.stderr}")
|
||||
return output.stdout.split()[0]
|
||||
|
||||
|
||||
# For some reason separators other than '/' don't work when using sed on system paths
|
||||
# This requires escaping '/' in pattern and target string
|
||||
def escape_sed_string(string: str, sed_replace: bool = False):
|
||||
|
@ -14,18 +14,14 @@ class File(FsItem):
|
||||
def __init__(self, full_path):
|
||||
FsItem.__init__(self, full_path)
|
||||
|
||||
def compare(self, other_file):
|
||||
return fs_utils.compare(str(self), str(other_file))
|
||||
def compare(self, other_file, timeout: timedelta = timedelta(minutes=30)):
|
||||
return fs_utils.compare(str(self), str(other_file), timeout)
|
||||
|
||||
def diff(self, other_file):
|
||||
return fs_utils.diff(str(self), str(other_file))
|
||||
def diff(self, other_file, timeout: timedelta = timedelta(minutes=30)):
|
||||
return fs_utils.diff(str(self), str(other_file), timeout)
|
||||
|
||||
def md5sum(self, binary=True):
|
||||
output = TestRun.executor.run(
|
||||
f"md5sum {'-b' if binary else ''} {self.full_path}")
|
||||
if output.exit_code != 0:
|
||||
raise Exception(f"Md5sum command execution failed! {output.stdout}\n{output.stderr}")
|
||||
return output.stdout.split()[0]
|
||||
def md5sum(self, binary=True, timeout: timedelta = timedelta(minutes=30)):
|
||||
return fs_utils.md5sum(str(self), binary, timeout)
|
||||
|
||||
def read(self):
|
||||
return fs_utils.read_file(str(self))
|
||||
|
Loading…
Reference in New Issue
Block a user