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

Side by Side Diff: chromecast/crash/linux/minidump_uploader_unittest.cc

Issue 2322573002: misc files A-P: Change ScopedTempDir::path() to GetPath() (Closed)
Patch Set: Just rebased Created 4 years, 3 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chromecast/crash/linux/minidump_uploader.h" 5 #include "chromecast/crash/linux/minidump_uploader.h"
6 6
7 #include <memory> 7 #include <memory>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/base_paths.h" 10 #include "base/base_paths.h"
(...skipping 57 matching lines...) Expand 10 before | Expand all | Expand 10 after
68 class MinidumpUploaderTest : public testing::Test { 68 class MinidumpUploaderTest : public testing::Test {
69 public: 69 public:
70 MinidumpUploaderTest() {} 70 MinidumpUploaderTest() {}
71 ~MinidumpUploaderTest() override {} 71 ~MinidumpUploaderTest() override {}
72 72
73 protected: 73 protected:
74 void SetUp() override { 74 void SetUp() override {
75 // Set up a temporary directory which will be used as our fake home dir. 75 // Set up a temporary directory which will be used as our fake home dir.
76 ASSERT_TRUE(fake_home_dir_.CreateUniqueTempDir()); 76 ASSERT_TRUE(fake_home_dir_.CreateUniqueTempDir());
77 path_override_.reset( 77 path_override_.reset(
78 new base::ScopedPathOverride(base::DIR_HOME, fake_home_dir_.path())); 78 new base::ScopedPathOverride(base::DIR_HOME, fake_home_dir_.GetPath()));
79 79
80 minidump_dir_ = fake_home_dir_.path().Append(kMinidumpSubdir); 80 minidump_dir_ = fake_home_dir_.GetPath().Append(kMinidumpSubdir);
81 lockfile_ = minidump_dir_.Append(kLockfileName); 81 lockfile_ = minidump_dir_.Append(kLockfileName);
82 metadata_ = minidump_dir_.Append(kMetadataName); 82 metadata_ = minidump_dir_.Append(kMetadataName);
83 83
84 // Create minidump directory. 84 // Create minidump directory.
85 ASSERT_TRUE(base::CreateDirectory(minidump_dir_)); 85 ASSERT_TRUE(base::CreateDirectory(minidump_dir_));
86 86
87 CastCrashdumpData data; 87 CastCrashdumpData data;
88 mock_crash_uploader_.reset(new StrictMock<MockCastCrashdumpUploader>(data)); 88 mock_crash_uploader_.reset(new StrictMock<MockCastCrashdumpUploader>(data));
89 } 89 }
90 90
(...skipping 274 matching lines...) Expand 10 before | Expand all | Expand 10 after
365 // Ensure dump files were deleted, lockfile was emptied. 365 // Ensure dump files were deleted, lockfile was emptied.
366 ASSERT_FALSE(base::PathExists(minidump_path)); 366 ASSERT_FALSE(base::PathExists(minidump_path));
367 ASSERT_FALSE(base::PathExists(logfile_path)); 367 ASSERT_FALSE(base::PathExists(logfile_path));
368 368
369 ASSERT_TRUE(base::GetFileSize(lockfile_, &size)); 369 ASSERT_TRUE(base::GetFileSize(lockfile_, &size));
370 ASSERT_EQ(size, 0); 370 ASSERT_EQ(size, 0);
371 } 371 }
372 372
373 } // namespace 373 } // namespace
374 } // namespace chromecast 374 } // namespace chromecast
OLDNEW
« no previous file with comments | « chromecast/crash/linux/dummy_minidump_generator_unittest.cc ('k') | chromecast/crash/linux/minidump_writer_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698