tests: Fix warnings from clippy::no_effect

Actually a nice logic error

Author: lutter

tests/runner-tests/data-source-revert/grafted.yaml

@@ -6,7 +6,7 @@ schema:
 graft:
   # This can be overwritten by `updateAndDeploy.js`.
   # Please commit this file when this happens.
-  base: QmcAL39QSKZvRssr2ToCJrav7XK9ggajxvBR7M1NNUCqdh
+  base: QmRfaNu6ymQVUiJVTayqWuT8ftLtvn7iE8kQSSAtTPA2J2
   block: 3
 dataSources:
   - kind: ethereum/contract

tests/tests/runner_tests.rs

@@ -37,14 +37,14 @@ fn assert_eq_ignore_backtrace(err: &SubgraphError, expected: &SubgraphError) {
             || err.handler != expected.handler
             || err.deterministic != expected.deterministic
         {
-            false;
-        }
-
-        // Ignore any WASM backtrace in the error message
-        let split_err: Vec<&str> = err.message.split("\\twasm backtrace:").collect();
-        let split_expected: Vec<&str> = expected.message.split("\\twasm backtrace:").collect();
+            false
+        } else {
+            // Ignore any WASM backtrace in the error message
+            let split_err: Vec<&str> = err.message.split("\\twasm backtrace:").collect();
+            let split_expected: Vec<&str> = expected.message.split("\\twasm backtrace:").collect();
 
-        split_err.first() == split_expected.first()
+            split_err.first() == split_expected.first()
+        }
     };
 
     if !equal {