From 4c5685786ce067f48272a160b154f1473f4258d2 Mon Sep 17 00:00:00 2001 From: Ian Jackson Date: Sat, 27 Feb 2021 20:57:58 +0000 Subject: [PATCH] tests: Use assert_eq! and assert! a lot more This is actually better because you can get a backtrace. Signed-off-by: Ian Jackson --- wdriver/wdt-altergame.rs | 6 +++--- wdriver/wdt-simple.rs | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/wdriver/wdt-altergame.rs b/wdriver/wdt-altergame.rs index 133e5033..28c1b101 100644 --- a/wdriver/wdt-altergame.rs +++ b/wdriver/wdt-altergame.rs @@ -28,10 +28,10 @@ impl Ctx { .iter() .filter(|(_e, txt)| txt == desc) .collect(); - ensure!(relevant.len() == url.iter().len()); + assert_eq!(relevant.len(), url.iter().len()); if let Some(url) = url { - ensure!(relevant[0].0.get_attribute("href")? - .as_ref().map(|s| s.as_str()) == Some(url)); + assert_eq!(relevant[0].0.get_attribute("href")? + .as_ref().map(|s| s.as_str()), Some(url)); } Ok::<_,AE>(()) })() diff --git a/wdriver/wdt-simple.rs b/wdriver/wdt-simple.rs index f38a211c..167235e7 100644 --- a/wdriver/wdt-simple.rs +++ b/wdriver/wdt-simple.rs @@ -34,7 +34,7 @@ impl Ctx { let p1g_new = p1.posg()?; dbg!(p1g_old, p1g_new); - ensure!( p1g_new != p1g_old ); + assert!( p1g_new != p1g_old ); w.synch()?; p1g_new -- 2.30.2