| Alex Deymo | aea4c1c | 2015-08-19 20:24:43 -0700 | [diff] [blame] | 1 | // | 
 | 2 | // Copyright (C) 2009 The Android Open Source Project | 
 | 3 | // | 
 | 4 | // Licensed under the Apache License, Version 2.0 (the "License"); | 
 | 5 | // you may not use this file except in compliance with the License. | 
 | 6 | // You may obtain a copy of the License at | 
 | 7 | // | 
 | 8 | //      http://www.apache.org/licenses/LICENSE-2.0 | 
 | 9 | // | 
 | 10 | // Unless required by applicable law or agreed to in writing, software | 
 | 11 | // distributed under the License is distributed on an "AS IS" BASIS, | 
 | 12 | // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | 
 | 13 | // See the License for the specific language governing permissions and | 
 | 14 | // limitations under the License. | 
 | 15 | // | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 16 |  | 
| Alex Deymo | 39910dc | 2015-11-09 17:04:30 -0800 | [diff] [blame] | 17 | #include "update_engine/common/action_processor.h" | 
| Alex Deymo | aab50e3 | 2014-11-10 19:55:35 -0800 | [diff] [blame] | 18 |  | 
| Alex Deymo | 39910dc | 2015-11-09 17:04:30 -0800 | [diff] [blame] | 19 | #include <string> | 
| Alex Deymo | 14fd1ec | 2016-02-24 22:03:57 -0800 | [diff] [blame] | 20 |  | 
 | 21 | #include <gtest/gtest.h> | 
 | 22 |  | 
| Alex Deymo | 39910dc | 2015-11-09 17:04:30 -0800 | [diff] [blame] | 23 | #include "update_engine/common/action.h" | 
| Alex Deymo | 14fd1ec | 2016-02-24 22:03:57 -0800 | [diff] [blame] | 24 | #include "update_engine/common/mock_action.h" | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 25 |  | 
| Andrew de los Reyes | 4fe15d0 | 2009-12-10 19:01:36 -0800 | [diff] [blame] | 26 | using std::string; | 
 | 27 |  | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 28 | namespace chromeos_update_engine { | 
 | 29 |  | 
 | 30 | using chromeos_update_engine::ActionPipe; | 
 | 31 |  | 
 | 32 | class ActionProcessorTestAction; | 
 | 33 |  | 
 | 34 | template<> | 
 | 35 | class ActionTraits<ActionProcessorTestAction> { | 
 | 36 |  public: | 
 | 37 |   typedef string OutputObjectType; | 
 | 38 |   typedef string InputObjectType; | 
 | 39 | }; | 
 | 40 |  | 
 | 41 | // This is a simple Action class for testing. | 
| Yunlian Jiang | a178e5e | 2013-04-05 14:41:56 -0700 | [diff] [blame] | 42 | class ActionProcessorTestAction : public Action<ActionProcessorTestAction> { | 
 | 43 |  public: | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 44 |   typedef string InputObjectType; | 
 | 45 |   typedef string OutputObjectType; | 
 | 46 |   ActionPipe<string>* in_pipe() { return in_pipe_.get(); } | 
 | 47 |   ActionPipe<string>* out_pipe() { return out_pipe_.get(); } | 
 | 48 |   ActionProcessor* processor() { return processor_; } | 
 | 49 |   void PerformAction() {} | 
 | 50 |   void CompleteAction() { | 
 | 51 |     ASSERT_TRUE(processor()); | 
| Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 52 |     processor()->ActionComplete(this, ErrorCode::kSuccess); | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 53 |   } | 
 | 54 |   string Type() const { return "ActionProcessorTestAction"; } | 
 | 55 | }; | 
 | 56 |  | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 57 | namespace { | 
 | 58 | class MyActionProcessorDelegate : public ActionProcessorDelegate { | 
 | 59 |  public: | 
 | 60 |   explicit MyActionProcessorDelegate(const ActionProcessor* processor) | 
| Andrew de los Reyes | 3270f74 | 2010-07-15 22:28:14 -0700 | [diff] [blame] | 61 |       : processor_(processor), | 
 | 62 |         processing_done_called_(false), | 
 | 63 |         processing_stopped_called_(false), | 
 | 64 |         action_completed_called_(false), | 
| Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 65 |         action_exit_code_(ErrorCode::kError) {} | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 66 |  | 
| Darin Petkov | c1a8b42 | 2010-07-19 11:34:49 -0700 | [diff] [blame] | 67 |   virtual void ProcessingDone(const ActionProcessor* processor, | 
| David Zeuthen | a99981f | 2013-04-29 13:42:47 -0700 | [diff] [blame] | 68 |                               ErrorCode code) { | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 69 |     EXPECT_EQ(processor_, processor); | 
 | 70 |     EXPECT_FALSE(processing_done_called_); | 
 | 71 |     processing_done_called_ = true; | 
 | 72 |   } | 
 | 73 |   virtual void ProcessingStopped(const ActionProcessor* processor) { | 
 | 74 |     EXPECT_EQ(processor_, processor); | 
 | 75 |     EXPECT_FALSE(processing_stopped_called_); | 
 | 76 |     processing_stopped_called_ = true; | 
 | 77 |   } | 
| adlr@google.com | c98a7ed | 2009-12-04 18:54:03 +0000 | [diff] [blame] | 78 |   virtual void ActionCompleted(ActionProcessor* processor, | 
 | 79 |                                AbstractAction* action, | 
| David Zeuthen | a99981f | 2013-04-29 13:42:47 -0700 | [diff] [blame] | 80 |                                ErrorCode code) { | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 81 |     EXPECT_EQ(processor_, processor); | 
 | 82 |     EXPECT_FALSE(action_completed_called_); | 
 | 83 |     action_completed_called_ = true; | 
| Darin Petkov | c1a8b42 | 2010-07-19 11:34:49 -0700 | [diff] [blame] | 84 |     action_exit_code_ = code; | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 85 |   } | 
 | 86 |  | 
 | 87 |   const ActionProcessor* processor_; | 
 | 88 |   bool processing_done_called_; | 
 | 89 |   bool processing_stopped_called_; | 
 | 90 |   bool action_completed_called_; | 
| David Zeuthen | a99981f | 2013-04-29 13:42:47 -0700 | [diff] [blame] | 91 |   ErrorCode action_exit_code_; | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 92 | }; | 
| Alex Vakulenko | d2779df | 2014-06-16 13:19:00 -0700 | [diff] [blame] | 93 | }  // namespace | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 94 |  | 
| Alex Deymo | 14fd1ec | 2016-02-24 22:03:57 -0800 | [diff] [blame] | 95 | class ActionProcessorTest : public ::testing::Test { | 
 | 96 |   void SetUp() override { | 
 | 97 |     action_processor_.set_delegate(&delegate_); | 
 | 98 |     // Silence Type() calls used for logging. | 
 | 99 |     EXPECT_CALL(mock_action_, Type()).Times(testing::AnyNumber()); | 
 | 100 |   } | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 101 |  | 
| Alex Deymo | 14fd1ec | 2016-02-24 22:03:57 -0800 | [diff] [blame] | 102 |   void TearDown() override { | 
 | 103 |     action_processor_.set_delegate(nullptr); | 
 | 104 |   } | 
 | 105 |  | 
 | 106 |  protected: | 
 | 107 |   // The ActionProcessor under test. | 
 | 108 |   ActionProcessor action_processor_; | 
 | 109 |  | 
 | 110 |   MyActionProcessorDelegate delegate_{&action_processor_}; | 
 | 111 |  | 
 | 112 |   // Common actions used during most tests. | 
 | 113 |   testing::StrictMock<MockAction> mock_action_; | 
 | 114 |   ActionProcessorTestAction action_; | 
 | 115 | }; | 
 | 116 |  | 
 | 117 | TEST_F(ActionProcessorTest, SimpleTest) { | 
 | 118 |   EXPECT_FALSE(action_processor_.IsRunning()); | 
 | 119 |   action_processor_.EnqueueAction(&action_); | 
 | 120 |   EXPECT_FALSE(action_processor_.IsRunning()); | 
 | 121 |   EXPECT_FALSE(action_.IsRunning()); | 
 | 122 |   action_processor_.StartProcessing(); | 
 | 123 |   EXPECT_TRUE(action_processor_.IsRunning()); | 
 | 124 |   EXPECT_TRUE(action_.IsRunning()); | 
 | 125 |   EXPECT_EQ(action_processor_.current_action(), &action_); | 
 | 126 |   action_.CompleteAction(); | 
 | 127 |   EXPECT_FALSE(action_processor_.IsRunning()); | 
 | 128 |   EXPECT_FALSE(action_.IsRunning()); | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 129 | } | 
 | 130 |  | 
| Alex Deymo | 14fd1ec | 2016-02-24 22:03:57 -0800 | [diff] [blame] | 131 | TEST_F(ActionProcessorTest, DelegateTest) { | 
 | 132 |   action_processor_.EnqueueAction(&action_); | 
 | 133 |   action_processor_.StartProcessing(); | 
 | 134 |   action_.CompleteAction(); | 
 | 135 |   EXPECT_TRUE(delegate_.processing_done_called_); | 
 | 136 |   EXPECT_TRUE(delegate_.action_completed_called_); | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 137 | } | 
 | 138 |  | 
| Alex Deymo | 14fd1ec | 2016-02-24 22:03:57 -0800 | [diff] [blame] | 139 | TEST_F(ActionProcessorTest, StopProcessingTest) { | 
 | 140 |   action_processor_.EnqueueAction(&action_); | 
 | 141 |   action_processor_.StartProcessing(); | 
 | 142 |   action_processor_.StopProcessing(); | 
 | 143 |   EXPECT_TRUE(delegate_.processing_stopped_called_); | 
 | 144 |   EXPECT_FALSE(delegate_.action_completed_called_); | 
 | 145 |   EXPECT_FALSE(action_processor_.IsRunning()); | 
 | 146 |   EXPECT_EQ(nullptr, action_processor_.current_action()); | 
 | 147 | } | 
 | 148 |  | 
 | 149 | TEST_F(ActionProcessorTest, ChainActionsTest) { | 
 | 150 |   // This test doesn't use a delegate since it terminates several actions. | 
 | 151 |   action_processor_.set_delegate(nullptr); | 
 | 152 |  | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 153 |   ActionProcessorTestAction action1, action2; | 
| Alex Deymo | 14fd1ec | 2016-02-24 22:03:57 -0800 | [diff] [blame] | 154 |   action_processor_.EnqueueAction(&action1); | 
 | 155 |   action_processor_.EnqueueAction(&action2); | 
 | 156 |   action_processor_.StartProcessing(); | 
 | 157 |   EXPECT_EQ(&action1, action_processor_.current_action()); | 
 | 158 |   EXPECT_TRUE(action_processor_.IsRunning()); | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 159 |   action1.CompleteAction(); | 
| Alex Deymo | 14fd1ec | 2016-02-24 22:03:57 -0800 | [diff] [blame] | 160 |   EXPECT_EQ(&action2, action_processor_.current_action()); | 
 | 161 |   EXPECT_TRUE(action_processor_.IsRunning()); | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 162 |   action2.CompleteAction(); | 
| Alex Deymo | 14fd1ec | 2016-02-24 22:03:57 -0800 | [diff] [blame] | 163 |   EXPECT_EQ(nullptr, action_processor_.current_action()); | 
 | 164 |   EXPECT_FALSE(action_processor_.IsRunning()); | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 165 | } | 
 | 166 |  | 
| Alex Deymo | 14fd1ec | 2016-02-24 22:03:57 -0800 | [diff] [blame] | 167 | TEST_F(ActionProcessorTest, DtorTest) { | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 168 |   ActionProcessorTestAction action1, action2; | 
 | 169 |   { | 
 | 170 |     ActionProcessor action_processor; | 
 | 171 |     action_processor.EnqueueAction(&action1); | 
 | 172 |     action_processor.EnqueueAction(&action2); | 
 | 173 |     action_processor.StartProcessing(); | 
 | 174 |   } | 
| Alex Vakulenko | 88b591f | 2014-08-28 16:48:57 -0700 | [diff] [blame] | 175 |   EXPECT_EQ(nullptr, action1.processor()); | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 176 |   EXPECT_FALSE(action1.IsRunning()); | 
| Alex Vakulenko | 88b591f | 2014-08-28 16:48:57 -0700 | [diff] [blame] | 177 |   EXPECT_EQ(nullptr, action2.processor()); | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 178 |   EXPECT_FALSE(action2.IsRunning()); | 
 | 179 | } | 
 | 180 |  | 
| Alex Deymo | 14fd1ec | 2016-02-24 22:03:57 -0800 | [diff] [blame] | 181 | TEST_F(ActionProcessorTest, DefaultDelegateTest) { | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 182 |   // Just make sure it doesn't crash | 
| Alex Deymo | 14fd1ec | 2016-02-24 22:03:57 -0800 | [diff] [blame] | 183 |   action_processor_.EnqueueAction(&action_); | 
 | 184 |   action_processor_.StartProcessing(); | 
 | 185 |   action_.CompleteAction(); | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 186 |  | 
| Alex Deymo | 14fd1ec | 2016-02-24 22:03:57 -0800 | [diff] [blame] | 187 |   action_processor_.EnqueueAction(&action_); | 
 | 188 |   action_processor_.StartProcessing(); | 
 | 189 |   action_processor_.StopProcessing(); | 
 | 190 | } | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 191 |  | 
| Alex Deymo | 14fd1ec | 2016-02-24 22:03:57 -0800 | [diff] [blame] | 192 | // This test suspends and resume the action processor while running one action_. | 
 | 193 | TEST_F(ActionProcessorTest, SuspendResumeTest) { | 
 | 194 |   action_processor_.EnqueueAction(&mock_action_); | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 195 |  | 
| Alex Deymo | 14fd1ec | 2016-02-24 22:03:57 -0800 | [diff] [blame] | 196 |   testing::InSequence s; | 
 | 197 |   EXPECT_CALL(mock_action_, PerformAction()); | 
 | 198 |   action_processor_.StartProcessing(); | 
 | 199 |  | 
 | 200 |   EXPECT_CALL(mock_action_, SuspendAction()); | 
 | 201 |   action_processor_.SuspendProcessing(); | 
 | 202 |   // Suspending the processor twice should not suspend the action twice. | 
 | 203 |   action_processor_.SuspendProcessing(); | 
 | 204 |  | 
 | 205 |   // IsRunning should return whether there's is an action doing some work, even | 
 | 206 |   // if it is suspended. | 
 | 207 |   EXPECT_TRUE(action_processor_.IsRunning()); | 
 | 208 |   EXPECT_EQ(&mock_action_, action_processor_.current_action()); | 
 | 209 |  | 
 | 210 |   EXPECT_CALL(mock_action_, ResumeAction()); | 
 | 211 |   action_processor_.ResumeProcessing(); | 
 | 212 |  | 
 | 213 |   // Calling ResumeProcessing twice should not affect the action_. | 
 | 214 |   action_processor_.ResumeProcessing(); | 
 | 215 |  | 
 | 216 |   action_processor_.ActionComplete(&mock_action_, ErrorCode::kSuccess); | 
 | 217 | } | 
 | 218 |  | 
 | 219 | // This test suspends an action that presumably doesn't support suspend/resume | 
 | 220 | // and it finished before being resumed. | 
 | 221 | TEST_F(ActionProcessorTest, ActionCompletedWhileSuspendedTest) { | 
 | 222 |   action_processor_.EnqueueAction(&mock_action_); | 
 | 223 |  | 
 | 224 |   testing::InSequence s; | 
 | 225 |   EXPECT_CALL(mock_action_, PerformAction()); | 
 | 226 |   action_processor_.StartProcessing(); | 
 | 227 |  | 
 | 228 |   EXPECT_CALL(mock_action_, SuspendAction()); | 
 | 229 |   action_processor_.SuspendProcessing(); | 
 | 230 |  | 
 | 231 |   // Simulate the action completion while suspended. No other call to | 
 | 232 |   // |mock_action_| is expected at this point. | 
 | 233 |   action_processor_.ActionComplete(&mock_action_, ErrorCode::kSuccess); | 
 | 234 |  | 
 | 235 |   // The processing should not be done since the ActionProcessor is suspended | 
 | 236 |   // and the processing is considered to be still running until resumed. | 
 | 237 |   EXPECT_FALSE(delegate_.processing_done_called_); | 
 | 238 |   EXPECT_TRUE(action_processor_.IsRunning()); | 
 | 239 |  | 
 | 240 |   action_processor_.ResumeProcessing(); | 
 | 241 |   EXPECT_TRUE(delegate_.processing_done_called_); | 
 | 242 |   EXPECT_FALSE(delegate_.processing_stopped_called_); | 
 | 243 | } | 
 | 244 |  | 
 | 245 | TEST_F(ActionProcessorTest, StoppedWhileSuspendedTest) { | 
 | 246 |   action_processor_.EnqueueAction(&mock_action_); | 
 | 247 |  | 
 | 248 |   testing::InSequence s; | 
 | 249 |   EXPECT_CALL(mock_action_, PerformAction()); | 
 | 250 |   action_processor_.StartProcessing(); | 
 | 251 |   EXPECT_CALL(mock_action_, SuspendAction()); | 
 | 252 |   action_processor_.SuspendProcessing(); | 
 | 253 |  | 
 | 254 |   EXPECT_CALL(mock_action_, TerminateProcessing()); | 
 | 255 |   action_processor_.StopProcessing(); | 
 | 256 |   // Stopping the processing should abort the current execution no matter what. | 
 | 257 |   EXPECT_TRUE(delegate_.processing_stopped_called_); | 
 | 258 |   EXPECT_FALSE(delegate_.processing_done_called_); | 
 | 259 |   EXPECT_FALSE(delegate_.action_completed_called_); | 
 | 260 |   EXPECT_FALSE(action_processor_.IsRunning()); | 
 | 261 |   EXPECT_EQ(nullptr, action_processor_.current_action()); | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 262 | } | 
 | 263 |  | 
 | 264 | }  // namespace chromeos_update_engine |