Merge #420: Add support for running integration tests via cargo test

139324a Remove integration test code (sanket1729)
90b5f10 Remove warnings (sanket1729)
967b95e Add support for running integration tests via cargo test (sanket1729)

Pull request description:

  There are still some annoying warnings present for unused functions when they are used.
  rust-lang/rust#46379

  Later commits will remove the hacky integration test setup.

  Running `cargo test` now should also run integration tests

  Our testing infrastructure now takes a long time to build because we have more than 100 dependencies (transitive). But all of these are dev dependencies, so our library isn't getting bloated.

  Fixes #361

ACKs for top commit:
  apoelstra:
    ACK 139324a

Tree-SHA512: debf68fd0ac98cffa9c8c89964d6d1c45ee542fe17eb2fdab6295357efc06eb43a0412e6ed5d0fd7264a987989984eebb0c5b3bbf7169ecb780d31dce887cb0b

Author: sanket1729

.github/workflows/rust.yml

@@ -69,21 +69,4 @@ jobs:
       - name: Running cargo
         env:
           DO_FEATURE_MATRIX: true
-        run: ./contrib/test.sh
-
-  IntTests:
-    name: Integration tests
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkout Crate
-        uses: actions/checkout@v2
-      - name: Checkout Toolchain
-        uses: actions-rs/toolchain@v1
-        with:
-          profile: minimal
-          toolchain: stable
-          override: true
-      - name: Running cargo
-        env:
-          BITCOINVERSION: '22.0'
-        run: ./contrib/test.sh
+        run: ./contrib/test.sh

Cargo.toml

@@ -22,6 +22,11 @@ rand = ["bitcoin/rand"]
 bitcoin = "0.28.0"
 serde = { version = "1.0", optional = true}
 
+[dev-dependencies]
+bitcoind = {version = "0.26.1", features=["22_0"]}
+actual-rand = { package = "rand", version = "0.8.4"}
+bitcoin = { version = "0.28", features = ["rand"]}
+
 [[example]]
 name = "htlc"
 required-features = ["compiler"]

contrib/test.sh

@@ -66,17 +66,4 @@ if [ "$DO_DOCS" = true ]; then
     RUSTDOCFLAGS="--cfg docsrs" cargo doc --all --features="$FEATURES"
 fi
 
-# Run Integration tests if told so
-if [ -n "$BITCOINVERSION" ]; then
-    set -e
-    cd integration_test
-    curl https://bitcoincore.org/bin/bitcoin-core-$BITCOINVERSION/bitcoin-$BITCOINVERSION-x86_64-linux-gnu.tar.gz | tar xvzf - bitcoin-$BITCOINVERSION/bin/bitcoind    # will abort if the check fails.
-    sha256sum --check bitcoin-core-$BITCOINVERSION.sha256sum
-    export PATH=$PATH:$(pwd)/bitcoin-$BITCOINVERSION/bin
-    ./run.sh
-    # Cleanups
-    rm -rf bitcoin-$BITCOINVERSION
-    exit 0
-fi
-
 exit 0

integration_test/Cargo.toml

@@ -0,0 +1,29 @@
+extern crate miniscript;
+
+use bitcoind::bitcoincore_rpc::RpcApi;
+use bitcoind::BitcoinD;
+use miniscript::bitcoin;
+
+pub mod test_util;
+
+// Launch an instance of bitcoind with
+pub fn setup() -> BitcoinD {
+    let exe_path = bitcoind::exe_path().unwrap();
+    let bitcoind = bitcoind::BitcoinD::new(exe_path).unwrap();
+    let cl = &bitcoind.client;
+    // generate to an address by the wallet. And wait for funds to mature
+    let addr = cl.get_new_address(None, None).unwrap();
+    let blks = cl.generate_to_address(101, &addr).unwrap();
+    assert_eq!(blks.len(), 101);
+
+    assert_eq!(
+        cl.get_balance(Some(1) /*min conf*/, None).unwrap(),
+        bitcoin::Amount::from_sat(100_000_000 * 50)
+    );
+    bitcoind
+}
+
+#[test]
+fn test_setup() {
+    setup();
+}