X-Git-Url: http://git.bitcoin.ninja/index.cgi?a=blobdiff_plain;f=lightning%2Fsrc%2Fln%2Freload_tests.rs;h=76dd78ae2ad6a712560c5d6d6aa1a1c0b2e182e5;hb=ad91fcd510d7620df438a4cf59f7ed7b080adb19;hp=1ac290383a48761fa1e892bbc01fc73685aad983;hpb=db81c650baf2faa3a551bab216981ac84149176a;p=rust-lightning

diff --git a/lightning/src/ln/reload_tests.rs b/lightning/src/ln/reload_tests.rs
index 1ac29038..76dd78ae 100644
--- a/lightning/src/ln/reload_tests.rs
+++ b/lightning/src/ln/reload_tests.rs
@@ -16,7 +16,7 @@ use crate::sign::EntropySource;
 use crate::chain::transaction::OutPoint;
 use crate::events::{ClosureReason, Event, HTLCDestination, MessageSendEvent, MessageSendEventsProvider};
 use crate::ln::channelmanager::{ChannelManager, ChannelManagerReadArgs, PaymentId, RecipientOnionFields};
-use crate::ln::msgs;
+use crate::ln::{msgs, ChannelId};
 use crate::ln::msgs::{ChannelMessageHandler, RoutingMessageHandler, ErrorAction};
 use crate::util::test_channel_signer::TestChannelSigner;
 use crate::util::test_utils;
@@ -201,7 +201,7 @@ fn test_no_txn_manager_serialize_deserialize() {
 	nodes[1].node.peer_disconnected(&nodes[0].node.get_our_node_id());
 
 	let chan_0_monitor_serialized =
-		get_monitor!(nodes[0], OutPoint { txid: tx.txid(), index: 0 }.to_channel_id()).encode();
+		get_monitor!(nodes[0], ChannelId::v1_from_funding_outpoint(OutPoint { txid: tx.txid(), index: 0 })).encode();
 	reload_node!(nodes[0], nodes[0].node.encode(), &[&chan_0_monitor_serialized], persister, new_chain_monitor, nodes_0_deserialized);
 
 	nodes[0].node.peer_connected(&nodes[1].node.get_our_node_id(), &msgs::Init {
@@ -446,7 +446,8 @@ fn test_manager_serialize_deserialize_inconsistent_monitor() {
 	assert!(nodes_0_read.is_empty());
 
 	for monitor in node_0_monitors.drain(..) {
-		assert_eq!(nodes[0].chain_monitor.watch_channel(monitor.get_funding_txo().0, monitor),
+		let funding_outpoint = monitor.get_funding_txo().0;
+		assert_eq!(nodes[0].chain_monitor.watch_channel(funding_outpoint, monitor),
 			Ok(ChannelMonitorUpdateStatus::Completed));
 		check_added_monitors!(nodes[0], 1);
 	}
@@ -829,8 +830,8 @@ fn do_test_partial_claim_before_restart(persist_both_monitors: bool) {
 	// monitors and ChannelManager, for use later, if we don't want to persist both monitors.
 	let mut original_monitor = test_utils::TestVecWriter(Vec::new());
 	if !persist_both_monitors {
-		for outpoint in nodes[3].chain_monitor.chain_monitor.list_monitors() {
-			if outpoint.to_channel_id() == chan_id_not_persisted {
+		for (outpoint, channel_id) in nodes[3].chain_monitor.chain_monitor.list_monitors() {
+			if channel_id == chan_id_not_persisted {
 				assert!(original_monitor.0.is_empty());
 				nodes[3].chain_monitor.chain_monitor.get_monitor(outpoint).unwrap().write(&mut original_monitor).unwrap();
 			}
@@ -849,16 +850,16 @@ fn do_test_partial_claim_before_restart(persist_both_monitors: bool) {
 	// crashed in between the two persistence calls - using one old ChannelMonitor and one new one,
 	// with the old ChannelManager.
 	let mut updated_monitor = test_utils::TestVecWriter(Vec::new());
-	for outpoint in nodes[3].chain_monitor.chain_monitor.list_monitors() {
-		if outpoint.to_channel_id() == chan_id_persisted {
+	for (outpoint, channel_id) in nodes[3].chain_monitor.chain_monitor.list_monitors() {
+		if channel_id == chan_id_persisted {
 			assert!(updated_monitor.0.is_empty());
 			nodes[3].chain_monitor.chain_monitor.get_monitor(outpoint).unwrap().write(&mut updated_monitor).unwrap();
 		}
 	}
 	// If `persist_both_monitors` is set, get the second monitor here as well
 	if persist_both_monitors {
-		for outpoint in nodes[3].chain_monitor.chain_monitor.list_monitors() {
-			if outpoint.to_channel_id() == chan_id_not_persisted {
+		for (outpoint, channel_id) in nodes[3].chain_monitor.chain_monitor.list_monitors() {
+			if channel_id == chan_id_not_persisted {
 				assert!(original_monitor.0.is_empty());
 				nodes[3].chain_monitor.chain_monitor.get_monitor(outpoint).unwrap().write(&mut original_monitor).unwrap();
 			}
@@ -1220,7 +1221,7 @@ fn test_reload_partial_funding_batch() {
 	assert_eq!(nodes[0].tx_broadcaster.txn_broadcast().len(), 0);
 
 	// Reload the node while a subset of the channels in the funding batch have persisted monitors.
-	let channel_id_1 = OutPoint { txid: tx.txid(), index: 0 }.to_channel_id();
+	let channel_id_1 = ChannelId::v1_from_funding_outpoint(OutPoint { txid: tx.txid(), index: 0 });
 	let node_encoded = nodes[0].node.encode();
 	let channel_monitor_1_serialized = get_monitor!(nodes[0], channel_id_1).encode();
 	reload_node!(nodes[0], node_encoded, &[&channel_monitor_1_serialized], new_persister, new_chain_monitor, new_channel_manager);