Test: Fix test not resetting on removing test strategy.

Author: mostlikely4r

playerbot/strategy/Action.h

@@ -72,7 +72,7 @@ namespace ai
         virtual NextAction** getContinuers() { return NULL; }
         virtual ActionThreatType getThreatType() { return ActionThreatType::ACTION_THREAT_NONE; }
         void Update() {}  //Nonfunctional see AiObjectContext::Update() to enable.
-        void Reset() {}
+        virtual void Reset() {}
         virtual Unit* GetTarget();
         virtual Value<Unit*>* GetTargetValue();
         virtual std::string GetTargetName() { return "self target"; }

playerbot/strategy/tests/TestAction.h

@@ -18,6 +18,7 @@ namespace ai
         virtual bool isUseful() override { return true; }
         virtual bool isPossible() override { return bot && bot->IsAlive(); }
 
+        virtual void Reset() override { ctx.Reset(); }
 #ifdef GenerateBotHelp
         virtual std::string GetHelpName() { return "test"; }
         virtual std::string GetHelpDescription()

playerbot/strategy/tests/TestStrategy.cpp

@@ -11,3 +11,14 @@ void TestStrategy::InitNonCombatTriggers(std::list<TriggerNode*>& triggers)
         "test ready",
         NextAction::array(0, new NextAction("test", 100.0f), NULL)));
 }
+
+void TestStrategy::OnStrategyRemoved(BotState state)
+{
+    AiObjectContext* context = ai->GetAiObjectContext();
+    Action* action = context->GetAction("test");
+
+    if (action)
+    {
+        action->Reset();
+    };
+}

playerbot/strategy/tests/TestStrategy.h

@@ -15,6 +15,8 @@ namespace ai
         virtual void InitCombatTriggers(std::list<TriggerNode*>& triggers) override { InitNonCombatTriggers(triggers); }
         virtual void InitDeadTriggers(std::list<TriggerNode*>& triggers) override { InitNonCombatTriggers(triggers); };
 
+
+        virtual void OnStrategyRemoved(BotState state);
 #ifdef GenerateBotHelp
         virtual std::string GetHelpName() { return "test"; }
         virtual std::string GetHelpDescription()