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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/chromeos/file_system_provider/service_unittest.cc

Issue 334263017: [fsp] Remember mounted file systems as soon as possible. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Updated a comment. 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
« no previous file with comments | « chrome/browser/chromeos/file_system_provider/service.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/file_system_provider/service_unittest.cc
diff --git a/chrome/browser/chromeos/file_system_provider/service_unittest.cc b/chrome/browser/chromeos/file_system_provider/service_unittest.cc
index e5b0a83cecfd69062c3a5059051ac78955ca4552..ebcf4a633e95b83cbe72d68cb0d1e24be60d7b67 100644
--- a/chrome/browser/chromeos/file_system_provider/service_unittest.cc
+++ b/chrome/browser/chromeos/file_system_provider/service_unittest.cc
@@ -32,9 +32,13 @@ namespace file_system_provider {
namespace {
const char kExtensionId[] = "mbflcebpggnecokmikipoihdbecnjfoj";
-const char kFileSystemId[] = "camera/pictures/id !@#$%^&*()_+";
const char kDisplayName[] = "Camera Pictures";
+// The dot in the file system ID is there in order to check that saving to
+// preferences works correctly. File System ID is used as a key in
+// a base::DictionaryValue, so it has to be stored without path expansion.
+const char kFileSystemId[] = "camera/pictures/id .!@#$%^&*()_+";
+
// Utility observer, logging events from file_system_provider::Service.
class LoggingObserver : public Observer {
public:
@@ -80,8 +84,10 @@ scoped_refptr<extensions::Extension> createFakeExtension(
const std::string& extension_id) {
base::DictionaryValue manifest;
std::string error;
- manifest.SetString(extensions::manifest_keys::kVersion, "1.0.0.0");
- manifest.SetString(extensions::manifest_keys::kName, "unused");
+ manifest.SetStringWithoutPathExpansion(extensions::manifest_keys::kVersion,
+ "1.0.0.0");
+ manifest.SetStringWithoutPathExpansion(extensions::manifest_keys::kName,
+ "unused");
return extensions::Extension::Create(base::FilePath(),
extensions::Manifest::UNPACKED,
manifest,
@@ -95,16 +101,18 @@ void RememberFakeFileSystem(TestingProfile* profile,
const std::string& extension_id,
const std::string& file_system_id,
const std::string& display_name) {
- TestingPrefServiceSyncable* pref_service = profile->GetTestingPrefService();
+ TestingPrefServiceSyncable* const pref_service =
+ profile->GetTestingPrefService();
ASSERT_TRUE(pref_service);
base::DictionaryValue extensions;
- base::ListValue* file_systems = new base::ListValue();
+ base::DictionaryValue* file_systems = new base::DictionaryValue();
base::DictionaryValue* file_system = new base::DictionaryValue();
- file_system->SetString(kPrefKeyFileSystemId, kFileSystemId);
- file_system->SetString(kPrefKeyDisplayName, kDisplayName);
- file_systems->Append(file_system);
- extensions.Set(kExtensionId, file_systems);
+ file_system->SetStringWithoutPathExpansion(kPrefKeyFileSystemId,
+ kFileSystemId);
+ file_system->SetStringWithoutPathExpansion(kPrefKeyDisplayName, kDisplayName);
+ file_systems->SetWithoutPathExpansion(kFileSystemId, file_system);
+ extensions.SetWithoutPathExpansion(kExtensionId, file_systems);
pref_service->Set(prefs::kFileSystemProviderMounted, extensions);
}
@@ -123,9 +131,8 @@ class FileSystemProviderServiceTest : public testing::Test {
user_manager_enabler_.reset(new ScopedUserManagerEnabler(user_manager_));
extension_registry_.reset(
new extensions::ExtensionRegistry(profile_.get()));
- file_system_provider_service_.reset(
- new Service(profile_.get(), extension_registry_.get()));
- file_system_provider_service_->SetFileSystemFactoryForTesting(
+ service_.reset(new Service(profile_.get(), extension_registry_.get()));
+ service_->SetFileSystemFactoryForTesting(
base::Bind(&FakeProvidedFileSystem::Create));
extension_ = createFakeExtension(kExtensionId);
}
@@ -135,17 +142,16 @@ class FileSystemProviderServiceTest : public testing::Test {
FakeUserManager* user_manager_;
scoped_ptr<ScopedUserManagerEnabler> user_manager_enabler_;
scoped_ptr<extensions::ExtensionRegistry> extension_registry_;
- scoped_ptr<Service> file_system_provider_service_;
+ scoped_ptr<Service> service_;
scoped_refptr<extensions::Extension> extension_;
};
TEST_F(FileSystemProviderServiceTest, MountFileSystem) {
LoggingObserver observer;
- file_system_provider_service_->AddObserver(&observer);
+ service_->AddObserver(&observer);
- const bool result = file_system_provider_service_->MountFileSystem(
- kExtensionId, kFileSystemId, kDisplayName);
- EXPECT_TRUE(result);
+ EXPECT_TRUE(
+ service_->MountFileSystem(kExtensionId, kFileSystemId, kDisplayName));
ASSERT_EQ(1u, observer.mounts.size());
EXPECT_EQ(kExtensionId, observer.mounts[0].file_system_info().extension_id());
@@ -160,77 +166,70 @@ TEST_F(FileSystemProviderServiceTest, MountFileSystem) {
ASSERT_EQ(0u, observer.unmounts.size());
std::vector<ProvidedFileSystemInfo> file_system_info_list =
- file_system_provider_service_->GetProvidedFileSystemInfoList();
+ service_->GetProvidedFileSystemInfoList();
ASSERT_EQ(1u, file_system_info_list.size());
- file_system_provider_service_->RemoveObserver(&observer);
+ service_->RemoveObserver(&observer);
}
TEST_F(FileSystemProviderServiceTest, MountFileSystem_UniqueIds) {
LoggingObserver observer;
- file_system_provider_service_->AddObserver(&observer);
-
- const bool result = file_system_provider_service_->MountFileSystem(
- kExtensionId, kFileSystemId, kDisplayName);
- EXPECT_TRUE(result);
+ service_->AddObserver(&observer);
- const bool second_result = file_system_provider_service_->MountFileSystem(
- kExtensionId, kFileSystemId, kDisplayName);
- EXPECT_FALSE(second_result);
+ EXPECT_TRUE(
+ service_->MountFileSystem(kExtensionId, kFileSystemId, kDisplayName));
+ EXPECT_FALSE(
+ service_->MountFileSystem(kExtensionId, kFileSystemId, kDisplayName));
ASSERT_EQ(2u, observer.mounts.size());
EXPECT_EQ(base::File::FILE_OK, observer.mounts[0].error());
EXPECT_EQ(base::File::FILE_ERROR_EXISTS, observer.mounts[1].error());
std::vector<ProvidedFileSystemInfo> file_system_info_list =
- file_system_provider_service_->GetProvidedFileSystemInfoList();
+ service_->GetProvidedFileSystemInfoList();
ASSERT_EQ(1u, file_system_info_list.size());
- file_system_provider_service_->RemoveObserver(&observer);
+ service_->RemoveObserver(&observer);
}
TEST_F(FileSystemProviderServiceTest, MountFileSystem_StressTest) {
LoggingObserver observer;
- file_system_provider_service_->AddObserver(&observer);
+ service_->AddObserver(&observer);
const size_t kMaxFileSystems = 16;
for (size_t i = 0; i < kMaxFileSystems; ++i) {
const std::string file_system_id =
std::string("test-") + base::IntToString(i);
- const bool result = file_system_provider_service_->MountFileSystem(
- kExtensionId, file_system_id, kDisplayName);
- EXPECT_TRUE(result);
+ EXPECT_TRUE(
+ service_->MountFileSystem(kExtensionId, file_system_id, kDisplayName));
}
ASSERT_EQ(kMaxFileSystems, observer.mounts.size());
// The next file system is out of limit, and registering it should fail.
- const bool result = file_system_provider_service_->MountFileSystem(
- kExtensionId, kFileSystemId, kDisplayName);
- EXPECT_FALSE(result);
+ EXPECT_FALSE(
+ service_->MountFileSystem(kExtensionId, kFileSystemId, kDisplayName));
ASSERT_EQ(kMaxFileSystems + 1, observer.mounts.size());
EXPECT_EQ(base::File::FILE_ERROR_TOO_MANY_OPENED,
observer.mounts[kMaxFileSystems].error());
std::vector<ProvidedFileSystemInfo> file_system_info_list =
- file_system_provider_service_->GetProvidedFileSystemInfoList();
+ service_->GetProvidedFileSystemInfoList();
ASSERT_EQ(kMaxFileSystems, file_system_info_list.size());
- file_system_provider_service_->RemoveObserver(&observer);
+ service_->RemoveObserver(&observer);
}
TEST_F(FileSystemProviderServiceTest, UnmountFileSystem) {
LoggingObserver observer;
- file_system_provider_service_->AddObserver(&observer);
+ service_->AddObserver(&observer);
- const bool result = file_system_provider_service_->MountFileSystem(
- kExtensionId, kFileSystemId, kDisplayName);
- EXPECT_TRUE(result);
+ EXPECT_TRUE(
+ service_->MountFileSystem(kExtensionId, kFileSystemId, kDisplayName));
ASSERT_EQ(1u, observer.mounts.size());
- const bool unmount_result = file_system_provider_service_->UnmountFileSystem(
- kExtensionId, kFileSystemId);
- EXPECT_TRUE(unmount_result);
+ EXPECT_TRUE(service_->UnmountFileSystem(
+ kExtensionId, kFileSystemId, Service::UNMOUNT_REASON_USER));
ASSERT_EQ(1u, observer.unmounts.size());
EXPECT_EQ(base::File::FILE_OK, observer.unmounts[0].error());
@@ -240,23 +239,22 @@ TEST_F(FileSystemProviderServiceTest, UnmountFileSystem) {
observer.unmounts[0].file_system_info().file_system_id());
std::vector<ProvidedFileSystemInfo> file_system_info_list =
- file_system_provider_service_->GetProvidedFileSystemInfoList();
+ service_->GetProvidedFileSystemInfoList();
ASSERT_EQ(0u, file_system_info_list.size());
- file_system_provider_service_->RemoveObserver(&observer);
+ service_->RemoveObserver(&observer);
}
TEST_F(FileSystemProviderServiceTest, UnmountFileSystem_OnExtensionUnload) {
LoggingObserver observer;
- file_system_provider_service_->AddObserver(&observer);
+ service_->AddObserver(&observer);
- const bool result = file_system_provider_service_->MountFileSystem(
- kExtensionId, kFileSystemId, kDisplayName);
- EXPECT_TRUE(result);
+ EXPECT_TRUE(
+ service_->MountFileSystem(kExtensionId, kFileSystemId, kDisplayName));
ASSERT_EQ(1u, observer.mounts.size());
// Directly call the observer's method.
- file_system_provider_service_->OnExtensionUnloaded(
+ service_->OnExtensionUnloaded(
profile_.get(),
extension_.get(),
extensions::UnloadedExtensionInfo::REASON_DISABLE);
@@ -270,55 +268,55 @@ TEST_F(FileSystemProviderServiceTest, UnmountFileSystem_OnExtensionUnload) {
observer.unmounts[0].file_system_info().file_system_id());
std::vector<ProvidedFileSystemInfo> file_system_info_list =
- file_system_provider_service_->GetProvidedFileSystemInfoList();
+ service_->GetProvidedFileSystemInfoList();
ASSERT_EQ(0u, file_system_info_list.size());
- file_system_provider_service_->RemoveObserver(&observer);
+ service_->RemoveObserver(&observer);
}
TEST_F(FileSystemProviderServiceTest, UnmountFileSystem_WrongExtensionId) {
LoggingObserver observer;
- file_system_provider_service_->AddObserver(&observer);
+ service_->AddObserver(&observer);
const std::string kWrongExtensionId = "helloworldhelloworldhelloworldhe";
- const bool result = file_system_provider_service_->MountFileSystem(
- kExtensionId, kFileSystemId, kDisplayName);
- EXPECT_TRUE(result);
+ EXPECT_TRUE(
+ service_->MountFileSystem(kExtensionId, kFileSystemId, kDisplayName));
ASSERT_EQ(1u, observer.mounts.size());
- ASSERT_EQ(
- 1u,
- file_system_provider_service_->GetProvidedFileSystemInfoList().size());
+ ASSERT_EQ(1u, service_->GetProvidedFileSystemInfoList().size());
- const bool unmount_result = file_system_provider_service_->UnmountFileSystem(
- kWrongExtensionId, kFileSystemId);
- EXPECT_FALSE(unmount_result);
+ EXPECT_FALSE(service_->UnmountFileSystem(
+ kWrongExtensionId, kFileSystemId, Service::UNMOUNT_REASON_USER));
ASSERT_EQ(1u, observer.unmounts.size());
EXPECT_EQ(base::File::FILE_ERROR_NOT_FOUND, observer.unmounts[0].error());
- ASSERT_EQ(
- 1u,
- file_system_provider_service_->GetProvidedFileSystemInfoList().size());
+ ASSERT_EQ(1u, service_->GetProvidedFileSystemInfoList().size());
std::vector<ProvidedFileSystemInfo> file_system_info_list =
- file_system_provider_service_->GetProvidedFileSystemInfoList();
+ service_->GetProvidedFileSystemInfoList();
ASSERT_EQ(1u, file_system_info_list.size());
- file_system_provider_service_->RemoveObserver(&observer);
+ service_->RemoveObserver(&observer);
}
TEST_F(FileSystemProviderServiceTest, RestoreFileSystem_OnExtensionLoad) {
- LoggingObserver observer;
- file_system_provider_service_->AddObserver(&observer);
-
// Create a fake entry in the preferences.
RememberFakeFileSystem(
profile_.get(), kExtensionId, kFileSystemId, kDisplayName);
+ // Create a new service instance in order to load remembered file systems
+ // from preferences.
+ scoped_ptr<Service> new_service(
+ new Service(profile_.get(), extension_registry_.get()));
+ LoggingObserver observer;
+ new_service->AddObserver(&observer);
+
+ new_service->SetFileSystemFactoryForTesting(
+ base::Bind(&FakeProvidedFileSystem::Create));
+
EXPECT_EQ(0u, observer.mounts.size());
// Directly call the observer's method.
- file_system_provider_service_->OnExtensionLoaded(profile_.get(),
- extension_.get());
+ new_service->OnExtensionLoaded(profile_.get(), extension_.get());
ASSERT_EQ(1u, observer.mounts.size());
EXPECT_EQ(base::File::FILE_OK, observer.mounts[0].error());
@@ -328,86 +326,129 @@ TEST_F(FileSystemProviderServiceTest, RestoreFileSystem_OnExtensionLoad) {
observer.mounts[0].file_system_info().file_system_id());
std::vector<ProvidedFileSystemInfo> file_system_info_list =
- file_system_provider_service_->GetProvidedFileSystemInfoList();
+ new_service->GetProvidedFileSystemInfoList();
ASSERT_EQ(1u, file_system_info_list.size());
- file_system_provider_service_->RemoveObserver(&observer);
+ new_service->RemoveObserver(&observer);
}
-TEST_F(FileSystemProviderServiceTest, ForgetFileSystem_OnExtensionUnload) {
+TEST_F(FileSystemProviderServiceTest, RememberFileSystem_OnMount) {
LoggingObserver observer;
- file_system_provider_service_->AddObserver(&observer);
-
- // Create a fake entry in the preferences.
- RememberFakeFileSystem(
- profile_.get(), kExtensionId, kFileSystemId, kDisplayName);
-
- // Directly call the observer's methods.
- file_system_provider_service_->OnExtensionLoaded(profile_.get(),
- extension_.get());
-
- file_system_provider_service_->OnExtensionUnloaded(
- profile_.get(),
- extension_.get(),
- extensions::UnloadedExtensionInfo::REASON_DISABLE);
+ service_->AddObserver(&observer);
+ EXPECT_TRUE(
+ service_->MountFileSystem(kExtensionId, kFileSystemId, kDisplayName));
ASSERT_EQ(1u, observer.mounts.size());
- EXPECT_EQ(base::File::FILE_OK, observer.mounts[0].error());
- ASSERT_EQ(1u, observer.unmounts.size());
- EXPECT_EQ(base::File::FILE_OK, observer.unmounts[0].error());
- TestingPrefServiceSyncable* pref_service = profile_->GetTestingPrefService();
+ TestingPrefServiceSyncable* const pref_service =
+ profile_->GetTestingPrefService();
ASSERT_TRUE(pref_service);
- const base::DictionaryValue* extensions =
+ const base::DictionaryValue* const extensions =
pref_service->GetDictionary(prefs::kFileSystemProviderMounted);
ASSERT_TRUE(extensions);
- const base::ListValue* file_systems;
- EXPECT_FALSE(extensions->GetList(kExtensionId, &file_systems));
+ const base::DictionaryValue* file_systems = NULL;
+ ASSERT_TRUE(extensions->GetDictionaryWithoutPathExpansion(kExtensionId,
+ &file_systems));
+ EXPECT_EQ(1u, file_systems->size());
+
+ const base::Value* file_system_value = NULL;
+ const base::DictionaryValue* file_system = NULL;
+ ASSERT_TRUE(
+ file_systems->GetWithoutPathExpansion(kFileSystemId, &file_system_value));
+ ASSERT_TRUE(file_system_value->GetAsDictionary(&file_system));
+
+ std::string file_system_id;
+ EXPECT_TRUE(file_system->GetStringWithoutPathExpansion(kPrefKeyFileSystemId,
+ &file_system_id));
+ EXPECT_EQ(kFileSystemId, file_system_id);
- file_system_provider_service_->RemoveObserver(&observer);
+ std::string display_name;
+ EXPECT_TRUE(file_system->GetStringWithoutPathExpansion(kPrefKeyDisplayName,
+ &display_name));
+ EXPECT_EQ(kDisplayName, display_name);
+
+ service_->RemoveObserver(&observer);
}
-TEST_F(FileSystemProviderServiceTest, RememberFileSystem_OnShutdown) {
- {
- scoped_ptr<Service> service(
- new Service(profile_.get(), extension_registry_.get()));
- service->SetFileSystemFactoryForTesting(
- base::Bind(&FakeProvidedFileSystem::Create));
+TEST_F(FileSystemProviderServiceTest, RememberFileSystem_OnUnmountOnShutdown) {
+ LoggingObserver observer;
+ service_->AddObserver(&observer);
- LoggingObserver observer;
- service->AddObserver(&observer);
+ TestingPrefServiceSyncable* const pref_service =
+ profile_->GetTestingPrefService();
+ ASSERT_TRUE(pref_service);
- const bool result =
- service->MountFileSystem(kExtensionId, kFileSystemId, kDisplayName);
- EXPECT_TRUE(result);
+ {
+ EXPECT_TRUE(
+ service_->MountFileSystem(kExtensionId, kFileSystemId, kDisplayName));
ASSERT_EQ(1u, observer.mounts.size());
- service->RemoveObserver(&observer);
+ const base::DictionaryValue* extensions =
+ pref_service->GetDictionary(prefs::kFileSystemProviderMounted);
+ ASSERT_TRUE(extensions);
+
+ const base::DictionaryValue* file_systems = NULL;
+ ASSERT_TRUE(extensions->GetDictionaryWithoutPathExpansion(kExtensionId,
+ &file_systems));
+ EXPECT_EQ(1u, file_systems->size());
+ }
+
+ {
+ EXPECT_TRUE(service_->UnmountFileSystem(
+ kExtensionId, kFileSystemId, Service::UNMOUNT_REASON_SHUTDOWN));
+
+ const base::DictionaryValue* const extensions =
+ pref_service->GetDictionary(prefs::kFileSystemProviderMounted);
+ ASSERT_TRUE(extensions);
+
+ const base::DictionaryValue* file_systems = NULL;
+ ASSERT_TRUE(extensions->GetDictionaryWithoutPathExpansion(kExtensionId,
+ &file_systems));
+ EXPECT_EQ(1u, file_systems->size());
}
- TestingPrefServiceSyncable* pref_service = profile_->GetTestingPrefService();
+ service_->RemoveObserver(&observer);
+}
+
+TEST_F(FileSystemProviderServiceTest, RememberFileSystem_OnUnmountByUser) {
+ LoggingObserver observer;
+ service_->AddObserver(&observer);
+
+ TestingPrefServiceSyncable* const pref_service =
+ profile_->GetTestingPrefService();
ASSERT_TRUE(pref_service);
- const base::DictionaryValue* extensions =
- pref_service->GetDictionary(prefs::kFileSystemProviderMounted);
- ASSERT_TRUE(extensions);
+ {
+ EXPECT_TRUE(
+ service_->MountFileSystem(kExtensionId, kFileSystemId, kDisplayName));
+ ASSERT_EQ(1u, observer.mounts.size());
- const base::ListValue* file_systems;
- ASSERT_TRUE(extensions->GetList(kExtensionId, &file_systems));
- ASSERT_EQ(1u, file_systems->GetSize());
+ const base::DictionaryValue* extensions =
+ pref_service->GetDictionary(prefs::kFileSystemProviderMounted);
+ ASSERT_TRUE(extensions);
- const base::DictionaryValue* file_system = NULL;
- ASSERT_TRUE(file_systems->GetDictionary(0, &file_system));
+ const base::DictionaryValue* file_systems = NULL;
+ ASSERT_TRUE(extensions->GetDictionaryWithoutPathExpansion(kExtensionId,
+ &file_systems));
+ EXPECT_EQ(1u, file_systems->size());
+ }
- std::string file_system_id;
- file_system->GetString(kPrefKeyFileSystemId, &file_system_id);
- EXPECT_EQ(kFileSystemId, file_system_id);
+ {
+ EXPECT_TRUE(service_->UnmountFileSystem(
+ kExtensionId, kFileSystemId, Service::UNMOUNT_REASON_USER));
- std::string display_name;
- file_system->GetString(kPrefKeyDisplayName, &display_name);
- EXPECT_EQ(kDisplayName, display_name);
+ const base::DictionaryValue* const extensions =
+ pref_service->GetDictionary(prefs::kFileSystemProviderMounted);
+ ASSERT_TRUE(extensions);
+
+ const base::DictionaryValue* file_systems = NULL;
+ EXPECT_FALSE(extensions->GetDictionaryWithoutPathExpansion(kExtensionId,
+ &file_systems));
+ }
+
+ service_->RemoveObserver(&observer);
}
} // namespace file_system_provider
« no previous file with comments | « chrome/browser/chromeos/file_system_provider/service.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698