diff options
author | msw@chromium.org <msw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-07-13 21:46:32 +0000 |
---|---|---|
committer | msw@chromium.org <msw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-07-13 21:46:32 +0000 |
commit | 62b4e52857f1fc8c0e26be90bc5e770e65f12aec (patch) | |
tree | 9101c313ebb60fda88a18f2aac014007011385dd /chrome/installer | |
parent | 36717d9b4e6ac635fe762ae9974fd57538df2fa8 (diff) | |
download | chromium_src-62b4e52857f1fc8c0e26be90bc5e770e65f12aec.zip chromium_src-62b4e52857f1fc8c0e26be90bc5e770e65f12aec.tar.gz chromium_src-62b4e52857f1fc8c0e26be90bc5e770e65f12aec.tar.bz2 |
Nix CommandLine::GetSwitchCount.
BUG=73195
TEST=none
Review URL: http://codereview.chromium.org/7342024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92423 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/installer')
-rw-r--r-- | chrome/installer/util/install_util_unittest.cc | 4 | ||||
-rw-r--r-- | chrome/installer/util/product_state_unittest.cc | 12 |
2 files changed, 8 insertions, 8 deletions
diff --git a/chrome/installer/util/install_util_unittest.cc b/chrome/installer/util/install_util_unittest.cc index b579746..9b12075d 100644 --- a/chrome/installer/util/install_util_unittest.cc +++ b/chrome/installer/util/install_util_unittest.cc @@ -41,9 +41,9 @@ TEST_F(InstallUtilTest, MakeUninstallCommand) { InstallUtil::MakeUninstallCommand(param.first, param.second, &command_line); EXPECT_EQ(param.first, command_line.GetProgram().value()); if (param.second.empty()) { - EXPECT_EQ(0U, command_line.GetSwitchCount()); + EXPECT_TRUE(command_line.GetSwitches().empty()); } else { - EXPECT_EQ(2U, command_line.GetSwitchCount()); + EXPECT_EQ(2U, command_line.GetSwitches().size()); EXPECT_TRUE(command_line.HasSwitch("do-something")); EXPECT_TRUE(command_line.HasSwitch("silly")); } diff --git a/chrome/installer/util/product_state_unittest.cc b/chrome/installer/util/product_state_unittest.cc index b6c228f..ad81ed8 100644 --- a/chrome/installer/util/product_state_unittest.cc +++ b/chrome/installer/util/product_state_unittest.cc @@ -284,7 +284,7 @@ TEST_F(ProductStateTest, InitializeUninstallCommand) { EXPECT_TRUE(state.Initialize(system_install_, dist_)); EXPECT_TRUE(state.GetSetupPath().empty()); EXPECT_TRUE(state.uninstall_command().command_line_string().empty()); - EXPECT_EQ(0U, state.uninstall_command().GetSwitchCount()); + EXPECT_TRUE(state.uninstall_command().GetSwitches().empty()); } // Empty values. @@ -294,7 +294,7 @@ TEST_F(ProductStateTest, InitializeUninstallCommand) { EXPECT_TRUE(state.Initialize(system_install_, dist_)); EXPECT_TRUE(state.GetSetupPath().empty()); EXPECT_TRUE(state.uninstall_command().command_line_string().empty()); - EXPECT_EQ(0U, state.uninstall_command().GetSwitchCount()); + EXPECT_TRUE(state.uninstall_command().GetSwitches().empty()); } // Uninstall command without exe. @@ -304,7 +304,7 @@ TEST_F(ProductStateTest, InitializeUninstallCommand) { EXPECT_TRUE(state.Initialize(system_install_, dist_)); EXPECT_TRUE(state.GetSetupPath().empty()); EXPECT_EQ(L" --uninstall", state.uninstall_command().command_line_string()); - EXPECT_EQ(1U, state.uninstall_command().GetSwitchCount()); + EXPECT_EQ(1U, state.uninstall_command().GetSwitches().size()); } // Uninstall command without args. @@ -314,7 +314,7 @@ TEST_F(ProductStateTest, InitializeUninstallCommand) { EXPECT_TRUE(state.Initialize(system_install_, dist_)); EXPECT_EQ(L"setup.exe", state.GetSetupPath().value()); EXPECT_EQ(L"setup.exe", state.uninstall_command().command_line_string()); - EXPECT_EQ(0U, state.uninstall_command().GetSwitchCount()); + EXPECT_TRUE(state.uninstall_command().GetSwitches().empty()); } // Uninstall command with exe that requires quoting. @@ -325,7 +325,7 @@ TEST_F(ProductStateTest, InitializeUninstallCommand) { EXPECT_EQ(L"set up.exe", state.GetSetupPath().value()); EXPECT_EQ(L"\"set up.exe\"", state.uninstall_command().command_line_string()); - EXPECT_EQ(0U, state.uninstall_command().GetSwitchCount()); + EXPECT_TRUE(state.uninstall_command().GetSwitches().empty()); } // Uninstall command with both exe and args. @@ -336,7 +336,7 @@ TEST_F(ProductStateTest, InitializeUninstallCommand) { EXPECT_EQ(L"setup.exe", state.GetSetupPath().value()); EXPECT_EQ(L"setup.exe --uninstall", state.uninstall_command().command_line_string()); - EXPECT_EQ(1U, state.uninstall_command().GetSwitchCount()); + EXPECT_EQ(1U, state.uninstall_command().GetSwitches().size()); } } |