make sudo --help and sudo --version print on stdout

Author: squell

src/sudo/mod.rs

@@ -81,11 +81,11 @@ fn sudo_process() -> Result<(), Error> {
     match SudoAction::from_env() {
         Ok(action) => match action {
             SudoAction::Help(_) => {
-                eprintln_ignore_io_error!("{}", long_help());
+                println_ignore_io_error!("{}", long_help());
                 std::process::exit(0);
             }
             SudoAction::Version(_) => {
-                eprintln_ignore_io_error!("sudo-rs {VERSION}");
+                println_ignore_io_error!("sudo-rs {VERSION}");
                 std::process::exit(0);
             }
             SudoAction::RemoveTimestamp(_) => {

test-framework/sudo-compliance-tests/src/sudo/flag_help.rs

@@ -16,3 +16,22 @@ fn does_not_panic_on_io_errors() -> Result<()> {
 
     Ok(())
 }
+
+#[test]
+fn prints_on_stdout() -> Result<()> {
+    let env = Env("").build();
+
+    let output = Command::new("sudo").args(["--version"]).output(&env);
+
+    let output = output.stdout();
+    assert_starts_with!(
+        output,
+        if sudo_test::is_original_sudo() {
+            "sudo - execute a command as another user"
+        } else {
+            "sudo - run commands as another user"
+        }
+    );
+
+    Ok(())
+}

test-framework/sudo-compliance-tests/src/sudo/flag_version.rs

@@ -19,3 +19,22 @@ fn does_not_panic_on_io_errors() -> Result<()> {
 
     Ok(())
 }
+
+#[test]
+fn prints_on_stdout() -> Result<()> {
+    let env = Env("").build();
+
+    let output = Command::new("sudo").args(["--version"]).output(&env);
+
+    let output = output.stdout();
+    assert_starts_with!(
+        output,
+        if sudo_test::is_original_sudo() {
+            "Sudo version"
+        } else {
+            "sudo-rs"
+        }
+    );
+
+    Ok(())
+}