diff options
| author | Bertrand SIMONNET <bsimonnet@chromium.org> | 2015-07-21 14:42:25 -0700 |
|---|---|---|
| committer | ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> | 2015-07-22 05:02:42 +0000 |
| commit | 738267b2659457100ac0af45b6a3e3d9c1f56d86 (patch) | |
| tree | 1a0f96acbf453a12577dd93f2a309627228b6254 | |
| parent | a8632e4a671d44cffa7b1f079baa0b9fdb48c5e3 (diff) | |
| download | libbrillo-738267b2659457100ac0af45b6a3e3d9c1f56d86.tar.gz | |
libchromeos: Remove incorrect test.
dca2caf2ea0d27d1abd0e47f1dbf509bb627dd6c changed the behaviour of
Process to not fail when the output file already exists.
The test should be remove too.
BUG=None
TEST=FEATURES=test emerge-gizmo libchromeos --nodeps
Change-Id: I6495190cf347c8f7dc8eb9f8bd57f01b6d15591e
Reviewed-on: https://chromium-review.googlesource.com/287403
Trybot-Ready: Bertrand Simonnet <bsimonnet@chromium.org>
Tested-by: Bertrand Simonnet <bsimonnet@chromium.org>
Reviewed-by: Christopher Wiley <wiley@chromium.org>
Reviewed-by: Dan Erat <derat@chromium.org>
Commit-Queue: Bertrand Simonnet <bsimonnet@chromium.org>
| -rw-r--r-- | chromeos/process_test.cc | 9 |
1 files changed, 0 insertions, 9 deletions
diff --git a/chromeos/process_test.cc b/chromeos/process_test.cc index eeaf14f..8cb503b 100644 --- a/chromeos/process_test.cc +++ b/chromeos/process_test.cc @@ -127,15 +127,6 @@ TEST_F(ProcessTest, BadOutputFile) { EXPECT_EQ(static_cast<pid_t>(Process::kErrorExitStatus), process_.Run()); } -TEST_F(ProcessTest, ExistingOutputFile) { - process_.AddArg(kBinEcho); - process_.AddArg("hello world"); - EXPECT_FALSE(base::PathExists(FilePath(output_file_))); - EXPECT_EQ(0, process_.Run()); - EXPECT_TRUE(base::PathExists(FilePath(output_file_))); - EXPECT_EQ(static_cast<pid_t>(Process::kErrorExitStatus), process_.Run()); -} - TEST_F(ProcessTest, BadExecutable) { process_.AddArg("false"); EXPECT_EQ(static_cast<pid_t>(Process::kErrorExitStatus), process_.Run()); |