coverage false negative: NVRO artifacts and defensive guards

Author: frs69wq

include/dtlmod/ActorRegistry.hpp

@@ -46,8 +46,8 @@ class ActorRegistry {
 
   [[nodiscard]] bool contains(sg4::ActorPtr actor) const noexcept
   {
-    if (!actor)
-      return false; // Safe handling for noexcept
+    if (!actor)     // LCOV_EXCL_LINE
+      return false; // LCOV_EXCL_LINE
     return actors_.find(actor) != actors_.end();
   }
 

src/FileEngine.cpp

@@ -77,7 +77,7 @@ std::shared_ptr<FileTransport> FileEngine::get_file_transport() const
   if (!transport)                                                                                // LCOV_EXCL_LINE
     throw TransportEngineMismatchException(XBT_THROW_POINT, "Transport is not a FileTransport"); // LCOV_EXCL_LINE
   return transport;
-}
+} // LCOV_EXCL_LINE
 
 std::string FileEngine::get_path_to_dataset() const
 {

src/StagingEngine.cpp

@@ -39,7 +39,7 @@ std::shared_ptr<StagingTransport> StagingEngine::get_staging_transport() const
   if (!transport)                                                                                   // LCOV_EXCL_LINE
     throw TransportEngineMismatchException(XBT_THROW_POINT, "Transport is not a StagingTransport"); // LCOV_EXCL_LINE
   return transport;
-}
+} // LCOV_EXCL_LINE
 
 void StagingEngine::begin_pub_transaction()
 {

src/Stream.cpp

@@ -334,7 +334,7 @@ std::vector<std::string> Stream::get_all_variables() const
   for (const auto& [name, var] : variables_)
     variable_names.push_back(name);
   return variable_names;
-}
+} // LCOV_EXCL_LINE
 
 std::shared_ptr<Variable> Stream::inquire_variable(std::string_view name) const
 {