Skip to content

Commit

Permalink
expr: Minor linting fix
Browse files Browse the repository at this point in the history
  • Loading branch information
sargas committed Apr 5, 2024
1 parent 2ae2511 commit e166030
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions src/uu/expr/src/syntax_tree.rs
Original file line number Diff line number Diff line change
Expand Up @@ -211,12 +211,12 @@ fn validate_regex(pattern: &str) -> ExprResult<()> {
invalid_content_error = true;

Check warning on line 211 in src/uu/expr/src/syntax_tree.rs

View check run for this annotation

Codecov / codecov/patch

src/uu/expr/src/syntax_tree.rs#L211

Added line #L211 was not covered by tests
}
(Some(x), None) | (Some(x), Some("")) => {
if !x.parse::<i16>().is_ok() {
if x.parse::<i16>().is_err() {
invalid_content_error = true;
}
}
(None, Some(x)) | (Some(""), Some(x)) => {
if !x.parse::<i16>().is_ok() {
if x.parse::<i16>().is_err() {
invalid_content_error = true;

Check warning on line 220 in src/uu/expr/src/syntax_tree.rs

View check run for this annotation

Codecov / codecov/patch

src/uu/expr/src/syntax_tree.rs#L220

Added line #L220 was not covered by tests
}
}
Expand Down Expand Up @@ -706,13 +706,13 @@ mod test {

#[test]
fn starting_stars_become_escaped() {
let result = AstNode::parse(&["yolo", ":", r"*yolo"])
let result = AstNode::parse(&["cats", ":", r"*cats"])
.unwrap()
.eval()
.unwrap();
assert_eq!(result.eval_as_string(), "0");

let result = AstNode::parse(&["*yolo", ":", r"*yolo"])
let result = AstNode::parse(&["*cats", ":", r"*cats"])
.unwrap()
.eval()
.unwrap();
Expand All @@ -721,7 +721,7 @@ mod test {

#[test]
fn only_match_in_beginning() {
let result = AstNode::parse(&["cowsay", ":", r"ow"])
let result = AstNode::parse(&["budget", ":", r"get"])
.unwrap()
.eval()
.unwrap();
Expand Down

0 comments on commit e166030

Please sign in to comment.