Run misspell linter with -w flag (#2320)

This runs the popular misspell linter with the -w flag to automatically correct spelling mistakes in go-algorand.

Author: cce

agreement/abstractions.go

@@ -51,7 +51,7 @@ type BlockValidator interface {
 	Validate(context.Context, bookkeeping.Block) (ValidatedBlock, error)
 }
 
-// A ValidatedBlock represents an Block that has been successfuly validated
+// A ValidatedBlock represents an Block that has been successfully validated
 // and can now be recorded in the ledger.  This is an optimized version of
 // calling EnsureBlock() on the Ledger.
 type ValidatedBlock interface {

agreement/fuzzer/fuzzer_test.go

@@ -229,7 +229,7 @@ func (n *Fuzzer) initAccountsAndBalances(rootSeed []byte, onlineNodes []bool) er
 	return nil
 }
 
-// Disconnect would disconnect node diconnectingNode from node disconnectedNode ensuring that no futher messages
+// Disconnect would disconnect node diconnectingNode from node disconnectedNode ensuring that no further messages
 // from disconnectedNode would reach diconnectingNode
 func (n *Fuzzer) Disconnect(diconnectingNode, disconnectedNode int) {
 	n.disconnectMu.Lock()

agreement/fuzzer/networkFacade_test.go

@@ -352,7 +352,7 @@ func (n *NetworkFacade) Zero() timers.Clock {
 	n.zeroClock = n.fuzzer.WallClock()
 
 	// we don't want to expire all the pending clocks here.
-	// this callback is coming *only* from the agreement sevice.
+	// this callback is coming *only* from the agreement service.
 	// it also means that we're not in the demux loop, so no one is blocking
 	// on any of the clocks.
 

agreement/fuzzer/tests_test.go

@@ -78,7 +78,7 @@ func printResults(t *testing.T, r *RunResult) {
 		r.PostRecoveryLowRound, r.PostRecoveryHighRound,
 	)
 	if r.PreRecoveryHighRound != r.PreRecoveryLowRound {
-		// network got disupted by the filters.
+		// network got disputed by the filters.
 		fmt.Printf("%v partitioned the network ( %d - %d ), but recovered correctly reaching round %d\n", t.Name(), r.PreRecoveryLowRound, r.PreRecoveryHighRound, r.PostRecoveryHighRound)
 	} else {
 		if r.PreRecoveryHighRound == r.StartLowRound {

agreement/fuzzer/validator_test.go

@@ -82,7 +82,7 @@ func (v *Validator) CheckNetworkRecovery() {
 		v.runResult.PostRecoveryLowRound, v.runResult.PostRecoveryHighRound,
 	)
 	if v.runResult.PreRecoveryHighRound != v.runResult.PreRecoveryLowRound {
-		// network got disupted by the filters.
+		// network got disputed by the filters.
 		fmt.Printf("%v partitioned the network ( %d - %d ), but recovered correctly reaching round %d\n", v.tb.Name(), v.runResult.PreRecoveryLowRound, v.runResult.PreRecoveryHighRound, v.runResult.PostRecoveryHighRound)
 	} else {
 		if v.runResult.PreRecoveryHighRound == v.runResult.StartLowRound {

agreement/player_test.go

@@ -2972,7 +2972,7 @@ func TestPlayer_CertThresholdCommitsFuturePeriodIfAlreadyHasBlock(t *testing.T)
 	require.Equalf(t, r+1, pWhite.Round, "player did not enter new round... bad!")
 	require.Equalf(t, period(0), pWhite.Period, "player should have entered period 0 of new round but didn't")
 	commitEvent := ev(ensureAction{Certificate: Certificate(bun), Payload: *payload})
-	require.Truef(t, pM.getTrace().Contains(commitEvent), "Player should have commited a block but didn't")
+	require.Truef(t, pM.getTrace().Contains(commitEvent), "Player should have committed a block but didn't")
 }
 
 func TestPlayer_PayloadAfterCertThresholdCommits(t *testing.T) {

agreement/proposalManager.go

@@ -23,7 +23,7 @@ import (
 )
 
 // A proposalManager is a proposalMachine which applies relay rules to incoming
-// proposal-votes and proposal payloads, absorbes threshold events, and emits
+// proposal-votes and proposal payloads, absorbs threshold events, and emits
 // proposalCommittable events as proposals become committable.
 //
 // It handles the following type(s) of event: vote{Present,Verified},

agreement/state_machine_test.go

@@ -164,7 +164,7 @@ type ioSafetyProp interface {
 	// returns bool whether trace is in the safety property. If false,
 	// optionally accompanied by an informational message pertaining to why
 	// the trace is not in the safety property. Err is set if we saw an
-	// unforseen error.
+	// unforeseen error.
 	containsTrace(trace ioTrace) (contains bool, info string, err error)
 
 	// every safety prop also exposes the option to check it dynamically
@@ -283,7 +283,7 @@ type ioAutomataConcrete struct {
 	listener
 
 	// listeners need additional context. For now, we keep it static.
-	routerCtx router // optional, set to router{} on defualt
+	routerCtx router // optional, set to router{} on default
 	playerCtx player // optional, set to player{} by default
 
 	// private

agreement/voteAuxiliary_test.go

@@ -99,7 +99,7 @@ func TestVoteTrackerPeriodStepCachedThresholdPrivate(t *testing.T) {
 	require.NoError(t, err)
 	require.NoErrorf(t, res, "Expected threshold event not relayed")
 
-	// now, given that a bottom threshold was emmited, make sure private state was set (so we can appropriately respond to queries)
+	// now, given that a bottom threshold was emitted, make sure private state was set (so we can appropriately respond to queries)
 	vt := perRouter.voteRoot.underlying().(*voteTrackerPeriod)
 	require.Truef(t, vt.Cached.Bottom, "VoteTrackerPeriod didn't set bottom to true")
 

agreement/voteTracker.go

@@ -216,7 +216,7 @@ func (tracker *voteTracker) handle(r routerHandle, p player, e0 event) event {
 
 			// We've just moved the vote around ( regular vote -> equivocator vote ) but that did not
 			// change the total weight. Since the total weight for the proposal in the vote wasn't altered,
-			// we know for sure that we haven't reached a threshold for that proposal. ( but maybe for a diffrent one )
+			// we know for sure that we haven't reached a threshold for that proposal. ( but maybe for a different one )
 
 			// at this point, we need to check if this is the very last vote or not.
 			// if we have no regular votes, we won't be generating a bundle so we can abort right here.