Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix coverage check #224

Closed
wants to merge 7 commits into from
Closed

Fix coverage check #224

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
8075aea
bump dependency version
wulfraem Jun 28, 2024
ecac961
fix clippy warning
wulfraem Jun 28, 2024
3f7e402
fix clippy warnings
wulfraem Jun 28, 2024
df04ad7
add changelog entry for previous PR
wulfraem Aug 14, 2024
f10f8e8
update coverage flag to be set as stable option
wulfraem Aug 14, 2024
dfab1d5
Merge branch 'iotaledger:dev' into chore/fix-coverage-check
wulfraem Aug 14, 2024
4b251ef
add updated dependencies to change note
wulfraem Aug 14, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .changes/bump-dependency-version-to-fix-vulnerability.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"iota-crypto": patch
---

Bump dependencies to fix vulnerability from `cargo audit` (`curve25519-dalek`: 3.2 -> 4.1.3, `x25519-dalek`: 1.1 -> 2.0.1, `age`: 0.9 -> 0.10).
4 changes: 2 additions & 2 deletions .github/workflows/scripts/coverage.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ mkdir coverage

# Run tests with profiling instrumentation
echo "Running instrumented unit tests..."
RUSTFLAGS="-Zinstrument-coverage" LLVM_PROFILE_FILE="crypto-rs-%m.profraw" cargo +nightly test --tests --all --all-features
RUSTFLAGS="-Cinstrument-coverage" LLVM_PROFILE_FILE="crypto-rs-%m.profraw" cargo +nightly test --tests --all --all-features

# Merge all .profraw files into "crypto-rs.profdata"
echo "Merging coverage data..."
Expand All @@ -19,7 +19,7 @@ BINARIES=""

for file in \
$( \
RUSTFLAGS="-Zinstrument-coverage" \
RUSTFLAGS="-Cinstrument-coverage" \
cargo +nightly test --tests --all --all-features --no-run --message-format=json \
| jq -r "select(.profile.test == true) | .filenames[]" \
| grep -v dSYM - \
Expand Down
Loading