From: Ian Jackson <ijackson@chiark.greenend.org.uk>
Date: Sat, 27 Feb 2021 20:53:09 +0000 (+0000)
Subject: tests: Use assert_eq! a lot more
X-Git-Tag: otter-0.4.0~294
X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=commitdiff_plain;h=f350d694b30aa2f33a348fdef6a477ebe1775393;p=otter.git

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 3e91ec7b..f68713e5 100644
--- 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)]