Merge pull request #8954 from cakebaker/cargo_toml_remove_two_features

Cargo.toml: remove two features

Cargo.toml

@@ -27,8 +27,6 @@ macos = ["feat_os_macos"]
 unix = ["feat_os_unix"]
 windows = ["feat_os_windows"]
 ## project-specific feature shortcodes
-nightly = []
-test_unimplemented = []
 expensive_tests = []
 # "test_risky_names" == enable tests that create problematic file names (would make a network share inaccessible to Windows, breaks SVN on Mac OS, etc.)
 test_risky_names = []

tests/by-util/test_comm.rs

@@ -309,7 +309,7 @@ fn zero_terminated_with_total() {
     }
 }
 
-#[cfg_attr(not(feature = "test_unimplemented"), ignore = "")]
+#[ignore = "not implemented"]
 #[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 = "")]
+#[ignore = "not implemented"]
 #[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 = "")]
+#[ignore = "not implemented"]
 #[test]
 fn defaultcheck_order() {
     let scene = TestScenario::new(util_name!());