fix SiStripLatency when empty

Author: mmusich

CondFormats/SiStripObjects/src/SiStripLatency.cc

@@ -77,6 +77,9 @@ std::pair<uint16_t, uint16_t> SiStripLatency::latencyAndMode(const uint32_t detI
 }
 
 uint16_t SiStripLatency::singleLatency() const {
+  if (latencies_.empty()) {
+    return 255;
+  }
   if (latencies_.size() == 1) {
     return latencies_[0].latency;
   }
@@ -94,6 +97,9 @@ uint16_t SiStripLatency::singleLatency() const {
 }
 
 uint16_t SiStripLatency::singleMode() const {
+  if (latencies_.empty()) {
+    return 0;
+  }
   if (latencies_.size() == 1) {
     return latencies_[0].mode;
   }

CondFormats/SiStripObjects/test/BuildFile.xml

@@ -9,6 +9,10 @@
 <bin file="testSerializationSiStripObjects.cpp">
 </bin>
 
-<bin file="test_catch2_*.cpp" name="test_catch2_SiStripBadStrip_Phase2">
+<bin file="test_catch2_main.cpp,test_catch2_SiStripBadStripForPhase2.cpp" name="test_catch2_SiStripBadStrip_Phase2">
+  <use name="catch2"/>
+</bin>
+
+<bin file="test_catch2_main.cpp,test_catch2_SiStripLatency.cpp" name="test_catch2_SiStripLatency">
   <use name="catch2"/>
 </bin>

CondFormats/SiStripObjects/test/test_catch2_SiStripLatency.cpp

@@ -0,0 +1,38 @@
+#include <sstream>
+#include "catch.hpp"
+#include <iostream>
+#include <iomanip>  // std::setw
+
+// Include the headers for SiStripLatency and TrackerTopology
+#include "CondFormats/SiStripObjects/interface/SiStripLatency.h"
+#include "DataFormats/TrackerCommon/interface/TrackerTopology.h"
+#include "FWCore/Utilities/interface/Exception.h"
+
+// Test case to check the SiStripLatency::printSummary and printDebug
+TEST_CASE("SiStripLatency basic test", "[SiStripLatency]") {
+  // Step 1: Create an empty SiStripLatency object
+  SiStripLatency latency;
+
+  // Step 2: Create a mock or dummy TrackerTopology object
+  TrackerTopology* trackerTopo = nullptr;  // Assuming null for now (replace with actual initialization if needed)
+
+  // Step 3: Create a stringstream to capture the output
+  std::stringstream ssSummary;
+  std::stringstream ssDebug;
+
+  // Step 4: Call printSummary and printDebug on the SiStripLatency object
+  try {
+    latency.printSummary(ssSummary, trackerTopo);
+    latency.printDebug(ssDebug, trackerTopo);
+  } catch (const cms::Exception& e) {
+    FAIL("Exception caught during printSummary or printDebug: " << e.what());
+  }
+
+  // Step 5: Optional - Check the output
+  REQUIRE(!ssSummary.str().empty());  // Ensure the summary output is not empty
+  REQUIRE(!ssDebug.str().empty());    // Ensure the debug output is not empty
+
+  // Print outputs for manual inspection
+  std::cout << "Summary Output:\n" << ssSummary.str() << std::endl;
+  std::cout << "Debug Output:\n" << ssDebug.str() << std::endl;
+}