Vitaly Buka | 4615e0d | 2015-10-14 15:35:12 -0700 | [diff] [blame] | 1 | // Copyright 2015 The Weave Authors. All rights reserved. |
Alex Vakulenko | aa3a559 | 2014-08-07 07:24:06 -0700 | [diff] [blame] | 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
Stefan Sauer | 2d16dfa | 2015-09-25 17:08:35 +0200 | [diff] [blame] | 5 | #include "src/commands/command_queue.h" |
Alex Deymo | f6cbe32 | 2014-11-10 19:55:35 -0800 | [diff] [blame] | 6 | |
Alex Vakulenko | 515b42b | 2014-08-07 15:46:31 -0700 | [diff] [blame] | 7 | #include <set> |
Alex Vakulenko | aa3a559 | 2014-08-07 07:24:06 -0700 | [diff] [blame] | 8 | #include <string> |
Alex Vakulenko | 515b42b | 2014-08-07 15:46:31 -0700 | [diff] [blame] | 9 | #include <vector> |
Alex Vakulenko | aa3a559 | 2014-08-07 07:24:06 -0700 | [diff] [blame] | 10 | |
Vitaly Buka | ae0f3a1 | 2015-05-11 16:27:30 -0700 | [diff] [blame] | 11 | #include <base/bind.h> |
| 12 | #include <base/memory/weak_ptr.h> |
Alex Vakulenko | 98d1fee | 2016-02-01 12:25:21 -0800 | [diff] [blame] | 13 | #include <gmock/gmock.h> |
Alex Vakulenko | aa3a559 | 2014-08-07 07:24:06 -0700 | [diff] [blame] | 14 | #include <gtest/gtest.h> |
Alex Vakulenko | 98d1fee | 2016-02-01 12:25:21 -0800 | [diff] [blame] | 15 | #include <weave/provider/test/fake_task_runner.h> |
Alex Vakulenko | aa3a559 | 2014-08-07 07:24:06 -0700 | [diff] [blame] | 16 | |
Alex Vakulenko | 98d1fee | 2016-02-01 12:25:21 -0800 | [diff] [blame] | 17 | #include "src/bind_lambda.h" |
Stefan Sauer | 2d16dfa | 2015-09-25 17:08:35 +0200 | [diff] [blame] | 18 | #include "src/string_utils.h" |
Alex Vakulenko | aa3a559 | 2014-08-07 07:24:06 -0700 | [diff] [blame] | 19 | |
Vitaly Buka | b6f015a | 2015-07-09 14:59:23 -0700 | [diff] [blame] | 20 | namespace weave { |
Alex Vakulenko | aa3a559 | 2014-08-07 07:24:06 -0700 | [diff] [blame] | 21 | |
Alex Vakulenko | 98d1fee | 2016-02-01 12:25:21 -0800 | [diff] [blame] | 22 | using testing::Return; |
| 23 | using testing::StrictMock; |
| 24 | |
Alex Vakulenko | 5ef7579 | 2015-03-19 15:50:44 -0700 | [diff] [blame] | 25 | class CommandQueueTest : public testing::Test { |
| 26 | public: |
Vitaly Buka | 2a9b30f | 2015-04-01 10:51:59 -0700 | [diff] [blame] | 27 | std::unique_ptr<CommandInstance> CreateDummyCommandInstance( |
| 28 | const std::string& name, |
| 29 | const std::string& id) { |
Vitaly Buka | 34668e7 | 2015-12-15 14:46:47 -0800 | [diff] [blame] | 30 | std::unique_ptr<CommandInstance> cmd{ |
| 31 | new CommandInstance{name, Command::Origin::kLocal, {}}}; |
Alex Vakulenko | 5ef7579 | 2015-03-19 15:50:44 -0700 | [diff] [blame] | 32 | cmd->SetID(id); |
| 33 | return cmd; |
| 34 | } |
| 35 | |
Vitaly Buka | 2a9b30f | 2015-04-01 10:51:59 -0700 | [diff] [blame] | 36 | bool Remove(const std::string& id) { return queue_.Remove(id); } |
| 37 | |
| 38 | void Cleanup(const base::TimeDelta& interval) { |
Alex Vakulenko | 98d1fee | 2016-02-01 12:25:21 -0800 | [diff] [blame] | 39 | return queue_.Cleanup(task_runner_.GetClock()->Now() + interval); |
Vitaly Buka | 2a9b30f | 2015-04-01 10:51:59 -0700 | [diff] [blame] | 40 | } |
| 41 | |
Alex Vakulenko | 98d1fee | 2016-02-01 12:25:21 -0800 | [diff] [blame] | 42 | std::string GetFirstCommandToBeRemoved() const { |
| 43 | return queue_.remove_queue_.top().second; |
| 44 | } |
| 45 | |
| 46 | StrictMock<provider::test::FakeTaskRunner> task_runner_; |
| 47 | CommandQueue queue_{&task_runner_, task_runner_.GetClock()}; |
Alex Vakulenko | 5ef7579 | 2015-03-19 15:50:44 -0700 | [diff] [blame] | 48 | }; |
Alex Vakulenko | aa3a559 | 2014-08-07 07:24:06 -0700 | [diff] [blame] | 49 | |
Vitaly Buka | ae0f3a1 | 2015-05-11 16:27:30 -0700 | [diff] [blame] | 50 | // Keeps track of commands being added to and removed from the queue_. |
Alex Vakulenko | 515b42b | 2014-08-07 15:46:31 -0700 | [diff] [blame] | 51 | // Aborts if duplicate commands are added or non-existent commands are removed. |
Vitaly Buka | ae0f3a1 | 2015-05-11 16:27:30 -0700 | [diff] [blame] | 52 | class FakeDispatcher { |
Alex Vakulenko | 515b42b | 2014-08-07 15:46:31 -0700 | [diff] [blame] | 53 | public: |
Vitaly Buka | ae0f3a1 | 2015-05-11 16:27:30 -0700 | [diff] [blame] | 54 | explicit FakeDispatcher(CommandQueue* queue) { |
Vitaly Buka | 553a762 | 2015-10-05 13:53:20 -0700 | [diff] [blame] | 55 | queue->AddCommandAddedCallback(base::Bind(&FakeDispatcher::OnCommandAdded, |
| 56 | weak_ptr_factory_.GetWeakPtr())); |
| 57 | queue->AddCommandRemovedCallback(base::Bind( |
Vitaly Buka | a647c85 | 2015-07-06 14:51:01 -0700 | [diff] [blame] | 58 | &FakeDispatcher::OnCommandRemoved, weak_ptr_factory_.GetWeakPtr())); |
Vitaly Buka | ae0f3a1 | 2015-05-11 16:27:30 -0700 | [diff] [blame] | 59 | } |
| 60 | |
Vitaly Buka | 12affd8 | 2015-07-23 18:45:35 -0700 | [diff] [blame] | 61 | void OnCommandAdded(Command* command) { |
| 62 | CHECK(ids_.insert(command->GetID()).second) << "Command ID already exists: " |
| 63 | << command->GetID(); |
| 64 | CHECK(commands_.insert(command).second) |
Alex Vakulenko | 515b42b | 2014-08-07 15:46:31 -0700 | [diff] [blame] | 65 | << "Command instance already exists"; |
| 66 | } |
| 67 | |
Vitaly Buka | 12affd8 | 2015-07-23 18:45:35 -0700 | [diff] [blame] | 68 | void OnCommandRemoved(Command* command) { |
| 69 | CHECK_EQ(1u, ids_.erase(command->GetID())) << "Command ID not found: " |
| 70 | << command->GetID(); |
| 71 | CHECK_EQ(1u, commands_.erase(command)) << "Command instance not found"; |
Alex Vakulenko | 515b42b | 2014-08-07 15:46:31 -0700 | [diff] [blame] | 72 | } |
| 73 | |
| 74 | // Get the comma-separated list of command IDs currently accumulated in the |
Vitaly Buka | 2a9b30f | 2015-04-01 10:51:59 -0700 | [diff] [blame] | 75 | // command queue_. |
Alex Vakulenko | 515b42b | 2014-08-07 15:46:31 -0700 | [diff] [blame] | 76 | std::string GetIDs() const { |
Vitaly Buka | db770e7 | 2015-03-10 19:33:33 -0700 | [diff] [blame] | 77 | return Join(",", std::vector<std::string>(ids_.begin(), ids_.end())); |
Alex Vakulenko | 515b42b | 2014-08-07 15:46:31 -0700 | [diff] [blame] | 78 | } |
| 79 | |
| 80 | private: |
| 81 | std::set<std::string> ids_; |
Vitaly Buka | 12affd8 | 2015-07-23 18:45:35 -0700 | [diff] [blame] | 82 | std::set<Command*> commands_; |
Vitaly Buka | ae0f3a1 | 2015-05-11 16:27:30 -0700 | [diff] [blame] | 83 | base::WeakPtrFactory<FakeDispatcher> weak_ptr_factory_{this}; |
Alex Vakulenko | 515b42b | 2014-08-07 15:46:31 -0700 | [diff] [blame] | 84 | }; |
| 85 | |
Alex Vakulenko | 5ef7579 | 2015-03-19 15:50:44 -0700 | [diff] [blame] | 86 | TEST_F(CommandQueueTest, Empty) { |
Vitaly Buka | 2a9b30f | 2015-04-01 10:51:59 -0700 | [diff] [blame] | 87 | EXPECT_TRUE(queue_.IsEmpty()); |
Vitaly Buka | 52d006a | 2015-11-21 17:14:51 -0800 | [diff] [blame] | 88 | EXPECT_EQ(0u, queue_.GetCount()); |
Alex Vakulenko | aa3a559 | 2014-08-07 07:24:06 -0700 | [diff] [blame] | 89 | } |
| 90 | |
Alex Vakulenko | 5ef7579 | 2015-03-19 15:50:44 -0700 | [diff] [blame] | 91 | TEST_F(CommandQueueTest, Add) { |
Vitaly Buka | 2a9b30f | 2015-04-01 10:51:59 -0700 | [diff] [blame] | 92 | queue_.Add(CreateDummyCommandInstance("base.reboot", "id1")); |
| 93 | queue_.Add(CreateDummyCommandInstance("base.reboot", "id2")); |
| 94 | queue_.Add(CreateDummyCommandInstance("base.reboot", "id3")); |
Vitaly Buka | 52d006a | 2015-11-21 17:14:51 -0800 | [diff] [blame] | 95 | EXPECT_EQ(3u, queue_.GetCount()); |
Vitaly Buka | 2a9b30f | 2015-04-01 10:51:59 -0700 | [diff] [blame] | 96 | EXPECT_FALSE(queue_.IsEmpty()); |
Alex Vakulenko | aa3a559 | 2014-08-07 07:24:06 -0700 | [diff] [blame] | 97 | } |
| 98 | |
Alex Vakulenko | 5ef7579 | 2015-03-19 15:50:44 -0700 | [diff] [blame] | 99 | TEST_F(CommandQueueTest, Remove) { |
Anton Muhin | 5191e81 | 2014-10-30 17:49:48 +0400 | [diff] [blame] | 100 | const std::string id1 = "id1"; |
| 101 | const std::string id2 = "id2"; |
Vitaly Buka | 2a9b30f | 2015-04-01 10:51:59 -0700 | [diff] [blame] | 102 | queue_.Add(CreateDummyCommandInstance("base.reboot", id1)); |
| 103 | queue_.Add(CreateDummyCommandInstance("base.reboot", id2)); |
| 104 | EXPECT_FALSE(queue_.IsEmpty()); |
| 105 | EXPECT_FALSE(Remove("dummy")); |
Vitaly Buka | 52d006a | 2015-11-21 17:14:51 -0800 | [diff] [blame] | 106 | EXPECT_EQ(2u, queue_.GetCount()); |
Vitaly Buka | 2a9b30f | 2015-04-01 10:51:59 -0700 | [diff] [blame] | 107 | EXPECT_TRUE(Remove(id1)); |
Vitaly Buka | 52d006a | 2015-11-21 17:14:51 -0800 | [diff] [blame] | 108 | EXPECT_EQ(1u, queue_.GetCount()); |
Vitaly Buka | 2a9b30f | 2015-04-01 10:51:59 -0700 | [diff] [blame] | 109 | EXPECT_FALSE(Remove(id1)); |
Vitaly Buka | 52d006a | 2015-11-21 17:14:51 -0800 | [diff] [blame] | 110 | EXPECT_EQ(1u, queue_.GetCount()); |
Vitaly Buka | 2a9b30f | 2015-04-01 10:51:59 -0700 | [diff] [blame] | 111 | EXPECT_TRUE(Remove(id2)); |
Vitaly Buka | 52d006a | 2015-11-21 17:14:51 -0800 | [diff] [blame] | 112 | EXPECT_EQ(0u, queue_.GetCount()); |
Vitaly Buka | 2a9b30f | 2015-04-01 10:51:59 -0700 | [diff] [blame] | 113 | EXPECT_FALSE(Remove(id2)); |
Vitaly Buka | 52d006a | 2015-11-21 17:14:51 -0800 | [diff] [blame] | 114 | EXPECT_EQ(0u, queue_.GetCount()); |
Vitaly Buka | 2a9b30f | 2015-04-01 10:51:59 -0700 | [diff] [blame] | 115 | EXPECT_TRUE(queue_.IsEmpty()); |
| 116 | } |
| 117 | |
Alex Vakulenko | 329ad80 | 2016-02-01 12:11:30 -0800 | [diff] [blame] | 118 | TEST_F(CommandQueueTest, RemoveLater) { |
Vitaly Buka | 2a9b30f | 2015-04-01 10:51:59 -0700 | [diff] [blame] | 119 | const std::string id1 = "id1"; |
| 120 | queue_.Add(CreateDummyCommandInstance("base.reboot", id1)); |
Vitaly Buka | 52d006a | 2015-11-21 17:14:51 -0800 | [diff] [blame] | 121 | EXPECT_EQ(1u, queue_.GetCount()); |
Vitaly Buka | 2a9b30f | 2015-04-01 10:51:59 -0700 | [diff] [blame] | 122 | |
Alex Vakulenko | 329ad80 | 2016-02-01 12:11:30 -0800 | [diff] [blame] | 123 | queue_.RemoveLater(id1); |
Vitaly Buka | 52d006a | 2015-11-21 17:14:51 -0800 | [diff] [blame] | 124 | EXPECT_EQ(1u, queue_.GetCount()); |
Vitaly Buka | 2a9b30f | 2015-04-01 10:51:59 -0700 | [diff] [blame] | 125 | |
| 126 | Cleanup(base::TimeDelta::FromMinutes(1)); |
Vitaly Buka | 52d006a | 2015-11-21 17:14:51 -0800 | [diff] [blame] | 127 | EXPECT_EQ(1u, queue_.GetCount()); |
Vitaly Buka | 2a9b30f | 2015-04-01 10:51:59 -0700 | [diff] [blame] | 128 | |
| 129 | Cleanup(base::TimeDelta::FromMinutes(15)); |
Vitaly Buka | 52d006a | 2015-11-21 17:14:51 -0800 | [diff] [blame] | 130 | EXPECT_EQ(0u, queue_.GetCount()); |
Alex Vakulenko | aa3a559 | 2014-08-07 07:24:06 -0700 | [diff] [blame] | 131 | } |
| 132 | |
Alex Vakulenko | 98d1fee | 2016-02-01 12:25:21 -0800 | [diff] [blame] | 133 | TEST_F(CommandQueueTest, RemoveLaterOnCleanupTask) { |
| 134 | const std::string id1 = "id1"; |
| 135 | queue_.Add(CreateDummyCommandInstance("base.reboot", id1)); |
| 136 | EXPECT_EQ(1u, queue_.GetCount()); |
| 137 | |
| 138 | queue_.RemoveLater(id1); |
| 139 | EXPECT_EQ(1u, queue_.GetCount()); |
| 140 | ASSERT_EQ(1u, task_runner_.GetTaskQueueSize()); |
| 141 | |
| 142 | task_runner_.RunOnce(); |
| 143 | |
| 144 | EXPECT_EQ(0u, queue_.GetCount()); |
| 145 | EXPECT_EQ(0u, task_runner_.GetTaskQueueSize()); |
| 146 | } |
| 147 | |
| 148 | TEST_F(CommandQueueTest, CleanupMultipleCommands) { |
| 149 | const std::string id1 = "id1"; |
| 150 | const std::string id2 = "id2"; |
| 151 | |
| 152 | queue_.Add(CreateDummyCommandInstance("base.reboot", id1)); |
| 153 | queue_.Add(CreateDummyCommandInstance("base.reboot", id2)); |
| 154 | auto remove_task = [this](const std::string& id) { queue_.RemoveLater(id); }; |
| 155 | remove_task(id1); |
| 156 | task_runner_.PostDelayedTask(FROM_HERE, base::Bind(remove_task, id2), |
| 157 | base::TimeDelta::FromSeconds(10)); |
| 158 | EXPECT_EQ(2u, queue_.GetCount()); |
| 159 | ASSERT_EQ(2u, task_runner_.GetTaskQueueSize()); |
| 160 | task_runner_.RunOnce(); // Executes "remove_task(id2) @ T+10s". |
| 161 | ASSERT_EQ(2u, queue_.GetCount()); |
| 162 | ASSERT_EQ(1u, task_runner_.GetTaskQueueSize()); |
| 163 | EXPECT_EQ(id1, GetFirstCommandToBeRemoved()); |
| 164 | task_runner_.RunOnce(); // Should remove task "id1" from queue. |
| 165 | ASSERT_EQ(1u, queue_.GetCount()); |
| 166 | ASSERT_EQ(1u, task_runner_.GetTaskQueueSize()); |
| 167 | EXPECT_EQ(id2, GetFirstCommandToBeRemoved()); |
| 168 | task_runner_.RunOnce(); // Should remove task "id2" from queue. |
| 169 | EXPECT_EQ(0u, queue_.GetCount()); |
| 170 | EXPECT_EQ(0u, task_runner_.GetTaskQueueSize()); |
| 171 | } |
| 172 | |
Alex Vakulenko | 5ef7579 | 2015-03-19 15:50:44 -0700 | [diff] [blame] | 173 | TEST_F(CommandQueueTest, Dispatch) { |
Vitaly Buka | ae0f3a1 | 2015-05-11 16:27:30 -0700 | [diff] [blame] | 174 | FakeDispatcher dispatch(&queue_); |
Anton Muhin | 5191e81 | 2014-10-30 17:49:48 +0400 | [diff] [blame] | 175 | const std::string id1 = "id1"; |
| 176 | const std::string id2 = "id2"; |
Vitaly Buka | 2a9b30f | 2015-04-01 10:51:59 -0700 | [diff] [blame] | 177 | queue_.Add(CreateDummyCommandInstance("base.reboot", id1)); |
| 178 | queue_.Add(CreateDummyCommandInstance("base.reboot", id2)); |
Alex Vakulenko | 515b42b | 2014-08-07 15:46:31 -0700 | [diff] [blame] | 179 | std::set<std::string> ids{id1, id2}; // Make sure they are sorted properly. |
Vitaly Buka | 24d6fd5 | 2015-08-13 23:22:48 -0700 | [diff] [blame] | 180 | std::string expected_set = |
| 181 | Join(",", std::vector<std::string>(ids.begin(), ids.end())); |
Alex Vakulenko | 515b42b | 2014-08-07 15:46:31 -0700 | [diff] [blame] | 182 | EXPECT_EQ(expected_set, dispatch.GetIDs()); |
Vitaly Buka | 2a9b30f | 2015-04-01 10:51:59 -0700 | [diff] [blame] | 183 | Remove(id1); |
Alex Vakulenko | 515b42b | 2014-08-07 15:46:31 -0700 | [diff] [blame] | 184 | EXPECT_EQ(id2, dispatch.GetIDs()); |
Vitaly Buka | 2a9b30f | 2015-04-01 10:51:59 -0700 | [diff] [blame] | 185 | Remove(id2); |
Alex Vakulenko | 515b42b | 2014-08-07 15:46:31 -0700 | [diff] [blame] | 186 | EXPECT_EQ("", dispatch.GetIDs()); |
| 187 | } |
| 188 | |
Alex Vakulenko | 5ef7579 | 2015-03-19 15:50:44 -0700 | [diff] [blame] | 189 | TEST_F(CommandQueueTest, Find) { |
Anton Muhin | 5191e81 | 2014-10-30 17:49:48 +0400 | [diff] [blame] | 190 | const std::string id1 = "id1"; |
| 191 | const std::string id2 = "id2"; |
Vitaly Buka | 2a9b30f | 2015-04-01 10:51:59 -0700 | [diff] [blame] | 192 | queue_.Add(CreateDummyCommandInstance("base.reboot", id1)); |
| 193 | queue_.Add(CreateDummyCommandInstance("base.shutdown", id2)); |
| 194 | EXPECT_EQ(nullptr, queue_.Find("dummy")); |
| 195 | auto cmd1 = queue_.Find(id1); |
Alex Vakulenko | aa3a559 | 2014-08-07 07:24:06 -0700 | [diff] [blame] | 196 | EXPECT_NE(nullptr, cmd1); |
| 197 | EXPECT_EQ("base.reboot", cmd1->GetName()); |
Alex Vakulenko | fedc487 | 2014-08-20 12:38:43 -0700 | [diff] [blame] | 198 | EXPECT_EQ(id1, cmd1->GetID()); |
Vitaly Buka | 2a9b30f | 2015-04-01 10:51:59 -0700 | [diff] [blame] | 199 | auto cmd2 = queue_.Find(id2); |
Alex Vakulenko | aa3a559 | 2014-08-07 07:24:06 -0700 | [diff] [blame] | 200 | EXPECT_NE(nullptr, cmd2); |
| 201 | EXPECT_EQ("base.shutdown", cmd2->GetName()); |
Alex Vakulenko | fedc487 | 2014-08-20 12:38:43 -0700 | [diff] [blame] | 202 | EXPECT_EQ(id2, cmd2->GetID()); |
Alex Vakulenko | aa3a559 | 2014-08-07 07:24:06 -0700 | [diff] [blame] | 203 | } |
Vitaly Buka | 2a9b30f | 2015-04-01 10:51:59 -0700 | [diff] [blame] | 204 | |
Vitaly Buka | b6f015a | 2015-07-09 14:59:23 -0700 | [diff] [blame] | 205 | } // namespace weave |