Document outbound payment amount invariant

Replace the success-path unwrap on payment amounts with an expect that explains why outbound payments must already have a recorded amount by the time LDK reports them as sent.

Co-Authored-By: HAL 9000

Author: tnull

src/event.rs

@@ -1092,8 +1092,9 @@ where
 				};
 
 				self.payment_store.get(&payment_id).map(|payment| {
-					#[allow(clippy::unwrap_used)]
-					let amount_msat = payment.amount_msat.unwrap();
+					let amount_msat = payment.amount_msat.expect(
+						"outbound payments should record their amount before they can succeed",
+					);
 					log_info!(
 						self.logger,
 						"Successfully sent payment of {}msat{} from \