Merge pull request #3632 from vkarak/bugfix/sourcecodeinfo-crash

[bugfix] Fix ReFrame crash when trying to retrieve source code context during errors and treat Unicode errors gracefully

Author: vkarak

reframe/core/exceptions.py

@@ -83,7 +83,11 @@ class SourceCodeInfo:
     def __init__(self, frame):
         self.__filename = frame.filename
         self.__lineno = frame.lineno
-        self.__line = frame.code_context[frame.index]
+        if frame.code_context:
+            self.__line = frame.code_context[frame.index]
+        else:
+            self.__line = None
+
         if sys.version_info >= (3, 11):
             self.__pos  = frame.positions
         else:
@@ -117,18 +121,20 @@ def lines(self):
 
     def __str__(self):
         prefix = '| '
-        ret = f'{prefix}{self.filename}:{self.lineno}\n{prefix}{self.line}'
-        if self.__pos is not None:
-            # Add precise column information
-            l_start, l_end = self.lines
-            c_start, c_end = self.columns
-            if l_start != l_end:
-                # If the code segment is multiline, the columns are not
-                # precise. So we use the full line as span.
-                c_end = len(self.line) - 1
-
-            span = c_end - c_start
-            ret += f'{prefix}{" "*c_start}^{"~"*(span-1)}'
+        ret = f'{prefix}{self.filename}:{self.lineno}'
+        if self.line:
+            ret += f'\n{prefix}{self.line}'
+            if self.__pos is not None:
+                # Add precise column information
+                l_start, l_end = self.lines
+                c_start, c_end = self.columns
+                if l_start != l_end:
+                    # If the code segment is multiline, the columns are not
+                    # precise. So we use the full line as span.
+                    c_end = len(self.line) - 1
+
+                span = c_end - c_start
+                ret += f'{prefix}{" "*c_start}^{"~"*(span-1)}'
 
         return ret
 

reframe/frontend/printer.py

@@ -113,8 +113,8 @@ def _file_info(filename, prefix):
             lines = [f'--- {filename} (last {num_lines} lines) ---\n']
             try:
                 lines += osext.tail(os.path.join(prefix, filename), num_lines)
-            except OSError as e:
-                lines += [f'--- {filename} ({e}) ---']
+            except (OSError, UnicodeError) as e:
+                lines += [f'--- {filename} (ERROR: {e}) ---']
             else:
                 lines += [f'--- {filename} ---']