fix: double spend ignored when parent is double spent

sn_node/src/put_validation.rs

@@ -734,6 +734,7 @@ impl Node {
                 Ok((spend, Err(NetworkError::Transfer(TransferError::DoubleSpentParent)))) => {
                     warn!("Parent of {spend:?} was double spent, keeping aside in case we're a double spend as well");
                     let _ = double_spent_parent.insert(spend.clone());
+                    let _ = all_verified_spends.insert(spend.clone());
                 }
                 Ok((spend, Err(e))) => {
                     // an error here most probably means the received spend is invalid

sn_node/tests/double_spend.rs

@@ -334,6 +334,7 @@ async fn parent_and_child_double_spends_should_lead_to_cashnote_being_invalid()
         reason.clone(),
     )?;
 
+    info!("spend B to C: {:?}", transfer_to_c.all_spend_requests);
     client
         .send_spends(transfer_to_c.all_spend_requests.iter(), false)
         .await?;
@@ -386,9 +387,18 @@ async fn parent_and_child_double_spends_should_lead_to_cashnote_being_invalid()
         wallet_b.address(),
         reason.clone(),
     )?; // reuse the old cash notes
+
+    info!("spend B to Y: {:?}", transfer_to_y.all_spend_requests);
     client
         .send_spends(transfer_to_y.all_spend_requests.iter(), false)
         .await?;
+    let spend_b_to_y = transfer_to_y
+        .all_spend_requests
+        .first()
+        .expect("should have one");
+    let b_spends = client.get_spend_from_network(spend_b_to_y.address()).await;
+    info!("B spends: {b_spends:?}");
+
     info!("Verifying the transfers from B -> Y wallet... It should error out.");
     let cash_notes_for_y: Vec<_> = transfer_to_y.cash_notes_for_recipient.clone();
     let result = client.verify_cashnote(&cash_notes_for_y[0]).await;