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

Unified Diff: chrome/tools/disable_outdated_build_detector/google_update_integration_unittest.cc

Issue 2193823002: A tool to disable the outdated build detector for organic installs of Chrome. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix chromium.fyi.json Created 4 years, 4 months 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: chrome/tools/disable_outdated_build_detector/google_update_integration_unittest.cc
diff --git a/chrome/tools/disable_outdated_build_detector/google_update_integration_unittest.cc b/chrome/tools/disable_outdated_build_detector/google_update_integration_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..7905259ff554a48e5ba8e879e1a3ec8411ee557f
--- /dev/null
+++ b/chrome/tools/disable_outdated_build_detector/google_update_integration_unittest.cc
@@ -0,0 +1,54 @@
+// Copyright 2016 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 "chrome/tools/disable_outdated_build_detector/google_update_integration.h"
+
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace {
+
+// Copied from chrome/browser/google/google_brand.cc.
+const wchar_t* const kOrganicBrands[] = {
+ L"CHCA", L"CHCB", L"CHCG", L"CHCH", L"CHCI", L"CHCJ", L"CHCK", L"CHCL",
+ L"CHFO", L"CHFT", L"CHHS", L"CHHM", L"CHMA", L"CHMB", L"CHME", L"CHMF",
+ L"CHMG", L"CHMH", L"CHMI", L"CHMQ", L"CHMV", L"CHNB", L"CHNC", L"CHNG",
+ L"CHNH", L"CHNI", L"CHOA", L"CHOB", L"CHOC", L"CHON", L"CHOO", L"CHOP",
+ L"CHOQ", L"CHOR", L"CHOS", L"CHOT", L"CHOU", L"CHOX", L"CHOY", L"CHOZ",
+ L"CHPD", L"CHPE", L"CHPF", L"CHPG", L"ECBA", L"ECBB", L"ECDA", L"ECDB",
+ L"ECSA", L"ECSB", L"ECVA", L"ECVB", L"ECWA", L"ECWB", L"ECWC", L"ECWD",
+ L"ECWE", L"ECWF", L"EUBB", L"EUBC", L"GGLA", L"GGLS",
+
+ // EUB*
+ L"EUBQ",
+
+ // EUC*
+ L"EUCQ",
+
+ // GGR*
+ L"GGRQ",
+};
+
+} // namespace
+
+// Test that all expected brands are considered organic.
+class IsOrganicTest : public ::testing::TestWithParam<const wchar_t*> {};
+
+TEST_P(IsOrganicTest, IsOrganicBrand) {
+ EXPECT_TRUE(IsOrganic(GetParam()));
+}
+
+INSTANTIATE_TEST_CASE_P(OrganicBrands,
+ IsOrganicTest,
+ ::testing::ValuesIn(kOrganicBrands));
+
+// Test that a smattering of non-organic brands are not considered organic.
+class IsNotOrganicTest : public ::testing::TestWithParam<const wchar_t*> {};
+
+TEST_P(IsNotOrganicTest, IsNotOrganicBrand) {
+ EXPECT_FALSE(IsOrganic(GetParam()));
+}
+
+INSTANTIATE_TEST_CASE_P(NonOrganicBrands,
+ IsNotOrganicTest,
+ ::testing::Values(L"AOHY", L"YAKS", L""));

Powered by Google App Engine
This is Rietveld 408576698