test: exclude ord in pe module for coverage

This is useless to cover
vthib · Feb 2, 2023 · 9258bb5 · 9258bb5
1 parent 863bb35
commit 9258bb5
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -127,9 +127,7 @@ jobs:
       CARGO_TERM_COLOR: always
     steps:
       - uses: actions/checkout@v3
-
-      - name: Install Rust
-        run: rustup update stable
+      - uses: dtolnay/rust-toolchain@nightly
 
       - name: Install cargo-llvm-cov
         uses: taiki-e/install-action@cargo-llvm-cov

diff --git a/boreal/src/lib.rs b/boreal/src/lib.rs
@@ -63,6 +63,7 @@
 #![allow(unsafe_code)]
 #![deny(missing_docs)]
 #![deny(clippy::cargo)]
+#![cfg_attr(coverage_nightly, feature(no_coverage))]
 
 // Used in integration tests, not in the library.
 // This is to remove the "unused_crate_dependencies" warning, maybe a better solution

diff --git a/boreal/src/module/pe/ord.rs b/boreal/src/module/pe/ord.rs
@@ -19,6 +19,7 @@ pub(super) fn ord_lookup(dll_name: &[u8], ord: u16) -> Vec<u8> {
     format!("ord{ord}").into_bytes()
 }
 
+#[cfg_attr(coverage_nightly, no_coverage)]
 fn wsock32_ord_lookup(ord: u16) -> Option<&'static [u8]> {
     match ord {
         1 => Some(b"accept"),
@@ -142,6 +143,7 @@ fn wsock32_ord_lookup(ord: u16) -> Option<&'static [u8]> {
     }
 }
 
+#[cfg_attr(coverage_nightly, no_coverage)]
 fn oleaut32_ord_lookup(ord: u16) -> Option<&'static [u8]> {
     match ord {
         2 => Some(b"SysAllocString"),