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

Unified Diff: components/sync/core_impl/sync_manager_impl_unittest.cc

Issue 2276943006: [USS] Move GetAllNodes from backend to controller (Closed)
Patch Set: Created 4 years, 4 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
Index: components/sync/core_impl/sync_manager_impl_unittest.cc
diff --git a/components/sync/core_impl/sync_manager_impl_unittest.cc b/components/sync/core_impl/sync_manager_impl_unittest.cc
index 9ce3c9c0c711c54dd2d8f828b1126744b7a02550..cd0461dbc39c2fe0a1475f6ce9ca3809d12675aa 100644
--- a/components/sync/core_impl/sync_manager_impl_unittest.cc
+++ b/components/sync/core_impl/sync_manager_impl_unittest.cc
@@ -1193,23 +1193,6 @@ class SyncManagerTest : public testing::Test,
MockUnrecoverableErrorHandler mock_unrecoverable_error_handler_;
};
-TEST_F(SyncManagerTest, GetAllNodesForTypeTest) {
- ModelSafeRoutingInfo routing_info;
- GetModelSafeRoutingInfo(&routing_info);
- sync_manager_.StartSyncingNormally(routing_info, base::Time());
-
- std::unique_ptr<base::ListValue> node_list(
- sync_manager_.GetAllNodesForType(syncer::PREFERENCES));
-
- // Should have one node: the type root node.
- ASSERT_EQ(1U, node_list->GetSize());
-
- const base::DictionaryValue* first_result;
- ASSERT_TRUE(node_list->GetDictionary(0, &first_result));
- EXPECT_TRUE(first_result->HasKey("ID"));
- EXPECT_TRUE(first_result->HasKey("NON_UNIQUE_NAME"));
-}
-
TEST_F(SyncManagerTest, RefreshEncryptionReady) {
EXPECT_TRUE(SetUpEncryption(WRITE_TO_NIGORI, DEFAULT_ENCRYPTION));
EXPECT_CALL(encryption_observer_, OnEncryptionComplete());

Powered by Google App Engine
This is Rietveld 408576698