Use shared workflow for running cargo-deny

.github/workflows/nightly.yaml

@@ -36,16 +36,7 @@ jobs:
     uses: eclipse-uprotocol/ci-cd/.github/workflows/rust-verify-latest-deps.yaml@main
 
   deny:
-    runs-on: ubuntu-latest
-    steps:
-    - uses: actions/checkout@v4
-      with:
-        submodules: "recursive"
-    - uses: EmbarkStudios/cargo-deny-action@v2
-      with: 
-        rust-version: ${{ env.RUST_TOOLCHAIN }}
-        command: check
-        arguments: --all-features
+    uses: eclipse-uprotocol/ci-cd/.github/workflows/rust-deny-check.yaml@main
 
   # [impl->req~up-language-ci-test~1]
   test-all-features:

Cargo.toml

@@ -95,3 +95,8 @@ required-features = ["communication", "util"]
 [[example]]
 name = "simple_rpc"
 required-features = ["communication", "util"]
+
+[lints.rust]
+# this prevents cargo from complaining about code blocks
+# excluded from tarpaulin coverage checks
+unexpected_cfgs = { level = "warn", check-cfg = ['cfg(tarpaulin_include)'] }

src/communication/notification.rs

@@ -30,6 +30,7 @@ pub enum NotificationError {
     NotifyError(UStatus),
 }
 
+#[cfg(not(tarpaulin_include))]
 impl Display for NotificationError {
     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
         match self {

src/communication/pubsub.rs

@@ -31,6 +31,7 @@ pub enum PubSubError {
     PublishError(UStatus),
 }
 
+#[cfg(not(tarpaulin_include))]
 impl Display for PubSubError {
     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
         match self {

src/communication/rpc.rs

@@ -291,6 +291,7 @@ pub trait RpcServer {
     ) -> Result<(), RegistrationError>;
 }
 
+#[cfg(not(tarpaulin_include))]
 #[cfg(any(test, feature = "test-util"))]
 mockall::mock! {
     /// This extra struct is necessary in order to comply with mockall's requirements regarding the parameter lifetimes
@@ -301,6 +302,7 @@ mockall::mock! {
     }
 }
 
+#[cfg(not(tarpaulin_include))]
 #[cfg(any(test, feature = "test-util"))]
 #[async_trait]
 /// This delegates the invocation of the UTransport functions to the mocked functions of the Transport struct.

src/utransport.rs

@@ -271,6 +271,7 @@ pub trait UTransport: Send + Sync {
     }
 }
 
+#[cfg(not(tarpaulin_include))]
 #[cfg(any(test, feature = "test-util"))]
 mockall::mock! {
     /// This extra struct is necessary in order to comply with mockall's requirements regarding the parameter lifetimes
@@ -282,6 +283,7 @@ mockall::mock! {
     }
 }
 
+#[cfg(not(tarpaulin_include))]
 #[cfg(any(test, feature = "test-util"))]
 #[async_trait]
 /// This delegates the invocation of the UTransport functions to the mocked functions of the Transport struct.