Unlocking of any wallet should refresh timeout for given session. Rel…

…ated to clive#235

programs/beekeeper/core/beekeeper_wallet_manager.cpp

@@ -85,6 +85,7 @@ void beekeeper_wallet_manager::lock( const std::string& token, const std::string
 
 void beekeeper_wallet_manager::unlock( const std::string& token, const std::string& wallet_name, const std::string& password )
 {
+  sessions->refresh_timeout( token );
   sessions->get_wallet_manager( token )->unlock( wallet_name, password );
 }
 

programs/beekeeper/core/include/core/session_base.hpp

@@ -38,6 +38,7 @@ class session_base
 
     void set_timeout( const std::chrono::seconds& t );
     void check_timeout();
+    void refresh_timeout();
 
     info get_info();
 

programs/beekeeper/core/include/core/session_manager_base.hpp

@@ -37,6 +37,7 @@ class session_manager_base
 
     void set_timeout( const std::string& token, const std::chrono::seconds& t );
     void check_timeout( const std::string& token );
+    void refresh_timeout( const std::string& token );
     info get_info( const std::string& token );
 
     std::shared_ptr<wallet_manager_impl> get_wallet_manager( const std::string& token );

programs/beekeeper/core/session_base.cpp

@@ -22,6 +22,11 @@ void session_base::check_timeout()
   refresh_timeout( true/*refresh_only_active*/ );
 }
 
+void session_base::refresh_timeout()
+{
+  refresh_timeout( false/*refresh_only_active*/ );
+}
+
 void session_base::refresh_timeout( bool refresh_only_active )
 {
   auto _now = std::chrono::system_clock::now();

programs/beekeeper/core/session_manager_base.cpp

@@ -72,6 +72,11 @@ void session_manager_base::check_timeout( const std::string& token )
   get_session( token )->check_timeout();
 }
 
+void session_manager_base::refresh_timeout( const std::string& token )
+{
+  get_session( token )->refresh_timeout();
+}
+
 info session_manager_base::get_info( const std::string& token )
 {
   return get_session( token )->get_info();

tests/unit/tests/beekeeper_tests.cpp

@@ -1553,6 +1553,25 @@ BOOST_AUTO_TEST_CASE(beekeeper_timeout_list_wallets)
       const size_t _wallet_cnt = 0;
       BOOST_REQUIRE_EQUAL( _wallets[_wallet_cnt].unlocked, false );
     }
+    {
+      const size_t _wallet_cnt = 1;
+      _beekeeper.unlock( _token, _wallets[_wallet_cnt].name, _wallets[_wallet_cnt].password );
+    }
+    {
+      auto _wallets = _beekeeper.list_wallets( _token );
+      BOOST_REQUIRE_EQUAL( _wallets.size(), 1 );
+      const size_t _wallet_cnt = 0;
+      BOOST_REQUIRE_EQUAL( _wallets[_wallet_cnt].unlocked, true );
+    }
+    {
+      std::this_thread::sleep_for( std::chrono::milliseconds(1200) );
+    }
+    {
+      auto _wallets = _beekeeper.list_wallets( _token );
+      BOOST_REQUIRE_EQUAL( _wallets.size(), 1 );
+      const size_t _wallet_cnt = 0;
+      BOOST_REQUIRE_EQUAL( _wallets[_wallet_cnt].unlocked, false );
+    }
   } FC_LOG_AND_RETHROW()
 }