Skip to content

Commit a86724d

Browse files
committed
Fix tests
1 parent 31ce18d commit a86724d

File tree

1 file changed

+4
-18
lines changed

1 file changed

+4
-18
lines changed

cloud/storage/core/libs/diagnostics/request_counters_ut.cpp

Lines changed: 4 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -766,7 +766,7 @@ Y_UNIT_TEST_SUITE(TRequestCountersTest)
766766
usGroup,
767767
TRequestUsTimeBuckets::MakeNames(),
768768
"000",
769-
options & EHistogramCounterOption::UseUsUnitsForTimeHistogram);
769+
!(options & EHistogramCounterOption::UseMsUnitsForTimeHistogram));
770770
validateCounters(
771771
msGroup,
772772
TRequestMsTimeBuckets::MakeNames(),
@@ -783,14 +783,7 @@ Y_UNIT_TEST_SUITE(TRequestCountersTest)
783783
Y_UNIT_TEST(ShouldReportCompoundTimeHistogram_UseUsUnitsForTimeHistogram)
784784
{
785785
ShouldReportCompoundTimeHistogramWithMultipleCounters(
786-
EHistogramCounterOption::UseUsUnitsForTimeHistogram);
787-
}
788-
789-
Y_UNIT_TEST(ShouldReportCompoundTimeHistogram_AllUnits)
790-
{
791-
ShouldReportCompoundTimeHistogramWithMultipleCounters(
792-
EHistogramCounterOption::UseUsUnitsForTimeHistogram |
793-
EHistogramCounterOption::UseMsUnitsForTimeHistogram);
786+
EHistogramCounterOptions());
794787
}
795788

796789
void ShouldReportCompoundTimeHistogramWithSingleCounter(
@@ -857,7 +850,7 @@ Y_UNIT_TEST_SUITE(TRequestCountersTest)
857850

858851
validateCounters(
859852
usGroup,
860-
options & EHistogramCounterOption::UseUsUnitsForTimeHistogram);
853+
!(options & EHistogramCounterOption::UseMsUnitsForTimeHistogram));
861854
validateCounters(
862855
msGroup,
863856
options & EHistogramCounterOption::UseMsUnitsForTimeHistogram);
@@ -874,14 +867,7 @@ Y_UNIT_TEST_SUITE(TRequestCountersTest)
874867
ShouldReportCompoundTimeHistogramWithSingleCounter_UseUsUnitsForTimeHistogram)
875868
{
876869
ShouldReportCompoundTimeHistogramWithSingleCounter(
877-
EHistogramCounterOption::UseUsUnitsForTimeHistogram);
878-
}
879-
880-
Y_UNIT_TEST(ShouldReportCompoundTimeHistogramWithSingleCounter_AllUnits)
881-
{
882-
ShouldReportCompoundTimeHistogramWithSingleCounter(
883-
EHistogramCounterOption::UseUsUnitsForTimeHistogram |
884-
EHistogramCounterOption::UseMsUnitsForTimeHistogram);
870+
EHistogramCounterOptions());
885871
}
886872

887873
Y_UNIT_TEST(ShouldReportHistogramAsMultipleSensors)

0 commit comments

Comments
 (0)