ci: enforce Rust contract coverage - #595
Conversation
|
@Mojisola001-tech is attempting to deploy a commit to the Collins' projects Team on Vercel. A member of the Team first needs to authorize it. |
|
@Mojisola001-tech Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
| working-directory: packages/contracts | ||
| run: >- | ||
| cargo llvm-cov | ||
| --package meridian-vault |
There was a problem hiding this comment.
adapter-common is missing from this --package list, so its coverage is never measured, confirmed directly in this run's own output: the printed table only has rows for blend-adapter/src/lib.rs, defindex-adapter/src/lib.rs, and vault/src/lib.rs, no adapter-common row at all. That crate has zero tests of its own (no #[cfg(test)] module) and still has two .unwrap()-based panics in require_vault_auth/get_usdc. #533's own motivation was closing exactly this kind of untested-panic gap in the code that custodies funds, so leaving out the one crate that's actually fully untested undercuts the point of adding the gate. Add --package adapter-common to the list.
Overview
Adds measured Rust contract coverage to Meridian's existing contracts CI job.
Related Issue
Closes #533
Changes
Contract coverage gate
cargo-llvm-covandllvm-tools-previewin the contracts workflow.meridian-vault,meridian-blend-adapter, andmeridian-defindex-adapter.Verification Results