chore: testify fix got, expected transpositions (#2666)

* chore: testify lint fixes

* chore: revert t.error replacements

* chore: revert errNotWebSocketAddress

---------

Co-authored-by: Sukun <sukunrt@gmail.com>

Author: dozyio

core/network/context_test.go

@@ -47,13 +47,13 @@ func TestSimultaneousConnect(t *testing.T) {
 		ok, isClient, reason := GetSimultaneousConnect(serverCtx)
 		require.True(t, ok)
 		require.False(t, isClient)
-		require.Equal(t, reason, "foobar")
+		require.Equal(t, "foobar", reason)
 	})
 	t.Run("for the client", func(t *testing.T) {
 		serverCtx := WithSimultaneousConnect(context.Background(), true, "foo")
 		ok, isClient, reason := GetSimultaneousConnect(serverCtx)
 		require.True(t, ok)
 		require.True(t, isClient)
-		require.Equal(t, reason, "foo")
+		require.Equal(t, "foo", reason)
 	})
 }

libp2p_test.go

@@ -325,8 +325,8 @@ func TestTransportCustomAddressWebTransport(t *testing.T) {
 	require.NotEqual(t, addrs[0], customAddr)
 	restOfAddr, lastComp := ma.SplitLast(addrs[0])
 	restOfAddr, secondToLastComp := ma.SplitLast(restOfAddr)
-	require.Equal(t, lastComp.Protocol().Code, ma.P_CERTHASH)
-	require.Equal(t, secondToLastComp.Protocol().Code, ma.P_CERTHASH)
+	require.Equal(t, ma.P_CERTHASH, lastComp.Protocol().Code)
+	require.Equal(t, ma.P_CERTHASH, secondToLastComp.Protocol().Code)
 	require.True(t, restOfAddr.Equal(customAddr))
 }
 
@@ -352,7 +352,7 @@ func TestTransportCustomAddressWebTransportDoesNotStall(t *testing.T) {
 	addrs := h.Addrs()
 	require.Len(t, addrs, 1)
 	_, lastComp := ma.SplitLast(addrs[0])
-	require.NotEqual(t, lastComp.Protocol().Code, ma.P_CERTHASH)
+	require.NotEqual(t, ma.P_CERTHASH, lastComp.Protocol().Code)
 	// We did not add the certhash to the multiaddr
 	require.Equal(t, addrs[0], customAddr)
 }

p2p/host/basic/basic_host_test.go

@@ -377,7 +377,7 @@ func TestHostProtoPreknowledge(t *testing.T) {
 
 	// This test implicitly relies on 1 connection. If a background identify
 	// completes after we set the stream handler below things break
-	require.Equal(t, 1, len(h1.Network().ConnsToPeer(h2.ID())))
+	require.Len(t, h1.Network().ConnsToPeer(h2.ID()), 1)
 
 	// wait for identify handshake to finish completely
 	select {
@@ -469,7 +469,7 @@ func TestNewStreamResolve(t *testing.T) {
 			break
 		}
 	}
-	assert.NotEqual(t, dialAddr, "")
+	assert.NotEqual(t, "", dialAddr)
 
 	// Add the DNS multiaddr to h1's peerstore.
 	maddr, err := ma.NewMultiaddr(dialAddr)
@@ -506,7 +506,7 @@ func TestProtoDowngrade(t *testing.T) {
 		defer s.Close()
 		result, err := io.ReadAll(s)
 		assert.NoError(t, err)
-		assert.Equal(t, string(result), "bar")
+		assert.Equal(t, "bar", string(result))
 		connectedOn <- s.Protocol()
 	})
 
@@ -527,7 +527,7 @@ func TestProtoDowngrade(t *testing.T) {
 		defer s.Close()
 		result, err := io.ReadAll(s)
 		assert.NoError(t, err)
-		assert.Equal(t, string(result), "foo")
+		assert.Equal(t, "foo", string(result))
 		connectedOn <- s.Protocol()
 	})
 

p2p/host/peerstore/pstoremem/sorting_test.go

@@ -16,5 +16,5 @@ func TestAddressSorting(t *testing.T) {
 
 	l := addrList{local, u1, u2l, norm}
 	sort.Sort(l)
-	require.Equal(t, l, addrList{u2l, u1, local, norm})
+	require.Equal(t, addrList{u2l, u1, local, norm}, l)
 }

p2p/host/peerstore/test/peerstore_suite.go

@@ -337,11 +337,11 @@ func testMetadata(ps pstore.Peerstore) func(t *testing.T) {
 			for _, p := range pids {
 				v, err := ps.Get(p, "AgentVersion")
 				require.NoError(t, err)
-				require.Equal(t, v, "string")
+				require.Equal(t, "string", v)
 
 				v, err = ps.Get(p, "bar")
 				require.NoError(t, err)
-				require.Equal(t, v, 1)
+				require.Equal(t, 1, v)
 			}
 		})
 
@@ -359,7 +359,7 @@ func testMetadata(ps pstore.Peerstore) func(t *testing.T) {
 			// make sure that entries for otherP were not deleted
 			val, err := ps.Get(otherP, "AgentVersion")
 			require.NoError(t, err)
-			require.Equal(t, val, "v1")
+			require.Equal(t, "v1", val)
 		})
 	}
 }

p2p/host/resource-manager/limit_test.go

@@ -110,7 +110,6 @@ func TestScaling(t *testing.T) {
 		require.Equal(t, 20+4*2, scaled.service["B"].Streams)
 		require.Equal(t, int64(200+4*3), scaled.service["B"].Memory)
 		require.Equal(t, 400, scaled.service["B"].FD)
-
 	})
 }
 
@@ -157,7 +156,7 @@ func TestJSONMarshalling(t *testing.T) {
 
 	jsonEncoded, err := json.Marshal(bl)
 	require.NoError(t, err)
-	require.Equal(t, string(jsonEncoded), `{"StreamsInbound":10,"StreamsOutbound":"blockAll","Conns":10,"ConnsOutbound":"unlimited","Memory":"unlimited"}`)
+	require.Equal(t, `{"StreamsInbound":10,"StreamsOutbound":"blockAll","Conns":10,"ConnsOutbound":"unlimited","Memory":"unlimited"}`, string(jsonEncoded))
 
 	// Roundtrip
 	var blDecoded ResourceLimits
@@ -175,7 +174,7 @@ func TestJSONRoundTripInt64(t *testing.T) {
 	jsonEncoded, err := json.Marshal(bl)
 	require.NoError(t, err)
 
-	require.Equal(t, string(jsonEncoded), `{"Memory":"9223372036854775807"}`)
+	require.Equal(t, `{"Memory":"9223372036854775807"}`, string(jsonEncoded))
 
 	// Roundtrip
 	var blDecoded ResourceLimits

p2p/http/libp2phttp_test.go

@@ -408,7 +408,7 @@ func TestCustomServeMux(t *testing.T) {
 	defer server.Close()
 
 	addrs := server.Addrs()
-	require.Equal(t, len(addrs), 1)
+	require.Len(t, addrs, 1)
 	var clientHttpHost libp2phttp.Host
 	rt, err := clientHttpHost.NewConstrainedRoundTripper(peer.AddrInfo{Addrs: addrs}, libp2phttp.PreferHTTPTransport)
 	require.NoError(t, err)
@@ -466,7 +466,7 @@ func TestSetHandlerAtPath(t *testing.T) {
 			}()
 			defer server.Close()
 			addrs := server.Addrs()
-			require.Equal(t, 1, len(addrs))
+			require.Len(t, addrs, 1)
 			port, err := addrs[0].ValueForProtocol(ma.P_TCP)
 			require.NoError(t, err)
 			httpAddr := fmt.Sprintf("http://127.0.0.1:%s", port)
@@ -483,5 +483,4 @@ func TestSetHandlerAtPath(t *testing.T) {
 			}
 		})
 	}
-
 }

p2p/net/connmgr/connmgr_test.go

@@ -662,7 +662,6 @@ func TestPeerProtectionMultipleTags(t *testing.T) {
 			t.Error("protected connection was closed by connection manager")
 		}
 	}
-
 }
 
 func TestPeerProtectionIdempotent(t *testing.T) {
@@ -835,15 +834,15 @@ func TestPeerInfoSorting(t *testing.T) {
 		p2 := &peerInfo{id: peer.ID("peer2"), temp: true}
 		pis := peerInfos{p1, p2}
 		pis.SortByValueAndStreams(makeSegmentsWithPeerInfos(pis), false)
-		require.Equal(t, pis, peerInfos{p2, p1})
+		require.Equal(t, peerInfos{p2, p1}, pis)
 	})
 
 	t.Run("starts with low-value connections", func(t *testing.T) {
 		p1 := &peerInfo{id: peer.ID("peer1"), value: 40}
 		p2 := &peerInfo{id: peer.ID("peer2"), value: 20}
 		pis := peerInfos{p1, p2}
 		pis.SortByValueAndStreams(makeSegmentsWithPeerInfos(pis), false)
-		require.Equal(t, pis, peerInfos{p2, p1})
+		require.Equal(t, peerInfos{p2, p1}, pis)
 	})
 
 	t.Run("prefer peers with no streams", func(t *testing.T) {
@@ -859,7 +858,7 @@ func TestPeerInfoSorting(t *testing.T) {
 		}
 		pis := peerInfos{p2, p1}
 		pis.SortByValueAndStreams(makeSegmentsWithPeerInfos(pis), false)
-		require.Equal(t, pis, peerInfos{p1, p2})
+		require.Equal(t, peerInfos{p1, p2}, pis)
 	})
 
 	t.Run("in a memory emergency, starts with incoming connections and higher streams", func(t *testing.T) {
@@ -902,7 +901,7 @@ func TestPeerInfoSorting(t *testing.T) {
 		// p3 is first because it is inactive (no streams).
 		// p4 is second because it has the most streams and we priortize killing
 		// connections with the higher number of streams.
-		require.Equal(t, pis, peerInfos{p3, p4, p2, p1})
+		require.Equal(t, peerInfos{p3, p4, p2, p1}, pis)
 	})
 
 	t.Run("in a memory emergency, starts with connections that have many streams", func(t *testing.T) {
@@ -921,7 +920,7 @@ func TestPeerInfoSorting(t *testing.T) {
 		}
 		pis := peerInfos{p1, p2}
 		pis.SortByValueAndStreams(makeSegmentsWithPeerInfos(pis), true)
-		require.Equal(t, pis, peerInfos{p2, p1})
+		require.Equal(t, peerInfos{p2, p1}, pis)
 	})
 }
 

p2p/net/connmgr/decay_test.go

@@ -49,7 +49,7 @@ func TestMultipleBumps(t *testing.T) {
 	require.NoError(t, tag.Bump(id, 5))
 
 	waitForTag(t, mgr, id)
-	require.Equal(t, mgr.GetTagInfo(id).Value, 10)
+	require.Equal(t, 10, mgr.GetTagInfo(id).Value)
 
 	require.NoError(t, tag.Bump(id, 100))
 	require.Eventually(t, func() bool { return mgr.GetTagInfo(id).Value == 20 }, 100*time.Millisecond, 10*time.Millisecond, "expected tag value to decay to 20")
@@ -77,7 +77,7 @@ func TestMultipleTagsNoDecay(t *testing.T) {
 
 	// all tags are upper-bounded, so the score must be 300
 	ti := mgr.GetTagInfo(id)
-	require.Equal(t, ti.Value, 300)
+	require.Equal(t, 300, ti.Value)
 
 	for _, s := range []string{"beep", "bop", "foo"} {
 		if v, ok := ti.Tags[s]; !ok || v != 100 {

p2p/net/swarm/swarm_dial_test.go

@@ -114,7 +114,7 @@ func TestDedupAddrsForDial(t *testing.T) {
 	mas, _, err := s.addrsForDial(ctx, otherPeer)
 	require.NoError(t, err)
 
-	require.Equal(t, 1, len(mas))
+	require.Len(t, mas, 1)
 }
 
 func newTestSwarmWithResolver(t *testing.T, resolver *madns.Resolver) *Swarm {
@@ -275,7 +275,7 @@ func TestAddrResolutionRecursiveTransportSpecific(t *testing.T) {
 	addrs, _, err := s.addrsForDial(tctx, p)
 	require.NoError(t, err)
 	require.Len(t, addrs, 1)
-	require.Equal(t, addrs[0].String(), "/ip4/1.2.3.4/tcp/443/tls/sni/sub.example.com/ws")
+	require.Equal(t, "/ip4/1.2.3.4/tcp/443/tls/sni/sub.example.com/ws", addrs[0].String())
 }
 
 func TestAddrsForDialFiltering(t *testing.T) {