diff options
author | pilgrim@chromium.org <pilgrim@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-04-12 05:10:24 +0000 |
---|---|---|
committer | pilgrim@chromium.org <pilgrim@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-04-12 05:10:24 +0000 |
commit | a507b82e40444679eff8d9367a4f5192f12d99d9 (patch) | |
tree | 6fd0bb43d40a45a8dd433a0694d2847045c97a49 /content/browser/appcache/appcache_disk_cache_unittest.cc | |
parent | 87d95a61d0ecbe5a52345502f340c48bf25a34a0 (diff) | |
download | chromium_src-a507b82e40444679eff8d9367a4f5192f12d99d9.zip chromium_src-a507b82e40444679eff8d9367a4f5192f12d99d9.tar.gz chromium_src-a507b82e40444679eff8d9367a4f5192f12d99d9.tar.bz2 |
Move all remaining appcache unittests from webkit/ to content/
BUG=338338
TBR=darin@chromium.org
Review URL: https://codereview.chromium.org/235763002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@263477 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/browser/appcache/appcache_disk_cache_unittest.cc')
-rw-r--r-- | content/browser/appcache/appcache_disk_cache_unittest.cc | 188 |
1 files changed, 188 insertions, 0 deletions
diff --git a/content/browser/appcache/appcache_disk_cache_unittest.cc b/content/browser/appcache/appcache_disk_cache_unittest.cc new file mode 100644 index 0000000..5e1e083 --- /dev/null +++ b/content/browser/appcache/appcache_disk_cache_unittest.cc @@ -0,0 +1,188 @@ +// Copyright 2014 The Chromium Authors. All rights reserved. +// Use of this source code is governed by a BSD-style license that can be +// found in the LICENSE file. + +#include "base/bind.h" +#include "base/file_util.h" +#include "base/files/scoped_temp_dir.h" +#include "base/message_loop/message_loop_proxy.h" +#include "base/run_loop.h" +#include "net/base/io_buffer.h" +#include "net/base/net_errors.h" +#include "testing/gtest/include/gtest/gtest.h" +#include "webkit/browser/appcache/appcache_disk_cache.h" + +using appcache::AppCacheDiskCache; + +namespace content { + +class AppCacheDiskCacheTest : public testing::Test { + public: + AppCacheDiskCacheTest() {} + + virtual void SetUp() OVERRIDE { + // Use the current thread for the DiskCache's cache_thread. + message_loop_.reset(new base::MessageLoopForIO()); + cache_thread_ = base::MessageLoopProxy::current(); + ASSERT_TRUE(directory_.CreateUniqueTempDir()); + completion_callback_ = base::Bind( + &AppCacheDiskCacheTest::OnComplete, + base::Unretained(this)); + } + + virtual void TearDown() OVERRIDE { + base::RunLoop().RunUntilIdle(); + message_loop_.reset(NULL); + } + + void FlushCacheTasks() { + base::RunLoop().RunUntilIdle(); + } + + void OnComplete(int err) { + completion_results_.push_back(err); + } + + base::ScopedTempDir directory_; + scoped_ptr<base::MessageLoop> message_loop_; + scoped_refptr<base::MessageLoopProxy> cache_thread_; + net::CompletionCallback completion_callback_; + std::vector<int> completion_results_; + + static const int k10MBytes = 10 * 1024 * 1024; +}; + +TEST_F(AppCacheDiskCacheTest, DisablePriorToInitCompletion) { + AppCacheDiskCache::Entry* entry = NULL; + + // Create an instance and start it initializing, queue up + // one of each kind of "entry" function. + scoped_ptr<AppCacheDiskCache> disk_cache(new AppCacheDiskCache); + EXPECT_FALSE(disk_cache->is_disabled()); + disk_cache->InitWithDiskBackend( + directory_.path(), k10MBytes, false, cache_thread_, + completion_callback_); + disk_cache->CreateEntry(1, &entry, completion_callback_); + disk_cache->OpenEntry(2, &entry, completion_callback_); + disk_cache->DoomEntry(3, completion_callback_); + + // Pull the plug on all that. + EXPECT_FALSE(disk_cache->is_disabled()); + disk_cache->Disable(); + EXPECT_TRUE(disk_cache->is_disabled()); + + FlushCacheTasks(); + + EXPECT_EQ(NULL, entry); + EXPECT_EQ(4u, completion_results_.size()); + for (std::vector<int>::const_iterator iter = completion_results_.begin(); + iter < completion_results_.end(); ++iter) { + EXPECT_EQ(net::ERR_ABORTED, *iter); + } + + // Ensure the directory can be deleted at this point. + EXPECT_TRUE(base::DirectoryExists(directory_.path())); + EXPECT_FALSE(base::IsDirectoryEmpty(directory_.path())); + EXPECT_TRUE(base::DeleteFile(directory_.path(), true)); + EXPECT_FALSE(base::DirectoryExists(directory_.path())); +} + +TEST_F(AppCacheDiskCacheTest, DisableAfterInitted) { + // Create an instance and let it fully init. + scoped_ptr<AppCacheDiskCache> disk_cache(new AppCacheDiskCache); + EXPECT_FALSE(disk_cache->is_disabled()); + disk_cache->InitWithDiskBackend( + directory_.path(), k10MBytes, false, cache_thread_, + completion_callback_); + FlushCacheTasks(); + EXPECT_EQ(1u, completion_results_.size()); + EXPECT_EQ(net::OK, completion_results_[0]); + + // Pull the plug + disk_cache->Disable(); + FlushCacheTasks(); + + // Ensure the directory can be deleted at this point. + EXPECT_TRUE(base::DirectoryExists(directory_.path())); + EXPECT_FALSE(base::IsDirectoryEmpty(directory_.path())); + EXPECT_TRUE(base::DeleteFile(directory_.path(), true)); + EXPECT_FALSE(base::DirectoryExists(directory_.path())); + + // Methods should return immediately when disabled and not invoke + // the callback at all. + AppCacheDiskCache::Entry* entry = NULL; + completion_results_.clear(); + EXPECT_EQ(net::ERR_ABORTED, + disk_cache->CreateEntry(1, &entry, completion_callback_)); + EXPECT_EQ(net::ERR_ABORTED, + disk_cache->OpenEntry(2, &entry, completion_callback_)); + EXPECT_EQ(net::ERR_ABORTED, + disk_cache->DoomEntry(3, completion_callback_)); + FlushCacheTasks(); + EXPECT_TRUE(completion_results_.empty()); +} + +// Flaky on Android: http://crbug.com/339534 +TEST_F(AppCacheDiskCacheTest, DISABLED_DisableWithEntriesOpen) { + // Create an instance and let it fully init. + scoped_ptr<AppCacheDiskCache> disk_cache(new AppCacheDiskCache); + EXPECT_FALSE(disk_cache->is_disabled()); + disk_cache->InitWithDiskBackend( + directory_.path(), k10MBytes, false, cache_thread_, + completion_callback_); + FlushCacheTasks(); + EXPECT_EQ(1u, completion_results_.size()); + EXPECT_EQ(net::OK, completion_results_[0]); + + // Note: We don't have detailed expectations of the DiskCache + // operations because on android it's really SimpleCache which + // does behave differently. + // + // What matters for the corruption handling and service reinitiazation + // is that the directory can be deleted after the calling Disable() method, + // and we do have expectations about that. + + // Create/open some entries. + AppCacheDiskCache::Entry* entry1 = NULL; + AppCacheDiskCache::Entry* entry2 = NULL; + disk_cache->CreateEntry(1, &entry1, completion_callback_); + disk_cache->CreateEntry(2, &entry2, completion_callback_); + FlushCacheTasks(); + EXPECT_TRUE(entry1); + EXPECT_TRUE(entry2); + + // Write something to one of the entries and flush it. + const char* kData = "Hello"; + const int kDataLen = strlen(kData) + 1; + scoped_refptr<net::IOBuffer> write_buf(new net::WrappedIOBuffer(kData)); + entry1->Write(0, 0, write_buf, kDataLen, completion_callback_); + FlushCacheTasks(); + + // Queue up a read and a write. + scoped_refptr<net::IOBuffer> read_buf = new net::IOBuffer(kDataLen); + entry1->Read(0, 0, read_buf.get(), kDataLen, completion_callback_); + entry2->Write(0, 0, write_buf.get(), kDataLen, completion_callback_); + + // Pull the plug + disk_cache->Disable(); + FlushCacheTasks(); + + // Ensure the directory can be deleted at this point. + EXPECT_TRUE(base::DirectoryExists(directory_.path())); + EXPECT_FALSE(base::IsDirectoryEmpty(directory_.path())); + EXPECT_TRUE(base::DeleteFile(directory_.path(), true)); + EXPECT_FALSE(base::DirectoryExists(directory_.path())); + + disk_cache.reset(NULL); + + // Also, new IO operations should fail immediately. + EXPECT_EQ( + net::ERR_ABORTED, + entry1->Read(0, 0, read_buf.get(), kDataLen, completion_callback_)); + entry1->Close(); + entry2->Close(); + + FlushCacheTasks(); +} + +} // namespace content |