Always check next_route in TestRouter is fully consumed

TheBlueMatt · TheBlueMatt · commit 12fd0beff669 · 2023-02-07T19:49:43.000Z
...rather than only in std.
diff --git a/lightning/src/util/test_utils.rs b/lightning/src/util/test_utils.rs
@@ -113,11 +113,12 @@ impl<'a> Router for TestRouter<'a> {
 	fn notify_payment_probe_failed(&self, _path: &[&RouteHop], _short_channel_id: u64) {}
 }
 
-#[cfg(feature = "std")] // If we put this on the `if`, we get "attributes are not yet allowed on `if` expressions" on 1.41.1
 impl<'a> Drop for TestRouter<'a> {
 	fn drop(&mut self) {
-		if std::thread::panicking() {
-			return;
+		#[cfg(feature = "std")] {
+			if std::thread::panicking() {
+				return;
+			}
 		}
 		assert!(self.next_routes.lock().unwrap().is_empty());
 	}

Original file line number	Diff line number	Diff line change
`@@ -113,11 +113,12 @@ impl<'a> Router for TestRouter<'a> {`
`113`	`113`	`fn notify_payment_probe_failed(&self, _path: &[&RouteHop], _short_channel_id: u64) {}`
`114`	`114`	`}`
`115`	`115`
`116`		-#[cfg(feature = "std")] // If we put this on the `if`, we get "attributes are not yet allowed on `if` expressions" on 1.41.1
`117`	`116`	`impl<'a> Drop for TestRouter<'a> {`
`118`	`117`	`fn drop(&mut self) {`
`119`		`- if std::thread::panicking() {`
`120`		`- return;`
	`118`	`+ #[cfg(feature = "std")] {`
	`119`	`+ if std::thread::panicking() {`
	`120`	`+ return;`
	`121`	`+ }`
`121`	`122`	`}`
`122`	`123`	`assert!(self.next_routes.lock().unwrap().is_empty());`
`123`	`124`	`}`