I had this note from the other gitter
oh… it is! apparently someone had a different idea than I did. I think it most important to test the shuffle alg in isolation and then the rest can be tested via the state tests. That said, I’m not entirely against having separate tests for
get_shuffling. I just don’t think we should be in the habit of testing all of these functions in isolation because no client should have to implement all the individual helpers exactly s in the spec.
permutated_indexwhich doesn't reflects in compiled tests repo, but current shuffling tests are useless and outdated. We'd either remove them at all and leave only permutated tests or update shuffling, so it will duplicate permutated but with a bit of shuffling logic a-top.
signed_rootdoesn't match, but I'm going to add generators for it, so it would be simpler for us and other clients to verify that we have the same logics for hashing. Next, for example,
test_proposer_slashingdoesn't match. I guess, there will be different results for
get_crosslink_committees_at_slot(slashing values match, but validator order differs) etc