This PR adds a fullCommit action to the test runner, which combines the commit, handlePendingCommit, handleCommit, and joinGroup. The motivation here was to remove the need for CommitResponse.EpochAuthenticator, which could be challenging for stacks to produce if they haven't moved into the new state. Just removing would introduce some ambiguity about how to verify that all the member agree on the epoch authenticator; the approach here keeps the epoch authenticator as local state across the individual steps.
Combining these steps also makes the scripts more terse, of course.
Note that with this, the only reason to keep handleCommit is to be able to test the case where a member creates a commit, but then another commit gets applied. We could probably safely delete the individual handlePendingCommit, handleCommit, and joinGroup actions, since these aren't required for this disaggregated case.
This PR adds a
fullCommitaction to the test runner, which combines thecommit,handlePendingCommit,handleCommit, andjoinGroup. The motivation here was to remove the need forCommitResponse.EpochAuthenticator, which could be challenging for stacks to produce if they haven't moved into the new state. Just removing would introduce some ambiguity about how to verify that all the member agree on the epoch authenticator; the approach here keeps the epoch authenticator as local state across the individual steps.Combining these steps also makes the scripts more terse, of course.
Note that with this, the only reason to keep
handleCommitis to be able to test the case where a member creates a commit, but then another commit gets applied. We could probably safely delete the individualhandlePendingCommit,handleCommit, andjoinGroupactions, since these aren't required for this disaggregated case.