MAGETWO-89567: Cannot set 'user' save handler by ini_set()

- fix integration test failure

Author: Joan He

dev/tests/integration/testsuite/Magento/Framework/Session/ConfigTest.php

@@ -317,8 +317,7 @@ public function testConstructor(
             $expectedSavePath,
             $givenSaveHandler,
             $expectedSaveHandler
-        )
-        {
+        ) {
             global $mockPHPFunctions;
             $mockPHPFunctions = $mockPHPFunctionNum;
 

dev/tests/integration/testsuite/Magento/Framework/Session/SessionManagerTest.php

@@ -12,7 +12,6 @@
 
     use Magento\Framework\App\DeploymentConfig;
     use Magento\Framework\App\State;
-    use Magento\Framework\Session\Config\ConfigInterface;
 
     // @codingStandardsIgnoreEnd
 
@@ -124,21 +123,6 @@ protected function setUp()
             ini_set('session.name', $this->sessionName);
 
             $this->objectManager = \Magento\TestFramework\Helper\Bootstrap::getObjectManager();
-            $deploymentConfigMock = $this->createMock(DeploymentConfig::class);
-            $deploymentConfigMock->method('get')
-                ->willReturnCallback(function ($configPath) {
-                    switch ($configPath) {
-                        case Config::PARAM_SESSION_SAVE_METHOD:
-                            return 'db';
-                        case Config::PARAM_SESSION_CACHE_LIMITER:
-                            return 'private_no_expire';
-                        case Config::PARAM_SESSION_SAVE_PATH:
-                            return 'explicit_save_path';
-                        default:
-                            return null;
-                    }
-                });
-            $this->objectManager->addSharedInstance($deploymentConfigMock, DeploymentConfig::class);
 
             /** @var \Magento\Framework\Session\SidResolverInterface $sidResolver */
             $this->appState = $this->getMockBuilder(State::class)
@@ -167,7 +151,6 @@ protected function tearDown()
                 $this->model->destroy();
                 $this->model = null;
             }
-            $this->objectManager->removeSharedInstance(DeploymentConfig::class);
         }
 
         public function testSessionNameFromIni()
@@ -321,13 +304,32 @@ public function testConstructor()
         {
             global $mockPHPFunctions;
             $mockPHPFunctions = true;
+
+            $deploymentConfigMock = $this->createMock(DeploymentConfig::class);
+            $deploymentConfigMock->method('get')
+                ->willReturnCallback(function ($configPath) {
+                    switch ($configPath) {
+                        case Config::PARAM_SESSION_SAVE_METHOD:
+                            return 'db';
+                        case Config::PARAM_SESSION_CACHE_LIMITER:
+                            return 'private_no_expire';
+                        case Config::PARAM_SESSION_SAVE_PATH:
+                            return 'explicit_save_path';
+                        default:
+                            return null;
+                    }
+                });
+            $sessionConfig = $this->objectManager->create(Config::class, ['deploymentConfig' => $deploymentConfigMock]);
+            $saveHandler = $this->objectManager->create(SaveHandler::class, ['sessionConfig' => $sessionConfig]);
+
             $this->model = $this->objectManager->create(
                 \Magento\Framework\Session\SessionManager::class,
                 [
-                    'sidResolver' => $this->sidResolver
+                    'sidResolver' => $this->sidResolver,
+                    'saveHandler' => $saveHandler,
+                    'sessionConfig' => $sessionConfig,
                 ]
             );
-            $sessionConfig = $this->objectManager->get(ConfigInterface::class);
             $this->assertEquals('db', $sessionConfig->getOption('session.save_handler'));
             $this->assertEquals('private_no_expire', $sessionConfig->getOption('session.cache_limiter'));
             $this->assertEquals('explicit_save_path', $sessionConfig->getOption('session.save_path'));