Score PaymentPathFailed with payment_failed_permanently as ProbeSuccess

Prior to this commit, we wouldn't score PaymentPathFailed events when the
destination explicitly failed the payment back when the blamed scid is None.
Instead, we should treat this as a ProbeSuccess because the payment reached the
destination.

Author: valentinewallace

lightning-background-processor/src/lib.rs

@@ -230,6 +230,12 @@ fn update_scorer<'a, S: 'static + Deref<Target = SC> + Send + Sync, SC: 'a + Wri
 			let path = path.iter().collect::<Vec<_>>();
 			score.payment_path_failed(&path, *scid);
 		},
+		Event::PaymentPathFailed { ref path, payment_failed_permanently: true, .. } => {
+			// Reached if the destination explicitly failed it back. We treat this as a successful probe
+			// because the payment made it all the way to the destination with sufficient liquidity.
+			let path = path.iter().collect::<Vec<_>>();
+			score.probe_successful(&path);
+		},
 		Event::PaymentPathSuccessful { path, .. } => {
 			let path = path.iter().collect::<Vec<_>>();
 			score.payment_path_successful(&path);