MAGETWO-51068: Admin User session expired despite Session Lifetime

Annotation

Author: Igor Melnikov

lib/internal/Magento/Framework/Session/SaveHandler/Redis/Config.php

@@ -101,19 +101,19 @@ class Config implements \Cm\RedisSession\Handler\ConfigInterface
     const PARAM_BREAK_AFTER             = 'session/redis/break_after';
 
     /**
-     * Cookie Lifetime config path
+     * Cookie lifetime config path
      */
     const XML_PATH_COOKIE_LIFETIME = 'web/cookie/cookie_lifetime';
 
     /**
-     * Admin Session Lifetime config path
+     * Admin session lifetime config path
      */
     const XML_PATH_ADMIN_SESSION_LIFETIME = 'admin/security/session_lifetime';
 
     /**
-     * Max Admin Session Lifetime
+     * Max admin session lifetime
      */
-    const ADMIN_SESSION_MAX_LIFETIME = 31536000; //One year
+    const ADMIN_SESSION_MAX_LIFETIME = 31536000;
 
     /**
      * Deployment config

lib/internal/Magento/Framework/Session/Test/Unit/SaveHandler/Redis/ConfigTest.php

@@ -12,22 +12,22 @@ class ConfigTest extends \PHPUnit_Framework_TestCase
     /**
      * @var \Magento\Framework\App\DeploymentConfig|\PHPUnit_Framework_MockObject_MockObject
      */
-    protected $deploymentConfigMock;
+    private $deploymentConfigMock;
 
     /**
      * @var \Magento\Framework\App\State|\PHPUnit_Framework_MockObject_MockObject
      */
-    protected $appStateMock;
+    private $appStateMock;
 
     /**
      * @var \Magento\Framework\App\Config|\PHPUnit_Framework_MockObject_MockObject
      */
-    protected $scopeConfigMock;
+    private $scopeConfigMock;
 
     /**
      * @var \Magento\Framework\Session\SaveHandler\Redis\Config
      */
-    protected $config;
+    private $config;
 
     public function setUp()
     {
@@ -218,8 +218,8 @@ public function testBreakAfter()
 
     public function testGetLifetimeAdmin()
     {
-        $expectedLifetime = 12345;
         $areaCode = 'adminhtml';
+        $expectedLifetime = 123;
         $this->appStateMock->expects($this->once())
             ->method('getAreaCode')
             ->willReturn($areaCode);
@@ -232,8 +232,8 @@ public function testGetLifetimeAdmin()
 
     public function testGetLifetimeFrontend()
     {
-        $expectedLifetime = 3010;
         $areaCode = 'frontend';
+        $expectedLifetime = 234;
         $this->appStateMock->expects($this->once())
             ->method('getAreaCode')
             ->willReturn($areaCode);