clippy says: fn new should usually return self <-- I picked ignore, since this is test code waiting to be moved. And likely also renamed from "new" to maybe "new_mock_block" to make sure nobody finds it and wants to use it to build a non-test block

core/src/core/block.rs

@@ -418,6 +418,7 @@ impl Block {
 	/// TODO - Move this somewhere where only tests will use it.
 	/// *** Only used in tests. ***
 	///
+	#[warn(clippy::new_ret_no_self)]
 	pub fn new(
 		prev: &BlockHeader,
 		txs: Vec<Transaction>,