[hist] Test ComputeLinearizedIndex exactly on the edges

Author: hahnjo

hist/histv7/test/hist_regular.cxx

@@ -64,6 +64,16 @@ TEST(RRegularAxis, ComputeLinearizedIndex)
       EXPECT_FALSE(linIndex.fValid);
    }
 
+   // Exactly the lower end of the axis interval
+   {
+      auto linIndex = axis.ComputeLinearizedIndex(0);
+      EXPECT_EQ(linIndex.fIndex, 0);
+      EXPECT_TRUE(linIndex.fValid);
+      linIndex = axisNoFlowBins.ComputeLinearizedIndex(0);
+      EXPECT_EQ(linIndex.fIndex, 0);
+      EXPECT_TRUE(linIndex.fValid);
+   }
+
    for (std::size_t i = 0; i < Bins; i++) {
       auto linIndex = axis.ComputeLinearizedIndex(i + 0.5);
       EXPECT_EQ(linIndex.fIndex, i);
@@ -73,6 +83,16 @@ TEST(RRegularAxis, ComputeLinearizedIndex)
       EXPECT_TRUE(linIndex.fValid);
    }
 
+   // Exactly the upper end of the axis interval
+   {
+      auto linIndex = axis.ComputeLinearizedIndex(Bins);
+      EXPECT_EQ(linIndex.fIndex, Bins + 1);
+      EXPECT_TRUE(linIndex.fValid);
+      linIndex = axisNoFlowBins.ComputeLinearizedIndex(Bins);
+      EXPECT_EQ(linIndex.fIndex, Bins + 1);
+      EXPECT_FALSE(linIndex.fValid);
+   }
+
    // Overflow
    static constexpr double PositiveInfinity = std::numeric_limits<double>::infinity();
    static constexpr double NaN = std::numeric_limits<double>::quiet_NaN();

hist/histv7/test/hist_variable.cxx

@@ -95,6 +95,26 @@ TEST(RVariableBinAxis, ComputeLinearizedIndex)
       EXPECT_TRUE(linIndex.fValid);
    }
 
+   // Exactly on the bin edges
+   for (std::size_t i = 0; i < Bins; i++) {
+      auto linIndex = axis.ComputeLinearizedIndex(i);
+      EXPECT_EQ(linIndex.fIndex, i);
+      EXPECT_TRUE(linIndex.fValid);
+      linIndex = axisNoFlowBins.ComputeLinearizedIndex(i);
+      EXPECT_EQ(linIndex.fIndex, i);
+      EXPECT_TRUE(linIndex.fValid);
+   }
+
+   // Exactly the upper end of the axis interval
+   {
+      auto linIndex = axis.ComputeLinearizedIndex(Bins);
+      EXPECT_EQ(linIndex.fIndex, Bins + 1);
+      EXPECT_TRUE(linIndex.fValid);
+      linIndex = axisNoFlowBins.ComputeLinearizedIndex(Bins);
+      EXPECT_EQ(linIndex.fIndex, Bins + 1);
+      EXPECT_FALSE(linIndex.fValid);
+   }
+
    // Overflow
    static constexpr double PositiveInfinity = std::numeric_limits<double>::infinity();
    static constexpr double NaN = std::numeric_limits<double>::quiet_NaN();