tests: Use assert_eq! a lot more

This is actually better because you can get a backtrace.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

diff --git a/apitest/at-otter.rs b/apitest/at-otter.rs
index 3e91ec7b8c7e77ec662d69e3f21f0ba73f122bd1..f68713e52cd4211ed97e2d2db42856cf584031c7 100644 (file)
--- a/apitest/at-otter.rs
+++ b/apitest/at-otter.rs
@@ -67,7 +67,7 @@ mod scraper_ext {
 
   #[throws(AE)]
   pub fn parse_html(resp: reqwest::blocking::Response) -> Html {
-    ensure_eq!(resp.status(), 200);
+    assert_eq!(resp.status(), 200);
     let body = resp.text()?;
     let dom = scraper::Html::parse_document(&body);
     //dbg!(&&dom);
@@ -219,7 +219,7 @@ impl Session {
         "op": op,
       }))
       .send()?;
-    ensure_eq!(resp.status(), 200);
+    assert_eq!(resp.status(), 200);
   }
 
   #[throws(AE)]
@@ -288,7 +288,7 @@ impl Ctx {
     )?;
     let add_err = self.otter(&command)
       .expect_err("library-add succeeded after reset!");
-    ensure_eq!(add_err.downcast::<ExitStatusError>()?.0.code(),
+    assert_eq!(add_err.downcast::<ExitStatusError>()?.0.code(),
                Some(EXIT_NOTFOUND));
 
     let mut session = self.connect_player(&self.alice)?;
@@ -320,7 +320,7 @@ impl Ctx {
       }
     )?;
     dbg!(&added);
-    ensure_eq!(added.len(), 6);
+    assert_eq!(added.len(), 6);
   }
 
   #[throws(AE)]