From ba05982d3a0a7f3c502b06d4a97c7dd655a77a92 Mon Sep 17 00:00:00 2001 From: Tom Tan Date: Fri, 26 Sep 2025 00:03:21 -0700 Subject: [PATCH] [TEST] Remove workaround for metrics cardinality limit test (#3663) --- CHANGELOG.md | 3 +++ sdk/test/metrics/sum_aggregation_test.cc | 9 ++------- 2 files changed, 5 insertions(+), 7 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index b1341a52ab..fa87cbc131 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -15,6 +15,9 @@ Increment the: ## [Unreleased] +* [TEST] Remove workaround for metrics cardinality limit test + [#3663](https://github.com/open-telemetry/opentelemetry-cpp/pull/3663) + ## [1.23 2025-09-25] * [CodeHealth] Fix clang-tidy warnings part 6 diff --git a/sdk/test/metrics/sum_aggregation_test.cc b/sdk/test/metrics/sum_aggregation_test.cc index 02421a9e26..5c29fe43e8 100644 --- a/sdk/test/metrics/sum_aggregation_test.cc +++ b/sdk/test/metrics/sum_aggregation_test.cc @@ -194,11 +194,7 @@ TEST(HistogramToSumFilterAttributesWithCardinalityLimit, 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()); + 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()); @@ -380,8 +376,7 @@ TEST(CounterToSumFilterAttributesWithCardinalityLimit, Double) // 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_GE(cardinality_limit, md.point_data_attr_.size()); - EXPECT_LT(cardinality_limit / 2, md.point_data_attr_.size()); + 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());