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

Side by Side Diff: chrome/browser/sync/sessions/status_controller_unittest.cc

Issue 9107055: Remove single direction conflict set code (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Pre-emptive rebase Created 8 years, 11 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/sync/sessions/sync_session.h" 5 #include "chrome/browser/sync/sessions/sync_session.h"
6 #include "chrome/browser/sync/test/engine/test_id_factory.h" 6 #include "chrome/browser/sync/test/engine/test_id_factory.h"
7 #include "testing/gtest/include/gtest/gtest.h" 7 #include "testing/gtest/include/gtest/gtest.h"
8 8
9 namespace browser_sync { 9 namespace browser_sync {
10 namespace sessions { 10 namespace sessions {
(...skipping 60 matching lines...) Expand 10 before | Expand all | Expand 10 after
71 status.set_unsynced_handles(v); 71 status.set_unsynced_handles(v);
72 EXPECT_TRUE(status.TestAndClearIsDirty()); 72 EXPECT_TRUE(status.TestAndClearIsDirty());
73 std::vector<int64> v2; 73 std::vector<int64> v2;
74 v2.push_back(1); 74 v2.push_back(1);
75 status.set_unsynced_handles(v2); 75 status.set_unsynced_handles(v2);
76 EXPECT_FALSE(status.TestAndClearIsDirty()); // Test for deep comparison. 76 EXPECT_FALSE(status.TestAndClearIsDirty()); // Test for deep comparison.
77 } 77 }
78 78
79 TEST_F(StatusControllerTest, StaysClean) { 79 TEST_F(StatusControllerTest, StaysClean) {
80 StatusController status(routes_); 80 StatusController status(routes_);
81 status.update_conflict_sets_built(true);
82 EXPECT_FALSE(status.TestAndClearIsDirty());
83 status.update_conflicts_resolved(true); 81 status.update_conflicts_resolved(true);
84 EXPECT_FALSE(status.TestAndClearIsDirty()); 82 EXPECT_FALSE(status.TestAndClearIsDirty());
85 83
86 status.set_items_committed(); 84 status.set_items_committed();
87 EXPECT_FALSE(status.TestAndClearIsDirty()); 85 EXPECT_FALSE(status.TestAndClearIsDirty());
88 86
89 OrderedCommitSet commits(routes_); 87 OrderedCommitSet commits(routes_);
90 commits.AddCommitItem(0, syncable::Id(), syncable::BOOKMARKS); 88 commits.AddCommitItem(0, syncable::Id(), syncable::BOOKMARKS);
91 status.set_commit_set(commits); 89 status.set_commit_set(commits);
92 EXPECT_FALSE(status.TestAndClearIsDirty()); 90 EXPECT_FALSE(status.TestAndClearIsDirty());
(...skipping 25 matching lines...) Expand all
118 EXPECT_EQ(13, status.num_server_changes_remaining()); 116 EXPECT_EQ(13, status.num_server_changes_remaining());
119 117
120 EXPECT_FALSE(status.syncer_status().invalid_store); 118 EXPECT_FALSE(status.syncer_status().invalid_store);
121 status.set_invalid_store(true); 119 status.set_invalid_store(true);
122 EXPECT_TRUE(status.syncer_status().invalid_store); 120 EXPECT_TRUE(status.syncer_status().invalid_store);
123 121
124 EXPECT_FALSE(status.conflicts_resolved()); 122 EXPECT_FALSE(status.conflicts_resolved());
125 status.update_conflicts_resolved(true); 123 status.update_conflicts_resolved(true);
126 EXPECT_TRUE(status.conflicts_resolved()); 124 EXPECT_TRUE(status.conflicts_resolved());
127 125
128 EXPECT_FALSE(status.conflict_sets_built());
129 status.update_conflict_sets_built(true);
130 EXPECT_TRUE(status.conflict_sets_built());
131
132 status.set_last_download_updates_result(SYNCER_OK); 126 status.set_last_download_updates_result(SYNCER_OK);
133 EXPECT_EQ(SYNCER_OK, status.error().last_download_updates_result); 127 EXPECT_EQ(SYNCER_OK, status.error().last_download_updates_result);
134 128
135 status.set_last_post_commit_result(SYNC_AUTH_ERROR); 129 status.set_last_post_commit_result(SYNC_AUTH_ERROR);
136 EXPECT_EQ(SYNC_AUTH_ERROR, status.error().last_post_commit_result); 130 EXPECT_EQ(SYNC_AUTH_ERROR, status.error().last_post_commit_result);
137 131
138 status.set_last_process_commit_response_result(SYNC_SERVER_ERROR); 132 status.set_last_process_commit_response_result(SYNC_SERVER_ERROR);
139 EXPECT_EQ(SYNC_SERVER_ERROR, 133 EXPECT_EQ(SYNC_SERVER_ERROR,
140 status.error().last_process_commit_response_result); 134 status.error().last_process_commit_response_result);
141 135
(...skipping 76 matching lines...) Expand 10 before | Expand all | Expand 10 after
218 status.num_server_changes_remaining(); 212 status.num_server_changes_remaining();
219 status.commit_ids(); 213 status.commit_ids();
220 status.HasBookmarkCommitActivity(); 214 status.HasBookmarkCommitActivity();
221 status.download_updates_succeeded(); 215 status.download_updates_succeeded();
222 status.ServerSaysNothingMoreToDownload(); 216 status.ServerSaysNothingMoreToDownload();
223 status.group_restriction(); 217 status.group_restriction();
224 } 218 }
225 219
226 } // namespace sessions 220 } // namespace sessions
227 } // namespace browser_sync 221 } // namespace browser_sync
OLDNEW
« no previous file with comments | « chrome/browser/sync/sessions/status_controller.cc ('k') | chrome/browser/sync/sessions/sync_session.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698