Fix regression test to match upstream macro signature

The expect_payment_successful_event! macro takes a bare PaymentId, not an
Option<PaymentId>. Adjust the #1033 regression test accordingly so it
compiles against current upstream/main.

Co-Authored-By: Claude <noreply@anthropic.com>

Author: elnafateh

tests/integration_tests_rust.rs

@@ -3473,7 +3473,7 @@ async fn unified_send_bolt11_duplicate_payment_no_onchain_fallback() {
 		Ok(other) => panic!("Expected Bolt11 result on first send, got: {:?}", other),
 		Err(e) => panic!("Expected Bolt11 result on first send, got error: {:?}", e),
 	};
-	expect_payment_successful_event!(node_a, Some(first_payment_id), None);
+	expect_payment_successful_event!(node_a, first_payment_id, None);
 
 	// Second send with the same URI: should return DuplicatePayment, NOT fall back to on-chain.
 	let second_result = node_a.unified_payment().send(uri_str_bolt11_only, None, None).await;