Make code coverage more accurate

Author: EpicEric

.github/workflows/validate.yml

@@ -51,7 +51,7 @@ jobs:
           rustup override set ${RUST_MSRV}-x86_64-unknown-linux-gnu
           cargo build --locked
   tests:
-    name: Run tests on MSRV
+    name: Run tests on nightly
     runs-on: ubuntu-latest
     env:
       CARGO_LLVM_COV_VERSION: 0.6.16
@@ -67,8 +67,8 @@ jobs:
           tar -xvzf /tmp/cargo-llvm-cov.tar.gz -C ${HOME}/.cargo/bin
       - name: Test
         run: |
-          rustup toolchain install ${RUST_MSRV}-x86_64-unknown-linux-gnu
-          rustup override set ${RUST_MSRV}-x86_64-unknown-linux-gnu
+          rustup toolchain install nightly-x86_64-unknown-linux-gnu
+          rustup override set nightly-x86_64-unknown-linux-gnu
           cargo llvm-cov --verbose --codecov --locked --output-path codecov.json
       - name: Upload coverage reports to Codecov
         uses: codecov/codecov-action@v5

Cargo.toml

@@ -68,3 +68,6 @@ regex = "1.11.1"
 tokio-tungstenite = "0.26.2"
 tower = "0.5.2"
 vt100-ctt = "0.16.0"
+
+[lints.rust]
+unexpected_cfgs = { level = "warn", check-cfg = ['cfg(coverage_nightly)'] }

src/addressing.rs

@@ -363,6 +363,7 @@ impl<R: Resolver> AddressDelegator<R> {
 }
 
 #[cfg(test)]
+#[cfg_attr(coverage_nightly, coverage(off))]
 mod address_delegator_tests {
     use std::{collections::HashSet, net::SocketAddr};
 

src/certificates.rs

@@ -227,6 +227,7 @@ impl ResolvesServerCert for CertificateResolver {
 }
 
 #[cfg(test)]
+#[cfg_attr(coverage_nightly, coverage(off))]
 mod certificate_resolver_tests {
     use std::sync::{Arc, RwLock};
 

src/config.rs

@@ -326,6 +326,7 @@ fn validate_txt_record_prefix(value: &str) -> Result<String, String> {
 }
 
 #[cfg(test)]
+#[cfg_attr(coverage_nightly, coverage(off))]
 mod application_config_tests {
     use std::str::FromStr;
 

src/connections.rs

@@ -264,6 +264,7 @@ impl ConnectionGetByHttpHost<Arc<SshTunnelHandler>> for Arc<HttpAliasingConnecti
     }
 }
 
+#[cfg_attr(coverage_nightly, coverage(off))]
 #[cfg(test)]
 mod connection_map_tests {
     use std::sync::Arc;

src/fingerprints.rs

@@ -241,6 +241,7 @@ impl FingerprintsValidator {
 }
 
 #[cfg(test)]
+#[cfg_attr(coverage_nightly, coverage(off))]
 mod fingerprints_validator_tests {
     use russh::keys::{parse_public_key_base64, HashAlg};
 

src/http.rs

@@ -388,6 +388,7 @@ where
 }
 
 #[cfg(test)]
+#[cfg_attr(coverage_nightly, coverage(off))]
 mod proxy_handler_tests {
     use bytes::Bytes;
     use futures_util::{SinkExt, StreamExt};

src/ip.rs

@@ -66,6 +66,7 @@ impl IpFilter {
 }
 
 #[cfg(test)]
+#[cfg_attr(coverage_nightly, coverage(off))]
 mod ip_filter_tests {
     use std::{net::IpAddr, str::FromStr};
 

src/lib.rs

@@ -1,3 +1,5 @@
+#![cfg_attr(coverage_nightly, feature(coverage_attribute))]
+
 //!
 #![doc = include_str!("../README.md")]
 //!

src/login.rs

@@ -195,6 +195,7 @@ impl<C: Configurer> ApiLogin<C> {
 }
 
 #[cfg(test)]
+#[cfg_attr(coverage_nightly, coverage(off))]
 mod api_login_tests {
     use std::{
         net::{IpAddr, Ipv4Addr, SocketAddr},

src/quota.rs

@@ -129,6 +129,7 @@ impl QuotaHandler for Arc<QuotaMap> {
 }
 
 #[cfg(test)]
+#[cfg_attr(coverage_nightly, coverage(off))]
 mod quota_map_tests {
     use std::sync::Arc;
 

src/telemetry.rs

@@ -88,6 +88,7 @@ impl Counter {
 }
 
 #[cfg(test)]
+#[cfg_attr(coverage_nightly, coverage(off))]
 mod counter_tests {
     use std::{thread::sleep, time::Duration};
 
@@ -245,6 +246,7 @@ impl Telemetry {
 }
 
 #[cfg(test)]
+#[cfg_attr(coverage_nightly, coverage(off))]
 mod telemetry_tests {
     use std::sync::Arc;