Address feedback

Author: ThomsonTan

sdk/test/metrics/sum_aggregation_test.cc

@@ -194,11 +194,11 @@ TEST(HistogramToSumFilterAttributesWithCardinaityLimit, Double)
       {
         for (const MetricData &md : smd.metric_data_)
         {
-          // Something weird about attributes hashmap. If cardinality is setup to n, it emits n-1
-          // including overflow. Just making the logic generic here to succeed for n or n-1 total
-          // cardinality.
-          EXPECT_GE(cardinality_limit, md.point_data_attr_.size());
-          EXPECT_LT(cardinality_limit / 2, md.point_data_attr_.size());
+          // When the number of unique attribute sets exceeds the cardinality limit, the
+          // implementation emits up to (cardinality_limit - 1) unique sets and one overflow set,
+          // resulting in a total of cardinality_limit sets. This test checks that the number of
+          // emitted attribute sets is within the expected range, accounting for the overflow behavior.
+          EXPECT_EQ(cardinality_limit, md.point_data_attr_.size());
           for (size_t i = 0; i < md.point_data_attr_.size(); i++)
           {
             EXPECT_EQ(1, md.point_data_attr_[i].attributes.size());