expr: Remove useless arg in enum variant

uutils · Feb 25, 2025 · 481a155 · 481a155
1 parent a172060
commit 481a155
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 29 deletions.
diff --git a/src/uu/expr/src/expr.rs b/src/uu/expr/src/expr.rs
@@ -48,8 +48,8 @@ pub enum ExprError {
     UnmatchedOpeningBrace,
     #[error("Unmatched ) or \\}}")]
     UnmatchedClosingBrace,
-    #[error("Invalid content of {0}")]
-    InvalidContent(String),
+    #[error("Invalid content of \\{{\\}}")]
+    InvalidContent,
 }
 
 impl UError for ExprError {

diff --git a/src/uu/expr/src/syntax_tree.rs b/src/uu/expr/src/syntax_tree.rs
@@ -264,7 +264,7 @@ fn check_posix_regex_errors(pattern: &str) -> ExprResult<()> {
         (true, true, false) => Ok(()),
         (_, false, _) => Err(ExprError::UnmatchedOpeningBrace),
         (false, _, _) => Err(ExprError::UnmatchedOpeningParenthesis),
-        (true, true, true) => Err(ExprError::InvalidContent(r"\{\}".to_string())),
+        (true, true, true) => Err(ExprError::InvalidContent),
     }
 }
 
@@ -793,38 +793,20 @@ mod test {
 
     #[test]
     fn check_regex_empty_repeating_pattern() {
-        assert_eq!(
-            check_posix_regex_errors("ab\\{\\}"),
-            Err(InvalidContent(r"\{\}".to_string()))
-        )
+        assert_eq!(check_posix_regex_errors("ab\\{\\}"), Err(InvalidContent))
     }
 
     #[test]
     fn check_regex_intervals_two_numbers() {
         assert_eq!(
             // out of order
             check_posix_regex_errors("ab\\{1,0\\}"),
-            Err(InvalidContent(r"\{\}".to_string()))
-        );
-        assert_eq!(
-            check_posix_regex_errors("ab\\{1,a\\}"),
-            Err(InvalidContent(r"\{\}".to_string()))
-        );
-        assert_eq!(
-            check_posix_regex_errors("ab\\{a,3\\}"),
-            Err(InvalidContent(r"\{\}".to_string()))
-        );
-        assert_eq!(
-            check_posix_regex_errors("ab\\{a,b\\}"),
-            Err(InvalidContent(r"\{\}".to_string()))
-        );
-        assert_eq!(
-            check_posix_regex_errors("ab\\{a,\\}"),
-            Err(InvalidContent(r"\{\}".to_string()))
-        );
-        assert_eq!(
-            check_posix_regex_errors("ab\\{,b\\}"),
-            Err(InvalidContent(r"\{\}".to_string()))
+            Err(InvalidContent)
         );
+        assert_eq!(check_posix_regex_errors("ab\\{1,a\\}"), Err(InvalidContent));
+        assert_eq!(check_posix_regex_errors("ab\\{a,3\\}"), Err(InvalidContent));
+        assert_eq!(check_posix_regex_errors("ab\\{a,b\\}"), Err(InvalidContent));
+        assert_eq!(check_posix_regex_errors("ab\\{a,\\}"), Err(InvalidContent));
+        assert_eq!(check_posix_regex_errors("ab\\{,b\\}"), Err(InvalidContent));
     }
 }
diff --git a/tests/by-util/test_expr.rs b/tests/by-util/test_expr.rs
@@ -688,7 +688,7 @@ mod gnu_expr {
             .stdout_only("\n");
     }
 
-    #[ignore]
+    #[ignore = "rust-onig bug, see https://github.com/rust-onig/rust-onig/issues/188"]
     #[test]
     fn test_bre10() {
         new_ucmd!()