#include "sync/engine/process_commit_response_command.h"
#include <vector>
#include "base/location.h"
#include "base/stringprintf.h"
#include "sync/protocol/bookmark_specifics.pb.h"
#include "sync/protocol/sync.pb.h"
#include "sync/sessions/sync_session.h"
#include "sync/syncable/entry.h"
#include "sync/syncable/mutable_entry.h"
#include "sync/syncable/read_transaction.h"
#include "sync/syncable/syncable_id.h"
#include "sync/syncable/syncable_proto_util.h"
#include "sync/syncable/write_transaction.h"
#include "sync/test/engine/fake_model_worker.h"
#include "sync/test/engine/syncer_command_test.h"
#include "sync/test/engine/test_id_factory.h"
#include "testing/gtest/include/gtest/gtest.h"
using std::string;
using sync_pb::ClientToServerMessage;
using sync_pb::CommitResponse;
namespace syncer {
using sessions::SyncSession;
using syncable::BASE_VERSION;
using syncable::Entry;
using syncable::IS_DIR;
using syncable::IS_UNSYNCED;
using syncable::Id;
using syncable::MutableEntry;
using syncable::NON_UNIQUE_NAME;
using syncable::UNITTEST;
using syncable::WriteTransaction;
class ProcessCommitResponseCommandTest : public SyncerCommandTest {
public:
virtual void SetUp() {
workers()->clear();
mutable_routing_info()->clear();
workers()->push_back(
make_scoped_refptr(new FakeModelWorker(GROUP_DB)));
workers()->push_back(
make_scoped_refptr(new FakeModelWorker(GROUP_UI)));
(*mutable_routing_info())[BOOKMARKS] = GROUP_UI;
(*mutable_routing_info())[PREFERENCES] = GROUP_UI;
(*mutable_routing_info())[AUTOFILL] = GROUP_DB;
SyncerCommandTest::SetUp();
}
protected:
ProcessCommitResponseCommandTest()
: next_old_revision_(1),
next_new_revision_(4000),
next_server_position_(10000) {
}
void CheckEntry(Entry* e, const std::string& name,
ModelType model_type, const Id& parent_id) {
EXPECT_TRUE(e->good());
ASSERT_EQ(name, e->Get(NON_UNIQUE_NAME));
ASSERT_EQ(model_type, e->GetModelType());
ASSERT_EQ(parent_id, e->Get(syncable::PARENT_ID));
ASSERT_LT(0, e->Get(BASE_VERSION))
<< "Item should have a valid (positive) server base revision";
}
void CreateUnsyncedItem(const Id& item_id,
const Id& parent_id,
const string& name,
bool is_folder,
ModelType model_type,
int64* metahandle_out) {
WriteTransaction trans(FROM_HERE, UNITTEST, directory());
Id predecessor_id;
ASSERT_TRUE(
directory()->GetLastChildIdForTest(&trans, parent_id, &predecessor_id));
MutableEntry entry(&trans, syncable::CREATE, parent_id, name);
ASSERT_TRUE(entry.good());
entry.Put(syncable::ID, item_id);
entry.Put(syncable::BASE_VERSION,
item_id.ServerKnows() ? next_old_revision_++ : 0);
entry.Put(syncable::IS_UNSYNCED, true);
entry.Put(syncable::IS_DIR, is_folder);
entry.Put(syncable::IS_DEL, false);
entry.Put(syncable::PARENT_ID, parent_id);
entry.PutPredecessor(predecessor_id);
sync_pb::EntitySpecifics default_specifics;
AddDefaultFieldValue(model_type, &default_specifics);
entry.Put(syncable::SPECIFICS, default_specifics);
if (item_id.ServerKnows()) {
entry.Put(syncable::SERVER_SPECIFICS, default_specifics);
entry.Put(syncable::SERVER_IS_DIR, is_folder);
entry.Put(syncable::SERVER_PARENT_ID, parent_id);
entry.Put(syncable::SERVER_IS_DEL, false);
}
if (metahandle_out)
*metahandle_out = entry.Get(syncable::META_HANDLE);
}
void CreateUnprocessedCommitResult(
const Id& item_id,
const Id& parent_id,
const string& name,
ModelType model_type,
sessions::OrderedCommitSet *commit_set,
sync_pb::ClientToServerMessage *commit,
sync_pb::ClientToServerResponse *response) {
bool is_folder = true;
int64 metahandle = 0;
CreateUnsyncedItem(item_id, parent_id, name, is_folder, model_type,
&metahandle);
commit_set->AddCommitItem(metahandle, item_id, model_type);
WriteTransaction trans(FROM_HERE, UNITTEST, directory());
MutableEntry entry(&trans, syncable::GET_BY_ID, item_id);
ASSERT_TRUE(entry.good());
entry.Put(syncable::SYNCING, true);
commit->set_message_contents(ClientToServerMessage::COMMIT);
sync_pb::SyncEntity* entity = commit->mutable_commit()->add_entries();
entity->set_non_unique_name(name);
entity->set_folder(is_folder);
entity->set_parent_id_string(SyncableIdToProto(parent_id));
entity->set_version(entry.Get(syncable::BASE_VERSION));
entity->mutable_specifics()->CopyFrom(entry.Get(syncable::SPECIFICS));
entity->set_id_string(SyncableIdToProto(item_id));
response->set_error_code(sync_pb::SyncEnums::SUCCESS);
sync_pb::CommitResponse_EntryResponse* entry_response =
response->mutable_commit()->add_entryresponse();
entry_response->set_response_type(CommitResponse::SUCCESS);
entry_response->set_name("Garbage.");
entry_response->set_non_unique_name(entity->name());
if (item_id.ServerKnows())
entry_response->set_id_string(entity->id_string());
else
entry_response->set_id_string(id_factory_.NewServerId().GetServerId());
entry_response->set_version(next_new_revision_++);
entry_response->set_position_in_parent(next_server_position_++);
entry_response->set_parent_id_string(entity->parent_id_string());
for (int i = 0; i < commit->commit().entries_size(); ++i) {
if (commit->commit().entries(i).id_string() ==
entity->parent_id_string()) {
entry_response->set_parent_id_string(
response->commit().entryresponse(i).id_string());
}
}
}
void SetLastErrorCode(sync_pb::CommitResponse::ResponseType error_code,
sync_pb::ClientToServerResponse* response) {
sync_pb::CommitResponse_EntryResponse* entry_response =
response->mutable_commit()->mutable_entryresponse(
response->mutable_commit()->entryresponse_size() - 1);
entry_response->set_response_type(error_code);
}
TestIdFactory id_factory_;
private:
int64 next_old_revision_;
int64 next_new_revision_;
int64 next_server_position_;
DISALLOW_COPY_AND_ASSIGN(ProcessCommitResponseCommandTest);
};
TEST_F(ProcessCommitResponseCommandTest, MultipleCommitIdProjections) {
sessions::OrderedCommitSet commit_set(session()->routing_info());
sync_pb::ClientToServerMessage request;
sync_pb::ClientToServerResponse response;
Id bookmark_folder_id = id_factory_.NewLocalId();
Id bookmark_id1 = id_factory_.NewLocalId();
Id bookmark_id2 = id_factory_.NewLocalId();
Id pref_id1 = id_factory_.NewLocalId(), pref_id2 = id_factory_.NewLocalId();
Id autofill_id1 = id_factory_.NewLocalId();
Id autofill_id2 = id_factory_.NewLocalId();
CreateUnprocessedCommitResult(bookmark_folder_id, id_factory_.root(),
"A bookmark folder", BOOKMARKS,
&commit_set, &request, &response);
CreateUnprocessedCommitResult(bookmark_id1, bookmark_folder_id,
"bookmark 1", BOOKMARKS,
&commit_set, &request, &response);
CreateUnprocessedCommitResult(bookmark_id2, bookmark_folder_id,
"bookmark 2", BOOKMARKS,
&commit_set, &request, &response);
CreateUnprocessedCommitResult(pref_id1, id_factory_.root(),
"Pref 1", PREFERENCES,
&commit_set, &request, &response);
CreateUnprocessedCommitResult(pref_id2, id_factory_.root(),
"Pref 2", PREFERENCES,
&commit_set, &request, &response);
CreateUnprocessedCommitResult(autofill_id1, id_factory_.root(),
"Autofill 1", AUTOFILL,
&commit_set, &request, &response);
CreateUnprocessedCommitResult(autofill_id2, id_factory_.root(),
"Autofill 2", AUTOFILL,
&commit_set, &request, &response);
ProcessCommitResponseCommand command(commit_set, request, response);
ExpectGroupsToChange(command, GROUP_UI, GROUP_DB);
command.ExecuteImpl(session());
syncable::ReadTransaction trans(FROM_HERE, directory());
Id new_fid;
ASSERT_TRUE(directory()->GetFirstChildId(
&trans, id_factory_.root(), &new_fid));
ASSERT_FALSE(new_fid.IsRoot());
EXPECT_TRUE(new_fid.ServerKnows());
EXPECT_FALSE(bookmark_folder_id.ServerKnows());
EXPECT_FALSE(new_fid == bookmark_folder_id);
Entry b_folder(&trans, syncable::GET_BY_ID, new_fid);
ASSERT_TRUE(b_folder.good());
ASSERT_EQ("A bookmark folder", b_folder.Get(NON_UNIQUE_NAME))
<< "Name of bookmark folder should not change.";
ASSERT_LT(0, b_folder.Get(BASE_VERSION))
<< "Bookmark folder should have a valid (positive) server base revision";
Id cid;
ASSERT_TRUE(directory()->GetFirstChildId(&trans, new_fid, &cid));
Entry b1(&trans, syncable::GET_BY_ID, cid);
Entry b2(&trans, syncable::GET_BY_ID, b1.Get(syncable::NEXT_ID));
CheckEntry(&b1, "bookmark 1", BOOKMARKS, new_fid);
CheckEntry(&b2, "bookmark 2", BOOKMARKS, new_fid);
ASSERT_TRUE(b2.Get(syncable::NEXT_ID).IsRoot());
Entry p1(&trans, syncable::GET_BY_ID, b_folder.Get(syncable::NEXT_ID));
Entry p2(&trans, syncable::GET_BY_ID, p1.Get(syncable::NEXT_ID));
CheckEntry(&p1, "Pref 1", PREFERENCES, id_factory_.root());
CheckEntry(&p2, "Pref 2", PREFERENCES, id_factory_.root());
Entry a1(&trans, syncable::GET_BY_ID, p2.Get(syncable::NEXT_ID));
Entry a2(&trans, syncable::GET_BY_ID, a1.Get(syncable::NEXT_ID));
CheckEntry(&a1, "Autofill 1", AUTOFILL, id_factory_.root());
CheckEntry(&a2, "Autofill 2", AUTOFILL, id_factory_.root());
ASSERT_TRUE(a2.Get(syncable::NEXT_ID).IsRoot());
}
TEST_F(ProcessCommitResponseCommandTest, NewFolderCommitKeepsChildOrder) {
sessions::OrderedCommitSet commit_set(session()->routing_info());
sync_pb::ClientToServerMessage request;
sync_pb::ClientToServerResponse response;
Id folder_id = id_factory_.NewLocalId();
CreateUnprocessedCommitResult(folder_id, id_factory_.root(), "A",
BOOKMARKS,
&commit_set, &request, &response);
{
syncable::ReadTransaction trans(FROM_HERE, directory());
Id child_id;
ASSERT_TRUE(directory()->GetFirstChildId(
&trans, id_factory_.root(), &child_id));
ASSERT_EQ(folder_id, child_id);
}
int batch_size = 25;
int i = 0;
for (; i < batch_size; ++i) {
Id id = (i % 4 < 2) ? id_factory_.NewLocalId() : id_factory_.NewServerId();
CreateUnprocessedCommitResult(
id, folder_id, base::StringPrintf("Item %d", i), BOOKMARKS,
&commit_set, &request, &response);
}
for (; i < 2*batch_size; ++i) {
Id id = (i % 4 < 2) ? id_factory_.NewLocalId() : id_factory_.NewServerId();
CreateUnsyncedItem(id, folder_id, base::StringPrintf("Item %d", i),
false, BOOKMARKS, NULL);
}
ProcessCommitResponseCommand command(commit_set, request, response);
ExpectGroupToChange(command, GROUP_UI);
command.ExecuteImpl(session());
syncable::ReadTransaction trans(FROM_HERE, directory());
Id new_fid;
ASSERT_TRUE(directory()->GetFirstChildId(
&trans, id_factory_.root(), &new_fid));
ASSERT_FALSE(new_fid.IsRoot());
EXPECT_TRUE(new_fid.ServerKnows());
EXPECT_FALSE(folder_id.ServerKnows());
EXPECT_TRUE(new_fid != folder_id);
Entry parent(&trans, syncable::GET_BY_ID, new_fid);
ASSERT_TRUE(parent.good());
ASSERT_EQ("A", parent.Get(NON_UNIQUE_NAME))
<< "Name of parent folder should not change.";
ASSERT_LT(0, parent.Get(BASE_VERSION))
<< "Parent should have a valid (positive) server base revision";
Id cid;
ASSERT_TRUE(directory()->GetFirstChildId(&trans, new_fid, &cid));
int child_count = 0;
while (!cid.IsRoot()) {
SCOPED_TRACE(::testing::Message("Examining item #") << child_count);
Entry c(&trans, syncable::GET_BY_ID, cid);
DCHECK(c.good());
ASSERT_EQ(base::StringPrintf("Item %d", child_count),
c.Get(NON_UNIQUE_NAME));
ASSERT_EQ(new_fid, c.Get(syncable::PARENT_ID));
if (child_count < batch_size) {
ASSERT_FALSE(c.Get(IS_UNSYNCED)) << "Item should be committed";
ASSERT_TRUE(cid.ServerKnows());
ASSERT_LT(0, c.Get(BASE_VERSION));
} else {
ASSERT_TRUE(c.Get(IS_UNSYNCED)) << "Item should be uncommitted";
if (child_count % 4 < 2) {
ASSERT_FALSE(cid.ServerKnows());
ASSERT_GE(0, c.Get(BASE_VERSION));
} else {
ASSERT_TRUE(cid.ServerKnows());
ASSERT_LT(0, c.Get(BASE_VERSION));
}
}
cid = c.Get(syncable::NEXT_ID);
child_count++;
}
ASSERT_EQ(batch_size*2, child_count)
<< "Too few or too many children in parent folder after commit.";
}
enum {
TEST_PARAM_BOOKMARK_ENABLE_BIT,
TEST_PARAM_AUTOFILL_ENABLE_BIT,
TEST_PARAM_BIT_COUNT
};
class MixedResult :
public ProcessCommitResponseCommandTest,
public ::testing::WithParamInterface<int> {
protected:
bool ShouldFailBookmarkCommit() {
return (GetParam() & (1 << TEST_PARAM_BOOKMARK_ENABLE_BIT)) == 0;
}
bool ShouldFailAutofillCommit() {
return (GetParam() & (1 << TEST_PARAM_AUTOFILL_ENABLE_BIT)) == 0;
}
};
INSTANTIATE_TEST_CASE_P(ProcessCommitResponse,
MixedResult,
testing::Range(0, 1 << TEST_PARAM_BIT_COUNT));
TEST_P(MixedResult, ExtensionActivity) {
sessions::OrderedCommitSet commit_set(session()->routing_info());
sync_pb::ClientToServerMessage request;
sync_pb::ClientToServerResponse response;
EXPECT_NE(routing_info().find(BOOKMARKS)->second,
routing_info().find(AUTOFILL)->second)
<< "To not be lame, this test requires more than one active group.";
CreateUnprocessedCommitResult(id_factory_.NewServerId(),
id_factory_.root(), "Some bookmark", BOOKMARKS,
&commit_set, &request, &response);
if (ShouldFailBookmarkCommit())
SetLastErrorCode(CommitResponse::TRANSIENT_ERROR, &response);
CreateUnprocessedCommitResult(id_factory_.NewServerId(),
id_factory_.root(), "Some autofill", AUTOFILL,
&commit_set, &request, &response);
if (ShouldFailAutofillCommit())
SetLastErrorCode(CommitResponse::TRANSIENT_ERROR, &response);
{
ExtensionsActivityMonitor::Records* records =
session()->mutable_extensions_activity();
(*records)["ABC"].extension_id = "ABC";
(*records)["ABC"].bookmark_write_count = 2049U;
(*records)["xyz"].extension_id = "xyz";
(*records)["xyz"].bookmark_write_count = 4U;
}
ProcessCommitResponseCommand command(commit_set, request, response);
command.ExecuteImpl(session());
ExpectGroupsToChange(command, GROUP_UI, GROUP_DB);
ExtensionsActivityMonitor::Records final_monitor_records;
context()->extensions_monitor()->GetAndClearRecords(&final_monitor_records);
if (ShouldFailBookmarkCommit()) {
ASSERT_EQ(2U, final_monitor_records.size())
<< "Should restore records after unsuccessful bookmark commit.";
EXPECT_EQ("ABC", final_monitor_records["ABC"].extension_id);
EXPECT_EQ("xyz", final_monitor_records["xyz"].extension_id);
EXPECT_EQ(2049U, final_monitor_records["ABC"].bookmark_write_count);
EXPECT_EQ(4U, final_monitor_records["xyz"].bookmark_write_count);
} else {
EXPECT_TRUE(final_monitor_records.empty())
<< "Should not restore records after successful bookmark commit.";
}
}
}