github/open-cas-linux
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #456 from imjfckm/fix-force-reinstall

Browse Source 
Change force-reinstall test option to behave properly
This commit is contained in:
Robert Baldyga 2020-07-12 09:28:01 +02:00 committed by GitHub
commit 885ca70699
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
test/functional/tests/conftest.py
View File

@ -120,8 +120,7 @@ def pytest_addoption(parser):
parser.addoption("--dut-config", action="store", default="None") parser.addoption("--dut-config", action="store", default="None")
parser.addoption("--log-path", action="store", parser.addoption("--log-path", action="store",
default=f"{os.path.join(os.path.dirname(__file__), '../results')}") default=f"{os.path.join(os.path.dirname(__file__), '../results')}")
parser.addoption("--force-reinstall", action="store", default="False") parser.addoption("--force-reinstall", action="store_true", default=False)
# TODO: investigate whether it is possible to pass the last param as bool
def unmount_cas_devices(): def unmount_cas_devices():
@ -147,7 +146,7 @@ def unmount_cas_devices():
def get_force_param(item): def get_force_param(item):
return item.config.getoption("--force-reinstall") is not "False" return item.config.getoption("--force-reinstall")
def base_prepare(item): def base_prepare(item):