Index: sync/engine/sync_scheduler_unittest.cc |
diff --git a/sync/engine/sync_scheduler_unittest.cc b/sync/engine/sync_scheduler_unittest.cc |
index 6c3e3eba2d87e7ca12fa445e195381d9cbbf5585..acac5db1bf64a2aa315b9c5394170f93a063b050 100644 |
--- a/sync/engine/sync_scheduler_unittest.cc |
+++ b/sync/engine/sync_scheduler_unittest.cc |
@@ -300,9 +300,7 @@ TEST_F(SyncSchedulerTest, Config) { |
SyncShareRecords records; |
const ModelTypeSet model_types(syncer::BOOKMARKS); |
- EXPECT_CALL(*syncer(), |
- SyncShare(_,_,_)) |
- .WillOnce(Invoke(sessions::test_util::SimulateSuccess)) |
+ EXPECT_CALL(*syncer(), SyncShare(_,_,_)) |
.WillOnce(DoAll(Invoke(sessions::test_util::SimulateSuccess), |
WithArg<0>(RecordSyncShare(&records)))); |
@@ -333,9 +331,7 @@ TEST_F(SyncSchedulerTest, ConfigWithBackingOff) { |
SyncShareRecords records; |
const ModelTypeSet model_types(syncer::BOOKMARKS); |
- EXPECT_CALL(*syncer(), |
- SyncShare(_,_,_)) |
- .WillOnce(Invoke(sessions::test_util::SimulateSuccess)) |
+ EXPECT_CALL(*syncer(), SyncShare(_,_,_)) |
.WillOnce(DoAll(Invoke(sessions::test_util::SimulateCommitFailed), |
WithArg<0>(RecordSyncShare(&records)))) |
.WillOnce(DoAll(Invoke(sessions::test_util::SimulateSuccess), |
@@ -721,11 +717,7 @@ TEST_F(SyncSchedulerTest, ThrottlingDoesThrottle) { |
TimeDelta throttle(TimeDelta::FromMinutes(10)); |
scheduler()->OnReceivedLongPollIntervalUpdate(poll); |
- EXPECT_CALL(*syncer(), |
- SyncShare(_, CLEANUP_DISABLED_TYPES, CLEANUP_DISABLED_TYPES)) |
- .WillOnce(Invoke(sessions::test_util::SimulateSuccess)); |
- EXPECT_CALL(*syncer(), SyncShare(_,Not(CLEANUP_DISABLED_TYPES), |
- Not(CLEANUP_DISABLED_TYPES))) |
+ EXPECT_CALL(*syncer(), SyncShare(_,_,_)) |
.WillOnce(WithArg<0>(sessions::test_util::SimulateThrottled(throttle))) |
.WillRepeatedly(AddFailureAndQuitLoopNow()); |
@@ -778,7 +770,6 @@ TEST_F(SyncSchedulerTest, ConfigurationMode) { |
SyncShareRecords records; |
scheduler()->OnReceivedLongPollIntervalUpdate(poll); |
EXPECT_CALL(*syncer(), SyncShare(_,_,_)) |
- .WillOnce(Invoke(sessions::test_util::SimulateSuccess)) |
.WillOnce(DoAll(Invoke(sessions::test_util::SimulateSuccess), |
WithArg<0>(RecordSyncShare(&records)))); |
@@ -903,10 +894,6 @@ TEST_F(SyncSchedulerTest, BackoffDropsJobs) { |
EXPECT_EQ(GetUpdatesCallerInfo::LOCAL, |
r.snapshots[1].source().updates_source); |
- // Cleanup is not affected by backoff, but it should not relieve it either. |
- EXPECT_CALL(*syncer(), |
- SyncShare(_, CLEANUP_DISABLED_TYPES, CLEANUP_DISABLED_TYPES)) |
- .WillOnce(Invoke(sessions::test_util::SimulateSuccess)); |
EXPECT_CALL(*delay(), GetDelay(_)).Times(0); |
StartSyncScheduler(SyncScheduler::CONFIGURATION_MODE); |
@@ -1078,10 +1065,7 @@ TEST_F(SyncSchedulerTest, SyncerSteps) { |
StopSyncScheduler(); |
Mock::VerifyAndClearExpectations(syncer()); |
- // Configuration (always includes a cleanup disabled types). |
- EXPECT_CALL(*syncer(), |
- SyncShare(_, CLEANUP_DISABLED_TYPES, CLEANUP_DISABLED_TYPES)) |
- .WillOnce(Invoke(sessions::test_util::SimulateSuccess)); |
+ // Configuration. |
EXPECT_CALL(*syncer(), SyncShare(_, DOWNLOAD_UPDATES, APPLY_UPDATES)) |
.WillOnce(Invoke(sessions::test_util::SimulateSuccess)); |
StartSyncScheduler(SyncScheduler::CONFIGURATION_MODE); |
@@ -1100,25 +1084,6 @@ TEST_F(SyncSchedulerTest, SyncerSteps) { |
StopSyncScheduler(); |
Mock::VerifyAndClearExpectations(syncer()); |
- // Cleanup disabled types. Because no types are being configured, we just |
- // perform the cleanup. |
- EXPECT_CALL(*syncer(), |
- SyncShare(_, CLEANUP_DISABLED_TYPES, CLEANUP_DISABLED_TYPES)). |
- WillOnce(Invoke(sessions::test_util::SimulateSuccess)); |
- StartSyncScheduler(SyncScheduler::CONFIGURATION_MODE); |
- |
- CallbackCounter counter2; |
- ConfigurationParams params2( |
- GetUpdatesCallerInfo::RECONFIGURATION, |
- ModelTypeSet(), |
- ModelSafeRoutingInfo(), |
- ConfigurationParams::KEYSTORE_KEY_UNNECESSARY, |
- base::Bind(&CallbackCounter::Callback, base::Unretained(&counter2))); |
- ASSERT_TRUE(scheduler()->ScheduleConfiguration(params2)); |
- ASSERT_EQ(1, counter2.times_called()); |
- StopSyncScheduler(); |
- Mock::VerifyAndClearExpectations(syncer()); |
- |
StartSyncScheduler(SyncScheduler::NORMAL_MODE); |
// Poll. |
@@ -1164,24 +1129,4 @@ TEST_F(SyncSchedulerTest, StartWhenNotConnected) { |
MessageLoop::current()->RunAllPending(); |
} |
-TEST_F(SyncSchedulerTest, SetsPreviousRoutingInfo) { |
- ModelSafeRoutingInfo info; |
- EXPECT_TRUE(info == context()->previous_session_routing_info()); |
- ModelSafeRoutingInfo expected(context()->routing_info()); |
- ASSERT_FALSE(expected.empty()); |
- EXPECT_CALL(*syncer(), SyncShare(_,_,_)).Times(1); |
- |
- StartSyncScheduler(SyncScheduler::NORMAL_MODE); |
- |
- scheduler()->ScheduleNudgeAsync( |
- zero(), NUDGE_SOURCE_LOCAL, ModelTypeSet(), FROM_HERE); |
- PumpLoop(); |
- // Pump again to run job. |
- PumpLoop(); |
- |
- StopSyncScheduler(); |
- |
- EXPECT_TRUE(expected == context()->previous_session_routing_info()); |
-} |
- |
-} // namespace syncer |
+} // namespace browser_sync |
tim (not reviewing)
2012/07/19 22:05:59
huh?
Nicolas Zea
2012/07/20 22:07:01
Done.
|