Skip to content

Commit 45a3b51

Browse files
committed
Use helper functions in Sugg tests.
1 parent 0b6d1fd commit 45a3b51

File tree

1 file changed

+14
-25
lines changed

1 file changed

+14
-25
lines changed

clippy_utils/src/sugg.rs

Lines changed: 14 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -1070,33 +1070,22 @@ mod test {
10701070
fn not_op() {
10711071
use AssocOp::{Add, Equal, Greater, GreaterEqual, LAnd, LOr, Less, LessEqual, NotEqual};
10721072

1073-
// Invert the comparison operator.
1074-
let sugg = Sugg::BinOp(Equal, "1".into(), "1".into());
1075-
assert_eq!("1 != 1", (!sugg).to_string());
1076-
1077-
let sugg = Sugg::BinOp(NotEqual, "1".into(), "1".into());
1078-
assert_eq!("1 == 1", (!sugg).to_string());
1079-
1080-
let sugg = Sugg::BinOp(Less, "1".into(), "1".into());
1081-
assert_eq!("1 >= 1", (!sugg).to_string());
1082-
1083-
let sugg = Sugg::BinOp(LessEqual, "1".into(), "1".into());
1084-
assert_eq!("1 > 1", (!sugg).to_string());
1085-
1086-
let sugg = Sugg::BinOp(Greater, "1".into(), "1".into());
1087-
assert_eq!("1 <= 1", (!sugg).to_string());
1073+
fn test_not(op: AssocOp, correct: &str) {
1074+
let sugg = Sugg::BinOp(op, "x".into(), "y".into());
1075+
assert_eq!((!sugg).to_string(), correct);
1076+
}
10881077

1089-
let sugg = Sugg::BinOp(GreaterEqual, "1".into(), "1".into());
1090-
assert_eq!("1 < 1", (!sugg).to_string());
1078+
// Invert the comparison operator.
1079+
test_not(Equal, "x != y");
1080+
test_not(NotEqual, "x == y");
1081+
test_not(Less, "x >= y");
1082+
test_not(LessEqual, "x > y");
1083+
test_not(Greater, "x <= y");
1084+
test_not(GreaterEqual, "x < y");
10911085

10921086
// Other operators are inverted like !(..).
1093-
let sugg = Sugg::BinOp(Add, "1".into(), "1".into());
1094-
assert_eq!("!(1 + 1)", (!sugg).to_string());
1095-
1096-
let sugg = Sugg::BinOp(LAnd, "1".into(), "1".into());
1097-
assert_eq!("!(1 && 1)", (!sugg).to_string());
1098-
1099-
let sugg = Sugg::BinOp(LOr, "1".into(), "1".into());
1100-
assert_eq!("!(1 || 1)", (!sugg).to_string());
1087+
test_not(Add, "!(x + y)");
1088+
test_not(LAnd, "!(x && y)");
1089+
test_not(LOr, "!(x || y)");
11011090
}
11021091
}

0 commit comments

Comments
 (0)