chore: fix golangci-lint errors

Author: galargh

core/record/envelope_test.go

@@ -91,8 +91,8 @@ func TestEnvelopeHappyPath(t *testing.T) {
 
 func TestConsumeTypedEnvelope(t *testing.T) {
 	var (
-		rec          = simpleRecord{message: "hello world!"}
-		priv, _, err = test.RandTestKeyPair(crypto.Ed25519, 256)
+		rec        = simpleRecord{message: "hello world!"}
+		priv, _, _ = test.RandTestKeyPair(crypto.Ed25519, 256)
 	)
 
 	envelope, err := Seal(&rec, priv)

p2p/discovery/backoff/backoff_test.go

@@ -21,9 +21,7 @@ func TestFixedBackoff(t *testing.T) {
 	delay := startDelay
 
 	bkf := NewFixedBackoff(delay)
-	delay *= 2
 	b1 := bkf()
-	delay *= 2
 	b2 := bkf()
 
 	if b1.Delay() != startDelay || b2.Delay() != startDelay {

p2p/host/autonat/autonat.go

@@ -270,7 +270,6 @@ func (as *AmbientAutoNAT) scheduleProbe(forceProbe bool) time.Duration {
 		// retry very quicky if forceProbe is true *and* we don't know our reachability
 		// limit all peers fetch from peerstore to 1 per second.
 		nextProbeAfter = 2 * time.Second
-		nextProbeAfter = 2 * time.Second
 	case currentStatus == network.ReachabilityUnknown,
 		as.confidence < maxConfidence,
 		currentStatus != network.ReachabilityPublic && receivedInbound:

p2p/http/example_test.go

@@ -181,7 +181,7 @@ func ExampleHost_overLibp2pStreams() {
 	// Make an HTTP request using the Go standard library, but over libp2p
 	// streams. If the server were listening on an HTTP transport, this could
 	// also make the request over the HTTP transport.
-	httpClient, err := client.NamespacedClient("/echo/1.0.0", peer.AddrInfo{ID: server.PeerID(), Addrs: server.Addrs()})
+	httpClient, _ := client.NamespacedClient("/echo/1.0.0", peer.AddrInfo{ID: server.PeerID(), Addrs: server.Addrs()})
 
 	// Only need to Post to "/" because this client is namespaced to the "/echo/1.0.0" protocol.
 	resp, err := httpClient.Post("/", "application/octet-stream", strings.NewReader("Hello HTTP"))

p2p/net/connmgr/connmgr_test.go

@@ -531,7 +531,7 @@ func TestPeerProtectionSingleTag(t *testing.T) {
 	}
 
 	// protect the first 5 peers.
-	var protected []network.Conn
+	protected := make([]network.Conn, 0, 5)
 	for _, c := range conns[0:5] {
 		cm.Protect(c.RemotePeer(), "global")
 		protected = append(protected, c)
@@ -610,7 +610,7 @@ func TestPeerProtectionMultipleTags(t *testing.T) {
 	}
 
 	// protect the first 5 peers under two tags.
-	var protected []network.Conn
+	protected := make([]network.Conn, 0, 5)
 	for _, c := range conns[0:5] {
 		cm.Protect(c.RemotePeer(), "tag1")
 		cm.Protect(c.RemotePeer(), "tag2")

p2p/net/nat/internal/nat/nat.go

@@ -29,7 +29,7 @@ func (e ErrNoNATFound) Unwrap() []error {
 }
 
 func (e ErrNoNATFound) Error() string {
-	var errStrs []string
+	errStrs := make([]string, 0, len(e.Errs))
 	for _, err := range e.Errs {
 		errStrs = append(errStrs, err.Error())
 	}

p2p/net/swarm/black_hole_detector_test.go

@@ -11,9 +11,8 @@ import (
 func TestBlackHoleSuccessCounterReset(t *testing.T) {
 	n := 10
 	bhf := &BlackHoleSuccessCounter{N: n, MinSuccesses: 2, Name: "test"}
-	var i = 0
 	// calls up to n should be probing
-	for i = 1; i <= n; i++ {
+	for i := 1; i <= n; i++ {
 		if bhf.HandleRequest() != blackHoleStateProbing {
 			t.Fatalf("expected calls up to n to be probes")
 		}
@@ -24,7 +23,7 @@ func TestBlackHoleSuccessCounterReset(t *testing.T) {
 	}
 
 	// after threshold calls every nth call should be a probe
-	for i = n + 1; i < 42; i++ {
+	for i := n + 1; i < 42; i++ {
 		result := bhf.HandleRequest()
 		if (i%n == 0 && result != blackHoleStateProbing) || (i%n != 0 && result != blackHoleStateBlocked) {
 			t.Fatalf("expected every nth dial to be a probe")
@@ -36,7 +35,7 @@ func TestBlackHoleSuccessCounterReset(t *testing.T) {
 
 	bhf.RecordResult(true)
 	// check if calls up to n are probes again
-	for i = 0; i < n; i++ {
+	for i := 0; i < n; i++ {
 		if bhf.HandleRequest() != blackHoleStateProbing {
 			t.Fatalf("expected black hole detector state to reset after success")
 		}

p2p/protocol/circuitv2/relay/relay.go

@@ -591,7 +591,7 @@ func makeReservationMsg(
 		return rsvp
 	}
 
-	var addrBytes [][]byte
+	addrBytes := make([][]byte, 0, len(selfAddrs))
 	for _, addr := range selfAddrs {
 		if !manet.IsPublicAddr(addr) {
 			continue

p2p/protocol/circuitv2/relay/relay_priv_test.go

@@ -42,7 +42,7 @@ func TestMakeReservationWithP2PAddrs(t *testing.T) {
 		"/ip4/1.2.3.4/tcp/1235/p2p/" + selfID.String(),
 	}
 
-	var addrsFromRsvp []string
+	addrsFromRsvp := make([]string, 0, len(rsvp.GetAddrs()))
 	for _, addr := range rsvp.GetAddrs() {
 		a, err := ma.NewMultiaddrBytes(addr)
 		require.NoError(t, err)