Merge pull request #5401 from cakebaker/clippy_suppress_cognitive_complexity_warning

sylvestre · web-flow · commit 3cd5f9d175e4 · 2023-10-13T15:41:35.000+02:00
clippy: suppress cognitive_complexity lint in some tests
diff --git a/src/uu/csplit/src/csplit.rs b/src/uu/csplit/src/csplit.rs
@@ -664,6 +664,7 @@ mod tests {
     use super::*;
 
     #[test]
+    #[allow(clippy::cognitive_complexity)]
     fn input_splitter() {
         let input = vec![
             Ok(String::from("aaa")),
@@ -736,6 +737,7 @@ mod tests {
     }
 
     #[test]
+    #[allow(clippy::cognitive_complexity)]
     fn input_splitter_interrupt_rewind() {
         let input = vec![
             Ok(String::from("aaa")),
diff --git a/src/uu/csplit/src/patterns.rs b/src/uu/csplit/src/patterns.rs
@@ -211,6 +211,7 @@ mod tests {
     }
 
     #[test]
+    #[allow(clippy::cognitive_complexity)]
     fn up_to_match_pattern() {
         let input: Vec<String> = vec![
             "/test1.*end$/",
@@ -264,6 +265,7 @@ mod tests {
     }
 
     #[test]
+    #[allow(clippy::cognitive_complexity)]
     fn skip_to_match_pattern() {
         let input: Vec<String> = vec![
             "%test1.*end$%",
diff --git a/src/uu/dd/src/numbers.rs b/src/uu/dd/src/numbers.rs
@@ -115,6 +115,7 @@ mod tests {
     }
 
     #[test]
+    #[allow(clippy::cognitive_complexity)]
     fn test_to_magnitude_and_suffix_not_powers_of_1024() {
         assert_eq!(to_magnitude_and_suffix(1, SuffixType::Si), "1.0 B");
         assert_eq!(to_magnitude_and_suffix(999, SuffixType::Si), "999 B");
diff --git a/src/uu/dd/src/parseargs/unit_tests.rs b/src/uu/dd/src/parseargs/unit_tests.rs
@@ -103,6 +103,7 @@ fn test_status_level_none() {
 }
 
 #[test]
+#[allow(clippy::cognitive_complexity)]
 fn test_all_top_level_args_no_leading_dashes() {
     let args = &[
         "if=foo.file",
diff --git a/src/uu/df/src/blocks.rs b/src/uu/df/src/blocks.rs
@@ -239,6 +239,7 @@ mod tests {
     }
 
     #[test]
+    #[allow(clippy::cognitive_complexity)]
     fn test_to_magnitude_and_suffix_not_powers_of_1024() {
         assert_eq!(to_magnitude_and_suffix(1, SuffixType::Si), "1B");
         assert_eq!(to_magnitude_and_suffix(999, SuffixType::Si), "999B");
diff --git a/src/uu/head/src/head.rs b/src/uu/head/src/head.rs
@@ -548,6 +548,7 @@ mod tests {
     }
 
     #[test]
+    #[allow(clippy::cognitive_complexity)]
     fn all_args_test() {
         assert!(options("--silent").unwrap().quiet);
         assert!(options("--quiet").unwrap().quiet);
diff --git a/src/uu/head/src/parse.rs b/src/uu/head/src/parse.rs
@@ -153,6 +153,7 @@ mod tests {
     }
 
     #[test]
+    #[allow(clippy::cognitive_complexity)]
     fn test_parse_numbers_obsolete() {
         assert_eq!(obsolete("-5"), obsolete_result(&["-n", "5"]));
         assert_eq!(obsolete("-100"), obsolete_result(&["-n", "100"]));
diff --git a/src/uu/nl/src/nl.rs b/src/uu/nl/src/nl.rs
@@ -394,6 +394,7 @@ mod test {
     use super::*;
 
     #[test]
+    #[allow(clippy::cognitive_complexity)]
     fn test_format() {
         assert_eq!(NumberFormat::Left.format(12, 1), "12");
         assert_eq!(NumberFormat::Left.format(-12, 1), "-12");
diff --git a/src/uu/numfmt/src/format.rs b/src/uu/numfmt/src/format.rs
@@ -430,6 +430,7 @@ mod tests {
     use super::*;
 
     #[test]
+    #[allow(clippy::cognitive_complexity)]
     fn test_round_with_precision() {
         let rm = RoundMethod::FromZero;
         assert_eq!(1.0, round_with_precision(0.12345, rm, 0));
diff --git a/src/uu/numfmt/src/options.rs b/src/uu/numfmt/src/options.rs
@@ -266,6 +266,7 @@ mod tests {
     }
 
     #[test]
+    #[allow(clippy::cognitive_complexity)]
     fn test_parse_format_with_invalid_formats() {
         assert!("".parse::<FormatOptions>().is_err());
         assert!("hello".parse::<FormatOptions>().is_err());
diff --git a/src/uu/od/src/prn_float.rs b/src/uu/od/src/prn_float.rs
@@ -198,6 +198,7 @@ fn test_format_flo64() {
 }
 
 #[test]
+#[allow(clippy::cognitive_complexity)]
 fn test_format_flo16() {
     assert_eq!(format_flo16(f16::from_bits(0x8400u16)), "-6.104e-5");
     assert_eq!(format_flo16(f16::from_bits(0x8401u16)), "-6.109e-5");
diff --git a/src/uu/seq/src/numberparse.rs b/src/uu/seq/src/numberparse.rs
@@ -538,6 +538,7 @@ mod tests {
     }
 
     #[test]
+    #[allow(clippy::cognitive_complexity)]
     fn test_num_integral_digits() {
         // no decimal, no exponent
         assert_eq!(num_integral_digits("123"), 3);
@@ -578,6 +579,7 @@ mod tests {
     }
 
     #[test]
+    #[allow(clippy::cognitive_complexity)]
     fn test_num_fractional_digits() {
         // no decimal, no exponent
         assert_eq!(num_fractional_digits("123"), 0);
diff --git a/src/uu/split/src/number.rs b/src/uu/split/src/number.rs
@@ -398,6 +398,7 @@ mod tests {
     }
 
     #[test]
+    #[allow(clippy::cognitive_complexity)]
     fn test_dynamic_width_number_display_alphabetic() {
         fn num(n: usize) -> Number {
             let mut number = Number::DynamicWidth(DynamicWidthNumber::new(26, 0));
@@ -443,6 +444,7 @@ mod tests {
     }
 
     #[test]
+    #[allow(clippy::cognitive_complexity)]
     fn test_dynamic_width_number_display_numeric_hexadecimal() {
         fn num(n: usize) -> Number {
             let mut number = Number::DynamicWidth(DynamicWidthNumber::new(16, 0));
@@ -467,6 +469,7 @@ mod tests {
     }
 
     #[test]
+    #[allow(clippy::cognitive_complexity)]
     fn test_fixed_width_number_increment() {
         let mut n = Number::FixedWidth(FixedWidthNumber::new(3, 2, 0).unwrap());
         assert_eq!(n.digits(), vec![0, 0]);
@@ -490,6 +493,7 @@ mod tests {
     }
 
     #[test]
+    #[allow(clippy::cognitive_complexity)]
     fn test_fixed_width_number_display_alphabetic() {
         fn num(n: usize) -> Result<Number, Overflow> {
             let mut number = Number::FixedWidth(FixedWidthNumber::new(26, 2, 0).unwrap());
diff --git a/src/uu/split/src/split.rs b/src/uu/split/src/split.rs
@@ -1846,6 +1846,7 @@ mod tests {
     }
 
     #[test]
+    #[allow(clippy::cognitive_complexity)]
     fn test_number_type_from_error() {
         assert_eq!(
             NumberType::from("xyz").unwrap_err(),
diff --git a/src/uu/stat/src/stat.rs b/src/uu/stat/src/stat.rs
@@ -947,6 +947,7 @@ mod tests {
     }
 
     #[test]
+    #[allow(clippy::cognitive_complexity)]
     fn test_group_num() {
         assert_eq!("12,379,821,234", group_num("12379821234"));
         assert_eq!("21,234", group_num("21234"));

Original file line number	Diff line number	Diff line change
`@@ -211,6 +211,7 @@ mod tests {`
`211`	`211`	`}`
`212`	`212`
`213`	`213`	`#[test]`
	`214`	`+ #[allow(clippy::cognitive_complexity)]`
`214`	`215`	`fn up_to_match_pattern() {`
`215`	`216`	`let input: Vec<String> = vec![`
`216`	`217`	`"/test1.*end$/",`
`@@ -264,6 +265,7 @@ mod tests {`
`264`	`265`	`}`
`265`	`266`
`266`	`267`	`#[test]`
	`268`	`+ #[allow(clippy::cognitive_complexity)]`
`267`	`269`	`fn skip_to_match_pattern() {`
`268`	`270`	`let input: Vec<String> = vec![`
`269`	`271`	`"%test1.*end$%",`
Original file line number	Diff line number	Diff line change
`@@ -115,6 +115,7 @@ mod tests {`
`115`	`115`	`}`
`116`	`116`
`117`	`117`	`#[test]`
	`118`	`+ #[allow(clippy::cognitive_complexity)]`
`118`	`119`	`fn test_to_magnitude_and_suffix_not_powers_of_1024() {`
`119`	`120`	`assert_eq!(to_magnitude_and_suffix(1, SuffixType::Si), "1.0 B");`
`120`	`121`	`assert_eq!(to_magnitude_and_suffix(999, SuffixType::Si), "999 B");`
Original file line number	Diff line number	Diff line change
`@@ -103,6 +103,7 @@ fn test_status_level_none() {`
`103`	`103`	`}`
`104`	`104`
`105`	`105`	`#[test]`
	`106`	`+#[allow(clippy::cognitive_complexity)]`
`106`	`107`	`fn test_all_top_level_args_no_leading_dashes() {`
`107`	`108`	`let args = &[`
`108`	`109`	`"if=foo.file",`
Original file line number	Diff line number	Diff line change
`@@ -239,6 +239,7 @@ mod tests {`
`239`	`239`	`}`
`240`	`240`
`241`	`241`	`#[test]`
	`242`	`+ #[allow(clippy::cognitive_complexity)]`
`242`	`243`	`fn test_to_magnitude_and_suffix_not_powers_of_1024() {`
`243`	`244`	`assert_eq!(to_magnitude_and_suffix(1, SuffixType::Si), "1B");`
`244`	`245`	`assert_eq!(to_magnitude_and_suffix(999, SuffixType::Si), "999B");`
Original file line number	Diff line number	Diff line change
`@@ -548,6 +548,7 @@ mod tests {`
`548`	`548`	`}`
`549`	`549`
`550`	`550`	`#[test]`
	`551`	`+ #[allow(clippy::cognitive_complexity)]`
`551`	`552`	`fn all_args_test() {`
`552`	`553`	`assert!(options("--silent").unwrap().quiet);`
`553`	`554`	`assert!(options("--quiet").unwrap().quiet);`
Original file line number	Diff line number	Diff line change
`@@ -153,6 +153,7 @@ mod tests {`
`153`	`153`	`}`
`154`	`154`
`155`	`155`	`#[test]`
	`156`	`+ #[allow(clippy::cognitive_complexity)]`
`156`	`157`	`fn test_parse_numbers_obsolete() {`
`157`	`158`	`assert_eq!(obsolete("-5"), obsolete_result(&["-n", "5"]));`
`158`	`159`	`assert_eq!(obsolete("-100"), obsolete_result(&["-n", "100"]));`
Original file line number	Diff line number	Diff line change
`@@ -266,6 +266,7 @@ mod tests {`
`266`	`266`	`}`
`267`	`267`
`268`	`268`	`#[test]`
	`269`	`+ #[allow(clippy::cognitive_complexity)]`
`269`	`270`	`fn test_parse_format_with_invalid_formats() {`
`270`	`271`	`assert!("".parse::<FormatOptions>().is_err());`
`271`	`272`	`assert!("hello".parse::<FormatOptions>().is_err());`