Update ef_tests
for new compare_fields
API
This commit is contained in:
parent
99ae734638
commit
686963bd34
@ -86,7 +86,9 @@ where
|
|||||||
pub fork: Fork,
|
pub fork: Fork,
|
||||||
|
|
||||||
// Validator registry
|
// Validator registry
|
||||||
|
#[compare_fields(as_slice)]
|
||||||
pub validator_registry: Vec<Validator>,
|
pub validator_registry: Vec<Validator>,
|
||||||
|
#[compare_fields(as_slice)]
|
||||||
pub balances: Vec<u64>,
|
pub balances: Vec<u64>,
|
||||||
|
|
||||||
// Randomness and committees
|
// Randomness and committees
|
||||||
|
@ -1,5 +1,5 @@
|
|||||||
use super::*;
|
use super::*;
|
||||||
use compare_fields::{CompareFields, FieldComparison};
|
use compare_fields::{CompareFields, Comparison, FieldComparison};
|
||||||
use std::fmt::Debug;
|
use std::fmt::Debug;
|
||||||
use types::BeaconState;
|
use types::BeaconState;
|
||||||
|
|
||||||
@ -51,13 +51,17 @@ where
|
|||||||
{
|
{
|
||||||
match (result, expected) {
|
match (result, expected) {
|
||||||
(Ok(result), Some(expected)) => {
|
(Ok(result), Some(expected)) => {
|
||||||
let mismatching_fields: Vec<FieldComparison> = expected
|
let mut mismatching_fields: Vec<Comparison> = expected
|
||||||
.compare_fields(result)
|
.compare_fields(result)
|
||||||
.into_iter()
|
.into_iter()
|
||||||
.filter(|c| !c.equal)
|
// Filter all out all fields that are equal.
|
||||||
// .map(|c| c.field_name)
|
.filter(Comparison::not_equal)
|
||||||
.collect();
|
.collect();
|
||||||
|
|
||||||
|
mismatching_fields
|
||||||
|
.iter_mut()
|
||||||
|
.for_each(|f| f.retain_children(FieldComparison::not_equal));
|
||||||
|
|
||||||
if !mismatching_fields.is_empty() {
|
if !mismatching_fields.is_empty() {
|
||||||
Err(Error::NotEqual(format!(
|
Err(Error::NotEqual(format!(
|
||||||
"Fields not equal (a = expected, b = result): {:#?}",
|
"Fields not equal (a = expected, b = result): {:#?}",
|
||||||
|
Loading…
Reference in New Issue
Block a user