nearcore
nearcore copied to clipboard
[stateless_validation] Refactor all tests to handle ChunkStateWitness and ChunkEndorsement network messages
Handling single client tests with stateless validation is currently very difficult as we don't have good ways to deal with the network messages to send the chunk state witness and chunk endorsements.
As a temporary solution, we're calling process_chunk_endorsement
bypassing the network stack for single client settings.
More context in this comment: https://github.com/near/nearcore/pull/10501#discussion_r1467117301
What's the current status?
@shreyan-gupta is this needed for MVP?
Is this still an issue after the new chunk witness distribution work? does the distribution logic has the same shortcoming in tests? I am interested in learning the details of this issue.
confirmed with @shreyan-gupta that this is not needed for MVP
This is actually quite an old issue from Jan. This was for when I had originally implemented the ChunkEndorsement logic for state validation, we didn't have a great way to test out the feature with multiple nodes.
Since then we have invested time in framework like TestLoop to get a set of reliable tests for end to end scenarios. Meanwhile we've also written custom handlers (and tests) for the normal test framework as well. I guess we can close this issue.