diff --git a/tests/ef_tests/src/lib.rs b/tests/ef_tests/src/lib.rs index 828e2615ce..bcce9a9526 100644 --- a/tests/ef_tests/src/lib.rs +++ b/tests/ef_tests/src/lib.rs @@ -1,8 +1,8 @@ use error::Error; +use ethereum_types::{U128, U256}; use serde_derive::Deserialize; use ssz::Decode; use std::fmt::Debug; -use ethereum_types::{U256, U128}; use test_decode::TestDecode; mod error; @@ -35,8 +35,7 @@ pub trait Test { impl Test for TestDoc { fn test(&self) -> Vec> { - self - .test_cases + self.test_cases .iter() .map(|tc| { if let Some(ssz) = &tc.ssz { @@ -47,9 +46,10 @@ impl Test for TestDoc { "uint64" => compare_decoding::(tc.valid, ssz, &tc.value), "uint128" => compare_decoding::(tc.valid, ssz, &tc.value), "uint256" => compare_decoding::(tc.valid, ssz, &tc.value), - _ => { - Err(Error::FailedToParseTest(format!("Unknown type: {}", tc.type_name))) - } + _ => Err(Error::FailedToParseTest(format!( + "Unknown type: {}", + tc.type_name + ))), } } else { // Skip tests that do not have an ssz field. @@ -66,8 +66,7 @@ fn compare_decoding(should_pass: bool, ssz: &String, value: &String) -> Resul where T: Decode + TestDecode + Debug + PartialEq, { - let ssz = hex::decode(&ssz[2..]) - .map_err(|e| Error::FailedToParseTest(format!("{:?}", e)))?; + let ssz = hex::decode(&ssz[2..]).map_err(|e| Error::FailedToParseTest(format!("{:?}", e)))?; let expected = T::test_decode(value)?; let decoded = T::from_ssz_bytes(&ssz); diff --git a/tests/ef_tests/tests/tests.rs b/tests/ef_tests/tests/tests.rs index 46fa4e9cb5..e549da0647 100644 --- a/tests/ef_tests/tests/tests.rs +++ b/tests/ef_tests/tests/tests.rs @@ -26,7 +26,7 @@ fn ssz() { let failures: Vec<(usize, &Result<_, _>)> = results .iter() .enumerate() - .filter(|(_i, r)| r.is_ok()) + .filter(|(_i, r)| r.is_err()) .collect(); if !failures.is_empty() {