Promote single string to list

Author: Flamefire

easybuild/tools/run.py

@@ -600,8 +600,10 @@ def extract_errors_from_log(log_txt, reg_exps):
     :return (warnings, errors) as lists of lines containing a match
     """
 
-    # Avoid accidentally passing a single element
-    assert isinstance(reg_exps, list), "reg_exps must be a list"
+    # promote single string value to list, since code below expects a list
+    if isinstance(reg_exps, string_type):
+        reg_exps = [reg_exps]
+
     re_tuples = []
     for cur in reg_exps:
         try:
@@ -614,6 +616,7 @@ def extract_errors_from_log(log_txt, reg_exps):
             re_tuples.append((re.compile(reg_exp), action))
         except Exception as e:
             raise EasyBuildError("Invalid input: No RegExp or tuple of RegExp and action '%s' (%s)", str(cur), e)
+
     warnings = []
     errors = []
     for line in log_txt.split('\n'):

test/framework/run.py

@@ -540,8 +540,13 @@ def test_check_log_for_errors(self):
         ])
         expected_error_msg = r"Found 2 error\(s\) in command output \(output: error found, the process crashed with 0\)"
 
+        # String promoted to list
+        self.assertErrorRegex(EasyBuildError, expected_error_msg, check_log_for_errors, input_text,
+                              r"\b(error|crashed)\b")
+        # List of string(s)
         self.assertErrorRegex(EasyBuildError, expected_error_msg, check_log_for_errors, input_text,
                               [r"\b(error|crashed)\b"])
+        # List of tuple(s)
         self.assertErrorRegex(EasyBuildError, expected_error_msg, check_log_for_errors, input_text,
                               [(r"\b(error|crashed)\b", ERROR)])