Merge rust-bitcoin/rust-bitcoin#1359: Add a test parsing transaction with a huge witness

d6ca7e4b9f Add a test parsing transaction with a huge witness (Martin Habovstiak)

Pull request description:

  This transaction broke past versions of `rust-bitcoin` and LND so this adds a test to avoid reintroducing the problem in the future.

  See also https://github.com/romanz/electrs/issues/783

  I'm publishing this immediately for research purposes. I can clean it up later if required (low on time rn) or we may even not merge it if there is a better test.

ACKs for top commit:
  tcharding:
    ACK d6ca7e4b9f

Tree-SHA512: cfa9f5c82be0885a82bddb0c15f3177e05feedb369a931e58bf48d90b26eab85bc501d84d17927fa72b42ad4f1016e6042ad318662403271c738eaa91cee7748

bitcoin/src/blockdata/transaction.rs

@@ -1343,6 +1343,12 @@ mod tests {
         }
     }
 
+    #[test]
+    fn test_huge_witness() {
+        use crate::hashes::hex::FromHex;
+        deserialize::<Transaction>(&Vec::from_hex(include_str!("../../tests/data/huge_witness.hex").trim()).unwrap()).unwrap();
+    }
+
     #[test]
     #[cfg(feature="bitcoinconsensus")]
     fn test_transaction_verify () {