Fine tune some tests.

tests/bool_op.rs

@@ -62,12 +62,10 @@ fn test_bool_op_short_circuit() -> Result<(), EvalAltResult> {
 }
 
 #[test]
-#[should_panic]
 fn test_bool_op_no_short_circuit1() {
     let mut engine = Engine::new();
 
-    assert_eq!(
+    assert!(engine
-        engine
         .eval::<bool>(
             r"
                     let this = true;
@@ -75,18 +73,14 @@ fn test_bool_op_no_short_circuit1() {
                     this | { throw; }
                 "
         )
-            .unwrap(),
+        .is_err());
-        false
-    );
 }
 
 #[test]
-#[should_panic]
 fn test_bool_op_no_short_circuit2() {
     let mut engine = Engine::new();
 
-    assert_eq!(
+    assert!(engine
-        engine
         .eval::<bool>(
             r"
                     let this = false;
@@ -94,7 +88,5 @@ fn test_bool_op_no_short_circuit2() {
                     this & { throw; }
                 "
         )
-            .unwrap(),
+        .is_err());
-        false
-    );
 }

tests/var_scope.rs

@@ -42,7 +42,15 @@ fn test_scope_eval() -> Result<(), EvalAltResult> {
     println!("result: {}", result); // should print 966
 
     // Variable y is changed in the script
-    assert_eq!(scope.get_value::<INT>("y").unwrap(), 1);
+    assert_eq!(
+        scope
+            .get_value::<INT>("y")
+            .ok_or(EvalAltResult::ErrorRuntime(
+                "variable y not found".into(),
+                Default::default()
+            ))?,
+        1
+    );
 
     Ok(())
 }