Skip to content

Commit d37fc8c

Browse files
committed
CI: Pull bitcoind test to separate script
Done in preparation for using the new maintainer tools script. Pull the bitoind test out into a separate script because it is specific to this repo. No change in test coverage.
1 parent 47b76d3 commit d37fc8c

File tree

3 files changed

+13
-13
lines changed

3 files changed

+13
-13
lines changed

.github/workflows/rust.yml

+1-3
Original file line numberDiff line numberDiff line change
@@ -47,9 +47,7 @@ jobs:
4747
toolchain: stable
4848
override: true
4949
- name: Running integration tests
50-
env:
51-
DO_BITCOIND_TESTS: true
52-
run: ./contrib/test.sh
50+
run: ./contrib/integration_test.sh
5351

5452
Tests:
5553
name: Tests

contrib/integration_test.sh

+12
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
#!/usr/bin/env bash
2+
#
3+
# Run the integration test optionally downloading Bitcoin Core binary if BITCOINVERSION is set.
4+
5+
set -euo pipefail
6+
7+
REPO_DIR=$(git rev-parse --show-toplevel)
8+
9+
# Make all cargo invocations verbose.
10+
export CARGO_TERM_VERBOSE=true
11+
12+
BITCOIND_EXE="$REPO_DIR/bitcoind-tests/bin/bitcoind" cargo test --verbose

contrib/test.sh

-10
Original file line numberDiff line numberDiff line change
@@ -27,16 +27,6 @@ then
2727
cargo fmt -- --check
2828
fi
2929

30-
# Test bitcoind integration tests if told to (this only works with the stable toolchain)
31-
if [ "$DO_BITCOIND_TESTS" = true ]; then
32-
cd bitcoind-tests
33-
BITCOIND_EXE="$(git rev-parse --show-toplevel)/bitcoind-tests/bin/bitcoind" \
34-
cargo test --verbose
35-
36-
# Exit integration tests, do not run other tests.
37-
exit 0
38-
fi
39-
4030
# Defaults / sanity checks
4131
cargo test
4232

0 commit comments

Comments
 (0)