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

Improve error handling and exception messages

Browse Source
This commit is contained in:
Katarzyna Lapinska 2019-11-08 15:42:10 +01:00
parent 54d6435656
commit 783daba8d4
2 changed files with 33 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
test/functional/test-framework

@ -1 +1 @@
Subproject commit 6cb2c928b05e62f9296e6e5223f96585eaa1816e Subproject commit 7c5c8fd5143497c56a8fb0cc1fe9a923f48a9db1

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

@ -3,7 +3,6 @@
# SPDX-License-Identifier: BSD-3-Clause-Clear # SPDX-License-Identifier: BSD-3-Clause-Clear
# #
import pytest
import os import os
import sys import sys
import yaml import yaml
@ -24,7 +23,8 @@ plugins_dir = os.path.join(os.path.dirname(__file__), "../plugins")
sys.path.append(plugins_dir) sys.path.append(plugins_dir)
try: try:
from test_wrapper import plugin as test_wrapper from test_wrapper import plugin as test_wrapper
except ImportError: except ImportError as e:
TestRun.LOGGER.info(str(e))
pass pass
@ -56,17 +56,25 @@ def pytest_runtest_setup(item):
with open(item.config.getoption('--dut-config')) as cfg: with open(item.config.getoption('--dut-config')) as cfg:
dut_config = yaml.safe_load(cfg) dut_config = yaml.safe_load(cfg)
except Exception: except Exception:
dut_config = {} TestRun.block("You need to specify DUT config. See the example_dut_config.py file.")
if 'test_wrapper' in sys.modules: if 'test_wrapper' in sys.modules:
if 'ip' in dut_config: if 'ip' in dut_config:
try: try:
IP(dut_config['ip']) IP(dut_config['ip'])
except ValueError: except ValueError:
raise Exception("IP address from configuration file is in invalid format.") TestRun.exception(
"IP address from configuration file is in invalid format.")
try:
dut_config = test_wrapper.prepare(dut_config) dut_config = test_wrapper.prepare(dut_config)
except Exception as ex:
TestRun.LOGGER.exception(f"Exception occurred on test wrapper prepare stage:\n"
f"{str(ex)}\n{traceback.format_exc()}")
try:
TestRun.setup(dut_config) TestRun.setup(dut_config)
except Exception as ex:
TestRun.LOGGER.exception(f"Exception occurred during test setup:\n"
f"{str(ex)}\n{traceback.format_exc()}")
if 'test_wrapper' in sys.modules: if 'test_wrapper' in sys.modules:
test_wrapper.try_setup_serial_log(dut_config) test_wrapper.try_setup_serial_log(dut_config)
@ -75,8 +83,9 @@ def pytest_runtest_setup(item):
repo_dir=os.path.join(os.path.dirname(__file__), "../../.."), repo_dir=os.path.join(os.path.dirname(__file__), "../../.."),
working_dir=dut_config['working_dir']) working_dir=dut_config['working_dir'])
except Exception as e: except Exception as exception:
TestRun.LOGGER.exception(f"{str(e)}\n{traceback.format_exc()}") TestRun.LOGGER.exception(f"Conftest prepare exception:\n"
f"{str(exception)}\n{traceback.format_exc()}")
TestRun.LOGGER.info(f"DUT info: {TestRun.dut}") TestRun.LOGGER.info(f"DUT info: {TestRun.dut}")
base_prepare(item) base_prepare(item)
@ -93,21 +102,25 @@ def pytest_runtest_teardown():
with TestRun.LOGGER.step("Cleanup after test"): with TestRun.LOGGER.step("Cleanup after test"):
try: try:
if TestRun.executor:
if TestRun.executor.is_active(): if TestRun.executor.is_active():
TestRun.executor.wait_for_connection() TestRun.executor.wait_for_connection()
Udev.enable() Udev.enable()
unmount_cas_devices() unmount_cas_devices()
casadm.stop_all_caches() casadm.stop_all_caches()
except Exception: except Exception as ex:
TestRun.LOGGER.warning("Exception occured during platform cleanup.") TestRun.LOGGER.warning(f"Exception occured during platform cleanup.\n"
f"{str(ex)}\n{traceback.format_exc()}")
if 'test_wrapper' in sys.modules: if 'test_wrapper' in sys.modules:
try: try:
test_wrapper.cleanup() test_wrapper.cleanup()
except Exception as e: except Exception as ex:
TestRun.LOGGER.warning(f"Exception occured during test wrapper cleanup.\n{str(e)}") TestRun.LOGGER.warning(f"Exception occured during test wrapper cleanup.\n{str(ex)}"
f"\n{traceback.format_exc()}")
TestRun.LOGGER.end() TestRun.LOGGER.end()
if TestRun.executor:
TestRun.LOGGER.get_additional_logs() TestRun.LOGGER.get_additional_logs()
Log.destroy() Log.destroy()