From 561337223fc5aa6d97457bdd133f7404b6ae562a Mon Sep 17 00:00:00 2001 From: Lukas Pustina Date: Tue, 22 Mar 2016 15:19:24 +0100 Subject: [PATCH] Fixes test which are now run due to should_panic Since I changed no_run to should_panic on some tests, the were run but two lacked an actual assertion. Further, I missed to check the return type on another test. --- src/libstd/process.rs | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/libstd/process.rs b/src/libstd/process.rs index d58e8ebab584..a542bf2c8d23 100644 --- a/src/libstd/process.rs +++ b/src/libstd/process.rs @@ -314,6 +314,8 @@ impl Command { /// println!("status: {}", output.status); /// println!("stdout: {}", String::from_utf8_lossy(&output.stdout)); /// println!("stderr: {}", String::from_utf8_lossy(&output.stderr)); + /// + /// assert!(output.status.success()); /// ``` #[stable(feature = "process", since = "1.0.0")] pub fn output(&mut self) -> io::Result { @@ -335,6 +337,8 @@ impl Command { /// .expect("failed to execute process"); /// /// println!("process exited with: {}", status); + /// + /// assert!(status.success()); /// ``` #[stable(feature = "process", since = "1.0.0")] pub fn status(&mut self) -> io::Result { @@ -518,7 +522,7 @@ impl Child { /// let ecode = child.wait_with_output() /// .expect("failed to wait on child"); /// - /// assert!(ecode.success()); + /// assert!(ecode.status.success()); /// ``` /// #[stable(feature = "process", since = "1.0.0")]