Revert "Replace AttachmentStore's StoreAttachments with UploadAttachments."

sync/internal_api/attachments/attachment_service_impl_unittest.cc

 // 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 "sync/internal_api/public/attachments/attachment_service_impl.h"
 
 #include "base/bind.h"
 #include "base/memory/weak_ptr.h"
 #include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
 #include "sync/internal_api/public/attachments/fake_attachment_downloader.h"
 #include "sync/internal_api/public/attachments/fake_attachment_uploader.h"
-#include "testing/gmock/include/gmock/gmock-matchers.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace syncer {
 
 class MockAttachmentStore : public AttachmentStore,
                             public base::SupportsWeakPtr<MockAttachmentStore> {
  public:
   MockAttachmentStore() {}
 
   virtual void Read(const AttachmentIdList& ids,
@@ skipping 166 matching lines @@
                                   delegate));
   }
 
   AttachmentService* attachment_service() { return attachment_service_.get(); }
 
   AttachmentService::GetOrDownloadCallback download_callback() {
     return base::Bind(&AttachmentServiceImplTest::DownloadDone,
                       base::Unretained(this));
   }
 
+  AttachmentService::StoreCallback store_callback() {
+    return base::Bind(&AttachmentServiceImplTest::StoreDone,
+                      base::Unretained(this));
+  }
+
   void DownloadDone(const AttachmentService::GetOrDownloadResult& result,
                     scoped_ptr<AttachmentMap> attachments) {
     download_results_.push_back(result);
     last_download_attachments_ = attachments.Pass();
   }
 
+  void StoreDone(const AttachmentService::StoreResult& result) {
+    store_results_.push_back(result);
+  }
+
   void RunLoop() {
     base::RunLoop run_loop;
     run_loop.RunUntilIdle();
   }
 
   const std::vector<AttachmentService::GetOrDownloadResult>&
   download_results() const {
     return download_results_;
   }
 
   const AttachmentMap& last_download_attachments() const {
     return *last_download_attachments_.get();
   }
 
+  const std::vector<AttachmentService::StoreResult>& store_results() const {
+    return store_results_;
+  }
+
   MockAttachmentStore* store() { return attachment_store_.get(); }
 
   MockAttachmentDownloader* downloader() {
     return attachment_downloader_.get();
   }
 
   MockAttachmentUploader* uploader() {
     return attachment_uploader_.get();
   }
 
   const std::vector<AttachmentId>& on_attachment_uploaded_list() const {
     return on_attachment_uploaded_list_;
   }
 
  private:
   base::MessageLoop message_loop_;
   base::WeakPtr<MockAttachmentStore> attachment_store_;
   base::WeakPtr<MockAttachmentDownloader> attachment_downloader_;
   base::WeakPtr<MockAttachmentUploader> attachment_uploader_;
   scoped_ptr<AttachmentService> attachment_service_;
 
   std::vector<AttachmentService::GetOrDownloadResult> download_results_;
   scoped_ptr<AttachmentMap> last_download_attachments_;
   std::vector<AttachmentId> on_attachment_uploaded_list_;
+
+  std::vector<AttachmentService::StoreResult> store_results_;
 };
 
-TEST_F(AttachmentServiceImplTest, GetStore) {
-  EXPECT_EQ(store(), attachment_service()->GetStore());
-}
-
 TEST_F(AttachmentServiceImplTest, GetOrDownload_EmptyAttachmentList) {
   AttachmentIdList attachment_ids;
   attachment_service()->GetOrDownloadAttachments(attachment_ids,
                                                  download_callback());
   store()->RespondToRead(AttachmentIdSet());
 
   RunLoop();
   EXPECT_EQ(1U, download_results().size());
   EXPECT_EQ(0U, last_download_attachments().size());
 }
@@ skipping 71 matching lines @@
   EXPECT_FALSE(store()->read_ids.empty());
 
   AttachmentIdSet local_attachments;
   store()->RespondToRead(local_attachments);
   RunLoop();
   ASSERT_EQ(1U, download_results().size());
   EXPECT_EQ(AttachmentService::GET_UNSPECIFIED_ERROR, download_results()[0]);
   EXPECT_TRUE(last_download_attachments().empty());
 }
 
-TEST_F(AttachmentServiceImplTest, UploadAttachments_Success) {
-  AttachmentIdSet attachment_ids;
-  const size_t num_attachments = 3;
-  for (unsigned i = 0; i < num_attachments; ++i) {
-    attachment_ids.insert(AttachmentId::Create());
-  }
-  attachment_service()->UploadAttachments(attachment_ids);
+TEST_F(AttachmentServiceImplTest, StoreAttachments_Success) {
+  scoped_refptr<base::RefCountedString> data = new base::RefCountedString();
+  Attachment attachment(Attachment::Create(data));
+  AttachmentList attachments;
+  attachments.push_back(attachment);
+  attachment_service()->StoreAttachments(attachments, store_callback());
+  EXPECT_EQ(1U, store()->write_attachments.size());
+  EXPECT_EQ(1U, uploader()->upload_requests.size());
+
+  store()->RespondToWrite(AttachmentStore::SUCCESS);
+  uploader()->RespondToUpload(attachment.GetId(),
+                              AttachmentUploader::UPLOAD_SUCCESS);
   RunLoop();
-  // See that the service has issued reads for the attachments, but not yet
-  // uploaded anything.
-  EXPECT_EQ(num_attachments, store()->read_ids.size());
-  EXPECT_EQ(0U, uploader()->upload_requests.size());
-  for (unsigned i = 0; i < num_attachments; ++i) {
-    store()->RespondToRead(attachment_ids);
-  }
-
-  RunLoop();
-  EXPECT_EQ(0U, store()->read_ids.size());
-  EXPECT_EQ(num_attachments, uploader()->upload_requests.size());
-  AttachmentIdSet::const_iterator iter = attachment_ids.begin();
-  const AttachmentIdSet::const_iterator end = attachment_ids.end();
-  for (; iter != end; ++iter) {
-    uploader()->RespondToUpload(*iter, AttachmentUploader::UPLOAD_SUCCESS);
-  }
-  RunLoop();
-
-  // See that all the attachments were uploaded.
-  ASSERT_EQ(attachment_ids.size(), on_attachment_uploaded_list().size());
-  for (iter = attachment_ids.begin(); iter != end; ++iter) {
-    EXPECT_THAT(on_attachment_uploaded_list(), testing::Contains(*iter));
-  }
+  ASSERT_EQ(1U, store_results().size());
+  EXPECT_EQ(AttachmentService::STORE_SUCCESS, store_results()[0]);
+  ASSERT_EQ(1U, on_attachment_uploaded_list().size());
+  EXPECT_EQ(attachment.GetId(), on_attachment_uploaded_list()[0]);
 }
 
-TEST_F(AttachmentServiceImplTest, UploadAttachments_Success_NoDelegate) {
+TEST_F(AttachmentServiceImplTest,
+       StoreAttachments_StoreFailsWithUnspecifiedError) {
+  scoped_refptr<base::RefCountedString> data = new base::RefCountedString();
+  Attachment attachment(Attachment::Create(data));
+  AttachmentList attachments;
+  attachments.push_back(attachment);
+  attachment_service()->StoreAttachments(attachments, store_callback());
+  EXPECT_EQ(1U, store()->write_attachments.size());
+  EXPECT_EQ(1U, uploader()->upload_requests.size());
+
+  store()->RespondToWrite(AttachmentStore::UNSPECIFIED_ERROR);
+  uploader()->RespondToUpload(attachment.GetId(),
+                              AttachmentUploader::UPLOAD_SUCCESS);
+  RunLoop();
+  ASSERT_EQ(1U, store_results().size());
+  EXPECT_EQ(AttachmentService::STORE_UNSPECIFIED_ERROR, store_results()[0]);
+  ASSERT_EQ(1U, on_attachment_uploaded_list().size());
+  EXPECT_EQ(attachment.GetId(), on_attachment_uploaded_list()[0]);
+}
+
+TEST_F(AttachmentServiceImplTest,
+       StoreAttachments_UploadFailsWithUnspecifiedError) {
+  scoped_refptr<base::RefCountedString> data = new base::RefCountedString();
+  Attachment attachment(Attachment::Create(data));
+  AttachmentList attachments;
+  attachments.push_back(attachment);
+  attachment_service()->StoreAttachments(attachments, store_callback());
+  EXPECT_EQ(1U, store()->write_attachments.size());
+  EXPECT_EQ(1U, uploader()->upload_requests.size());
+
+  store()->RespondToWrite(AttachmentStore::SUCCESS);
+  uploader()->RespondToUpload(attachment.GetId(),
+                              AttachmentUploader::UPLOAD_UNSPECIFIED_ERROR);
+  RunLoop();
+  ASSERT_EQ(1U, store_results().size());
+  // Even though the upload failed, the Store operation is successful.
+  EXPECT_EQ(AttachmentService::STORE_SUCCESS, store_results()[0]);
+  EXPECT_TRUE(on_attachment_uploaded_list().empty());
+}
+
+TEST_F(AttachmentServiceImplTest, StoreAttachments_NoDelegate) {
   InitializeAttachmentService(make_scoped_ptr(new MockAttachmentUploader()),
                               make_scoped_ptr(new MockAttachmentDownloader()),
                               NULL);  // No delegate.
 
-  AttachmentIdSet attachment_ids;
-  attachment_ids.insert(AttachmentId::Create());
-  attachment_service()->UploadAttachments(attachment_ids);
-  RunLoop();
-  EXPECT_EQ(1U, store()->read_ids.size());
-  EXPECT_EQ(0U, uploader()->upload_requests.size());
-  store()->RespondToRead(attachment_ids);
-  RunLoop();
-  EXPECT_EQ(0U, store()->read_ids.size());
+  scoped_refptr<base::RefCountedString> data = new base::RefCountedString();
+  Attachment attachment(Attachment::Create(data));
+  AttachmentList attachments;
+  attachments.push_back(attachment);
+  attachment_service()->StoreAttachments(attachments, store_callback());
+  EXPECT_EQ(1U, store()->write_attachments.size());
   EXPECT_EQ(1U, uploader()->upload_requests.size());
-  uploader()->RespondToUpload(*attachment_ids.begin(),
+
+  store()->RespondToWrite(AttachmentStore::SUCCESS);
+  uploader()->RespondToUpload(attachment.GetId(),
                               AttachmentUploader::UPLOAD_SUCCESS);
   RunLoop();
-  ASSERT_TRUE(on_attachment_uploaded_list().empty());
+  ASSERT_EQ(1U, store_results().size());
+  EXPECT_EQ(AttachmentService::STORE_SUCCESS, store_results()[0]);
+  EXPECT_TRUE(on_attachment_uploaded_list().empty());
 }
 
-TEST_F(AttachmentServiceImplTest, UploadAttachments_SomeMissingFromStore) {
-  AttachmentIdSet attachment_ids;
-  attachment_ids.insert(AttachmentId::Create());
-  attachment_ids.insert(AttachmentId::Create());
-
-  attachment_service()->UploadAttachments(attachment_ids);
-  RunLoop();
-  EXPECT_EQ(2U, store()->read_ids.size());
-  EXPECT_EQ(0U, uploader()->upload_requests.size());
-  store()->RespondToRead(attachment_ids);
-  EXPECT_EQ(1U, store()->read_ids.size());
-  // Not found!
-  store()->RespondToRead(AttachmentIdSet());
-  EXPECT_EQ(0U, store()->read_ids.size());
-  RunLoop();
-
-  // One attachment went missing so we should see only one upload request.
-  EXPECT_EQ(1U, uploader()->upload_requests.size());
-  uploader()->RespondToUpload(uploader()->upload_requests.begin()->first,
-                              AttachmentUploader::UPLOAD_SUCCESS);
-  RunLoop();
-
-  // See that the delegate was called for only one.
-  ASSERT_EQ(1U, on_attachment_uploaded_list().size());
-}
-
-TEST_F(AttachmentServiceImplTest, UploadAttachments_AllMissingFromStore) {
-  AttachmentIdSet attachment_ids;
-  attachment_ids.insert(AttachmentId::Create());
-  attachment_ids.insert(AttachmentId::Create());
-
-  attachment_service()->UploadAttachments(attachment_ids);
-  RunLoop();
-  EXPECT_EQ(2U, store()->read_ids.size());
-  EXPECT_EQ(0U, uploader()->upload_requests.size());
-  // None found!
-  store()->RespondToRead(AttachmentIdSet());
-  store()->RespondToRead(AttachmentIdSet());
-  EXPECT_EQ(0U, store()->read_ids.size());
-  RunLoop();
-
-  // Nothing uploaded.
-  EXPECT_EQ(0U, uploader()->upload_requests.size());
-  RunLoop();
-
-  // See that the delegate was never called.
-  ASSERT_EQ(0U, on_attachment_uploaded_list().size());
-}
-
-TEST_F(AttachmentServiceImplTest, UploadAttachments_NoUploader) {
+TEST_F(AttachmentServiceImplTest, StoreAttachments_NoUploader) {
+  // No uploader.
   InitializeAttachmentService(make_scoped_ptr<MockAttachmentUploader>(NULL),
                               make_scoped_ptr(new MockAttachmentDownloader()),
                               this);
 
-  AttachmentIdSet attachment_ids;
-  attachment_ids.insert(AttachmentId::Create());
-  attachment_service()->UploadAttachments(attachment_ids);
+  scoped_refptr<base::RefCountedString> data = new base::RefCountedString();
+  Attachment attachment(Attachment::Create(data));
+  AttachmentList attachments;
+  attachments.push_back(attachment);
+  attachment_service()->StoreAttachments(attachments, store_callback());
+  EXPECT_EQ(1U, store()->write_attachments.size());
+
+  store()->RespondToWrite(AttachmentStore::SUCCESS);
   RunLoop();
-  EXPECT_EQ(0U, store()->read_ids.size());
-  ASSERT_EQ(0U, on_attachment_uploaded_list().size());
-}
-
-// Upload three attachments.  For one of them, server responds with error.
-TEST_F(AttachmentServiceImplTest, UploadAttachments_OneUploadFails) {
-  AttachmentIdSet attachment_ids;
-  attachment_ids.insert(AttachmentId::Create());
-  attachment_ids.insert(AttachmentId::Create());
-  attachment_ids.insert(AttachmentId::Create());
-
-  attachment_service()->UploadAttachments(attachment_ids);
-  RunLoop();
-  EXPECT_EQ(3U, store()->read_ids.size());
-  EXPECT_EQ(0U, uploader()->upload_requests.size());
-
-  // All attachments found.
-  store()->RespondToRead(attachment_ids);
-  store()->RespondToRead(attachment_ids);
-  store()->RespondToRead(attachment_ids);
-  RunLoop();
-
-  EXPECT_EQ(3U, uploader()->upload_requests.size());
-  uploader()->RespondToUpload(uploader()->upload_requests.begin()->first,
-                              AttachmentUploader::UPLOAD_SUCCESS);
-  uploader()->RespondToUpload(uploader()->upload_requests.begin()->first,
-                              AttachmentUploader::UPLOAD_UNSPECIFIED_ERROR);
-  uploader()->RespondToUpload(uploader()->upload_requests.begin()->first,
-                              AttachmentUploader::UPLOAD_SUCCESS);
-  EXPECT_EQ(0U, uploader()->upload_requests.size());
-  RunLoop();
-
-  EXPECT_EQ(2U, on_attachment_uploaded_list().size());
+  ASSERT_EQ(1U, store_results().size());
+  EXPECT_EQ(AttachmentService::STORE_SUCCESS, store_results()[0]);
+  EXPECT_TRUE(on_attachment_uploaded_list().empty());
 }
 
 }  // namespace syncer