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

Unified Diff: chrome/install_static/product_install_details_unittest.cc

Issue 2601443002: Remove multi-install from install_static. (Closed)
Patch Set: Created 4 years 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/install_static/product_install_details_unittest.cc
diff --git a/chrome/install_static/product_install_details_unittest.cc b/chrome/install_static/product_install_details_unittest.cc
index 15bb5ec200510709489065ed00f32bab1b1442c8..693730565ee5259679b5515da6d39529e1345607 100644
--- a/chrome/install_static/product_install_details_unittest.cc
+++ b/chrome/install_static/product_install_details_unittest.cc
@@ -185,37 +185,26 @@ class MakeProductDetailsTest : public testing::TestWithParam<TestData> {
const TestData& test_data() const { return test_data_; }
void SetUninstallArguments(const wchar_t* value) {
- ASSERT_THAT(
- base::win::RegKey(root_key_, GetClientStateKeyPath(false).c_str(),
- KEY_WOW64_32KEY | KEY_SET_VALUE)
- .WriteValue(L"UninstallArguments", value),
- Eq(ERROR_SUCCESS));
+ ASSERT_THAT(base::win::RegKey(root_key_, GetClientStateKeyPath().c_str(),
+ KEY_WOW64_32KEY | KEY_SET_VALUE)
+ .WriteValue(L"UninstallArguments", value),
+ Eq(ERROR_SUCCESS));
}
- void SetAp(const wchar_t* value, bool binaries) {
- ASSERT_TRUE(!binaries ||
- kInstallModes[test_data().index].supports_multi_install);
- ASSERT_THAT(
- base::win::RegKey(root_key_, GetClientStateKeyPath(binaries).c_str(),
- KEY_WOW64_32KEY | KEY_SET_VALUE)
- .WriteValue(L"ap", value),
- Eq(ERROR_SUCCESS));
+ void SetAp(const wchar_t* value) {
+ ASSERT_THAT(base::win::RegKey(root_key_, GetClientStateKeyPath().c_str(),
+ KEY_WOW64_32KEY | KEY_SET_VALUE)
+ .WriteValue(L"ap", value),
+ Eq(ERROR_SUCCESS));
}
private:
// Returns the registry path for the product's ClientState key.
- std::wstring GetClientStateKeyPath(bool binaries) {
- EXPECT_TRUE(!binaries ||
- kInstallModes[test_data().index].supports_multi_install);
+ std::wstring GetClientStateKeyPath() {
std::wstring result(L"Software\\");
if (kUseGoogleUpdateIntegration) {
result.append(L"Google\\Update\\ClientState\\");
- if (binaries)
- result.append(kBinariesAppGuid);
- else
- result.append(kInstallModes[test_data().index].app_guid);
- } else if (binaries) {
- result.append(kBinariesPathName);
+ result.append(kInstallModes[test_data().index].app_guid);
} else {
result.append(kProductPathName);
}
@@ -251,32 +240,6 @@ TEST_P(MakeProductDetailsTest, DefaultChannel) {
EXPECT_THAT(details->channel(), StrEq(test_data().channel));
}
-// Test that multi-install is properly parsed out of the registry.
-TEST_P(MakeProductDetailsTest, MultiInstall) {
- {
- std::unique_ptr<PrimaryInstallDetails> details(
- MakeProductDetails(test_data().path));
- EXPECT_FALSE(details->multi_install());
- }
-
- {
- SetUninstallArguments(L"--uninstall");
- std::unique_ptr<PrimaryInstallDetails> details(
- MakeProductDetails(test_data().path));
- EXPECT_FALSE(details->multi_install());
- }
-
- if (!kInstallModes[test_data().index].supports_multi_install)
- return;
-
- {
- SetUninstallArguments(L"--uninstall --multi-install --chrome");
- std::unique_ptr<PrimaryInstallDetails> details(
- MakeProductDetails(test_data().path));
- EXPECT_TRUE(details->multi_install());
- }
-}
-
// Test that the channel name is properly parsed out of additional parameters.
TEST_P(MakeProductDetailsTest, AdditionalParametersChannels) {
const std::pair<const wchar_t*, const wchar_t*> kApChannels[] = {
@@ -317,25 +280,7 @@ TEST_P(MakeProductDetailsTest, AdditionalParametersChannels) {
};
for (const auto& ap_and_channel : kApChannels) {
- SetAp(ap_and_channel.first, false);
- std::unique_ptr<PrimaryInstallDetails> details(
- MakeProductDetails(test_data().path));
- if (kInstallModes[test_data().index].channel_strategy ==
- ChannelStrategy::ADDITIONAL_PARAMETERS) {
- EXPECT_THAT(details->channel(), StrEq(ap_and_channel.second));
- } else {
- // "ap" is ignored for this mode.
- EXPECT_THAT(details->channel(), StrEq(test_data().channel));
- }
- }
-
- if (!kInstallModes[test_data().index].supports_multi_install)
- return;
-
- // For multi-install modes, "ap" is pulled from the binaries' key.
- for (const auto& ap_and_channel : kApChannels) {
- SetAp(ap_and_channel.first, true);
- SetUninstallArguments(L"--uninstall --multi-install --chrome");
+ SetAp(ap_and_channel.first);
std::unique_ptr<PrimaryInstallDetails> details(
MakeProductDetails(test_data().path));
if (kInstallModes[test_data().index].channel_strategy ==
« chrome/install_static/install_util.cc ('K') | « chrome/install_static/product_install_details.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698