[AF] Fix StrikeDatabase test

I forgot to set expiry_time_micros_, but it passed the unittest
because I also forgot to call RemoveExpiredStrikes() in the unittest.

Bug: 1061639
Change-Id: I80570e03202c22cca069cb2084351849a4979484
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2594313Reviewed-by: Jared Saul <jsaul@google.com>
Commit-Queue: Anne Lim <annelim@google.com>
Cr-Commit-Position: refs/heads/master@{#837419}

components/autofill/core/browser/payments/strike_database_integrator_base.h

@@ -62,6 +62,8 @@ class StrikeDatabaseIntegratorBase {
  private:
   FRIEND_TEST_ALL_PREFIXES(ChromeBrowsingDataRemoverDelegateTest,
                            StrikeDatabaseEmptyOnAutofillRemoveEverything);
+  FRIEND_TEST_ALL_PREFIXES(StrikeDatabaseIntegratorTestStrikeDatabaseTest,
+                           NonExpiringStrikesDoNotExpire);
   FRIEND_TEST_ALL_PREFIXES(StrikeDatabaseIntegratorTestStrikeDatabaseTest,
                            RemoveExpiredStrikesTest);
   FRIEND_TEST_ALL_PREFIXES(StrikeDatabaseIntegratorTestStrikeDatabaseTest,

components/autofill/core/browser/payments/strike_database_integrator_test_strike_database.cc

@@ -15,7 +15,9 @@ StrikeDatabaseIntegratorTestStrikeDatabase::
     StrikeDatabaseIntegratorTestStrikeDatabase(
         StrikeDatabase* strike_database,
         base::Optional<int64_t> expiry_time_micros)
-    : StrikeDatabaseIntegratorTestStrikeDatabase(strike_database) {}
+    : StrikeDatabaseIntegratorTestStrikeDatabase(strike_database) {
+  expiry_time_micros_ = expiry_time_micros;
+}
 
 StrikeDatabaseIntegratorTestStrikeDatabase::
     StrikeDatabaseIntegratorTestStrikeDatabase(StrikeDatabase* strike_database)

components/autofill/core/browser/payments/strike_database_integrator_test_strike_database_unittest.cc

@@ -127,6 +127,8 @@ TEST_F(StrikeDatabaseIntegratorTestStrikeDatabaseTest,
   // Advance clock very far into the future.
   test_clock.Advance(base::TimeDelta::FromDays(INT_MAX));
 
+  no_expiry_strike_database_->RemoveExpiredStrikes();
+
   // Strike should not be removed.
   EXPECT_EQ(1, no_expiry_strike_database_->GetStrikes());
 }