twister: add trace print in exception

scripts/pylib/twister/twisterlib/error.py

@@ -2,28 +2,30 @@
 #
 # Copyright (c) 2018-2022 Intel Corporation
 # SPDX-License-Identifier: Apache-2.0
+import logging
+import traceback
 
-class TwisterException(Exception):
-    pass
+logger = logging.getLogger('twister')
+logger.setLevel(logging.DEBUG)
 
+class TwisterException(Exception):
+    def __init__(self, message="TwisterException"):
+        super().__init__(message)
+        logger.error(''.join(["Twister call stack dump:\n"] + traceback.format_stack()[:-1]))
 
 class TwisterRuntimeError(TwisterException):
     pass
 
-
 class ConfigurationError(TwisterException):
     def __init__(self, cfile, message):
         TwisterException.__init__(self, str(cfile) + ": " + message)
 
-
 class BuildError(TwisterException):
     pass
 
-
 class ExecutionError(TwisterException):
     pass
 
-
 class StatusAttributeError(TwisterException):
     def __init__(self, cls : type, value):
         msg = f'{cls.__name__} assigned status {value}, which could not be cast to a TwisterStatus.'

scripts/tests/twister/test_errors.py

@@ -28,8 +28,8 @@ def test_configurationerror():
 def test_status_value_error():
     harness = Test()
 
-    expected_err = 'Test assigned status None,' \
+    expected_err = 'Test assigned status OK,' \
                    ' which could not be cast to a TwisterStatus.'
 
     with pytest.raises(StatusAttributeError, match=expected_err):
-        harness.status = None
+        harness.status = "OK"

scripts/tests/twister/test_handlers.py

@@ -131,7 +131,7 @@ def test_handler_final_handle_actions(mocked_instance):
     handler.suite_name_check = True
 
     harness = twisterlib.harness.Test()
-    harness.status = 'NONE'
+    harness.status = TwisterStatus.NONE
     harness.detected_suite_names = mock.Mock()
     harness.matched_run_id = False
     harness.run_id_exists = True

scripts/tests/twister/test_testsuite.py

@@ -648,7 +648,10 @@ def mock_sf(filename, *args, **kwargs):
 
     def mock_stat(filename, *args, **kwargs):
         result = mock.Mock()
-        type(result).st_size = sizes[filename]
+        # as we may call os.stat in code
+        # some protection need add here
+        if filename in sizes:
+            type(result).st_size = sizes[filename]
 
         return result