Skip to content

Commit

Permalink
Fix SampleVector tests to expect CHECK rather than DCHECK deaths.
Browse files Browse the repository at this point in the history
Bug: 832728
Change-Id: I9a2d2d330aa4d250c925b7a614780acbf40dce3f
Reviewed-on: https://chromium-review.googlesource.com/1012732
Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
Commit-Queue: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551483}
  • Loading branch information
Wez authored and Commit Bot committed Apr 17, 2018
1 parent 256d8bc commit f0acd4f
Showing 1 changed file with 11 additions and 11 deletions.
22 changes: 11 additions & 11 deletions base/metrics/sample_vector_unittest.cc
Original file line number Diff line number Diff line change
Expand Up @@ -143,9 +143,9 @@ TEST_F(SampleVectorTest, BucketIndexDeath) {
EXPECT_EQ(3, samples.GetCount(65));

// Extreme case.
EXPECT_DCHECK_DEATH(samples.Accumulate(INT_MIN, 100));
EXPECT_DCHECK_DEATH(samples.Accumulate(-1, 100));
EXPECT_DCHECK_DEATH(samples.Accumulate(INT_MAX, 100));
EXPECT_DEATH_IF_SUPPORTED(samples.Accumulate(INT_MIN, 100), "");
EXPECT_DEATH_IF_SUPPORTED(samples.Accumulate(-1, 100), "");
EXPECT_DEATH_IF_SUPPORTED(samples.Accumulate(INT_MAX, 100), "");

// Custom buckets: [1, 5) [5, 10)
// Note, this is not a valid BucketRanges for Histogram because it does not
Expand All @@ -165,8 +165,8 @@ TEST_F(SampleVectorTest, BucketIndexDeath) {
EXPECT_EQ(4, samples2.GetCount(5));

// Extreme case.
EXPECT_DCHECK_DEATH(samples2.Accumulate(0, 100));
EXPECT_DCHECK_DEATH(samples2.Accumulate(10, 100));
EXPECT_DEATH_IF_SUPPORTED(samples2.Accumulate(0, 100), "");
EXPECT_DEATH_IF_SUPPORTED(samples2.Accumulate(10, 100), "");
}

TEST_F(SampleVectorTest, AddSubtractBucketNotMatchDeath) {
Expand All @@ -192,20 +192,20 @@ TEST_F(SampleVectorTest, AddSubtractBucketNotMatchDeath) {

// Extra bucket in the beginning.
samples2.Accumulate(0, 100);
EXPECT_DCHECK_DEATH(samples1.Add(samples2));
EXPECT_DCHECK_DEATH(samples1.Subtract(samples2));
EXPECT_DEATH_IF_SUPPORTED(samples1.Add(samples2), "");
EXPECT_DEATH_IF_SUPPORTED(samples1.Subtract(samples2), "");

// Extra bucket in the end.
samples2.Accumulate(0, -100);
samples2.Accumulate(6, 100);
EXPECT_DCHECK_DEATH(samples1.Add(samples2));
EXPECT_DCHECK_DEATH(samples1.Subtract(samples2));
EXPECT_DEATH_IF_SUPPORTED(samples1.Add(samples2), "");
EXPECT_DEATH_IF_SUPPORTED(samples1.Subtract(samples2), "");

// Bucket not match: [3, 5) VS [3, 6)
samples2.Accumulate(6, -100);
samples2.Accumulate(3, 100);
EXPECT_DCHECK_DEATH(samples1.Add(samples2));
EXPECT_DCHECK_DEATH(samples1.Subtract(samples2));
EXPECT_DEATH_IF_SUPPORTED(samples1.Add(samples2), "");
EXPECT_DEATH_IF_SUPPORTED(samples1.Subtract(samples2), "");
}

TEST_F(SampleVectorTest, Iterate) {
Expand Down

0 comments on commit f0acd4f

Please sign in to comment.