wasmtime / PR #13209 Add a `cargo test` coverage script · git-wasmtime

Stream: git-wasmtime

Topic: wasmtime / PR #13209 Add a `cargo test` coverage script

Wasmtime GitHub notifications bot (Apr 27 2026 at 16:21):

fitzgen opened PR #13209 from fitzgen:coverage-script to bytecodealliance:main:

Runs cargo test ... with LLVM instrumentation-based coverage enabled, merges the resulting .profraw files, and produces and HTML coverage report at report/index.html.

Wasmtime GitHub notifications bot (Apr 27 2026 at 16:21):

fitzgen requested alexcrichton for a review on PR #13209.

Wasmtime GitHub notifications bot (Apr 27 2026 at 16:21):

fitzgen requested wasmtime-default-reviewers for a review on PR #13209.

Wasmtime GitHub notifications bot (Apr 27 2026 at 16:25):

:thumbs_up: alexcrichton submitted PR review.

Wasmtime GitHub notifications bot (Apr 27 2026 at 16:25):

alexcrichton has enabled auto merge for PR #13209.

Wasmtime GitHub notifications bot (Apr 27 2026 at 16:35):

alexcrichton added PR #13209 Add a cargo test coverage script to the merge queue.

Wasmtime GitHub notifications bot (Apr 27 2026 at 17:01):

github-merge-queue[bot] removed PR #13209 Add a cargo test coverage script from the merge queue.

Wasmtime GitHub notifications bot (Apr 27 2026 at 17:12):

fitzgen updated PR #13209.

Wasmtime GitHub notifications bot (Apr 27 2026 at 17:12):

fitzgen has enabled auto merge for PR #13209.

Wasmtime GitHub notifications bot (Apr 27 2026 at 17:26):

fitzgen added PR #13209 Add a cargo test coverage script to the merge queue.

Wasmtime GitHub notifications bot (Apr 27 2026 at 17:38):

fitzgen commented on PR #13209:

Hrm, getting errors about wasm32-unkown-unknown missing in CI, but we also add that in the run step just before this one

https://github.com/bytecodealliance/wasmtime/actions/runs/25009689583/job/73241911350#step:10:737

https://github.com/bytecodealliance/wasmtime/actions/runs/25009689583/job/73241911350#step:8:1

Wasmtime GitHub notifications bot (Apr 27 2026 at 17:44):

alexcrichton commented on PR #13209:

I think that's due to the script hardcoding +nightly which isn't in-use on this builder (it's some oss-fuzz nightly snapshot). Could the +nightly be removed? (I realize that'd probably remove the ./the/script.rs aspect but that seems like a minor loss)

Wasmtime GitHub notifications bot (Apr 27 2026 at 17:51):

github-merge-queue[bot] removed PR #13209 Add a cargo test coverage script from the merge queue.

Wasmtime GitHub notifications bot (Apr 27 2026 at 18:03):

fitzgen updated PR #13209.

Wasmtime GitHub notifications bot (Apr 27 2026 at 18:04):

fitzgen commented on PR #13209:

I think that's due to the script hardcoding +nightly which isn't in-use on this builder (it's some oss-fuzz nightly snapshot). Could the +nightly be removed? (I realize that'd probably remove the ./the/script.rs aspect but that seems like a minor loss)

Ah good catch. I just removed +nightly but left the script stuff. Non-nightly-default folks can just compile and run separately.