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

Unified Diff: chrome/browser/chromeos/file_system_provider/fileapi/provider_async_file_util_unittest.cc

Issue 375543002: [fsp] Add support for deleting entries. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased + cleaned up. Created 6 years, 5 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
Index: chrome/browser/chromeos/file_system_provider/fileapi/provider_async_file_util_unittest.cc
diff --git a/chrome/browser/chromeos/file_system_provider/fileapi/provider_async_file_util_unittest.cc b/chrome/browser/chromeos/file_system_provider/fileapi/provider_async_file_util_unittest.cc
index 08e5f4cb1d4564548ea53127a0fecbb3e2c2667e..cf85ed8887e54a3211f6d79a27de7d1ff6783ba6 100644
--- a/chrome/browser/chromeos/file_system_provider/fileapi/provider_async_file_util_unittest.cc
+++ b/chrome/browser/chromeos/file_system_provider/fileapi/provider_async_file_util_unittest.cc
@@ -377,9 +377,10 @@ TEST_F(FileSystemProviderProviderAsyncFileUtilTest, DeleteFile) {
CreateOperationContext(),
file_url_,
base::Bind(&EventLogger::OnStatus, logger.GetWeakPtr()));
+ base::RunLoop().RunUntilIdle();
ASSERT_TRUE(logger.result());
- EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.result());
+ EXPECT_EQ(base::File::FILE_OK, *logger.result());
}
TEST_F(FileSystemProviderProviderAsyncFileUtilTest, DeleteDirectory) {
@@ -389,9 +390,10 @@ TEST_F(FileSystemProviderProviderAsyncFileUtilTest, DeleteDirectory) {
CreateOperationContext(),
directory_url_,
base::Bind(&EventLogger::OnStatus, logger.GetWeakPtr()));
+ base::RunLoop().RunUntilIdle();
ASSERT_TRUE(logger.result());
- EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.result());
+ EXPECT_EQ(base::File::FILE_OK, *logger.result());
}
TEST_F(FileSystemProviderProviderAsyncFileUtilTest, DeleteRecursively) {
@@ -401,9 +403,10 @@ TEST_F(FileSystemProviderProviderAsyncFileUtilTest, DeleteRecursively) {
CreateOperationContext(),
directory_url_,
base::Bind(&EventLogger::OnStatus, logger.GetWeakPtr()));
+ base::RunLoop().RunUntilIdle();
ASSERT_TRUE(logger.result());
- EXPECT_EQ(base::File::FILE_ERROR_ACCESS_DENIED, *logger.result());
+ EXPECT_EQ(base::File::FILE_OK, *logger.result());
}
TEST_F(FileSystemProviderProviderAsyncFileUtilTest, CreateSnapshotFile) {

Powered by Google App Engine
This is Rietveld 408576698