hybrid_platforms_conductor 33.0.0 → 33.0.1

data/spec/hybrid_platforms_conductor_test/api/actions_executor/actions_spec.rb

@@ -1,17 +1,15 @@
 describe HybridPlatformsConductor::ActionsExecutor do
 
-  context 'checking actions handling' do
+  context 'when checking actions handling' do
 
     # Instantiate a test platform, with the test action registered in Actions Executor.
     #
     # Parameters::
-    # * Proc: Code called with the environment ready
+    # * *block* (Proc): Code called with the environment ready
     #   * Parameters::
     #     * *repository* (String): Path to the repository
-    def with_test_platform_for_actions
-      with_test_platform_for_executor(nodes: { 'node1' => {}, 'node2' => {}, 'node3' => {} }) do |repository|
-        yield repository
-      end
+    def with_test_platform_for_actions(&block)
+      with_test_platform_for_executor(nodes: { 'node1' => {}, 'node2' => {}, 'node3' => {} }, &block)
     end
 
     it 'executes a simple action on 1 node' do
@@ -40,11 +38,13 @@ describe HybridPlatformsConductor::ActionsExecutor do
 
     it 'executes several actions on 1 node' do
       with_test_platform_for_actions do
-        test_actions_executor.execute_actions('node1' => [
-          { test_action: 'Action 1 executed' },
-          { test_action: 'Action 2 executed' },
-          { test_action: 'Action 3 executed' }
-        ])
+        test_actions_executor.execute_actions(
+          'node1' => [
+            { test_action: 'Action 1 executed' },
+            { test_action: 'Action 2 executed' },
+            { test_action: 'Action 3 executed' }
+          ]
+        )
         expect(action_executions).to eq [
           { node: 'node1', message: 'Action 1 executed' },
           { node: 'node1', message: 'Action 2 executed' },
@@ -71,26 +71,29 @@ describe HybridPlatformsConductor::ActionsExecutor do
     it 'executes several actions of different types' do
       with_test_platform_for_actions do
         actions_executed = []
-        expect(test_actions_executor.execute_actions('node1' => [
-          { ruby: proc do |stdout, stderr|
-            stdout << 'action1_stdout '
-            stderr << 'action1_stderr '
-            actions_executed << 'action1'
-          end },
-          { bash: 'echo action2_stdout' },
-          { ruby: proc do |stdout, stderr|
-            stdout << 'action3_stdout'
-            stderr << 'action3_stderr'
-            actions_executed << 'action3'
-          end }
-        ])).to eq('node1' => [0, "action1_stdout action2_stdout\naction3_stdout", 'action1_stderr action3_stderr'])
+        expect(
+          test_actions_executor.execute_actions(
+            'node1' => [
+              { ruby: proc do |stdout, stderr|
+                stdout << 'action1_stdout '
+                stderr << 'action1_stderr '
+                actions_executed << 'action1'
+              end },
+              { bash: 'echo action2_stdout' },
+              { ruby: proc do |stdout, stderr|
+                stdout << 'action3_stdout'
+                stderr << 'action3_stderr'
+                actions_executed << 'action3'
+              end }
+            ]
+          )
+        ).to eq('node1' => [0, "action1_stdout action2_stdout\naction3_stdout", 'action1_stderr action3_stderr'])
         expect(actions_executed).to eq %w[action1 action3]
       end
     end
 
     it 'executes several actions on 1 node specified using different selectors' do
       with_test_platform_for_actions do
-        actions_executed = []
         test_actions_executor.execute_actions(
           'node1' => { test_action: 'Action 1 executed' },
           '/node1/' => { test_action: 'Action 2 executed' }
@@ -125,11 +128,13 @@ describe HybridPlatformsConductor::ActionsExecutor do
 
     it 'returns errors without failing for actions having command issues' do
       with_test_platform_for_actions do
-        expect(test_actions_executor.execute_actions(
-          'node1' => { test_action: { code: proc { |stdout| stdout << 'Action 1' } } },
-          'node2' => { test_action: { code: proc { raise HybridPlatformsConductor::CmdRunner::UnexpectedExitCodeError, 'Command returned 1' } } },
-          'node3' => { test_action: { code: proc { |stdout| stdout << 'Action 3' } } }
-        )).to eq(
+        expect(
+          test_actions_executor.execute_actions(
+            'node1' => { test_action: { code: proc { |stdout| stdout << 'Action 1' } } },
+            'node2' => { test_action: { code: proc { raise HybridPlatformsConductor::CmdRunner::UnexpectedExitCodeError, 'Command returned 1' } } },
+            'node3' => { test_action: { code: proc { |stdout| stdout << 'Action 3' } } }
+          )
+        ).to eq(
           'node1' => [0, 'Action 1', ''],
           'node2' => [:failed_command, '', "Command returned 1\n"],
           'node3' => [0, 'Action 3', '']
@@ -139,11 +144,13 @@ describe HybridPlatformsConductor::ActionsExecutor do
 
     it 'returns errors without failing for actions having timeout issues' do
       with_test_platform_for_actions do
-        expect(test_actions_executor.execute_actions(
-          'node1' => { test_action: { code: proc { |stdout| stdout << 'Action 1' } } },
-          'node2' => { test_action: { code: proc { raise HybridPlatformsConductor::CmdRunner::TimeoutError } } },
-          'node3' => { test_action: { code: proc { |stdout| stdout << 'Action 3' } } }
-        )).to eq(
+        expect(
+          test_actions_executor.execute_actions(
+            'node1' => { test_action: { code: proc { |stdout| stdout << 'Action 1' } } },
+            'node2' => { test_action: { code: proc { raise HybridPlatformsConductor::CmdRunner::TimeoutError } } },
+            'node3' => { test_action: { code: proc { |stdout| stdout << 'Action 3' } } }
+          )
+        ).to eq(
           'node1' => [0, 'Action 1', ''],
           'node2' => [:timeout, '', ''],
           'node3' => [0, 'Action 3', '']
@@ -153,11 +160,13 @@ describe HybridPlatformsConductor::ActionsExecutor do
 
     it 'returns errors without failing for actions having connection issues' do
       with_test_platform_for_actions do
-        expect(test_actions_executor.execute_actions(
-          'node1' => { test_action: { code: proc { |stdout| stdout << 'Action 1' } } },
-          'node2' => { test_action: { code: proc { raise HybridPlatformsConductor::ActionsExecutor::ConnectionError, 'Can\'t connect' } } },
-          'node3' => { test_action: { code: proc { |stdout| stdout << 'Action 3' } } }
-        )).to eq(
+        expect(
+          test_actions_executor.execute_actions(
+            'node1' => { test_action: { code: proc { |stdout| stdout << 'Action 1' } } },
+            'node2' => { test_action: { code: proc { raise HybridPlatformsConductor::ActionsExecutor::ConnectionError, 'Can\'t connect' } } },
+            'node3' => { test_action: { code: proc { |stdout| stdout << 'Action 3' } } }
+          )
+        ).to eq(
           'node1' => [0, 'Action 1', ''],
           'node2' => [:connection_error, '', "Can't connect\n"],
           'node3' => [0, 'Action 3', '']
@@ -167,11 +176,13 @@ describe HybridPlatformsConductor::ActionsExecutor do
 
     it 'returns errors without failing for actions having unhandled exceptions' do
       with_test_platform_for_actions do
-        expect(test_actions_executor.execute_actions(
-          'node1' => { test_action: { code: proc { |stdout| stdout << 'Action 1' } } },
-          'node2' => { test_action: { code: proc { raise 'Strange error' } } },
-          'node3' => { test_action: { code: proc { |stdout| stdout << 'Action 3' } } }
-        )).to eq(
+        expect(
+          test_actions_executor.execute_actions(
+            'node1' => { test_action: { code: proc { |stdout| stdout << 'Action 1' } } },
+            'node2' => { test_action: { code: proc { raise 'Strange error' } } },
+            'node3' => { test_action: { code: proc { |stdout| stdout << 'Action 3' } } }
+          )
+        ).to eq(
           'node1' => [0, 'Action 1', ''],
           'node2' => [:failed_action, '', "Strange error\n"],
           'node3' => [0, 'Action 3', '']
@@ -182,11 +193,13 @@ describe HybridPlatformsConductor::ActionsExecutor do
     it 'returns errors without failing for actions on nodes having no connectors' do
       with_test_platform_for_actions do
         test_actions_executor.connector(:test_connector).accept_nodes = %w[node1 node3]
-        expect(test_actions_executor.execute_actions(
-          'node1' => { test_action: { need_connector: true, code: proc { |stdout| stdout << 'Action 1' } } },
-          'node2' => { test_action: { need_connector: true, code: proc { |stdout| stdout << 'Action 2' } } },
-          'node3' => { test_action: { need_connector: true, code: proc { |stdout| stdout << 'Action 3' } } }
-        )).to eq(
+        expect(
+          test_actions_executor.execute_actions(
+            'node1' => { test_action: { need_connector: true, code: proc { |stdout| stdout << 'Action 1' } } },
+            'node2' => { test_action: { need_connector: true, code: proc { |stdout| stdout << 'Action 2' } } },
+            'node3' => { test_action: { need_connector: true, code: proc { |stdout| stdout << 'Action 3' } } }
+          )
+        ).to eq(
           'node1' => [0, 'Action 1', ''],
           'node2' => [:no_connector, '', 'Unable to get a connector to node2'],
           'node3' => [0, 'Action 3', '']

data/spec/hybrid_platforms_conductor_test/api/actions_executor/connection_spec.rb

@@ -1,22 +1,22 @@
 describe HybridPlatformsConductor::ActionsExecutor do
 
-  context 'checking connections handling' do
+  context 'when checking connections handling' do
 
     # Get a test platform to test connection, using the test_connector
     #
     # Parameters::
-    # * Proc: Code called with platform setup
+    # * *block* (Proc): Code called with platform setup
     #   * Parameters::
     #     * *repository* (String): Repository where the platform has been setup
-    def with_test_platform_for_connections
-      with_test_platform_for_executor(nodes: {
-        'node1' => {},
-        'node2' => {},
-        'node3' => {},
-        'node4' => {}
-      }) do |repository|
-        yield repository
-      end
+    def with_test_platform_for_connections(&block)
+      with_test_platform_for_executor(
+        nodes: {
+          'node1' => {},
+          'node2' => {},
+          'node3' => {},
+          'node4' => {}
+        }, &block
+      )
     end
 
     it 'connects on a node before executing actions needing connection' do
@@ -25,8 +25,8 @@ describe HybridPlatformsConductor::ActionsExecutor do
         test_actions_executor.execute_actions('node1' => { test_action: { need_connector: true } })
         expect(action_executions).to eq [{ node: 'node1', message: 'Action executed' }]
         expect(test_actions_executor.connector(:test_connector).calls).to eq [
-         [:connectable_nodes_from, ['node1']],
-         [:with_connection_to, ['node1'], { no_exception: true }]
+          [:connectable_nodes_from, ['node1']],
+          [:with_connection_to, ['node1'], { no_exception: true }]
         ]
       end
     end
@@ -50,8 +50,8 @@ describe HybridPlatformsConductor::ActionsExecutor do
           { node: 'node4', message: 'Action executed' }
         ]
         expect(test_actions_executor.connector(:test_connector).calls).to eq [
-         [:connectable_nodes_from, %w[node1 node2 node3 node4]],
-         [:with_connection_to, %w[node1 node2 node3 node4], { no_exception: true }]
+          [:connectable_nodes_from, %w[node1 node2 node3 node4]],
+          [:with_connection_to, %w[node1 node2 node3 node4], { no_exception: true }]
         ]
       end
     end
@@ -67,8 +67,8 @@ describe HybridPlatformsConductor::ActionsExecutor do
           { node: 'node4', message: 'Action executed' }
         ]
         expect(test_actions_executor.connector(:test_connector).calls).to eq [
-         [:connectable_nodes_from, %w[node1 node2 node3 node4]],
-         [:with_connection_to, %w[node1 node2 node3 node4], { no_exception: true }]
+          [:connectable_nodes_from, %w[node1 node2 node3 node4]],
+          [:with_connection_to, %w[node1 node2 node3 node4], { no_exception: true }]
         ]
       end
     end
@@ -87,8 +87,8 @@ describe HybridPlatformsConductor::ActionsExecutor do
           { node: 'node4', message: 'Action executed' }
         ]
         expect(test_actions_executor.connector(:test_connector).calls).to eq [
-         [:connectable_nodes_from, %w[node1 node3]],
-         [:with_connection_to, %w[node1 node3], { no_exception: true }]
+          [:connectable_nodes_from, %w[node1 node3]],
+          [:with_connection_to, %w[node1 node3], { no_exception: true }]
         ]
       end
     end
@@ -119,12 +119,12 @@ describe HybridPlatformsConductor::ActionsExecutor do
           { node: 'node4', message: 'Action executed' }
         ]
         expect(test_actions_executor.connector(:test_connector).calls).to eq [
-         [:connectable_nodes_from, %w[node1 node2 node3 node4]],
-         [:with_connection_to, %w[node1 node3], { no_exception: true }]
+          [:connectable_nodes_from, %w[node1 node2 node3 node4]],
+          [:with_connection_to, %w[node1 node3], { no_exception: true }]
         ]
         expect(test_actions_executor.connector(:test_connector_2).calls).to eq [
-         [:connectable_nodes_from, %w[node2 node4]],
-         [:with_connection_to, %w[node2 node4], { no_exception: true }]
+          [:connectable_nodes_from, %w[node2 node4]],
+          [:with_connection_to, %w[node2 node4], { no_exception: true }]
         ]
       end
     end
@@ -141,12 +141,12 @@ describe HybridPlatformsConductor::ActionsExecutor do
           { node: 'node4', message: 'Action executed' }
         ]
         expect(test_actions_executor.connector(:test_connector).calls).to eq [
-         [:connectable_nodes_from, %w[node1 node2 node3 node4]],
-         [:with_connection_to, %w[node1 node2 node3], { no_exception: true }]
+          [:connectable_nodes_from, %w[node1 node2 node3 node4]],
+          [:with_connection_to, %w[node1 node2 node3], { no_exception: true }]
         ]
         expect(test_actions_executor.connector(:test_connector_2).calls).to eq [
-         [:connectable_nodes_from, %w[node4]],
-         [:with_connection_to, %w[node4], { no_exception: true }]
+          [:connectable_nodes_from, %w[node4]],
+          [:with_connection_to, %w[node4], { no_exception: true }]
         ]
       end
     end
@@ -157,12 +157,12 @@ describe HybridPlatformsConductor::ActionsExecutor do
         test_actions_executor.connector(:test_connector_2).accept_nodes = %w[node2 node4]
         test_actions_executor.with_connections_prepared_to(%w[node1 node2 node3 node4]) do |connected_nodes|
           expect(test_actions_executor.connector(:test_connector).calls).to eq [
-           [:connectable_nodes_from, %w[node1 node2 node3 node4]],
-           [:with_connection_to, %w[node1 node3], { no_exception: false }]
+            [:connectable_nodes_from, %w[node1 node2 node3 node4]],
+            [:with_connection_to, %w[node1 node3], { no_exception: false }]
           ]
           expect(test_actions_executor.connector(:test_connector_2).calls).to eq [
-           [:connectable_nodes_from, %w[node2 node4]],
-           [:with_connection_to, %w[node2 node4], { no_exception: false }]
+            [:connectable_nodes_from, %w[node2 node4]],
+            [:with_connection_to, %w[node2 node4], { no_exception: false }]
           ]
           expect(connected_nodes).to eq(
             'node1' => test_actions_executor.connector(:test_connector),
@@ -190,12 +190,12 @@ describe HybridPlatformsConductor::ActionsExecutor do
         test_actions_executor.connector(:test_connector_2).accept_nodes = %w[node2]
         test_actions_executor.with_connections_prepared_to(%w[node1 node2 node3 node4], no_exception: true) do |connected_nodes|
           expect(test_actions_executor.connector(:test_connector).calls).to eq [
-           [:connectable_nodes_from, %w[node1 node2 node3 node4]],
-           [:with_connection_to, %w[node1 node3], { no_exception: true }]
+            [:connectable_nodes_from, %w[node1 node2 node3 node4]],
+            [:with_connection_to, %w[node1 node3], { no_exception: true }]
           ]
           expect(test_actions_executor.connector(:test_connector_2).calls).to eq [
-           [:connectable_nodes_from, %w[node2 node4]],
-           [:with_connection_to, %w[node2], { no_exception: true }]
+            [:connectable_nodes_from, %w[node2 node4]],
+            [:with_connection_to, %w[node2], { no_exception: true }]
           ]
           expect(connected_nodes).to eq(
             'node1' => test_actions_executor.connector(:test_connector),

data/spec/hybrid_platforms_conductor_test/api/actions_executor/connectors/local/connectable_nodes_spec.rb

@@ -1,8 +1,8 @@
 describe HybridPlatformsConductor::ActionsExecutor do
 
-  context 'checking connector plugin local' do
+  context 'when checking connector plugin local' do
 
-    context 'checking connectable nodes selection' do
+    context 'when checking connectable nodes selection' do
 
       # Return the connector to be tested
       #
@@ -13,12 +13,16 @@ describe HybridPlatformsConductor::ActionsExecutor do
       end
 
       it 'selects connectable nodes correctly' do
-        with_test_platform(nodes: {
-          'node1' => { meta: { host_ip: '192.168.42.42' } },
-          'node2' => {},
-          'node3' => { meta: { host_ip: '127.0.0.1', local_node: true } },
-          'node4' => { meta: { local_node: true } }
-        }) do
+        with_test_platform(
+          {
+            nodes: {
+              'node1' => { meta: { host_ip: '192.168.42.42' } },
+              'node2' => {},
+              'node3' => { meta: { host_ip: '127.0.0.1', local_node: true } },
+              'node4' => { meta: { local_node: true } }
+            }
+          }
+        ) do
           expect(test_connector.connectable_nodes_from(%w[node1 node2 node3 node4]).sort).to eq %w[node3 node4].sort
         end
       end

data/spec/hybrid_platforms_conductor_test/api/actions_executor/connectors/local/remote_actions_spec.rb

@@ -1,8 +1,8 @@
 describe HybridPlatformsConductor::ActionsExecutor do
 
-  context 'checking connector plugin local' do
+  context 'when checking connector plugin local' do
 
-    context 'checking remote actions' do
+    context 'when checking remote actions' do
 
       # Return the connector to be tested
       #
@@ -19,7 +19,6 @@ describe HybridPlatformsConductor::ActionsExecutor do
       # * *expected_stdout* (String): Expected stdout after client code execution [default: '']
       # * *expected_stderr* (String): Expected stderr after client code execution [default: '']
       # * *timeout* (Integer or nil): Timeout to prepare the connector for [default: nil]
-      # * *password* (String or nil): Password to set for the node, or nil for none [default: nil]
       # * *additional_config* (String): Additional config [default: '']
       # * Proc: Client code to execute testing
       def with_test_platform_for_remote_testing(
@@ -27,13 +26,11 @@ describe HybridPlatformsConductor::ActionsExecutor do
         expected_stdout: '',
         expected_stderr: '',
         timeout: nil,
-        password: nil,
         additional_config: ''
       )
         with_test_platform(
           { nodes: { 'node' => { meta: { local_node: true } } } },
-          false,
-          additional_config
+          additional_config: additional_config
         ) do
           with_cmd_runner_mocked(expected_cmds) do
             test_connector.with_connection_to(['node']) do
@@ -62,7 +59,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
           expected_cmds: [
             [
               'cd /tmp/hpc_local_workspaces/node ; bash_cmd.bash',
-              proc do |cmd, log_to_file: nil, log_to_stdout: true, log_stdout_to_io: nil, log_stderr_to_io: nil, expected_code: 0, timeout: nil, no_exception: false|
+              proc do |_cmd, log_to_file: nil, log_to_stdout: true, log_stdout_to_io: nil, log_stderr_to_io: nil, expected_code: 0, timeout: nil, no_exception: false|
                 expect(timeout).to eq 5
                 [0, '', '']
               end

data/spec/hybrid_platforms_conductor_test/api/actions_executor/connectors/ssh/cli_options_spec.rb

@@ -1,8 +1,8 @@
 describe HybridPlatformsConductor::ActionsExecutor do
 
-  context 'checking connector plugin ssh' do
+  context 'when checking connector plugin ssh' do
 
-    context 'checking command-line interface options' do
+    context 'when checking command-line interface options' do
 
       # Setup a platform for tests
       #
@@ -13,11 +13,10 @@ describe HybridPlatformsConductor::ActionsExecutor do
       def with_test_platform_for_cli
         with_test_platform(
           { nodes: { 'node' => {} } },
-          false,
-          "
+          additional_config: <<~EO_CONFIG
             gateway :test_gateway, 'Host test_gateway'
-            gateway :test_gateway2, 'Host test_gateway2'
-          "
+            gateway :test_gateway_2, 'Host test_gateway_2'
+          EO_CONFIG
         ) do |repository|
           ENV['hpc_ssh_gateways_conf'] = 'test_gateway'
           yield repository
@@ -34,10 +33,10 @@ describe HybridPlatformsConductor::ActionsExecutor do
 
       it 'selects the correct gateway user' do
         with_test_platform_for_cli do
-          expect_actions_executor_runs([proc do |actions, timeout: nil, concurrent: false, log_to_dir: 'run_logs', log_to_stdout: true|
+          expect_actions_executor_runs [proc do
             expect(test_connector.ssh_gateway_user).to eq 'another_user'
             {}
-          end])
+          end]
           exit_code, stdout, stderr = run 'run', '--node', 'node', '--command', 'echo Hello', '--ssh-gateway-user', 'another_user'
           expect(exit_code).to eq 0
           expect(stdout).to eq ''
@@ -47,11 +46,11 @@ describe HybridPlatformsConductor::ActionsExecutor do
 
       it 'selects the correct gateway conf' do
         with_test_platform_for_cli do
-          expect_actions_executor_runs([proc do |actions, timeout: nil, concurrent: false, log_to_dir: 'run_logs', log_to_stdout: true|
-            expect(test_connector.ssh_gateways_conf).to eq :test_gateway2
+          expect_actions_executor_runs [proc do
+            expect(test_connector.ssh_gateways_conf).to eq :test_gateway_2
             {}
-          end])
-          exit_code, stdout, stderr = run 'run', '--node', 'node', '--command', 'echo Hello', '--ssh-gateways-conf', 'test_gateway2'
+          end]
+          exit_code, stdout, stderr = run 'run', '--node', 'node', '--command', 'echo Hello', '--ssh-gateways-conf', 'test_gateway_2'
           expect(exit_code).to eq 0
           expect(stdout).to eq ''
           expect(stderr).to eq ''
@@ -60,10 +59,10 @@ describe HybridPlatformsConductor::ActionsExecutor do
 
       it 'does not use the SSH control master' do
         with_test_platform_for_cli do
-          expect_actions_executor_runs([proc do |actions, timeout: nil, concurrent: false, log_to_dir: 'run_logs', log_to_stdout: true|
+          expect_actions_executor_runs [proc do
             expect(test_connector.ssh_use_control_master).to eq false
             {}
-          end])
+          end]
           exit_code, stdout, stderr = run 'run', '--node', 'node', '--command', 'echo Hello', '--ssh-no-control-master'
           expect(exit_code).to eq 0
           expect(stdout).to eq ''
@@ -73,10 +72,10 @@ describe HybridPlatformsConductor::ActionsExecutor do
 
       it 'does not use strict host key checking' do
         with_test_platform_for_cli do
-          expect_actions_executor_runs([proc do |actions, timeout: nil, concurrent: false, log_to_dir: 'run_logs', log_to_stdout: true|
+          expect_actions_executor_runs [proc do
             expect(test_connector.ssh_strict_host_key_checking).to eq false
             {}
-          end])
+          end]
           exit_code, stdout, stderr = run 'run', '--node', 'node', '--command', 'echo Hello', '--ssh-no-host-key-checking'
           expect(exit_code).to eq 0
           expect(stdout).to eq ''
@@ -86,10 +85,10 @@ describe HybridPlatformsConductor::ActionsExecutor do
 
       it 'uses a different SSH user name' do
         with_test_platform_for_cli do
-          expect_actions_executor_runs([proc do |actions, timeout: nil, concurrent: false, log_to_dir: 'run_logs', log_to_stdout: true|
+          expect_actions_executor_runs [proc do
             expect(test_connector.ssh_user).to eq 'ssh_new_user'
             {}
-          end])
+          end]
           exit_code, stdout, stderr = run 'run', '--node', 'node', '--command', 'echo Hello', '--ssh-user', 'ssh_new_user'
           expect(exit_code).to eq 0
           expect(stdout).to eq ''
@@ -101,9 +100,9 @@ describe HybridPlatformsConductor::ActionsExecutor do
         ENV.delete 'hpc_ssh_user'
         ENV.delete 'USER'
         with_test_platform_for_cli do
-          with_cmd_runner_mocked([
+          with_cmd_runner_mocked [
             ['whoami', proc { [0, '', ''] }]
-          ]) do
+          ] do
             expect { run 'run', '--node', 'node', '--command', 'echo Hello' }.to raise_error(RuntimeError, 'No SSH user name specified. Please use --ssh-user option or hpc_ssh_user environment variable to set it.')
           end
         end
@@ -111,10 +110,10 @@ describe HybridPlatformsConductor::ActionsExecutor do
 
       it 'expects passwords to be input' do
         with_test_platform_for_cli do
-          expect_actions_executor_runs([proc do |actions, timeout: nil, concurrent: false, log_to_dir: 'run_logs', log_to_stdout: true|
+          expect_actions_executor_runs [proc do
             expect(test_connector.auth_password).to eq true
             {}
-          end])
+          end]
           exit_code, stdout, stderr = run 'run', '--node', 'node', '--command', 'echo Hello', '--password'
           expect(exit_code).to eq 0
           expect(stdout).to eq ''