summaryrefslogtreecommitdiff
path: root/chromium/components/safe_browsing/content/browser/triggers/trigger_throttler_unittest.cc
blob: b760b99f7777ca588cea8b20a888306b31469721 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
// Copyright (c) 2017 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

#include "components/safe_browsing/content/browser/triggers/trigger_throttler.h"

#include "base/strings/stringprintf.h"
#include "base/test/scoped_feature_list.h"
#include "base/test/simple_test_clock.h"
#include "components/prefs/testing_pref_service.h"
#include "components/safe_browsing/core/common/features.h"
#include "components/safe_browsing/core/common/safe_browsing_prefs.h"
#include "testing/gmock/include/gmock/gmock.h"
#include "testing/gtest/include/gtest/gtest.h"

using testing::ElementsAre;

namespace safe_browsing {

class TriggerThrottlerTest : public ::testing::Test {
 public:
  TriggerThrottlerTest() : trigger_throttler_(nullptr) {}

  void SetUp() override {
    safe_browsing::RegisterLocalStatePrefs(pref_service_.registry());
    trigger_throttler_.ResetPrefsForTesting(&pref_service_);
  }

  void SetQuotaForTriggerType(TriggerType trigger_type, size_t max_quota) {
    SetQuotaForTriggerType(&trigger_throttler_, trigger_type, max_quota);
  }

  void SetQuotaForTriggerType(TriggerThrottler* throttler,
                              TriggerType trigger_type,
                              size_t max_quota) {
    throttler->trigger_type_and_quota_list_.push_back(
        std::make_pair(trigger_type, max_quota));
  }

  TriggerThrottler* throttler() { return &trigger_throttler_; }

  void SetTestClock(base::Clock* clock) {
    trigger_throttler_.SetClockForTesting(clock);
  }

  std::vector<base::Time> GetEventTimestampsForTriggerType(
      TriggerType trigger_type) {
    return trigger_throttler_.trigger_events_[trigger_type];
  }

  PrefService* get_pref_service() { return &pref_service_; }

 private:
  TestingPrefServiceSimple pref_service_;
  TriggerThrottler trigger_throttler_;
};

TEST_F(TriggerThrottlerTest, SecurityInterstitialsHaveUnlimitedQuota) {
  // Make sure that security interstitials never run out of quota.
  for (int i = 0; i < 1000; ++i) {
    throttler()->TriggerFired(TriggerType::SECURITY_INTERSTITIAL);
    EXPECT_TRUE(
        throttler()->TriggerCanFire(TriggerType::SECURITY_INTERSTITIAL));
  }
}

TEST_F(TriggerThrottlerTest, SecurityInterstitialQuotaCanNotBeOverwritten) {
  // Make sure that security interstitials never run out of quota, even if we
  // try to configure quota for this trigger type.
  SetQuotaForTriggerType(TriggerType::SECURITY_INTERSTITIAL, 3);
  for (int i = 0; i < 1000; ++i) {
    throttler()->TriggerFired(TriggerType::SECURITY_INTERSTITIAL);
    EXPECT_TRUE(
        throttler()->TriggerCanFire(TriggerType::SECURITY_INTERSTITIAL));
  }
}

TEST_F(TriggerThrottlerTest, TriggerQuotaSetToOne) {
  // This is a corner case where we can exceed array bounds for triggers that
  // have quota set to 1 report per day. This can happen when quota is 1 and
  // exactly one event has fired. When deciding whether another event can fire,
  // we look at the Nth-from-last event to check if it was recent or not - in
  // this scenario, Nth-from-last is 1st-from-last (because quota is 1). An
  // off-by-one error in this calculation can cause us to look at position 1
  // instead of position 0 in the even list.
  SetQuotaForTriggerType(TriggerType::AD_SAMPLE, 1);

  // Fire the trigger, first event will be allowed.
  EXPECT_TRUE(throttler()->TriggerCanFire(TriggerType::AD_SAMPLE));
  throttler()->TriggerFired(TriggerType::AD_SAMPLE);

  // Ensure that checking whether this trigger can fire again does not cause
  // an error and also returns the expected result.
  EXPECT_FALSE(throttler()->TriggerCanFire(TriggerType::AD_SAMPLE));
}

TEST_F(TriggerThrottlerTest, TriggerExceedsQuota) {
  // Ensure that a trigger can't fire more than its quota allows.
  SetQuotaForTriggerType(TriggerType::AD_SAMPLE, 2);

  // First two triggers should work
  EXPECT_TRUE(throttler()->TriggerCanFire(TriggerType::AD_SAMPLE));
  throttler()->TriggerFired(TriggerType::AD_SAMPLE);
  EXPECT_TRUE(throttler()->TriggerCanFire(TriggerType::AD_SAMPLE));
  throttler()->TriggerFired(TriggerType::AD_SAMPLE);

  // Third attempt will fail since we're out of quota.
  EXPECT_FALSE(throttler()->TriggerCanFire(TriggerType::AD_SAMPLE));
}

TEST_F(TriggerThrottlerTest, TriggerQuotaResetsAfterOneDay) {
  // Ensure that trigger events older than a day are cleaned up and triggers can
  // resume firing.

  // We initialize the test clock to several days ago and fire some events to
  // use up quota. We then advance the clock by a day and ensure quota is
  // available again.
  base::SimpleTestClock test_clock;
  test_clock.SetNow(base::Time::Now() - base::Days(10));
  base::Time base_ts = test_clock.Now();

  SetTestClock(&test_clock);
  SetQuotaForTriggerType(TriggerType::AD_SAMPLE, 2);

  // First two triggers should work
  EXPECT_TRUE(throttler()->TriggerCanFire(TriggerType::AD_SAMPLE));
  throttler()->TriggerFired(TriggerType::AD_SAMPLE);
  EXPECT_TRUE(throttler()->TriggerCanFire(TriggerType::AD_SAMPLE));
  throttler()->TriggerFired(TriggerType::AD_SAMPLE);

  // Third attempt will fail since we're out of quota.
  EXPECT_FALSE(throttler()->TriggerCanFire(TriggerType::AD_SAMPLE));

  // Also confirm that the throttler contains two event timestamps for the above
  // two events - since we use a test clock, it doesn't move unless we tell it
  // to.
  EXPECT_THAT(GetEventTimestampsForTriggerType(TriggerType::AD_SAMPLE),
              ElementsAre(base_ts, base_ts));

  // Move the clock forward by 1 day (and a bit) and try the trigger again,
  // quota should be available now.
  test_clock.Advance(base::Days(1) + base::Seconds(1));
  base::Time advanced_ts = test_clock.Now();
  EXPECT_TRUE(throttler()->TriggerCanFire(TriggerType::AD_SAMPLE));

  // The previous time stamps should remain in the throttler.
  EXPECT_THAT(GetEventTimestampsForTriggerType(TriggerType::AD_SAMPLE),
              ElementsAre(base_ts, base_ts));

  // Firing the trigger will clean up the expired timestamps and insert the new
  // timestamp.
  throttler()->TriggerFired(TriggerType::AD_SAMPLE);
  EXPECT_THAT(GetEventTimestampsForTriggerType(TriggerType::AD_SAMPLE),
              ElementsAre(advanced_ts));
}

TEST_F(TriggerThrottlerTest, TriggerQuotaPersistence) {
  // Test that trigger quota is persisted in prefs when triggers fire, and
  // retrieved from prefs on startup.

  // Set some low quotas for two triggers
  SetQuotaForTriggerType(TriggerType::AD_SAMPLE, 3);
  SetQuotaForTriggerType(TriggerType::SUSPICIOUS_SITE, 3);

  // Ensure each trigger can fire.
  EXPECT_TRUE(throttler()->TriggerCanFire(TriggerType::AD_SAMPLE));
  EXPECT_TRUE(throttler()->TriggerCanFire(TriggerType::SUSPICIOUS_SITE));

  // Fire each trigger twice to store some events.
  throttler()->TriggerFired(TriggerType::AD_SAMPLE);
  throttler()->TriggerFired(TriggerType::AD_SAMPLE);
  throttler()->TriggerFired(TriggerType::AD_SAMPLE);
  throttler()->TriggerFired(TriggerType::SUSPICIOUS_SITE);
  throttler()->TriggerFired(TriggerType::SUSPICIOUS_SITE);

  // The AD_SAMPLE trigger is now out of quota, while SUSPICIOUS_SITE can still
  // fire one more time.
  EXPECT_FALSE(throttler()->TriggerCanFire(TriggerType::AD_SAMPLE));
  EXPECT_TRUE(throttler()->TriggerCanFire(TriggerType::SUSPICIOUS_SITE));

  // Check the pref directly, it should reflect the events for each trigger.
  PrefService* prefs = get_pref_service();
  const base::Value* event_dict =
      prefs->GetDictionary(prefs::kSafeBrowsingTriggerEventTimestamps);

  const std::string kAdSampleKey = "2";
  const base::Value* ad_sample_events = event_dict->FindKey(kAdSampleKey);
  EXPECT_EQ(3u, ad_sample_events->GetListDeprecated().size());

  const std::string kSuspiciousSiteKey = "4";
  const base::Value* suspicious_site_events =
      event_dict->FindKey(kSuspiciousSiteKey);
  EXPECT_EQ(2u, suspicious_site_events->GetListDeprecated().size());

  // To simulate a new startup of the browser, we can create another throttler
  // using the same quota configuration and pref store. It should read the
  // events from prefs and and reflect the same status for each trigger.
  TriggerThrottler throttler2(prefs);
  SetQuotaForTriggerType(&throttler2, TriggerType::AD_SAMPLE, 3);
  SetQuotaForTriggerType(&throttler2, TriggerType::SUSPICIOUS_SITE, 3);
  EXPECT_FALSE(throttler2.TriggerCanFire(TriggerType::AD_SAMPLE));
  EXPECT_TRUE(throttler2.TriggerCanFire(TriggerType::SUSPICIOUS_SITE));
}

class TriggerThrottlerTestFinch : public ::testing::Test {
 public:
  void SetupQuotaParams(const TriggerType trigger_type,
                        const std::string& group_name,
                        int quota,
                        base::test::ScopedFeatureList* scoped_feature_list) {
    const base::Feature* feature = nullptr;
    std::string param_name = "";
    GetFeatureAndParamForTrigger(trigger_type, &feature, &param_name);

    base::FieldTrialParams feature_params;
    feature_params[param_name] =
        GetQuotaParamValueForTrigger(trigger_type, quota);
    scoped_feature_list->InitAndEnableFeatureWithParameters(*feature,
                                                            feature_params);
  }

  size_t GetDailyQuotaForTrigger(const TriggerThrottler& throttler,
                                 const TriggerType trigger_type) {
    return throttler.GetDailyQuotaForTrigger(trigger_type);
  }

 private:
  void GetFeatureAndParamForTrigger(const TriggerType trigger_type,
                                    const base::Feature** out_feature,
                                    std::string* out_param) {
    switch (trigger_type) {
      case TriggerType::AD_SAMPLE:
        *out_feature = &safe_browsing::kTriggerThrottlerDailyQuotaFeature;
        *out_param = safe_browsing::kTriggerTypeAndQuotaParam;
        break;

      case TriggerType::SUSPICIOUS_SITE:
        *out_feature = &safe_browsing::kSuspiciousSiteTriggerQuotaFeature;
        *out_param = safe_browsing::kSuspiciousSiteTriggerQuotaParam;
        break;

      default:
        NOTREACHED() << "Unhandled trigger type: "
                     << static_cast<int>(trigger_type);
    }
  }

  std::string GetQuotaParamValueForTrigger(const TriggerType trigger_type,
                                           int quota) {
    if (trigger_type == TriggerType::AD_SAMPLE)
      return base::StringPrintf("%d,%d", trigger_type, quota);
    else
      return base::StringPrintf("%d", quota);
  }
};

TEST_F(TriggerThrottlerTestFinch, ConfigureQuotaViaFinch) {
  base::test::ScopedFeatureList scoped_feature_list;
  SetupQuotaParams(TriggerType::AD_SAMPLE, "Group_ConfigureQuotaViaFinch", 3,
                   &scoped_feature_list);
  // Make sure that setting the quota param via Finch params works as expected.

  // The throttler has been configured (above) to allow ad samples to fire three
  // times per day.
  TriggerThrottler throttler(nullptr);

  // First three triggers should work
  EXPECT_TRUE(throttler.TriggerCanFire(TriggerType::AD_SAMPLE));
  throttler.TriggerFired(TriggerType::AD_SAMPLE);
  EXPECT_TRUE(throttler.TriggerCanFire(TriggerType::AD_SAMPLE));
  throttler.TriggerFired(TriggerType::AD_SAMPLE);
  EXPECT_TRUE(throttler.TriggerCanFire(TriggerType::AD_SAMPLE));
  throttler.TriggerFired(TriggerType::AD_SAMPLE);

  // Fourth attempt will fail since we're out of quota.
  EXPECT_FALSE(throttler.TriggerCanFire(TriggerType::AD_SAMPLE));
}

TEST_F(TriggerThrottlerTestFinch, AdSamplerDefaultQuota) {
  // Make sure that the ad sampler gets its own default quota when no finch
  // config exists, but the quota can be overwritten through Finch.
  TriggerThrottler throttler_default(nullptr);
  EXPECT_EQ(kAdSamplerTriggerDefaultQuota,
            GetDailyQuotaForTrigger(throttler_default, TriggerType::AD_SAMPLE));
  EXPECT_TRUE(throttler_default.TriggerCanFire(TriggerType::AD_SAMPLE));

  base::test::ScopedFeatureList scoped_feature_list;
  SetupQuotaParams(TriggerType::AD_SAMPLE, "Group_AdSamplerDefaultQuota", 4,
                   &scoped_feature_list);
  TriggerThrottler throttler_finch(nullptr);
  EXPECT_EQ(4u,
            GetDailyQuotaForTrigger(throttler_finch, TriggerType::AD_SAMPLE));
}

TEST_F(TriggerThrottlerTestFinch, SuspiciousSiteTriggerDefaultQuota) {
  // Ensure that suspicious site trigger is enabled with default quota.
  TriggerThrottler throttler_default(nullptr);
  EXPECT_EQ(
      kSuspiciousSiteTriggerDefaultQuota,
      GetDailyQuotaForTrigger(throttler_default, TriggerType::SUSPICIOUS_SITE));
  EXPECT_TRUE(throttler_default.TriggerCanFire(TriggerType::SUSPICIOUS_SITE));

  base::test::ScopedFeatureList scoped_feature_list;
  SetupQuotaParams(TriggerType::SUSPICIOUS_SITE,
                   "Group_SuspiciousSiteTriggerDefaultQuota", 7,
                   &scoped_feature_list);
  TriggerThrottler throttler_finch(nullptr);
  EXPECT_EQ(7u, GetDailyQuotaForTrigger(throttler_finch,
                                        TriggerType::SUSPICIOUS_SITE));
}

}  // namespace safe_browsing