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

Unified Diff: chrome/browser/chromeos/file_system_provider/operations/create_directory_unittest.cc

Issue 427493002: [fsp] Do not allow write operations on read only file systems. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: 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/operations/create_directory_unittest.cc
diff --git a/chrome/browser/chromeos/file_system_provider/operations/create_directory_unittest.cc b/chrome/browser/chromeos/file_system_provider/operations/create_directory_unittest.cc
index 6d1869a18b4bf946c9a8118854735eb63f620348..e4f191bb91a607063512f808089d721c5f93d399 100644
--- a/chrome/browser/chromeos/file_system_provider/operations/create_directory_unittest.cc
+++ b/chrome/browser/chromeos/file_system_provider/operations/create_directory_unittest.cc
@@ -115,6 +115,31 @@ TEST_F(FileSystemProviderOperationsCreateDirectoryTest, Execute_NoListener) {
EXPECT_FALSE(create_directory.Execute(kRequestId));
}
+TEST_F(FileSystemProviderOperationsCreateDirectoryTest, Execute_ReadOnly) {
+ util::LoggingDispatchEventImpl dispatcher(true /* dispatch_reply */);
+ util::StatusCallbackLog callback_log;
+
+ const ProvidedFileSystemInfo read_only_file_system_info(
+ kExtensionId,
+ kFileSystemId,
+ "" /* file_system_name */,
+ false /* writable */,
+ base::FilePath() /* mount_path */);
+
+ CreateDirectory create_directory(
+ NULL,
+ read_only_file_system_info,
+ base::FilePath::FromUTF8Unsafe(kDirectoryPath),
+ false /* exclusive */,
+ true /* recursive */,
+ base::Bind(&util::LogStatusCallback, &callback_log));
+ create_directory.SetDispatchEventImplForTesting(
+ base::Bind(&util::LoggingDispatchEventImpl::OnDispatchEventImpl,
+ base::Unretained(&dispatcher)));
+
+ EXPECT_FALSE(create_directory.Execute(kRequestId));
+}
+
TEST_F(FileSystemProviderOperationsCreateDirectoryTest, OnSuccess) {
util::LoggingDispatchEventImpl dispatcher(true /* dispatch_reply */);
util::StatusCallbackLog callback_log;

Powered by Google App Engine
This is Rietveld 408576698