Merge #1786

1786: Folloup for !1778, remove some of the less helpful error msgs r=asomers a=pacak



Co-authored-by: Michael Baikov <manpacket@gmail.com>

src/sys/signalfd.rs

@@ -147,18 +147,17 @@ mod tests {
     #[test]
     fn create_signalfd() {
         let mask = SigSet::empty();
-        let fd = SignalFd::new(&mask);
-        fd.expect("assert failed");
+        SignalFd::new(&mask).unwrap();
     }
 
     #[test]
     fn create_signalfd_with_opts() {
         let mask = SigSet::empty();
-        let fd = SignalFd::with_flags(
+        SignalFd::with_flags(
             &mask,
             SfdFlags::SFD_CLOEXEC | SfdFlags::SFD_NONBLOCK,
-        );
-        fd.expect("assert failed");
+        )
+        .unwrap();
     }
 
     #[test]

test/sys/test_aio.rs

@@ -101,8 +101,7 @@ mod aio_fsync {
             0,
             SigevNotify::SigevNone,
         ));
-        let err = aiof.as_mut().submit();
-        err.expect("assert failed");
+        aiof.as_mut().submit().unwrap();
         poll_aio!(&mut aiof).unwrap();
         aiof.as_mut().aio_return().unwrap();
     }
@@ -148,8 +147,7 @@ mod aio_read {
             Box::pin(AioRead::new(fd, 2, &mut rbuf, 0, SigevNotify::SigevNone));
         aior.as_mut().submit().unwrap();
 
-        let cancelstat = aior.as_mut().cancel();
-        cancelstat.expect("assert failed");
+        aior.as_mut().cancel().unwrap();
 
         // Wait for aiow to complete, but don't care whether it succeeded
         let _ = poll_aio!(&mut aior);
@@ -341,8 +339,7 @@ mod aio_write {
         let err = aiow.as_mut().error();
         assert!(err == Ok(()) || err == Err(Errno::EINPROGRESS));
 
-        let cancelstat = aiow.as_mut().cancel();
-        cancelstat.expect("assert failed");
+        aiow.as_mut().cancel().unwrap();
 
         // Wait for aiow to complete, but don't care whether it succeeded
         let _ = poll_aio!(&mut aiow);
@@ -564,8 +561,7 @@ fn test_aio_cancel_all() {
     let err = aiocb.as_mut().error();
     assert!(err == Ok(()) || err == Err(Errno::EINPROGRESS));
 
-    let cancelstat = aio_cancel_all(f.as_raw_fd());
-    cancelstat.expect("assert failed");
+    aio_cancel_all(f.as_raw_fd()).unwrap();
 
     // Wait for aiocb to complete, but don't care whether it succeeded
     let _ = poll_aio!(&mut aiocb);

test/sys/test_termios.rs

@@ -22,7 +22,7 @@ fn test_tcgetattr_pty() {
     let _m = crate::PTSNAME_MTX.lock();
 
     let pty = openpty(None, None).expect("openpty failed");
-    termios::tcgetattr(pty.slave).expect("assert failed");
+    termios::tcgetattr(pty.slave).unwrap();
     close(pty.master).expect("closing the master failed");
     close(pty.slave).expect("closing the slave failed");
 }

test/test_time.rs

@@ -29,18 +29,18 @@ pub fn test_clock_gettime() {
 #[test]
 pub fn test_clock_getcpuclockid() {
     let clock_id = clock_getcpuclockid(nix::unistd::Pid::this()).unwrap();
-    clock_gettime(clock_id).expect("assert failed");
+    clock_gettime(clock_id).unwrap();
 }
 
 #[cfg(not(target_os = "redox"))]
 #[test]
 pub fn test_clock_id_res() {
-    ClockId::CLOCK_REALTIME.res().expect("assert failed");
+    ClockId::CLOCK_REALTIME.res().unwrap();
 }
 
 #[test]
 pub fn test_clock_id_now() {
-    ClockId::CLOCK_REALTIME.now().expect("assert failed");
+    ClockId::CLOCK_REALTIME.now().unwrap();
 }
 
 #[cfg(any(
@@ -54,6 +54,6 @@ pub fn test_clock_id_now() {
 pub fn test_clock_id_pid_cpu_clock_id() {
     ClockId::pid_cpu_clock_id(nix::unistd::Pid::this())
         .map(ClockId::now)
-        .expect("assert failed")
-        .expect("assert failed");
+        .unwrap()
+        .unwrap();
 }