07576d3ed0
It turns out that "cargo test --release" doesn't use `[profile.release]`; it uses `[profile.bench]` instead; see [here](https://doc.rust-lang.org/cargo/reference/manifest.html) for details. So the plan to run the tests in optimized mode but with debug-assertions enabled didn't actually work, and we had an actual failing unit test that was hidden because assertions were disabled. So, this makes test-all.sh just run the unit tests in debug mode, as is the norm for Rust packages, and fixes the buggy test. This also removes the `[profile.release]` override from the top-level Cargo.toml file too. We don't need it now that we're not running tests in release mode, and it had confused multiple people because it made Cretonne's in-tree builds different from how Cretonne is built when used as a dependency in other projects.
69 lines
1.7 KiB
Bash
Executable File
69 lines
1.7 KiB
Bash
Executable File
#!/bin/bash
|
|
set -euo pipefail
|
|
|
|
# This is the top-level test script:
|
|
#
|
|
# - Make a debug build.
|
|
# - Make a release build.
|
|
# - Run unit tests for all Rust crates (including the filetests)
|
|
# - Build API documentation.
|
|
#
|
|
# All tests run by this script should be passing at all times.
|
|
|
|
# Disable generation of .pyc files because they cause trouble for vendoring
|
|
# scripts, and this is a build step that isn't run very often anyway.
|
|
export PYTHONDONTWRITEBYTECODE=1
|
|
|
|
# Repository top-level directory.
|
|
cd $(dirname "$0")
|
|
topdir=$(pwd)
|
|
|
|
function banner() {
|
|
echo "====== $@ ======"
|
|
}
|
|
|
|
# Run rustfmt if we have it.
|
|
if $topdir/check-rustfmt.sh; then
|
|
banner "Rust formatting"
|
|
$topdir/format-all.sh --write-mode=diff
|
|
fi
|
|
|
|
# Check if any Python files have changed since we last checked them.
|
|
tsfile=$topdir/target/meta-checked
|
|
if [ -f $tsfile ]; then
|
|
needcheck=$(find $topdir/lib/codegen/meta -name '*.py' -newer $tsfile)
|
|
else
|
|
needcheck=yes
|
|
fi
|
|
if [ -n "$needcheck" ]; then
|
|
banner "$(python --version 2>&1), $(python3 --version 2>&1)"
|
|
$topdir/lib/codegen/meta/check.sh
|
|
touch $tsfile || echo no target directory
|
|
fi
|
|
|
|
# Make sure the code builds in release mode.
|
|
banner "Rust release build"
|
|
cargo build --release
|
|
|
|
# Make sure the code builds in debug mode.
|
|
banner "Rust debug build"
|
|
cargo build
|
|
|
|
# Run the tests. We run these in debug mode so that assertions are enabled.
|
|
banner "Rust unit tests"
|
|
cargo test --all
|
|
|
|
# Make sure the documentation builds.
|
|
banner "Rust documentation: $topdir/target/doc/cretonne/index.html"
|
|
cargo doc
|
|
|
|
# Run clippy if we have it.
|
|
banner "Rust linter"
|
|
if $topdir/check-clippy.sh; then
|
|
$topdir/clippy-all.sh --write-mode=diff
|
|
else
|
|
echo "\`cargo +nightly install clippy\` for optional rust linting"
|
|
fi
|
|
|
|
banner "OK"
|