fixup! Add end-to-end test for HRN resolution

Author: chuksys

.github/workflows/kotlin.yml

@@ -41,20 +41,28 @@ jobs:
           ./gradlew ktlintCheck
 
       - name: Generate Kotlin JVM
+        env:
+          RUSTFLAGS: "--cfg hrn_tests"
         run: ./scripts/uniffi_bindgen_generate_kotlin.sh
 
       - name: Generate Kotlin Android
+        env:
+          RUSTFLAGS: "--cfg hrn_tests"
         run: ./scripts/uniffi_bindgen_generate_kotlin_android.sh
 
       - name: Start bitcoind and electrs
         run: docker compose up -d
 
       - name: Run ldk-node-jvm tests
+        env:
+            RUSTFLAGS: "--cfg hrn_tests"
         run: |
           cd $LDK_NODE_JVM_DIR
           ./gradlew test -Penv=ci
 
       - name: Run ldk-node-android tests
+        env:
+            RUSTFLAGS: "--cfg hrn_tests"
         run: |
           cd $LDK_NODE_ANDROID_DIR
           ./gradlew test

.github/workflows/python.yml

@@ -28,6 +28,7 @@ jobs:
 
       - name: Run Python unit tests
         env:
+          RUSTFLAGS: "--cfg hrn_tests"
           BITCOIN_CLI_BIN: "docker exec ldk-node-bitcoin-1 bitcoin-cli"
           BITCOIND_RPC_USER: "user"
           BITCOIND_RPC_PASSWORD: "pass"

src/payment/unified.rs

@@ -337,6 +337,7 @@ impl UnifiedPayment {
 	/// Sets a test offer to be used in the `send` method when the `hrn_tests` config flag is enabled.
 	/// This is necessary to test sending Bolt12 payments via the unified payment handler in HRN tests,
 	/// as we cannot rely on the offer being present in the parsed URI.
+	#[uniffi::export]
 	#[cfg(hrn_tests)]
 	pub fn set_test_offer(&self, offer: Offer) {
 		let mut guard = self.test_offer.lock().unwrap();

tests/integration_tests_hrn.rs

@@ -39,6 +39,7 @@ async fn unified_send_to_hrn() {
 
 	node_a.sync_wallets().unwrap();
 	open_channel(&node_a, &node_b, 4_000_000, true, &electrsd).await;
+	tokio::time::sleep(std::time::Duration::from_secs(1)).await;
 	generate_blocks_and_wait(&bitcoind.client, &electrsd.client, 6).await;
 
 	node_a.sync_wallets().unwrap();