Skip to content

Commit 8b136ec

Browse files
committed
Keep the size-limited VEX test out of coverage
The regression test runs the CLI under RLIMIT_FSIZE = 512 bytes. In the coverage job that limit also truncated the child's exit-time .profraw, and llvm-profdata refused to merge the corrupt profile, so the coverage job failed. The limited run now writes its profile to /dev/null, which the limit does not cover. Assisted-by: Claude Code:claude-opus-5-5
1 parent a0032a3 commit 8b136ec

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

‎crates/socket-patch-cli/tests/e2e_vex.rs‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1587,6 +1587,10 @@ fn failed_output_write_leaves_no_partial_document() {
15871587
"pkg:npm/app@1.0.0",
15881588
]);
15891589
if let Some(limit) = limit {
1590+
// The limit also caps a coverage build's exit-time `.profraw`
1591+
// write, and a truncated profile fails the coverage merge: send
1592+
// this run's profile to a device, which the limit does not cover.
1593+
cmd.env("LLVM_PROFILE_FILE", "/dev/null");
15901594
// SAFETY: only async-signal-safe libc calls between fork and exec.
15911595
unsafe {
15921596
cmd.pre_exec(move || {

0 commit comments

Comments
 (0)