twister: test: update test case

update test cases for twister
1. test_errors.py add protection.
2. test_handlers.py change call to status
3. test_testsuite.py change call to status

The log traces for TwisterException objects only.
And the stack trace output follow the same rules for all exceptions,
but StatusAttributeError with its dedicated handlers.

Signed-off-by: Hake Huang <[email protected]>

scripts/pylib/twister/twisterlib/error.py

@@ -11,9 +11,7 @@
 class TwisterException(Exception):
     def __init__(self, message="TwisterException"):
         super().__init__(message)
-        for line in traceback.format_stack():
-            logger.info(line.strip())
-        logger.warning("======call stack dump end============")
+        logger.error(''.join(["Twister call stack dump:\n"] + traceback.format_stack()[:-1]))
 
 class TwisterRuntimeError(TwisterException):
     pass

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