accounts: fix function names in doc

Author: gzliudan

accounts/abi/abigen/bind_test.go

@@ -2084,8 +2084,8 @@ var bindTests = []struct {
 	},
 }
 
-// Tests that packages generated by the binder can be successfully compiled and
-// the requested tester run against it.
+// TestBindings tests that packages generated by the binder can be successfully
+// compiled and the requested tester run against it.
 func TestBindings(t *testing.T) {
 	t.Parallel()
 	// Skip the test if no Go command can be found

accounts/abi/abigen/bindv2.go

@@ -186,9 +186,9 @@ func (cb *contractBinder) bindMethod(original abi.Method) error {
 	return nil
 }
 
-// normalize a set of arguments by stripping underscores, giving a generic name
-// in the case where the arg name collides with a reserved Go keyword, and finally
-// converting to camel-case.
+// normalizeArgs normalizes a set of arguments by stripping underscores,
+// giving a generic name in the case where the arg name collides with
+// a reserved Go keyword, and finally converting to camel-case.
 func normalizeArgs(args abi.Arguments) abi.Arguments {
 	args = slices.Clone(args)
 	used := make(map[string]bool)

accounts/abi/abigen/bindv2_test.go

@@ -323,7 +323,8 @@ func TestNormalizeArgs(t *testing.T) {
 	}
 }
 
-// returns a "pretty diff" on two strings. Useful if the strings are large.
+// prettyDiff returns a "pretty diff" on two strings. Useful if the strings
+// are large.
 func prettyDiff(have, want string) string {
 	if have == want {
 		return ""

accounts/abi/abigen/source2.go.tpl

@@ -225,7 +225,7 @@ var (
 			{{capitalise .Name}} {{if .Indexed}}{{bindtopictype .Type $structs}}{{else}}{{bindtype .Type $structs}}{{end}}; {{end}}
 		}
 
-		// ErrorID returns the hash of canonical representation of the error's signature.
+		// {{$contract.Type}}{{.Normalized.Name}}ErrorID returns the hash of canonical representation of the error's signature.
 		//
 		// Solidity: {{.Original.String}}
 		func {{$contract.Type}}{{.Normalized.Name}}ErrorID() common.Hash {

accounts/abi/bind/v2/internal/contracts/solc_errors/bindings.go

@@ -73,8 +73,8 @@ func makeTestDeployerWithNonceAssignment(backend simulated.Client) func(input, d
 	return bind.DeployerWithNonceAssignment(bind.NewKeyedTransactor(testKey, chainId), backend)
 }
 
-// test that deploying a contract with library dependencies works,
-// verifying by calling method on the deployed contract.
+// TestDeploymentLibraries tests that deploying a contract with library
+// dependencies works, verifying by calling method on the deployed contract.
 func TestDeploymentLibraries(t *testing.T) {
 	bindBackend, err := testSetup()
 	if err != nil {
@@ -117,8 +117,8 @@ func TestDeploymentLibraries(t *testing.T) {
 	}
 }
 
-// Same as TestDeployment.  However, stagger the deployments with overrides:
-// first deploy the library deps and then the contract.
+// TestDeploymentWithOverrides is same as TestDeployment. However, stagger the
+// deployments with overrides: first deploy the library deps and then the contract.
 func TestDeploymentWithOverrides(t *testing.T) {
 	bindBackend, err := testSetup()
 	if err != nil {
@@ -185,7 +185,7 @@ func TestDeploymentWithOverrides(t *testing.T) {
 	}
 }
 
-// returns transaction auth to send a basic transaction from testAddr
+// defaultTxAuth returns transaction auth to send a basic tx from testAddr
 func defaultTxAuth() *bind.TransactOpts {
 	signer := types.LatestSigner(params.AllDevChainProtocolChanges)
 	opts := &bind.TransactOpts{

accounts/abi/bind/v2/lib_test.go

@@ -29,7 +29,7 @@ import (
 // the stringer interface to allow printing type details in the tests below.
 type typeWithoutStringer Type
 
-// Tests that all allowed types get recognized by the type parser.
+// TestTypeRegexp tests that all allowed types get recognized by the type parser.
 func TestTypeRegexp(t *testing.T) {
 	t.Parallel()
 	tests := []struct {

accounts/abi/type_test.go

@@ -121,9 +121,11 @@ func readBool(word []byte) (bool, error) {
 	}
 }
 
-// A function type is simply the address with the function selection signature at the end.
+// readFunctionType enforces that standard by always presenting it as
+// a 24-array (address + sig = 24 bytes)
 //
-// readFunctionType enforces that standard by always presenting it as a 24-array (address + sig = 24 bytes)
+// A function type is simply the address with the function selection
+// signature at the end.
 func readFunctionType(t Type, word []byte) (funcTy [24]byte, err error) {
 	if t.T != FunctionTy {
 		return [24]byte{}, errors.New("abi: invalid type in call to make function type byte array")

accounts/abi/unpack.go

@@ -22,8 +22,8 @@ import (
 	"testing"
 )
 
-// Tests that HD derivation paths can be correctly parsed into our internal binary
-// representation.
+// TestHDPathParsing tests that HD derivation paths can be correctly
+// parsed into our internal binary representation.
 func TestHDPathParsing(t *testing.T) {
 	t.Parallel()
 	tests := []struct {

accounts/hd_test.go

@@ -190,7 +190,7 @@ func TestOverrideUnlock(t *testing.T) {
 	}
 }
 
-// This test should fail under -race if signing races the expiration goroutine.
+// TestSignRace should fail under -race if signing races the expiration goroutine.
 func TestSignRace(t *testing.T) {
 	t.Parallel()
 	_, ks := tmpKeyStore(t)
@@ -233,8 +233,8 @@ func waitForKsUpdating(t *testing.T, ks *KeyStore, wantStatus bool, maxTime time
 	return false
 }
 
-// Tests that the wallet notifier loop starts and stops correctly based on the
-// addition and removal of wallet event subscriptions.
+// TestWalletNotifierLifecycle tests that the wallet notifier loop starts and stops correctly
+// based on the addition and removal of wallet event subscriptions.
 func TestWalletNotifierLifecycle(t *testing.T) {
 	t.Parallel()
 	// Create a temporary keystore to test with
@@ -280,8 +280,8 @@ type walletEvent struct {
 	a accounts.Account
 }
 
-// Tests that wallet notifications and correctly fired when accounts are added
-// or deleted from the keystore.
+// TestWalletNotifications tests that wallet notifications and correctly fired
+// when accounts are added or deleted from the keystore.
 func TestWalletNotifications(t *testing.T) {
 	t.Parallel()
 	_, ks := tmpKeyStore(t)