fix(stress test): ignore InvalidSource errors when sending probes

That errors means the node name changed due to relocation but we haven't yet seen the `Relocated` event for it. No reason to abort the test for this. Just skip the node when sending the probes and try again next time.

examples/stress.rs

@@ -21,7 +21,8 @@ use rand::{
 };
 use serde::{Deserialize, Serialize};
 use sn_routing::{
-    Config, DstLocation, Event as RoutingEvent, Routing, SrcLocation, TransportConfig,
+    Config, DstLocation, Error as RoutingError, Event as RoutingEvent, Routing, SrcLocation,
+    TransportConfig,
 };
 use std::{
     collections::BTreeMap,
@@ -450,10 +451,14 @@ impl Network {
         };
         let bytes = bincode::serialize(&message)?.into();
 
-        node.send_message(SrcLocation::Node(src), DstLocation::Section(dst), bytes)
-            .await?;
-
-        Ok(true)
+        match node
+            .send_message(SrcLocation::Node(src), DstLocation::Section(dst), bytes)
+            .await
+        {
+            Ok(()) => Ok(true),
+            Err(RoutingError::InvalidSource) => Ok(false), // node name changed
+            Err(error) => Err(error.into()),
+        }
     }
 
     fn try_print_status(&mut self) {

src/routing/approved.rs

@@ -1793,15 +1793,15 @@ impl Approved {
                 "Not sending user message {:?} -> {:?}: not part of the source location",
                 src, dst
             );
-            return Err(Error::BadLocation);
+            return Err(Error::InvalidSource);
         }
 
         if matches!(dst, DstLocation::Direct) {
             error!(
                 "Not sending user message {:?} -> {:?}: direct dst not supported",
                 src, dst
             );
-            return Err(Error::BadLocation);
+            return Err(Error::InvalidDestination);
         }
 
         let variant = Variant::UserMessage(content);