comment - 3

WIPACrepo · Dec 6, 2024 · 41e0c55 · 41e0c55
1 parent ba899a2
commit 41e0c55
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/rest_tools/server/arghandler.py b/rest_tools/server/arghandler.py
@@ -62,7 +62,7 @@ def _universal_to_bool(val: Any) -> bool:
 INVALID_CHOICE_PATTERN = re.compile(r"(argument .+: invalid choice: .+)")
 
 # argument --reco_algo: cannot be empty string / whitespace
-CATCH_MOST_PATTERN = re.compile(r"(argument .+: .+)")
+FROM_ARGUMENT_TYPE_ERROR_PATTERN = re.compile(r"(argument .+: .+)")
 
 
 ###############################################################################
@@ -213,9 +213,9 @@ def _translate_error(
             elif match := INVALID_CHOICE_PATTERN.search(err_msg):
                 return match.group(1).replace("--", "")
 
-            # CATCH MOST (not quite 'catch all') -- covers errors in this known format
+            # argparse.ArgumentTypeError errors -- covers errors in this known format
             # -> this is matched when the server code raises argparse.ArgumentTypeError
-            elif match := CATCH_MOST_PATTERN.search(err_msg):
+            elif match := FROM_ARGUMENT_TYPE_ERROR_PATTERN.search(err_msg):
                 return match.group(1).replace("--", "")
 
         # FALL-THROUGH -- log unknown exception