Prompt for expiration date for local cards.

components/autofill/core/browser/credit_card_unittest.cc

 // Copyright 2013 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 <stddef.h>
 
 #include "base/guid.h"
 #include "base/macros.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/utf_string_conversions.h"
+#include "base/time/time.h"
 #include "build/build_config.h"
 #include "components/autofill/core/browser/autofill_test_utils.h"
 #include "components/autofill/core/browser/autofill_type.h"
 #include "components/autofill/core/browser/credit_card.h"
 #include "components/autofill/core/browser/validation.h"
 #include "components/autofill/core/common/form_field_data.h"
 #include "grit/components_scaled_resources.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using base::ASCIIToUTF16;
@@ skipping 704 matching lines @@
 TEST(CreditCardTest, CanBuildFromCardNumberAndExpirationDate) {
   base::string16 card_number = base::ASCIIToUTF16("test");
   int month = 1;
   int year = 3000;
   CreditCard card(card_number, month, year);
   EXPECT_EQ(card_number, card.number());
   EXPECT_EQ(month, card.expiration_month());
   EXPECT_EQ(year, card.expiration_year());
 }
 
+// A card that has expired last year should be updated.

[sebsg, 2016/04/27 14:59:02]

These tests seem to have a lot in common. Do you think you could make one test
and iterate through an array of test cases? Like this for example:
https://code.google.com/p/chromium/codesearch#chromium/src/components/autofil...
or
https://code.google.com/p/chromium/codesearch#chromium/src/components/autofil...

Thanks!

[please use gerrit instead, 2016/04/27 18:05:44]

Done. Sorry for the wall of tests. I attempted to make them as readable as
possible.

+TEST(CreditCardTest, CardExpiringLastYear) {
+  base::Time::Exploded today;
+  base::Time::Now().LocalExplode(&today);
+
+  // Local expired card.
+  CreditCard card(base::ASCIIToUTF16("1234"), 12, today.year - 1);
+  EXPECT_EQ(CreditCard::LOCAL_CARD, card.record_type());
+
+  EXPECT_TRUE(card.ShouldUpdateExpiration());
+
+  // Masked expired server card with OK server status.
+  card.set_record_type(CreditCard::MASKED_SERVER_CARD);
+  card.SetServerStatus(CreditCard::OK);
+
+  EXPECT_TRUE(card.ShouldUpdateExpiration());
+
+  // Masked expired server card with EXPIRED server status.
+  card.SetServerStatus(CreditCard::EXPIRED);
+
+  EXPECT_TRUE(card.ShouldUpdateExpiration());
+
+  // Full expired server card with OK server status.
+  card.set_record_type(CreditCard::FULL_SERVER_CARD);
+  card.SetServerStatus(CreditCard::OK);
+
+  EXPECT_TRUE(card.ShouldUpdateExpiration());
+
+  // Full expired server card with EXPIRED server status.
+  card.SetServerStatus(CreditCard::EXPIRED);
+
+  EXPECT_TRUE(card.ShouldUpdateExpiration());
+}
+
+// A card that has expired last month should be updated.
+TEST(CreditCardTest, CardExpiringLastMonth) {
+  base::Time::Exploded last_month;
+  (base::Time::Now() - base::TimeDelta::FromDays(31)).LocalExplode(&last_month);
+
+  // Local expired card.
+  CreditCard card(base::ASCIIToUTF16("1234"), last_month.month,
+                  last_month.year);
+  EXPECT_EQ(CreditCard::LOCAL_CARD, card.record_type());
+
+  EXPECT_TRUE(card.ShouldUpdateExpiration());
+
+  // Masked expired server card with OK server status.
+  card.set_record_type(CreditCard::MASKED_SERVER_CARD);
+  card.SetServerStatus(CreditCard::OK);
+
+  EXPECT_TRUE(card.ShouldUpdateExpiration());
+
+  // Masked expired server card with EXPIRED server status.
+  card.SetServerStatus(CreditCard::EXPIRED);
+
+  EXPECT_TRUE(card.ShouldUpdateExpiration());
+
+  // Full expired server card with OK server status.
+  card.set_record_type(CreditCard::FULL_SERVER_CARD);
+  card.SetServerStatus(CreditCard::OK);
+
+  EXPECT_TRUE(card.ShouldUpdateExpiration());
+
+  // Full expired server card with EXPIRED server status.
+  card.SetServerStatus(CreditCard::EXPIRED);
+
+  EXPECT_TRUE(card.ShouldUpdateExpiration());
+}
+
+// A card expiring this month should not be updated, unless it has EXPIRED
+// server status.
+TEST(CreditCardTest, CardExpiringThisMonth) {
+  base::Time::Exploded this_month;
+  base::Time::Now().LocalExplode(&this_month);
+
+  // Local card expiring this month.
+  CreditCard card(base::ASCIIToUTF16("1234"), this_month.month,
+                  this_month.year);
+  EXPECT_EQ(CreditCard::LOCAL_CARD, card.record_type());
+
+  EXPECT_FALSE(card.ShouldUpdateExpiration());
+
+  // Masked server card expiring this month with OK server status.
+  card.set_record_type(CreditCard::MASKED_SERVER_CARD);
+  card.SetServerStatus(CreditCard::OK);
+
+  EXPECT_FALSE(card.ShouldUpdateExpiration());
+
+  // Masked server card expiring this month with EXPIRED server status.
+  card.SetServerStatus(CreditCard::EXPIRED);
+
+  EXPECT_TRUE(card.ShouldUpdateExpiration());
+
+  // Full server card expiring this month with OK server status.
+  card.set_record_type(CreditCard::FULL_SERVER_CARD);
+  card.SetServerStatus(CreditCard::OK);
+
+  EXPECT_FALSE(card.ShouldUpdateExpiration());
+
+  // Full server card expiring this month with EXPIRED server status.
+  card.SetServerStatus(CreditCard::EXPIRED);
+
+  EXPECT_TRUE(card.ShouldUpdateExpiration());
+}
+
+// A card expiring next month should not be updated, unless it has EXPIRED
+// server status.
+TEST(CreditCardTest, CardExpiringNextMonth) {
+  base::Time::Exploded next_month;
+  (base::Time::Now() + base::TimeDelta::FromDays(31)).LocalExplode(&next_month);
+
+  // Local card expiring next month.
+  CreditCard card(base::ASCIIToUTF16("1234"), next_month.month,
+                  next_month.year);
+  EXPECT_EQ(CreditCard::LOCAL_CARD, card.record_type());
+
+  EXPECT_FALSE(card.ShouldUpdateExpiration());
+
+  // Masked server card expiring next month with OK server status.
+  card.set_record_type(CreditCard::MASKED_SERVER_CARD);
+  card.SetServerStatus(CreditCard::OK);
+
+  EXPECT_FALSE(card.ShouldUpdateExpiration());
+
+  // Masked server card expiring next month with EXPIRED server status.
+  card.SetServerStatus(CreditCard::EXPIRED);
+
+  EXPECT_TRUE(card.ShouldUpdateExpiration());
+
+  // Full server card expiring next month with OK server status.
+  card.set_record_type(CreditCard::FULL_SERVER_CARD);
+  card.SetServerStatus(CreditCard::OK);
+
+  EXPECT_FALSE(card.ShouldUpdateExpiration());
+
+  // Full server card expiring next month with EXPIRED server status.
+  card.SetServerStatus(CreditCard::EXPIRED);
+
+  EXPECT_TRUE(card.ShouldUpdateExpiration());
+}
+
+// A card expiring next year should not be updated, unless it has EXPIRED server
+// status.
+TEST(CreditCardTest, CardExpiringNextYear) {
+  base::Time::Exploded today;
+  base::Time::Now().LocalExplode(&today);
+
+  // Local card expiring next year.
+  CreditCard card(base::ASCIIToUTF16("1234"), 1, today.year + 1);
+  EXPECT_EQ(CreditCard::LOCAL_CARD, card.record_type());
+
+  EXPECT_FALSE(card.ShouldUpdateExpiration());
+
+  // Masked server card expiring next year with OK server status.
+  card.set_record_type(CreditCard::MASKED_SERVER_CARD);
+  card.SetServerStatus(CreditCard::OK);
+
+  EXPECT_FALSE(card.ShouldUpdateExpiration());
+
+  // Masked server card expiring next year with EXPIRED server status.
+  card.SetServerStatus(CreditCard::EXPIRED);
+
+  EXPECT_TRUE(card.ShouldUpdateExpiration());
+
+  // Full server card expiring next year with OK server status.
+  card.set_record_type(CreditCard::FULL_SERVER_CARD);
+  card.SetServerStatus(CreditCard::OK);
+
+  EXPECT_FALSE(card.ShouldUpdateExpiration());
+
+  // Full server card expiring next year with EXPIRED server status.
+  card.SetServerStatus(CreditCard::EXPIRED);
+
+  EXPECT_TRUE(card.ShouldUpdateExpiration());
+}
+
+
 }  // namespace autofill