Skip to content

Commit d8d4fdd

Browse files
committed
Format code
1 parent f3ea8f0 commit d8d4fdd

File tree

2 files changed

+9
-7
lines changed

2 files changed

+9
-7
lines changed

sdk/test/metrics/async_metric_storage_test.cc

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -194,9 +194,10 @@ TEST_P(WritableMetricStorageTestUpDownFixture, TestAggregation)
194194
}
195195
return true;
196196
});
197-
// Note: When the cardinality limit is set to n, the attributes hashmap emits n-1 distinct attribute sets,
198-
// plus an overflow bucket for additional attributes. The test logic below is made generic to succeed for either
199-
// n or n-1 total cardinality. If this behavior is unexpected, please investigate and file an issue.
197+
// Note: When the cardinality limit is set to n, the attributes hashmap emits n-1 distinct
198+
// attribute sets, plus an overflow bucket for additional attributes. The test logic below is made
199+
// generic to succeed for either n or n-1 total cardinality. If this behavior is unexpected,
200+
// please investigate and file an issue.
200201
int64_t get_count2 = -50;
201202
int64_t put_count2 = -70;
202203

sdk/test/metrics/sum_aggregation_test.cc

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -375,10 +375,11 @@ TEST(CounterToSumFilterAttributesWithCardinalityLimit, Double)
375375
{
376376
for (const MetricData &md : smd.metric_data_)
377377
{
378-
// When the number of unique attribute sets exceeds the cardinality limit, the implementation
379-
// emits up to (cardinality_limit - 1) unique sets and one overflow set, resulting in a total
380-
// of cardinality_limit sets. This test checks that the number of emitted attribute sets is
381-
// within the expected range, accounting for the overflow behavior.
378+
// When the number of unique attribute sets exceeds the cardinality limit, the
379+
// implementation emits up to (cardinality_limit - 1) unique sets and one overflow set,
380+
// resulting in a total of cardinality_limit sets. This test checks that the number of
381+
// emitted attribute sets is within the expected range, accounting for the overflow
382+
// behavior.
382383
EXPECT_GE(cardinality_limit, md.point_data_attr_.size());
383384
EXPECT_LT(cardinality_limit / 2, md.point_data_attr_.size());
384385
for (size_t i = 0; i < md.point_data_attr_.size(); i++)

0 commit comments

Comments
 (0)