summaryrefslogtreecommitdiff
path: root/filesystem_copier_action_unittest.cc
diff options
context:
space:
mode:
authorAlex Vakulenko <avakulenko@chromium.org>2014-06-16 13:19:00 -0700
committerchrome-internal-fetch <chrome-internal-fetch@google.com>2014-06-18 01:39:59 +0000
commitd2779df63aaad8b65fc5d4badee7dbc9bed7f2b6 (patch)
treee34a511ee4068d8d9cd46a992a4e147f3afbd351 /filesystem_copier_action_unittest.cc
parentb683327ed3f50ef89324069fc87ee076b65bee46 (diff)
update_engine: fixed warnings from cpplint
Fixed all the cpplint warnings in update engine. BUG=None TEST=Unit tests still pass. Change-Id: I285ae858eec8abe0b26ff203b99a42a200ceb71c Reviewed-on: https://chromium-review.googlesource.com/204027 Reviewed-by: Alex Vakulenko <avakulenko@chromium.org> Tested-by: Alex Vakulenko <avakulenko@chromium.org> Commit-Queue: Alex Vakulenko <avakulenko@chromium.org>
Diffstat (limited to 'filesystem_copier_action_unittest.cc')
-rw-r--r--filesystem_copier_action_unittest.cc3
1 files changed, 2 insertions, 1 deletions
diff --git a/filesystem_copier_action_unittest.cc b/filesystem_copier_action_unittest.cc
index 3e5f1b93..4d67b5e2 100644
--- a/filesystem_copier_action_unittest.cc
+++ b/filesystem_copier_action_unittest.cc
@@ -78,6 +78,7 @@ class FilesystemCopierActionTestDelegate : public ActionProcessorDelegate {
}
bool ran() const { return ran_; }
ErrorCode code() const { return code_; }
+
private:
GMainLoop* loop_;
FilesystemCopierAction* action_;
@@ -420,7 +421,7 @@ TEST_F(FilesystemCopierActionTest, RunAsRootDetermineFilesystemSizeTest) {
EXPECT_EQ(kint64max, action.filesystem_size_);
{
int fd = HANDLE_EINTR(open(img.c_str(), O_RDONLY));
- EXPECT_TRUE(fd > 0);
+ EXPECT_GT(fd, 0);
ScopedFdCloser fd_closer(&fd);
action.DetermineFilesystemSize(fd);
}