Noissue - remove JsEngine as a member of UpdateCheckTest

test/UpdateCheck.cpp

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2017 eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 20 matching lines @@
       source.replace(pos, find.size(), replace);
   }
 
   class UpdateCheckTest : public ::testing::Test
   {
   protected:
     AdblockPlus::AppInfo appInfo;
     AdblockPlus::ServerResponse webRequestResponse;
     DelayedWebRequest::SharedTasks webRequestTasks;
     DelayedTimer::SharedTasks timerTasks;
-    AdblockPlus::JsEnginePtr jsEngine;
     FilterEnginePtr filterEngine;
 
     bool eventCallbackCalled;
     AdblockPlus::JsValueList eventCallbackParams;
     bool updateCallbackCalled;
     std::string updateError;
 
     void SetUp()
     {
       eventCallbackCalled = false;
       updateCallbackCalled = false;
     }
 
     void CreateFilterEngine()
     {
       JsEngineCreationParameters jsEngineParams;
       jsEngineParams.appInfo = appInfo;
       jsEngineParams.logSystem.reset(new LazyLogSystem());
       jsEngineParams.fileSystem.reset(new LazyFileSystem());
       jsEngineParams.timer = DelayedTimer::New(timerTasks);
       jsEngineParams.webRequest = DelayedWebRequest::New(webRequestTasks);
-      jsEngine = CreateJsEngine(std::move(jsEngineParams));
+      auto jsEngine = CreateJsEngine(std::move(jsEngineParams));
       jsEngine->SetEventCallback("updateAvailable", [this](JsValueList&& params)
       {
         eventCallbackCalled = true;
         eventCallbackParams = std::move(params);
       });
 
       filterEngine = AdblockPlus::FilterEngine::Create(jsEngine);
     }
 
     // Returns a URL or the empty string if there is no such request.
@@ skipping 39 matching lines @@
   CreateFilterEngine();
   ForceUpdateCheck();
 
   auto requestUrl = ProcessPendingUpdateWebRequest();
 
   ASSERT_FALSE(eventCallbackCalled);
   ASSERT_TRUE(updateCallbackCalled);
   ASSERT_FALSE(updateError.empty());
 
   std::string expectedUrl(filterEngine->GetPref("update_url_release").AsString());
-  std::string platform = jsEngine->Evaluate("require('info').platform").AsString();
-  std::string platformVersion = jsEngine->Evaluate("require('info').platformVersion").AsString();
+  std::string platform = filterEngine->GetJsEngine()->Evaluate("require('info').platform").AsString();
+  std::string platformVersion = filterEngine->GetJsEngine()->Evaluate("require('info').platformVersion").AsString();
 
   FindAndReplace(expectedUrl, "%NAME%", appInfo.name);
   FindAndReplace(expectedUrl, "%TYPE%", "1");   // manual update
   expectedUrl += "&addonName=" + appInfo.name +
                  "&addonVersion=" + appInfo.version +
                  "&application=" + appInfo.application +
                  "&applicationVersion=" + appInfo.applicationVersion +
                  "&platform=" + platform +
                  "&platformVersion=" + platformVersion +
                  "&lastVersion=0&downloadCount=0";
@@ skipping 17 matching lines @@
 
   auto requestUrl = ProcessPendingUpdateWebRequest();
 
   ASSERT_TRUE(eventCallbackCalled);
   ASSERT_EQ(1u, eventCallbackParams.size());
   ASSERT_EQ("https://foo.bar/", eventCallbackParams[0].AsString());
   ASSERT_TRUE(updateCallbackCalled);
   ASSERT_TRUE(updateError.empty());
 
   std::string expectedUrl(filterEngine->GetPref("update_url_devbuild").AsString());
-  std::string platform = jsEngine->Evaluate("require('info').platform").AsString();
-  std::string platformVersion = jsEngine->Evaluate("require('info').platformVersion").AsString();
+  std::string platform = filterEngine->GetJsEngine()->Evaluate("require('info').platform").AsString();
+  std::string platformVersion = filterEngine->GetJsEngine()->Evaluate("require('info').platformVersion").AsString();
 
   FindAndReplace(expectedUrl, "%NAME%", appInfo.name);
   FindAndReplace(expectedUrl, "%TYPE%", "1");   // manual update
   expectedUrl += "&addonName=" + appInfo.name +
                  "&addonVersion=" + appInfo.version +
                  "&application=" + appInfo.application +
                  "&applicationVersion=" + appInfo.applicationVersion +
                  "&platform=" + platform +
                  "&platformVersion=" + platformVersion +
                  "&lastVersion=0&downloadCount=0";
@@ skipping 126 matching lines @@
 
   // ensure that the was the corresponding request
   EXPECT_FALSE(ProcessPendingUpdateWebRequest().empty());
 
   EXPECT_FALSE(eventCallbackCalled);
   EXPECT_EQ(1, timesCalled);
 
   // previous handler is not restored
   EXPECT_FALSE(eventCallbackCalled);
 }