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

Unified Diff: chrome/browser/sync/sessions/status_controller_unittest.cc

Issue 9158004: Detect sync server communication errors (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Update for review comments 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/sync/sessions/status_controller.cc ('k') | chrome/browser/sync/sessions/sync_session.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync/sessions/status_controller_unittest.cc
diff --git a/chrome/browser/sync/sessions/status_controller_unittest.cc b/chrome/browser/sync/sessions/status_controller_unittest.cc
index 90e986004ba08053b21a7f18b13b3b996d660df1..cf720e593b1b5dae9c43e2d501abcb0fdf73fae7 100644
--- a/chrome/browser/sync/sessions/status_controller_unittest.cc
+++ b/chrome/browser/sync/sessions/status_controller_unittest.cc
@@ -130,6 +130,16 @@ TEST_F(StatusControllerTest, ReadYourWrites) {
status.set_syncer_stuck(true);
EXPECT_TRUE(status.syncer_status().syncer_stuck);
+ status.set_last_download_updates_result(SYNCER_OK);
+ EXPECT_EQ(SYNCER_OK, status.error().last_download_updates_result);
+
+ status.set_last_post_commit_result(SYNC_AUTH_ERROR);
+ EXPECT_EQ(SYNC_AUTH_ERROR, status.error().last_post_commit_result);
+
+ status.set_last_process_commit_response_result(SYNC_SERVER_ERROR);
+ EXPECT_EQ(SYNC_SERVER_ERROR,
+ status.error().last_process_commit_response_result);
+
for (int i = 0; i < 14; i++)
status.increment_num_successful_commits();
EXPECT_EQ(14, status.syncer_status().num_successful_commits);
« no previous file with comments | « chrome/browser/sync/sessions/status_controller.cc ('k') | chrome/browser/sync/sessions/sync_session.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698