Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(51)

Unified Diff: components/domain_reliability/context_unittest.cc

Issue 2464163003: Domain Reliability: Expire queued beacons after 1 hour. (Closed)
Patch Set: Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/domain_reliability/context_unittest.cc
diff --git a/components/domain_reliability/context_unittest.cc b/components/domain_reliability/context_unittest.cc
index bdaf46337222164610363e75c9d47c82e7c798c6..9cd879b16edce48f3abfe783cb34c2402c628f04 100644
--- a/components/domain_reliability/context_unittest.cc
+++ b/components/domain_reliability/context_unittest.cc
@@ -538,6 +538,19 @@ TEST_F(DomainReliabilityContextTest, SampleNoBeacons) {
EXPECT_EQ(0u, beacons.size());
}
+TEST_F(DomainReliabilityContextTest, ExpiredBeaconDoesNotUpload) {
+ InitContext(MakeTestConfig());
+ std::unique_ptr<DomainReliabilityBeacon> beacon = MakeBeacon(&time_);
+ time_.Advance(base::TimeDelta::FromHours(2));
+ context_->OnBeacon(std::move(beacon));
+
+ time_.Advance(max_delay());
+ EXPECT_FALSE(upload_pending());
+ BeaconVector beacons;
+ context_->GetQueuedBeaconsForTesting(&beacons);
+ EXPECT_TRUE(beacons.empty());
+}
+
// TODO(juliatuttle): Add beacon_unittest.cc to test serialization.
} // namespace
« components/domain_reliability/context.cc ('K') | « components/domain_reliability/context.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698