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

Unified Diff: webrtc/base/fileutils_unittest.cc

Issue 2445733002: Delete unused features of rtc::FilesystemInterface. (Closed)
Patch Set: Created 4 years, 2 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webrtc/base/fileutils_mock.h ('k') | webrtc/base/unixfilesystem.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/base/fileutils_unittest.cc
diff --git a/webrtc/base/fileutils_unittest.cc b/webrtc/base/fileutils_unittest.cc
index 756f511f7a94c7b6d8ebb63d0def7ade2aa4d470..e9999040f21b352366daf534cc77069aaf7a376b 100644
--- a/webrtc/base/fileutils_unittest.cc
+++ b/webrtc/base/fileutils_unittest.cc
@@ -70,28 +70,6 @@ TEST(MAYBE_FilesystemTest, TestOpenBadFile) {
EXPECT_FALSE(fs != NULL);
}
-// Test that CreatePrivateFile fails for existing files and succeeds for
-// non-existent ones.
-TEST(MAYBE_FilesystemTest, TestCreatePrivateFile) {
- Pathname path;
- EXPECT_TRUE(Filesystem::GetTemporaryFolder(path, true, NULL));
- path.SetFilename("private_file_test");
-
- // First call should succeed because the file doesn't exist yet.
- EXPECT_TRUE(Filesystem::CreatePrivateFile(path));
- // Next call should fail, because now it exists.
- EXPECT_FALSE(Filesystem::CreatePrivateFile(path));
-
- // Verify that we have permission to open the file for reading and writing.
- std::unique_ptr<FileStream> fs(Filesystem::OpenFile(path, "wb"));
- EXPECT_TRUE(fs.get() != NULL);
- // Have to close the file on Windows before it will let us delete it.
- fs.reset();
-
- // Verify that we have permission to delete the file.
- EXPECT_TRUE(Filesystem::DeleteFile(path));
-}
-
// Test checking for free disk space.
TEST(MAYBE_FilesystemTest, TestGetDiskFreeSpace) {
// Note that we should avoid picking any file/folder which could be located
@@ -125,16 +103,4 @@ TEST(MAYBE_FilesystemTest, TestGetDiskFreeSpace) {
EXPECT_GT(free3, 0);
}
-// Tests that GetCurrentDirectory() returns something.
-TEST(MAYBE_FilesystemTest, TestGetCurrentDirectory) {
- EXPECT_FALSE(Filesystem::GetCurrentDirectory().empty());
-}
-
-// Tests that GetAppPathname returns something.
-TEST(MAYBE_FilesystemTest, TestGetAppPathname) {
- Pathname path;
- EXPECT_TRUE(Filesystem::GetAppPathname(&path));
- EXPECT_FALSE(path.empty());
-}
-
} // namespace rtc
« no previous file with comments | « webrtc/base/fileutils_mock.h ('k') | webrtc/base/unixfilesystem.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698