fix(kad): allow for different order of events in assertion

thomaseizinger · web-flow · commit f335697573e2 · 2023-06-28T12:26:44.000Z
Fixes the test failure witnessed in https://github.com/libp2p/rust-libp2p/actions/runs/5392115337/jobs/9790198947?pr=4128#step:6:162. Pull-Request: #4129.
diff --git a/protocols/kad/tests/client_mode.rs b/protocols/kad/tests/client_mode.rs
@@ -61,6 +61,10 @@ async fn two_servers_add_each_other_to_routing_table() {
         (
             [Identify(_), Kad(UnroutablePeer { .. }), Identify(_), Kad(RoutingUpdated { peer: peer2, .. }), Identify(_)],
             [Identify(_), Identify(_)],
+        )
+        | (
+            [Identify(_), Kad(UnroutablePeer { .. }), Identify(_), Identify(_), Kad(RoutingUpdated { peer: peer2, .. })],
+            [Identify(_), Identify(_)],
         ) => {
             assert_eq!(peer2, server1_peer_id);
         }

Original file line number	Diff line number	Diff line change
`@@ -61,6 +61,10 @@ async fn two_servers_add_each_other_to_routing_table() {`
`61`	`61`	`(`
`62`	`62`	`[Identify(_), Kad(UnroutablePeer { .. }), Identify(_), Kad(RoutingUpdated { peer: peer2, .. }), Identify(_)],`
`63`	`63`	`[Identify(_), Identify(_)],`
	`64`	`+ )`
	`65`	`+ \| (`
	`66`	`+ [Identify(_), Kad(UnroutablePeer { .. }), Identify(_), Identify(_), Kad(RoutingUpdated { peer: peer2, .. })],`
	`67`	`+ [Identify(_), Identify(_)],`
`64`	`68`	`) => {`
`65`	`69`	`assert_eq!(peer2, server1_peer_id);`
`66`	`70`	`}`