Chromium Code Reviews| Index: chrome/browser/google/google_update_settings_unittest.cc |
| diff --git a/chrome/browser/google/google_update_settings_unittest.cc b/chrome/browser/google/google_update_settings_unittest.cc |
| index 8b3572aa9a62544928f7c5674ff92d929ff3cc74..31c7fda4ccd347401462d68fb307edc4147f79d4 100644 |
| --- a/chrome/browser/google/google_update_settings_unittest.cc |
| +++ b/chrome/browser/google/google_update_settings_unittest.cc |
| @@ -50,6 +50,12 @@ TEST_F(GoogleUpdateTest, LastRunTime) { |
| #endif // defined(OS_WIN) |
| +TEST_F(GoogleUpdateTest, IsOrganic) { |
| + // Test some brand codes to ensure that future changes to this method won't |
| + // go unnoticed. |
| + EXPECT_FALSE(google_brand::IsOrganic("GGRV")); |
|
Peter Kasting
2017/04/12 20:05:20
Nit: Also EXPECT_TRUE for GGRsomething_else?
Roger Tawa OOO till Jul 10th
2017/04/12 21:51:43
Do you think I should do all the letters?
Peter Kasting
2017/04/12 22:46:03
Not necessarily, I was mainly thinking of catching
|
| +} |
| + |
| TEST_F(GoogleUpdateTest, IsOrganicFirstRunBrandCodes) { |
| // Test some brand codes to ensure that future changes to this method won't |
| // go unnoticed. |