fix clippy (#8275)

* fix clippy

* update

* update

tests/by-util/test_cat.rs

@@ -562,7 +562,7 @@ fn test_write_fast_fallthrough_uses_flush() {
 
 #[test]
 #[cfg(unix)]
-#[ignore]
+#[ignore = ""]
 fn test_domain_socket() {
     use std::io::prelude::*;
     use std::os::unix::net::UnixListener;

tests/by-util/test_cksum.rs

@@ -1856,7 +1856,7 @@ mod gnu_cksum_c {
     }
 
     #[test]
-    #[ignore]
+    #[ignore = "todo"]
     fn test_signed_checksums() {
         todo!()
     }

tests/by-util/test_comm.rs

@@ -309,7 +309,7 @@ fn zero_terminated_with_total() {
     }
 }
 
-#[cfg_attr(not(feature = "test_unimplemented"), ignore)]
+#[cfg_attr(not(feature = "test_unimplemented"), ignore = "")]
 #[test]
 fn check_order() {
     let scene = TestScenario::new(util_name!());
@@ -324,7 +324,7 @@ fn check_order() {
         .stderr_is("error to be defined");
 }
 
-#[cfg_attr(not(feature = "test_unimplemented"), ignore)]
+#[cfg_attr(not(feature = "test_unimplemented"), ignore = "")]
 #[test]
 fn nocheck_order() {
     let scene = TestScenario::new(util_name!());
@@ -340,7 +340,7 @@ fn nocheck_order() {
 // when neither --check-order nor --no-check-order is provided,
 // stderr and the error code behaves like check order, but stdout
 // behaves like nocheck_order. However with some quirks detailed below.
-#[cfg_attr(not(feature = "test_unimplemented"), ignore)]
+#[cfg_attr(not(feature = "test_unimplemented"), ignore = "")]
 #[test]
 fn defaultcheck_order() {
     let scene = TestScenario::new(util_name!());

tests/by-util/test_dd.rs

@@ -396,7 +396,7 @@ fn test_null_fullblock() {
 }
 
 #[cfg(unix)]
-#[ignore] // See note below before using this test.
+#[ignore = "See note below before using this test."]
 #[test]
 fn test_fullblock() {
     let tname = "fullblock-from-urand";
@@ -555,7 +555,7 @@ fn test_ascii_521k_to_file() {
     );
 }
 
-#[ignore]
+#[ignore = ""]
 #[cfg(unix)]
 #[test]
 fn test_ascii_5_gibi_to_file() {

tests/by-util/test_ls.rs

@@ -1898,7 +1898,7 @@ fn test_ls_long_ctime() {
 }
 
 #[test]
-#[ignore]
+#[ignore = ""]
 fn test_ls_order_birthtime() {
     let scene = TestScenario::new(util_name!());
     let at = &scene.fixtures;

tests/by-util/test_seq.rs

@@ -943,7 +943,7 @@ fn test_parse_out_of_bounds_exponents() {
         .stdout_only("-0\n1\n");
 }
 
-#[ignore]
+#[ignore = ""]
 #[test]
 fn test_parse_valid_hexadecimal_float_format_issues() {
     // These tests detect differences in the representation of floating-point values with GNU seq.

tests/by-util/test_tail.rs

@@ -4888,7 +4888,7 @@ fn test_following_with_pid() {
 // should fail with any command that takes piped input.
 // See also https://github.com/uutils/coreutils/issues/3895
 #[test]
-#[cfg_attr(not(feature = "expensive_tests"), ignore)]
+#[cfg_attr(not(feature = "expensive_tests"), ignore = "")]
 fn test_when_piped_input_then_no_broken_pipe() {
     let ts = TestScenario::new("tail");
     for i in 0..10000 {