| Index: components/invalidation/p2p_invalidator_unittest.cc
|
| diff --git a/components/invalidation/p2p_invalidator_unittest.cc b/components/invalidation/p2p_invalidator_unittest.cc
|
| index 150a91a96cf7a68d00ee66a9a52d48455f73a10d..0ad7afc71f519b7b06ec637e538f2222f436807c 100644
|
| --- a/components/invalidation/p2p_invalidator_unittest.cc
|
| +++ b/components/invalidation/p2p_invalidator_unittest.cc
|
| @@ -11,7 +11,6 @@
|
| #include "components/invalidation/notifier_reason_util.h"
|
| #include "jingle/notifier/listener/fake_push_client.h"
|
| #include "sync/internal_api/public/base/invalidator_state.h"
|
| -#include "sync/internal_api/public/base/model_type.h"
|
| #include "testing/gtest/include/gtest/gtest.h"
|
|
|
| namespace syncer {
|
| @@ -86,6 +85,8 @@ class P2PInvalidatorTest : public testing::Test {
|
| protected:
|
| P2PInvalidatorTest()
|
| : next_sent_notification_to_reflect_(0) {
|
| + default_enabled_ids_.insert(invalidation::ObjectId(10, "A"));
|
| + default_enabled_ids_.insert(invalidation::ObjectId(10, "B"));
|
| delegate_.CreateInvalidator("sender",
|
| "fake_state",
|
| base::WeakPtr<InvalidationStateTracker>());
|
| @@ -96,9 +97,7 @@ class P2PInvalidatorTest : public testing::Test {
|
| delegate_.GetInvalidator()->UnregisterHandler(&fake_handler_);
|
| }
|
|
|
| - ObjectIdInvalidationMap MakeInvalidationMap(ModelTypeSet types) {
|
| - ObjectIdInvalidationMap invalidations;
|
| - ObjectIdSet ids = ModelTypeSetToObjectIdSet(types);
|
| + ObjectIdInvalidationMap MakeInvalidationMap(ObjectIdSet ids) {
|
| return ObjectIdInvalidationMap::InvalidateAll(ids);
|
| }
|
|
|
| @@ -114,6 +113,8 @@ class P2PInvalidatorTest : public testing::Test {
|
| next_sent_notification_to_reflect_ = sent_notifications.size();
|
| }
|
|
|
| + ObjectIdSet default_enabled_ids_;
|
| +
|
| FakeInvalidationHandler fake_handler_;
|
| P2PInvalidatorTestDelegate delegate_;
|
|
|
| @@ -180,8 +181,7 @@ TEST_F(P2PInvalidatorTest, P2PNotificationDataDefault) {
|
| // non-default-constructed P2PNotificationData.
|
| TEST_F(P2PInvalidatorTest, P2PNotificationDataNonDefault) {
|
| ObjectIdInvalidationMap invalidation_map =
|
| - ObjectIdInvalidationMap::InvalidateAll(
|
| - ModelTypeSetToObjectIdSet(ModelTypeSet(BOOKMARKS, THEMES)));
|
| + ObjectIdInvalidationMap::InvalidateAll(default_enabled_ids_);
|
| const P2PNotificationData notification_data("sender",
|
| NOTIFY_ALL,
|
| invalidation_map);
|
| @@ -193,11 +193,12 @@ TEST_F(P2PInvalidatorTest, P2PNotificationDataNonDefault) {
|
| EXPECT_EQ(
|
| "{\"invalidations\":["
|
| "{\"isUnknownVersion\":true,"
|
| - "\"objectId\":{\"name\":\"BOOKMARK\",\"source\":1004}},"
|
| + "\"objectId\":{\"name\":\"A\",\"source\":10}},"
|
| "{\"isUnknownVersion\":true,"
|
| - "\"objectId\":{\"name\":\"THEME\",\"source\":1004}}"
|
| + "\"objectId\":{\"name\":\"B\",\"source\":10}}"
|
| "],\"notificationType\":\"notifyAll\","
|
| - "\"senderId\":\"sender\"}", notification_data_str);
|
| + "\"senderId\":\"sender\"}",
|
| + notification_data_str);
|
|
|
| P2PNotificationData notification_data_parsed;
|
| EXPECT_TRUE(notification_data_parsed.ResetFromString(notification_data_str));
|
| @@ -209,13 +210,10 @@ TEST_F(P2PInvalidatorTest, P2PNotificationDataNonDefault) {
|
| // observer should receive only a notification from the call to
|
| // UpdateEnabledTypes().
|
| TEST_F(P2PInvalidatorTest, NotificationsBasic) {
|
| - const ModelTypeSet enabled_types(BOOKMARKS, PREFERENCES);
|
| -
|
| P2PInvalidator* const invalidator = delegate_.GetInvalidator();
|
| notifier::FakePushClient* const push_client = delegate_.GetPushClient();
|
|
|
| - invalidator->UpdateRegisteredIds(&fake_handler_,
|
| - ModelTypeSetToObjectIdSet(enabled_types));
|
| + invalidator->UpdateRegisteredIds(&fake_handler_, default_enabled_ids_);
|
|
|
| const char kEmail[] = "foo@bar.com";
|
| const char kToken[] = "token";
|
| @@ -237,14 +235,12 @@ TEST_F(P2PInvalidatorTest, NotificationsBasic) {
|
|
|
| ReflectSentNotifications();
|
| EXPECT_EQ(1, fake_handler_.GetInvalidationCount());
|
| - EXPECT_THAT(
|
| - MakeInvalidationMap(enabled_types),
|
| - Eq(fake_handler_.GetLastInvalidationMap()));
|
| + EXPECT_THAT(MakeInvalidationMap(default_enabled_ids_),
|
| + Eq(fake_handler_.GetLastInvalidationMap()));
|
|
|
| // Sent with target NOTIFY_OTHERS so should not be propagated to
|
| // |fake_handler_|.
|
| - invalidator->SendInvalidation(
|
| - ModelTypeSetToObjectIdSet(ModelTypeSet(THEMES, APPS)));
|
| + invalidator->SendInvalidation(default_enabled_ids_);
|
|
|
| ReflectSentNotifications();
|
| EXPECT_EQ(1, fake_handler_.GetInvalidationCount());
|
| @@ -254,18 +250,26 @@ TEST_F(P2PInvalidatorTest, NotificationsBasic) {
|
| // target settings. The notifications received by the observer should
|
| // be consistent with the target settings.
|
| TEST_F(P2PInvalidatorTest, SendNotificationData) {
|
| - const ModelTypeSet enabled_types(BOOKMARKS, PREFERENCES, THEMES);
|
| - const ModelTypeSet changed_types(THEMES, APPS);
|
| - const ModelTypeSet expected_types(THEMES);
|
| + ObjectIdSet enabled_ids;
|
| + ObjectIdSet changed_ids;
|
| + ObjectIdSet expected_ids;
|
| +
|
| + enabled_ids.insert(invalidation::ObjectId(20, "A"));
|
| + enabled_ids.insert(invalidation::ObjectId(20, "B"));
|
| + enabled_ids.insert(invalidation::ObjectId(20, "C"));
|
| +
|
| + changed_ids.insert(invalidation::ObjectId(20, "A"));
|
| + changed_ids.insert(invalidation::ObjectId(20, "Z"));
|
| +
|
| + expected_ids.insert(invalidation::ObjectId(20, "A"));
|
|
|
| const ObjectIdInvalidationMap& invalidation_map =
|
| - MakeInvalidationMap(changed_types);
|
| + MakeInvalidationMap(changed_ids);
|
|
|
| P2PInvalidator* const invalidator = delegate_.GetInvalidator();
|
| notifier::FakePushClient* const push_client = delegate_.GetPushClient();
|
|
|
| - invalidator->UpdateRegisteredIds(&fake_handler_,
|
| - ModelTypeSetToObjectIdSet(enabled_types));
|
| + invalidator->UpdateRegisteredIds(&fake_handler_, enabled_ids);
|
|
|
| invalidator->UpdateCredentials("foo@bar.com", "fake_token");
|
|
|
| @@ -275,16 +279,13 @@ TEST_F(P2PInvalidatorTest, SendNotificationData) {
|
|
|
| ReflectSentNotifications();
|
| EXPECT_EQ(1, fake_handler_.GetInvalidationCount());
|
| - EXPECT_EQ(ModelTypeSetToObjectIdSet(enabled_types),
|
| - fake_handler_.GetLastInvalidationMap().GetObjectIds());
|
| + EXPECT_EQ(enabled_ids, fake_handler_.GetLastInvalidationMap().GetObjectIds());
|
|
|
| // Should be dropped.
|
| invalidator->SendNotificationDataForTest(P2PNotificationData());
|
| ReflectSentNotifications();
|
| EXPECT_EQ(1, fake_handler_.GetInvalidationCount());
|
|
|
| - const ObjectIdSet& expected_ids = ModelTypeSetToObjectIdSet(expected_types);
|
| -
|
| // Should be propagated.
|
| invalidator->SendNotificationDataForTest(
|
| P2PNotificationData("sender", NOTIFY_SELF, invalidation_map));
|
|
|