Skip to content

Commit

Permalink
Test nop rebuilds of entire workspace
Browse files Browse the repository at this point in the history
With an upcoming change we will add more conditional build logic to
build scripts that has the potential for screwing with rebuilds. To
preempt any screw ups from the start, make sure that our nop-rebuilds
job checks the entire workspace.

Signed-off-by: Daniel Müller <deso@posteo.net>
  • Loading branch information
d-e-s-o authored and danielocfb committed Jan 30, 2024
1 parent 47564c2 commit 07fdf1c
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -104,13 +104,15 @@ jobs:
- name: Install required tools
run: sudo apt-get install -y llvm-14
- name: Check incremental rebuilds
env:
CARGO_OPTS: --features=generate-unit-test-files --workspace --quiet --tests
run: |
cargo check --features=generate-unit-test-files --quiet --tests
cargo check ${CARGO_OPTS}
# We need another build here to have the reference `output` file
# present. As long as we converge eventually it's probably good
# enough...
cargo check --features=generate-unit-test-files --quiet --tests
output=$(CARGO_LOG=cargo::core::compiler::fingerprint=info cargo check --features=generate-unit-test-files --quiet --tests 2>&1)
cargo check ${CARGO_OPTS}
output=$(CARGO_LOG=cargo::core::compiler::fingerprint=info cargo check ${CARGO_OPTS} 2>&1)
[ -z "${output}" ] || (echo "!!!! cargo check --tests rebuild was not a no-op: ${output} !!!!" && false)
test-coverage:
name: Test and coverage
Expand Down

0 comments on commit 07fdf1c

Please sign in to comment.