ci: delete *test.sh files
These are not run in CI since #2353 and are likely to go out of date. If we want a script that users can run locally then we should create a new script that wraps our current CI.
This commit is contained in:
Andrew Poelstra
parent
162094322f
commit
e386cbfadf
|
|
@ -6,8 +6,10 @@ test:
|
||||||
- any-glob-to-any-file: fuzz/**
|
- any-glob-to-any-file: fuzz/**
|
||||||
- any-glob-to-any-file: '*/tests/**'
|
- any-glob-to-any-file: '*/tests/**'
|
||||||
- any-glob-to-any-file: 'dep_test'
|
- any-glob-to-any-file: 'dep_test'
|
||||||
- any-glob-to-any-file: 'contrib/test.sh'
|
- any-glob-to-any-file: 'contrib/run_task.sh'
|
||||||
- any-glob-to-any-file: '*/contrib/test.sh'
|
- any-glob-to-any-file: 'contrib/test_vars.sh'
|
||||||
|
- any-glob-to-any-file: '*/contrib/extra_tests.sh'
|
||||||
|
- any-glob-to-any-file: '*/contrib/test_vars.sh'
|
||||||
doc:
|
doc:
|
||||||
- changed-files:
|
- changed-files:
|
||||||
- any-glob-to-any-file: '**/*.md'
|
- any-glob-to-any-file: '**/*.md'
|
||||||
|
|
|
||||||
|
|
@ -124,19 +124,6 @@ NB: reviewers may run more complex test/CI scripts, thus, satisfying all the
|
||||||
requirements above is just a preliminary, but not necessary sufficient step for
|
requirements above is just a preliminary, but not necessary sufficient step for
|
||||||
getting the PR accepted as a valid candidate PR for the `master` branch.
|
getting the PR accepted as a valid candidate PR for the `master` branch.
|
||||||
|
|
||||||
PR authors may also find it useful to run the following script locally in order
|
|
||||||
to check that each of the commits within the PR satisfies the requirements
|
|
||||||
above, before submitting the PR to review:
|
|
||||||
```shell script
|
|
||||||
RUSTUP_TOOLCHAIN=1.41.1 ./contrib/test.sh
|
|
||||||
```
|
|
||||||
Please replace the value in `RUSTUP_TOOLCHAIN=1.41.1` with the current MSRV from
|
|
||||||
[README.md].
|
|
||||||
|
|
||||||
NB: Please keep in mind that the script above replaces `Cargo.lock` file, which
|
|
||||||
is necessary to support current MSRV, incompatible with `stable` and newer cargo
|
|
||||||
versions.
|
|
||||||
|
|
||||||
### Peer review
|
### Peer review
|
||||||
|
|
||||||
Anyone may participate in peer review which is expressed by comments in the pull
|
Anyone may participate in peer review which is expressed by comments in the pull
|
||||||
|
|
|
||||||
|
|
@ -1,131 +0,0 @@
|
||||||
#!/usr/bin/env bash
|
|
||||||
|
|
||||||
set -ex
|
|
||||||
|
|
||||||
FEATURES="std rand-std rand serde secp-recovery bitcoinconsensus-std base64 bitcoinconsensus"
|
|
||||||
|
|
||||||
cargo --version
|
|
||||||
rustc --version
|
|
||||||
|
|
||||||
# Some tests require certain toolchain types.
|
|
||||||
NIGHTLY=false
|
|
||||||
STABLE=true
|
|
||||||
if cargo --version | grep nightly; then
|
|
||||||
STABLE=false
|
|
||||||
NIGHTLY=true
|
|
||||||
fi
|
|
||||||
if cargo --version | grep beta; then
|
|
||||||
STABLE=false
|
|
||||||
fi
|
|
||||||
|
|
||||||
# Make all cargo invocations verbose
|
|
||||||
export CARGO_TERM_VERBOSE=true
|
|
||||||
|
|
||||||
# Defaults / sanity checks
|
|
||||||
cargo build
|
|
||||||
cargo test
|
|
||||||
|
|
||||||
if [ "$DO_LINT" = true ]
|
|
||||||
then
|
|
||||||
cargo clippy --locked --all-features --all-targets -- -D warnings
|
|
||||||
cargo clippy --locked --example bip32 -- -D warnings
|
|
||||||
cargo clippy --locked --example handshake --features=rand-std -- -D warnings
|
|
||||||
cargo clippy --locked --example ecdsa-psbt --features=bitcoinconsensus -- -D warnings
|
|
||||||
cargo clippy --locked --example sign-tx-segwit-v0 --features=rand-std -- -D warnings
|
|
||||||
cargo clippy --locked --example sign-tx-taproot --features=rand-std -- -D warnings
|
|
||||||
cargo clippy --locked --example taproot-psbt --features=rand-std,bitcoinconsensus -- -D warnings
|
|
||||||
|
|
||||||
# We should not have any duplicate dependencies. This catches mistakes made upgrading dependencies
|
|
||||||
# in one crate and not in another (e.g. upgrade bitcoin_hashes in bitcoin but not in secp).
|
|
||||||
duplicate_dependencies=$(
|
|
||||||
# Only show the actual duplicated deps, not their reverse tree, then
|
|
||||||
# whitelist the 'syn' crate which is duplicated but it's not our fault.
|
|
||||||
#
|
|
||||||
# Whitelist `bitcoin_hashes` while we release it and until secp v0.28.0 comes out.
|
|
||||||
cargo tree --target=all --all-features --duplicates \
|
|
||||||
| grep '^[0-9A-Za-z]' \
|
|
||||||
| grep -v 'syn' \
|
|
||||||
| grep -v 'bitcoin_hashes' \
|
|
||||||
| wc -l
|
|
||||||
)
|
|
||||||
if [ "$duplicate_dependencies" -ne 0 ]; then
|
|
||||||
echo "Dependency tree is broken, contains duplicates"
|
|
||||||
cargo tree --target=all --all-features --duplicates
|
|
||||||
exit 1
|
|
||||||
fi
|
|
||||||
fi
|
|
||||||
|
|
||||||
if [ "$DO_FEATURE_MATRIX" = true ]; then
|
|
||||||
cargo build --locked --no-default-features
|
|
||||||
cargo test --locked --no-default-features
|
|
||||||
|
|
||||||
# All features
|
|
||||||
cargo build --locked --no-default-features --features="$FEATURES"
|
|
||||||
cargo test --locked --no-default-features --features="$FEATURES"
|
|
||||||
# Single features
|
|
||||||
for feature in ${FEATURES}
|
|
||||||
do
|
|
||||||
cargo build --locked --no-default-features --features="$feature"
|
|
||||||
cargo test --locked --no-default-features --features="$feature"
|
|
||||||
# All combos of two features
|
|
||||||
for featuretwo in ${FEATURES}; do
|
|
||||||
cargo build --locked --no-default-features --features="$feature $featuretwo"
|
|
||||||
cargo test --locked --no-default-features --features="$feature $featuretwo"
|
|
||||||
done
|
|
||||||
done
|
|
||||||
fi
|
|
||||||
|
|
||||||
cargo run --locked --example bip32 7934c09359b234e076b9fa5a1abfd38e3dc2a9939745b7cc3c22a48d831d14bd
|
|
||||||
cargo run --locked --no-default-features --example bip32 7934c09359b234e076b9fa5a1abfd38e3dc2a9939745b7cc3c22a48d831d14bd
|
|
||||||
|
|
||||||
cargo run --locked --example ecdsa-psbt --features=bitcoinconsensus
|
|
||||||
cargo run --locked --example sign-tx-segwit-v0 --features=rand-std -- -D warnings
|
|
||||||
cargo run --locked --example sign-tx-taproot --features=rand-std -- -D warnings
|
|
||||||
cargo run --locked --example taproot-psbt --features=rand-std,bitcoinconsensus
|
|
||||||
|
|
||||||
# Build the docs if told to (this only works with the nightly toolchain)
|
|
||||||
if [ "$DO_DOCSRS" = true ]; then
|
|
||||||
RUSTDOCFLAGS="--cfg docsrs -D warnings -D rustdoc::broken-intra-doc-links" cargo +nightly doc --all-features
|
|
||||||
fi
|
|
||||||
|
|
||||||
# Build the docs with a stable toolchain, in unison with the DO_DOCSRS command
|
|
||||||
# above this checks that we feature guarded docs imports correctly.
|
|
||||||
if [ "$DO_DOCS" = true ]; then
|
|
||||||
RUSTDOCFLAGS="-D warnings" cargo +stable doc --all-features
|
|
||||||
fi
|
|
||||||
|
|
||||||
# Run formatter if told to.
|
|
||||||
if [ "$DO_FMT" = true ]; then
|
|
||||||
if [ "$NIGHTLY" = false ]; then
|
|
||||||
echo "DO_FMT requires a nightly toolchain (consider using RUSTUP_TOOLCHAIN)"
|
|
||||||
exit 1
|
|
||||||
fi
|
|
||||||
rustup component add rustfmt
|
|
||||||
cargo fmt --check
|
|
||||||
fi
|
|
||||||
|
|
||||||
# Bench if told to, only works with non-stable toolchain (nightly, beta).
|
|
||||||
if [ "$DO_BENCH" = true ]
|
|
||||||
then
|
|
||||||
if [ "$STABLE" = true ]; then
|
|
||||||
if [ -n "$RUSTUP_TOOLCHAIN" ]; then
|
|
||||||
echo "RUSTUP_TOOLCHAIN is set to a stable toolchain but DO_BENCH requires a non-stable (beta, nightly) toolchain"
|
|
||||||
else
|
|
||||||
echo "DO_BENCH requires a non-stable (beta, nightly) toolchain"
|
|
||||||
fi
|
|
||||||
exit 1
|
|
||||||
fi
|
|
||||||
RUSTFLAGS='--cfg=bench' cargo bench
|
|
||||||
fi
|
|
||||||
|
|
||||||
# Use as dependency if told to
|
|
||||||
if [ "$AS_DEPENDENCY" = true ]
|
|
||||||
then
|
|
||||||
cargo new dep_test 2> /dev/null # Mute warning about workspace, fixed below.
|
|
||||||
cd dep_test
|
|
||||||
echo 'bitcoin = { path = "..", features = ["serde"] }\n\n' >> Cargo.toml
|
|
||||||
# Adding an empty workspace section excludes this crate from the rust-bitcoin workspace.
|
|
||||||
echo '[workspace]\n\n' >> Cargo.toml
|
|
||||||
|
|
||||||
cargo test --verbose
|
|
||||||
fi
|
|
||||||
|
|
@ -1,31 +0,0 @@
|
||||||
#!/usr/bin/env bash
|
|
||||||
|
|
||||||
set -ex
|
|
||||||
|
|
||||||
CRATES="bitcoin hashes units internals fuzz"
|
|
||||||
DEPS="recent minimal"
|
|
||||||
|
|
||||||
for dep in $DEPS
|
|
||||||
do
|
|
||||||
cp "Cargo-$dep.lock" Cargo.lock
|
|
||||||
for crate in ${CRATES}
|
|
||||||
do
|
|
||||||
(
|
|
||||||
cd "$crate"
|
|
||||||
./contrib/test.sh
|
|
||||||
)
|
|
||||||
done
|
|
||||||
if [ "$dep" = recent ];
|
|
||||||
then
|
|
||||||
# We always test committed dependencies but we want to warn if they could've been updated
|
|
||||||
cargo update
|
|
||||||
if diff Cargo-recent.lock Cargo.lock;
|
|
||||||
then
|
|
||||||
echo Dependencies are up to date
|
|
||||||
else
|
|
||||||
echo "::warning file=Cargo-recent.lock::Dependencies could be updated"
|
|
||||||
fi
|
|
||||||
fi
|
|
||||||
done
|
|
||||||
|
|
||||||
exit 0
|
|
||||||
|
|
@ -1,44 +0,0 @@
|
||||||
#!/usr/bin/env bash
|
|
||||||
|
|
||||||
set -ex
|
|
||||||
|
|
||||||
FEATURES=""
|
|
||||||
|
|
||||||
cargo --version
|
|
||||||
rustc --version
|
|
||||||
|
|
||||||
# Make all cargo invocations verbose
|
|
||||||
export CARGO_TERM_VERBOSE=true
|
|
||||||
|
|
||||||
# Pin dependencies as required if we are using MSRV toolchain.
|
|
||||||
if cargo --version | grep "1\.41"; then
|
|
||||||
# 1.0.157 uses syn 2.0 which requires edition 2021
|
|
||||||
cargo update -p serde --precise 1.0.156
|
|
||||||
# 1.0.108 uses `matches!` macro so does not work with Rust 1.41.1, bad `syn` no biscuit.
|
|
||||||
cargo update -p syn --precise 1.0.107
|
|
||||||
# Half 1.8 uses edition 2021 features
|
|
||||||
cargo update -p half --precise 1.7.1
|
|
||||||
fi
|
|
||||||
|
|
||||||
if [ "$DO_LINT" = true ]
|
|
||||||
then
|
|
||||||
cargo clippy --all-features --all-targets -- -D warnings
|
|
||||||
fi
|
|
||||||
|
|
||||||
# Defaults / sanity checks
|
|
||||||
cargo build
|
|
||||||
cargo test
|
|
||||||
|
|
||||||
# Address Sanitizer
|
|
||||||
if [ "$DO_ASAN" = true ]; then
|
|
||||||
cargo clean
|
|
||||||
CC='clang -fsanitize=address -fno-omit-frame-pointer' \
|
|
||||||
RUSTFLAGS='-Zsanitizer=address -Clinker=clang -Cforce-frame-pointers=yes' \
|
|
||||||
ASAN_OPTIONS='detect_leaks=1 detect_invalid_pointer_pairs=1 detect_stack_use_after_return=1' \
|
|
||||||
cargo test --lib --no-default-features --features="$FEATURES" -Zbuild-std --target x86_64-unknown-linux-gnu
|
|
||||||
cargo clean
|
|
||||||
CC='clang -fsanitize=memory -fno-omit-frame-pointer' \
|
|
||||||
RUSTFLAGS='-Zsanitizer=memory -Zsanitizer-memory-track-origins -Cforce-frame-pointers=yes' \
|
|
||||||
cargo test --lib --no-default-features --features="$FEATURES" -Zbuild-std --target x86_64-unknown-linux-gnu
|
|
||||||
fi
|
|
||||||
|
|
||||||
|
|
@ -1,64 +0,0 @@
|
||||||
#!/usr/bin/env bash
|
|
||||||
|
|
||||||
set -ex
|
|
||||||
|
|
||||||
FEATURES="std alloc"
|
|
||||||
|
|
||||||
cargo --version
|
|
||||||
rustc --version
|
|
||||||
|
|
||||||
# Work out if we are using a nightly toolchain.
|
|
||||||
NIGHTLY=false
|
|
||||||
if cargo --version | grep nightly >/dev/null; then
|
|
||||||
NIGHTLY=true
|
|
||||||
fi
|
|
||||||
|
|
||||||
# Make all cargo invocations verbose
|
|
||||||
export CARGO_TERM_VERBOSE=true
|
|
||||||
|
|
||||||
# Defaults / sanity checks
|
|
||||||
cargo --locked build
|
|
||||||
cargo --locked test
|
|
||||||
|
|
||||||
if [ "$DO_LINT" = true ]
|
|
||||||
then
|
|
||||||
cargo clippy --locked --all-features --all-targets -- -D warnings
|
|
||||||
fi
|
|
||||||
|
|
||||||
if [ "$DO_FEATURE_MATRIX" = true ]; then
|
|
||||||
# No features
|
|
||||||
cargo build --locked --no-default-features
|
|
||||||
cargo test --locked --no-default-features
|
|
||||||
|
|
||||||
# All features
|
|
||||||
cargo build --locked --no-default-features --features="$FEATURES"
|
|
||||||
cargo test --locked --no-default-features --features="$FEATURES"
|
|
||||||
|
|
||||||
# Single features
|
|
||||||
for feature in ${FEATURES}
|
|
||||||
do
|
|
||||||
cargo build --locked --no-default-features --features="$feature"
|
|
||||||
cargo test --locked --no-default-features --features="$feature"
|
|
||||||
done
|
|
||||||
fi
|
|
||||||
|
|
||||||
# Build the docs if told to (this only works with the nightly toolchain)
|
|
||||||
if [ "$DO_DOCSRS" = true ]; then
|
|
||||||
RUSTDOCFLAGS="--cfg docsrs -D warnings -D rustdoc::broken-intra-doc-links" cargo +nightly doc --all-features
|
|
||||||
fi
|
|
||||||
|
|
||||||
# Build the docs with a stable toolchain, in unison with the DO_DOCSRS command
|
|
||||||
# above this checks that we feature guarded docs imports correctly.
|
|
||||||
if [ "$DO_DOCS" = true ]; then
|
|
||||||
RUSTDOCFLAGS="-D warnings" cargo +stable doc --locked --all-features
|
|
||||||
fi
|
|
||||||
|
|
||||||
# Run formatter if told to.
|
|
||||||
if [ "$DO_FMT" = true ]; then
|
|
||||||
if [ "$NIGHTLY" = false ]; then
|
|
||||||
echo "DO_FMT requires a nightly toolchain (consider using RUSTUP_TOOLCHAIN)"
|
|
||||||
exit 1
|
|
||||||
fi
|
|
||||||
rustup component add rustfmt
|
|
||||||
cargo fmt --check
|
|
||||||
fi
|
|
||||||
Loading…
Reference in New Issue