Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 1 | // Copyright 2018 The Chromium Authors. All rights reserved. |
| 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
| 5 | #include <memory> |
| 6 | |
| 7 | #include "base/strings/strcat.h" |
| 8 | #include "base/test/test_mock_time_task_runner.h" |
| 9 | #include "components/drive/drive_notification_manager.h" |
| 10 | #include "components/drive/drive_notification_observer.h" |
| 11 | #include "components/invalidation/impl/fake_invalidation_service.h" |
Maksim Moskvitin | 1817bfa | 2020-03-03 11:01:16 | [diff] [blame] | 12 | #include "components/invalidation/public/topic_invalidation_map.h" |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 13 | #include "testing/gtest/include/gtest/gtest.h" |
| 14 | |
| 15 | namespace drive { |
| 16 | |
| 17 | namespace { |
| 18 | |
Maksim Moskvitin | 1817bfa | 2020-03-03 11:01:16 | [diff] [blame] | 19 | const syncer::Topic kDefaultCorpusTopic = "Drive"; |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 20 | |
| 21 | struct ShutdownHelper { |
| 22 | template <typename T> |
| 23 | void operator()(T* object) const { |
| 24 | if (object) { |
| 25 | object->Shutdown(); |
| 26 | delete object; |
| 27 | } |
| 28 | } |
| 29 | }; |
| 30 | |
| 31 | class FakeDriveNotificationObserver : public DriveNotificationObserver { |
| 32 | public: |
| 33 | ~FakeDriveNotificationObserver() override {} |
| 34 | |
| 35 | // DriveNotificationObserver overrides |
Sam McNally | e4bd01c | 2018-10-17 23:51:11 | [diff] [blame] | 36 | void OnNotificationReceived( |
| 37 | const std::map<std::string, int64_t>& ids) override { |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 38 | notification_ids_ = ids; |
| 39 | } |
| 40 | void OnNotificationTimerFired() override {} |
| 41 | void OnPushNotificationEnabled(bool enabled) override {} |
| 42 | |
Sam McNally | e4bd01c | 2018-10-17 23:51:11 | [diff] [blame] | 43 | const std::map<std::string, int64_t> GetNotificationIds() const { |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 44 | return notification_ids_; |
| 45 | } |
| 46 | |
| 47 | void ClearNotificationIds() { notification_ids_.clear(); } |
| 48 | |
| 49 | private: |
Sam McNally | e4bd01c | 2018-10-17 23:51:11 | [diff] [blame] | 50 | std::map<std::string, int64_t> notification_ids_; |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 51 | }; |
| 52 | |
Maksim Moskvitin | 1817bfa | 2020-03-03 11:01:16 | [diff] [blame] | 53 | syncer::Topic CreateTeamDriveInvalidationTopic( |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 54 | const std::string& team_drive_id) { |
Maksim Moskvitin | 1817bfa | 2020-03-03 11:01:16 | [diff] [blame] | 55 | return base::StrCat({"team-drive-", team_drive_id}); |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 56 | } |
| 57 | |
| 58 | } // namespace |
| 59 | |
| 60 | class DriveNotificationManagerTest : public testing::Test { |
| 61 | protected: |
| 62 | void SetUp() override { |
| 63 | task_runner_ = base::MakeRefCounted<base::TestMockTimeTaskRunner>( |
| 64 | base::TestMockTimeTaskRunner::Type::kBoundToThread); |
| 65 | fake_invalidation_service_ = |
| 66 | std::make_unique<invalidation::FakeInvalidationService>(); |
| 67 | drive_notification_observer_ = |
| 68 | std::make_unique<FakeDriveNotificationObserver>(); |
| 69 | |
| 70 | // Can't use make_unique with a custom deleter. |
| 71 | drive_notification_manager_.reset( |
| 72 | new DriveNotificationManager(fake_invalidation_service_.get())); |
| 73 | |
| 74 | drive_notification_manager_->AddObserver( |
| 75 | drive_notification_observer_.get()); |
| 76 | } |
| 77 | |
| 78 | void TearDown() override { |
| 79 | drive_notification_manager_->RemoveObserver( |
| 80 | drive_notification_observer_.get()); |
| 81 | } |
| 82 | |
| 83 | scoped_refptr<base::TestMockTimeTaskRunner> task_runner_; |
| 84 | std::unique_ptr<invalidation::FakeInvalidationService> |
| 85 | fake_invalidation_service_; |
| 86 | std::unique_ptr<FakeDriveNotificationObserver> drive_notification_observer_; |
| 87 | std::unique_ptr<DriveNotificationManager, ShutdownHelper> |
| 88 | drive_notification_manager_; |
| 89 | }; |
| 90 | |
| 91 | TEST_F(DriveNotificationManagerTest, RegisterTeamDrives) { |
| 92 | // By default, we should have registered for default corpus notifications on |
| 93 | // initialization. |
Maksim Moskvitin | d705f6a | 2020-02-18 11:16:55 | [diff] [blame] | 94 | auto subscribed_topics = fake_invalidation_service_->invalidator_registrar() |
| 95 | .GetAllSubscribedTopics(); |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 96 | |
Maksim Moskvitin | 1817bfa | 2020-03-03 11:01:16 | [diff] [blame] | 97 | // TODO(crbug.com/1029698): replace syncer::Topics with syncer::TopicSet once |
| 98 | // |is_public| become the part of dedicated syncer::Topic struct. This should |
| 99 | // simplify this test. |
| 100 | syncer::Topics expected_topics; |
| 101 | expected_topics.emplace(kDefaultCorpusTopic, |
| 102 | syncer::TopicMetadata{/*is_public=*/false}); |
Maksim Moskvitin | d705f6a | 2020-02-18 11:16:55 | [diff] [blame] | 103 | EXPECT_EQ(expected_topics, subscribed_topics); |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 104 | |
| 105 | const std::string team_drive_id_1 = "td_id_1"; |
Maksim Moskvitin | 1817bfa | 2020-03-03 11:01:16 | [diff] [blame] | 106 | const auto team_drive_1_topic = |
| 107 | CreateTeamDriveInvalidationTopic(team_drive_id_1); |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 108 | |
| 109 | // Add the team drive |
| 110 | drive_notification_manager_->UpdateTeamDriveIds({team_drive_id_1}, {}); |
Maksim Moskvitin | d705f6a | 2020-02-18 11:16:55 | [diff] [blame] | 111 | subscribed_topics = fake_invalidation_service_->invalidator_registrar() |
| 112 | .GetAllSubscribedTopics(); |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 113 | |
Maksim Moskvitin | 1817bfa | 2020-03-03 11:01:16 | [diff] [blame] | 114 | expected_topics.emplace(team_drive_1_topic, |
| 115 | syncer::TopicMetadata{/*is_public=*/true}); |
Maksim Moskvitin | d705f6a | 2020-02-18 11:16:55 | [diff] [blame] | 116 | EXPECT_EQ(expected_topics, subscribed_topics); |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 117 | |
| 118 | // Remove the team drive. |
| 119 | drive_notification_manager_->UpdateTeamDriveIds({}, {team_drive_id_1}); |
Maksim Moskvitin | d705f6a | 2020-02-18 11:16:55 | [diff] [blame] | 120 | subscribed_topics = fake_invalidation_service_->invalidator_registrar() |
| 121 | .GetAllSubscribedTopics(); |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 122 | |
Maksim Moskvitin | 1817bfa | 2020-03-03 11:01:16 | [diff] [blame] | 123 | expected_topics.erase(team_drive_1_topic); |
Maksim Moskvitin | d705f6a | 2020-02-18 11:16:55 | [diff] [blame] | 124 | EXPECT_EQ(expected_topics, subscribed_topics); |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 125 | |
| 126 | const std::string team_drive_id_2 = "td_id_2"; |
Maksim Moskvitin | 1817bfa | 2020-03-03 11:01:16 | [diff] [blame] | 127 | const auto team_drive_2_topic = |
| 128 | CreateTeamDriveInvalidationTopic(team_drive_id_2); |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 129 | |
Maksim Moskvitin | 1817bfa | 2020-03-03 11:01:16 | [diff] [blame] | 130 | // Add two team drives. |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 131 | drive_notification_manager_->UpdateTeamDriveIds( |
| 132 | {team_drive_id_1, team_drive_id_2}, {}); |
Maksim Moskvitin | d705f6a | 2020-02-18 11:16:55 | [diff] [blame] | 133 | subscribed_topics = fake_invalidation_service_->invalidator_registrar() |
| 134 | .GetAllSubscribedTopics(); |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 135 | |
Maksim Moskvitin | 1817bfa | 2020-03-03 11:01:16 | [diff] [blame] | 136 | expected_topics.emplace(team_drive_1_topic, |
| 137 | syncer::TopicMetadata{/*is_public=*/true}); |
| 138 | expected_topics.emplace(team_drive_2_topic, |
| 139 | syncer::TopicMetadata{/*is_public=*/true}); |
Maksim Moskvitin | d705f6a | 2020-02-18 11:16:55 | [diff] [blame] | 140 | EXPECT_EQ(expected_topics, subscribed_topics); |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 141 | |
Maksim Moskvitin | 1817bfa | 2020-03-03 11:01:16 | [diff] [blame] | 142 | // Remove the first team drive. |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 143 | drive_notification_manager_->UpdateTeamDriveIds({}, {team_drive_id_1}); |
Maksim Moskvitin | d705f6a | 2020-02-18 11:16:55 | [diff] [blame] | 144 | subscribed_topics = fake_invalidation_service_->invalidator_registrar() |
| 145 | .GetAllSubscribedTopics(); |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 146 | |
Maksim Moskvitin | 1817bfa | 2020-03-03 11:01:16 | [diff] [blame] | 147 | expected_topics.erase(team_drive_1_topic); |
Maksim Moskvitin | d705f6a | 2020-02-18 11:16:55 | [diff] [blame] | 148 | EXPECT_EQ(expected_topics, subscribed_topics); |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 149 | |
| 150 | // Remove a team drive that doesn't exists with no changes. |
| 151 | const std::string team_drive_id_3 = "td_id_3"; |
| 152 | drive_notification_manager_->UpdateTeamDriveIds({}, {team_drive_id_3}); |
Maksim Moskvitin | d705f6a | 2020-02-18 11:16:55 | [diff] [blame] | 153 | subscribed_topics = fake_invalidation_service_->invalidator_registrar() |
| 154 | .GetAllSubscribedTopics(); |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 155 | |
Maksim Moskvitin | d705f6a | 2020-02-18 11:16:55 | [diff] [blame] | 156 | EXPECT_EQ(expected_topics, subscribed_topics); |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 157 | } |
| 158 | |
| 159 | TEST_F(DriveNotificationManagerTest, TestBatchInvalidation) { |
| 160 | // By default we'll be registered for the default change notification. |
| 161 | |
| 162 | // Emitting an invalidation should not call our observer until the timer |
| 163 | // expires. |
| 164 | fake_invalidation_service_->EmitInvalidationForTest( |
Maksim Moskvitin | 1817bfa | 2020-03-03 11:01:16 | [diff] [blame] | 165 | syncer::Invalidation::InitUnknownVersion(kDefaultCorpusTopic)); |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 166 | EXPECT_TRUE(drive_notification_observer_->GetNotificationIds().empty()); |
| 167 | |
Stuart Langley | 6477fb06 | 2018-11-15 03:03:36 | [diff] [blame] | 168 | task_runner_->FastForwardBy(base::TimeDelta::FromSeconds(30)); |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 169 | |
| 170 | // Default corpus is has the id "" when sent to observers. |
Sam McNally | 9840fa7 | 2018-10-19 01:33:59 | [diff] [blame] | 171 | std::map<std::string, int64_t> expected_ids = {{"", -1}}; |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 172 | EXPECT_EQ(expected_ids, drive_notification_observer_->GetNotificationIds()); |
| 173 | drive_notification_observer_->ClearNotificationIds(); |
| 174 | |
| 175 | // Register a team drive for notifications |
| 176 | const std::string team_drive_id_1 = "td_id_1"; |
Maksim Moskvitin | 1817bfa | 2020-03-03 11:01:16 | [diff] [blame] | 177 | const auto team_drive_1_topic = |
| 178 | CreateTeamDriveInvalidationTopic(team_drive_id_1); |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 179 | drive_notification_manager_->UpdateTeamDriveIds({team_drive_id_1}, {}); |
| 180 | |
| 181 | // Emit invalidation for default corpus, should not emit a team drive |
| 182 | // invalidation. |
| 183 | fake_invalidation_service_->EmitInvalidationForTest( |
Maksim Moskvitin | 1817bfa | 2020-03-03 11:01:16 | [diff] [blame] | 184 | syncer::Invalidation::Init(kDefaultCorpusTopic, 1, "")); |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 185 | EXPECT_TRUE(drive_notification_observer_->GetNotificationIds().empty()); |
| 186 | |
Stuart Langley | 6477fb06 | 2018-11-15 03:03:36 | [diff] [blame] | 187 | task_runner_->FastForwardBy(base::TimeDelta::FromSeconds(30)); |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 188 | |
| 189 | // Default corpus is has the id "" when sent to observers. |
Sam McNally | 9438e4b | 2018-10-30 08:14:01 | [diff] [blame] | 190 | expected_ids = {{"", 2}}; |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 191 | EXPECT_EQ(expected_ids, drive_notification_observer_->GetNotificationIds()); |
| 192 | drive_notification_observer_->ClearNotificationIds(); |
| 193 | |
| 194 | // Emit team drive invalidation |
| 195 | fake_invalidation_service_->EmitInvalidationForTest( |
Maksim Moskvitin | 1817bfa | 2020-03-03 11:01:16 | [diff] [blame] | 196 | syncer::Invalidation::Init(team_drive_1_topic, 2, "")); |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 197 | EXPECT_TRUE(drive_notification_observer_->GetNotificationIds().empty()); |
| 198 | |
Stuart Langley | 6477fb06 | 2018-11-15 03:03:36 | [diff] [blame] | 199 | task_runner_->FastForwardBy(base::TimeDelta::FromSeconds(30)); |
Sam McNally | e4bd01c | 2018-10-17 23:51:11 | [diff] [blame] | 200 | expected_ids = {{team_drive_id_1, 2}}; |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 201 | EXPECT_EQ(expected_ids, drive_notification_observer_->GetNotificationIds()); |
| 202 | drive_notification_observer_->ClearNotificationIds(); |
| 203 | |
| 204 | // Emit both default corpus and team drive. |
| 205 | fake_invalidation_service_->EmitInvalidationForTest( |
Maksim Moskvitin | 1817bfa | 2020-03-03 11:01:16 | [diff] [blame] | 206 | syncer::Invalidation::Init(kDefaultCorpusTopic, 1, "")); |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 207 | fake_invalidation_service_->EmitInvalidationForTest( |
Maksim Moskvitin | 1817bfa | 2020-03-03 11:01:16 | [diff] [blame] | 208 | syncer::Invalidation::Init(team_drive_1_topic, 2, "")); |
Sam McNally | 9438e4b | 2018-10-30 08:14:01 | [diff] [blame] | 209 | |
| 210 | // Emit with an earlier version. This should be ignored. |
| 211 | fake_invalidation_service_->EmitInvalidationForTest( |
Maksim Moskvitin | 1817bfa | 2020-03-03 11:01:16 | [diff] [blame] | 212 | syncer::Invalidation::Init(kDefaultCorpusTopic, 0, "")); |
Sam McNally | 9438e4b | 2018-10-30 08:14:01 | [diff] [blame] | 213 | |
| 214 | // Emit without a version. This should be ignored too. |
| 215 | fake_invalidation_service_->EmitInvalidationForTest( |
Maksim Moskvitin | 1817bfa | 2020-03-03 11:01:16 | [diff] [blame] | 216 | syncer::Invalidation::InitUnknownVersion(kDefaultCorpusTopic)); |
Sam McNally | 9438e4b | 2018-10-30 08:14:01 | [diff] [blame] | 217 | |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 218 | EXPECT_TRUE(drive_notification_observer_->GetNotificationIds().empty()); |
| 219 | |
Stuart Langley | 6477fb06 | 2018-11-15 03:03:36 | [diff] [blame] | 220 | task_runner_->FastForwardBy(base::TimeDelta::FromSeconds(30)); |
Sam McNally | 9438e4b | 2018-10-30 08:14:01 | [diff] [blame] | 221 | expected_ids = {{"", 2}, {team_drive_id_1, 2}}; |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 222 | EXPECT_EQ(expected_ids, drive_notification_observer_->GetNotificationIds()); |
| 223 | } |
| 224 | |
Austin Tankiang | 6f52aa0 | 2020-09-30 14:44:36 | [diff] [blame] | 225 | TEST_F(DriveNotificationManagerTest, UnregisterOnNoObservers) { |
| 226 | // By default, we should have registered for default corpus notifications on |
| 227 | // initialization. |
| 228 | auto subscribed_topics = fake_invalidation_service_->invalidator_registrar() |
| 229 | .GetAllSubscribedTopics(); |
| 230 | |
| 231 | // TODO(crbug.com/1029698): replace syncer::Topics with syncer::TopicSet once |
| 232 | // |is_public| become the part of dedicated syncer::Topic struct. |
| 233 | syncer::Topics expected_topics; |
| 234 | expected_topics.emplace(kDefaultCorpusTopic, |
| 235 | syncer::TopicMetadata{/*is_public=*/false}); |
| 236 | EXPECT_EQ(expected_topics, subscribed_topics); |
| 237 | |
| 238 | // Stop observing drive notification manager. |
| 239 | drive_notification_manager_->RemoveObserver( |
| 240 | drive_notification_observer_.get()); |
| 241 | |
| 242 | subscribed_topics = fake_invalidation_service_->invalidator_registrar() |
| 243 | .GetAllSubscribedTopics(); |
| 244 | EXPECT_EQ(syncer::Topics(), subscribed_topics); |
| 245 | |
| 246 | // Start observing drive notification manager again. It should subscribe to |
| 247 | // the previously subscried topics. |
| 248 | drive_notification_manager_->AddObserver(drive_notification_observer_.get()); |
| 249 | subscribed_topics = fake_invalidation_service_->invalidator_registrar() |
| 250 | .GetAllSubscribedTopics(); |
| 251 | EXPECT_EQ(expected_topics, subscribed_topics); |
| 252 | } |
| 253 | |
Stuart Langley | 8084586 | 2018-09-05 00:47:29 | [diff] [blame] | 254 | } // namespace drive |