hybrid_platforms_conductor 33.0.0 → 33.0.1

data/spec/hybrid_platforms_conductor_test/api/actions_executor/connectors/ssh/node_helpers_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 additional helpers on prepared nodes' do
+    context 'when checking additional helpers on prepared nodes' do
 
       it 'provides an SSH executable wrapping the node\'s SSH config' do
         with_test_platform_for_remote_testing do
@@ -21,7 +21,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
         with_test_platform_for_remote_testing(password: 'PaSsWoRd') do
           expect(`#{test_connector.ssh_exec} -V 2>&1`).to eq `ssh -V 2>&1`
           expect(`#{test_connector.ssh_exec} -G hpc.node`.split("\n").find { |line| line =~ /^hostname .+$/ }).to eq 'hostname 192.168.42.42'
-          expect(File.read(test_connector.ssh_exec)).to match /^sshpass -pPaSsWoRd ssh .+$/
+          expect(File.read(test_connector.ssh_exec)).to match(/^sshpass -pPaSsWoRd ssh .+$/)
         end
       end
 

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

@@ -1,12 +1,12 @@
 describe HybridPlatformsConductor::ActionsExecutor do
 
-  context 'checking connector plugin ssh' do
+  context 'when checking connector plugin ssh' do
 
-    context 'checking remote actions' do
+    context 'when checking remote actions' do
 
       it 'executes bash commands remotely' do
         with_test_platform_for_remote_testing(
-          expected_cmds: [[/.+\/ssh hpc\.node \/bin\/bash <<'HPC_EOF'\nbash_cmd.bash\nHPC_EOF/, proc { [0, 'Bash commands executed on node', ''] }]],
+          expected_cmds: [[%r{.+/ssh hpc\.node /bin/bash <<'HPC_EOF'\nbash_cmd.bash\nHPC_EOF}, proc { [0, 'Bash commands executed on node', ''] }]],
           expected_stdout: 'Bash commands executed on node'
         ) do
           test_connector.remote_bash('bash_cmd.bash')
@@ -17,8 +17,8 @@ describe HybridPlatformsConductor::ActionsExecutor do
         with_test_platform_for_remote_testing(
           expected_cmds: [
             [
-              /.+\/ssh hpc\.node \/bin\/bash <<'HPC_EOF'\nbash_cmd.bash\nHPC_EOF/,
-              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|
+              %r{.+/ssh hpc\.node /bin/bash <<'HPC_EOF'\nbash_cmd.bash\nHPC_EOF},
+              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
@@ -33,7 +33,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
       it 'executes interactive commands remotely' do
         with_test_platform_for_remote_testing do
           expect(test_connector).to receive(:system) do |cmd|
-            expect(cmd).to match /^.+\/ssh hpc\.node$/
+            expect(cmd).to match(%r{^.+/ssh hpc\.node$})
           end
           test_connector.remote_interactive
         end
@@ -43,7 +43,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
         with_test_platform_for_remote_testing(
           expected_cmds: [
             [
-              /cd \/path\/to && tar\s+--create\s+--gzip\s+--file -\s+src.file \| \/.+\/ssh\s+hpc\.node\s+"tar\s+--extract\s+--gunzip\s+--file -\s+--directory \/remote_path\/to\/dst.dir\s+--owner root\s+"/,
+              %r{cd /path/to && tar\s+--create\s+--gzip\s+--file -\s+src.file \| /.+/ssh\s+hpc\.node\s+"tar\s+--extract\s+--gunzip\s+--file -\s+--directory /remote_path/to/dst.dir\s+--owner root\s+"},
               proc { [0, '', ''] }
             ]
           ]
@@ -56,8 +56,8 @@ describe HybridPlatformsConductor::ActionsExecutor do
         with_test_platform_for_remote_testing(
           expected_cmds: [
             [
-              /cd \/path\/to && tar\s+--create\s+--gzip\s+--file -\s+src.file \| \/.+\/ssh\s+hpc\.node\s+"tar\s+--extract\s+--gunzip\s+--file -\s+--directory \/remote_path\/to\/dst.dir\s+--owner root\s+"/,
-              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|
+              %r{cd /path/to && tar\s+--create\s+--gzip\s+--file -\s+src.file \| /.+/ssh\s+hpc\.node\s+"tar\s+--extract\s+--gunzip\s+--file -\s+--directory /remote_path/to/dst.dir\s+--owner root\s+"},
+              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
@@ -74,9 +74,9 @@ describe HybridPlatformsConductor::ActionsExecutor do
         with_test_platform_for_remote_testing(
           expected_cmds: [
             [
-              /.+\/hpc_temp_cmds_.+\.sh$/,
+              %r{.+/hpc_temp_cmds_.+\.sh$},
               proc do |received_cmd|
-                expect(File.read(received_cmd)).to match /.+\/ssh hpc\.node \/bin\/bash <<'HPC_EOF'\n#{Regexp.escape(cmd)}\nHPC_EOF/
+                expect(File.read(received_cmd)).to match(%r{.+/ssh hpc\.node /bin/bash <<'HPC_EOF'\n#{Regexp.escape(cmd)}\nHPC_EOF})
                 [0, 'Bash commands executed on node', '']
               end
             ]
@@ -92,7 +92,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
         with_test_platform_for_remote_testing(
           expected_cmds: [
             [
-              /cd \/path\/to && tar\s+--create\s+--gzip\s+--file -\s+src.file \| \/.+\/ssh\s+hpc\.node\s+"sudo -u root tar\s+--extract\s+--gunzip\s+--file -\s+--directory \/remote_path\/to\/dst.dir\s+--owner root\s+"/,
+              %r{cd /path/to && tar\s+--create\s+--gzip\s+--file -\s+src.file \| /.+/ssh\s+hpc\.node\s+"sudo -u root tar\s+--extract\s+--gunzip\s+--file -\s+--directory /remote_path/to/dst.dir\s+--owner root\s+"},
               proc { [0, '', ''] }
             ]
           ]
@@ -105,11 +105,13 @@ describe HybridPlatformsConductor::ActionsExecutor do
         with_test_platform_for_remote_testing(
           expected_cmds: [
             [
-              /cd \/path\/to && tar\s+--create\s+--gzip\s+--file -\s+src.file \| \/.+\/ssh\s+hpc\.node\s+"other_sudo --user root tar\s+--extract\s+--gunzip\s+--file -\s+--directory \/remote_path\/to\/dst.dir\s+--owner root\s+"/,
+              %r{cd /path/to && tar\s+--create\s+--gzip\s+--file -\s+src.file \| /.+/ssh\s+hpc\.node\s+"other_sudo --user root tar\s+--extract\s+--gunzip\s+--file -\s+--directory /remote_path/to/dst.dir\s+--owner root\s+"},
               proc { [0, '', ''] }
             ]
           ],
-          additional_config: 'sudo_for { |user| "other_sudo --user #{user}" }'
+          additional_config: <<~'EO_CONFIG'
+            sudo_for { |user| "other_sudo --user #{user}" }
+          EO_CONFIG
         ) do
           test_connector.remote_copy('/path/to/src.file', '/remote_path/to/dst.dir', sudo: true)
         end
@@ -119,7 +121,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
         with_test_platform_for_remote_testing(
           expected_cmds: [
             [
-              /cd \/path\/to && tar\s+--create\s+--gzip\s+--file -\s+--owner remote_user\s+src.file \| \/.+\/ssh\s+hpc\.node\s+"tar\s+--extract\s+--gunzip\s+--file -\s+--directory \/remote_path\/to\/dst.dir\s+--owner root\s+"/,
+              %r{cd /path/to && tar\s+--create\s+--gzip\s+--file -\s+--owner remote_user\s+src.file \| /.+/ssh\s+hpc\.node\s+"tar\s+--extract\s+--gunzip\s+--file -\s+--directory /remote_path/to/dst.dir\s+--owner root\s+"},
               proc { [0, '', ''] }
             ]
           ]
@@ -132,7 +134,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
         with_test_platform_for_remote_testing(
           expected_cmds: [
             [
-              /cd \/path\/to && tar\s+--create\s+--gzip\s+--file -\s+--group remote_group\s+src.file \| \/.+\/ssh\s+hpc\.node\s+"tar\s+--extract\s+--gunzip\s+--file -\s+--directory \/remote_path\/to\/dst.dir\s+--owner root\s+"/,
+              %r{cd /path/to && tar\s+--create\s+--gzip\s+--file -\s+--group remote_group\s+src.file \| /.+/ssh\s+hpc\.node\s+"tar\s+--extract\s+--gunzip\s+--file -\s+--directory /remote_path/to/dst.dir\s+--owner root\s+"},
               proc { [0, '', ''] }
             ]
           ]
@@ -143,7 +145,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
 
       it 'executes bash commands remotely without Session Exec capabilities' do
         with_test_platform_for_remote_testing(
-          expected_cmds: [[/^\{ cat \| .+\/ssh hpc\.node -T; } <<'HPC_EOF'\nbash_cmd.bash\nHPC_EOF$/, proc { [0, 'Bash commands executed on node', ''] }]],
+          expected_cmds: [[%r{^\{ cat \| .+/ssh hpc\.node -T; \} <<'HPC_EOF'\nbash_cmd.bash\nHPC_EOF$}, proc { [0, 'Bash commands executed on node', ''] }]],
           expected_stdout: 'Bash commands executed on node',
           session_exec: false
         ) do
@@ -155,7 +157,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
         with_test_platform_for_remote_testing(
           expected_cmds: [
             [
-              /^scp -S .+\/ssh \/path\/to\/src.file hpc\.node:\/remote_path\/to\/dst.dir$/,
+              %r{^scp -S .+/ssh /path/to/src.file hpc\.node:/remote_path/to/dst.dir$},
               proc { [0, '', ''] }
             ]
           ],
@@ -168,12 +170,12 @@ describe HybridPlatformsConductor::ActionsExecutor do
       it 'copies files remotely without Session Exec capabilities and with sudo' do
         with_test_platform_for_remote_testing(
           expected_cmds: [
-            [/^\{ cat \| .+\/ssh hpc\.node -T; } <<'HPC_EOF'\nmkdir -p hpc_tmp_scp\nHPC_EOF$/, proc { [0, '', ''] }],
+            [%r{^\{ cat \| .+/ssh hpc\.node -T; \} <<'HPC_EOF'\nmkdir -p hpc_tmp_scp\nHPC_EOF$}, proc { [0, '', ''] }],
             [
-              /^scp -S .+\/ssh \/path\/to\/src.file hpc\.node:\.\/hpc_tmp_scp$/,
+              %r{^scp -S .+/ssh /path/to/src.file hpc\.node:\./hpc_tmp_scp$},
               proc { [0, '', ''] }
             ],
-            [/^\{ cat \| .+\/ssh hpc\.node -T; } <<'HPC_EOF'\nsudo -u root mv \.\/hpc_tmp_scp\/src\.file \/remote_path\/to\/dst\.dir\nHPC_EOF$/, proc { [0, '', ''] }]
+            [%r{^\{ cat \| .+/ssh hpc\.node -T; \} <<'HPC_EOF'\nsudo -u root mv \./hpc_tmp_scp/src\.file /remote_path/to/dst\.dir\nHPC_EOF$}, proc { [0, '', ''] }]
           ],
           session_exec: false
         ) do
@@ -184,14 +186,16 @@ describe HybridPlatformsConductor::ActionsExecutor do
       it 'copies files remotely without Session Exec capabilities and with a different sudo' do
         with_test_platform_for_remote_testing(
           expected_cmds: [
-            [/^\{ cat \| .+\/ssh hpc\.node -T; } <<'HPC_EOF'\nmkdir -p hpc_tmp_scp\nHPC_EOF$/, proc { [0, '', ''] }],
+            [%r{^\{ cat \| .+/ssh hpc\.node -T; \} <<'HPC_EOF'\nmkdir -p hpc_tmp_scp\nHPC_EOF$}, proc { [0, '', ''] }],
             [
-              /^scp -S .+\/ssh \/path\/to\/src.file hpc\.node:\.\/hpc_tmp_scp$/,
+              %r{^scp -S .+/ssh /path/to/src.file hpc\.node:\./hpc_tmp_scp$},
               proc { [0, '', ''] }
             ],
-            [/^\{ cat \| .+\/ssh hpc\.node -T; } <<'HPC_EOF'\nother_sudo --user root mv \.\/hpc_tmp_scp\/src\.file \/remote_path\/to\/dst\.dir\nHPC_EOF$/, proc { [0, '', ''] }]
+            [%r{^\{ cat \| .+/ssh hpc\.node -T; \} <<'HPC_EOF'\nother_sudo --user root mv \./hpc_tmp_scp/src\.file /remote_path/to/dst\.dir\nHPC_EOF$}, proc { [0, '', ''] }]
           ],
-          additional_config: 'sudo_for { |user| "other_sudo --user #{user}" }',
+          additional_config: <<~'EO_CONFIG',
+            sudo_for { |user| "other_sudo --user #{user}" }
+          EO_CONFIG
           session_exec: false
         ) do
           test_connector.remote_copy('/path/to/src.file', '/remote_path/to/dst.dir', sudo: true)

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

@@ -1,28 +1,30 @@
 describe HybridPlatformsConductor::ActionsExecutor do
 
-  context 'checking logging facilities' do
+  context 'when checking logging facilities' 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_logging
-      with_test_platform_for_executor(nodes: { 'node1' => {}, 'node2' => {}, 'node3' => {} }) do |repository|
-        yield repository
-      end
+    def with_test_platform_for_logging(&block)
+      with_test_platform_for_executor(nodes: { 'node1' => {}, 'node2' => {}, 'node3' => {} }, &block)
     end
 
     it 'captures stdout and stderr from action correctly' do
       with_test_platform_for_logging do
         expect(
-          test_actions_executor.execute_actions('node1' => { test_action: {
-            code: proc do |stdout, stderr|
-              stdout << 'action_stdout'
-              stderr << 'action_stderr'
-            end
-          } })
+          test_actions_executor.execute_actions(
+            'node1' => {
+              test_action: {
+                code: proc do |stdout, stderr|
+                  stdout << 'action_stdout'
+                  stderr << 'action_stderr'
+                end
+              }
+            }
+          )
         ).to eq('node1' => [0, 'action_stdout', 'action_stderr'])
       end
     end
@@ -30,13 +32,17 @@ describe HybridPlatformsConductor::ActionsExecutor do
     it 'captures stdout and stderr from action correctly even when the action is failing' do
       with_test_platform_for_logging do
         expect(
-          test_actions_executor.execute_actions('node1' => { test_action: {
-            code: proc do |stdout, stderr|
-              stdout << "action_stdout\n"
-              stderr << "action_stderr\n"
-              raise 'Failing action'
-            end
-          } })
+          test_actions_executor.execute_actions(
+            'node1' => {
+              test_action: {
+                code: proc do |stdout, stderr|
+                  stdout << "action_stdout\n"
+                  stderr << "action_stderr\n"
+                  raise 'Failing action'
+                end
+              }
+            }
+          )
         ).to eq('node1' => [:failed_action, "action_stdout\n", "action_stderr\nFailing action\n"])
       end
     end
@@ -74,34 +80,38 @@ describe HybridPlatformsConductor::ActionsExecutor do
       with_test_platform_for_logging do
         expect(
           test_actions_executor.execute_actions('node1' => { test_action: { run_cmd: 'echo action_stdout && >&2 echo action_stderr && exit 1' } })
-        ).to eq('node1' => [
-          :failed_command,
-          "action_stdout\n",
-          "action_stderr\nCommand 'echo action_stdout && >&2 echo action_stderr && exit 1' returned error code 1 (expected 0).\n"
-        ])
+        ).to eq(
+          'node1' => [
+            :failed_command,
+            "action_stdout\n",
+            "action_stderr\nCommand 'echo action_stdout && >&2 echo action_stderr && exit 1' returned error code 1 (expected 0).\n"
+          ]
+        )
       end
     end
 
     it 'captures stdout and stderr sequentially from several actions' do
       with_test_platform_for_logging do
         expect(
-          test_actions_executor.execute_actions('node1' => [
-            { test_action: {
-              code: proc do |stdout, stderr|
-                stdout << 'action1_stdout '
-                stderr << 'action1_stderr '
-              end
-            } },
-            { test_action: {
-              run_cmd: 'echo action2_stdout && >&2 echo action2_stderr'
-            } },
-            { test_action: {
-              code: proc do |stdout, stderr|
-                stdout << 'action3_stdout'
-                stderr << 'action3_stderr'
-              end
-            } }
-          ])
+          test_actions_executor.execute_actions(
+            'node1' => [
+              { test_action: {
+                code: proc do |stdout, stderr|
+                  stdout << 'action1_stdout '
+                  stderr << 'action1_stderr '
+                end
+              } },
+              { test_action: {
+                run_cmd: 'echo action2_stdout && >&2 echo action2_stderr'
+              } },
+              { test_action: {
+                code: proc do |stdout, stderr|
+                  stdout << 'action3_stdout'
+                  stderr << 'action3_stderr'
+                end
+              } }
+            ]
+          )
         ).to eq('node1' => [0, "action1_stdout action2_stdout\naction3_stdout", "action1_stderr action2_stderr\naction3_stderr"])
       end
     end
@@ -187,56 +197,61 @@ describe HybridPlatformsConductor::ActionsExecutor do
     it 'executes several actions on several nodes and returns the corresponding stdout and stderr correctly in files' do
       with_repository('logs') do |logs_repository|
         with_test_platform_for_logging do
-          expect(test_actions_executor.execute_actions({
-            'node1' => [
-              { test_action: { code: proc do |stdout, stderr|
-                stdout << 'node1_action1_stdout '
-                stderr << 'node1_action1_stderr '
-                sleep 1
-              end } },
-              { test_action: { code: proc do |stdout, stderr|
-                stdout << 'node1_action2_stdout '
-                stderr << 'node1_action2_stderr '
-                sleep 1
-              end } },
-              { test_action: { code: proc do |stdout, stderr|
-                stdout << 'node1_action3_stdout'
-                stderr << 'node1_action3_stderr'
-              end } }
-            ],
-            'node2' => [
-              { test_action: { code: proc do |stdout, stderr|
-                stdout << 'node2_action1_stdout '
-                stderr << 'node2_action1_stderr '
-                sleep 1
-              end } },
-              { test_action: { code: proc do |stdout, stderr|
-                stdout << 'node2_action2_stdout '
-                stderr << 'node2_action2_stderr '
-                sleep 1
-              end } },
-              { test_action: { code: proc do |stdout, stderr|
-                stdout << 'node2_action3_stdout'
-                stderr << 'node2_action3_stderr'
-              end } }
-            ],
-            'node3' => [
-              { test_action: { code: proc do |stdout, stderr|
-                stdout << 'node3_action1_stdout '
-                stderr << 'node3_action1_stderr '
-                sleep 1
-              end } },
-              { test_action: { code: proc do |stdout, stderr|
-                stdout << 'node3_action2_stdout '
-                stderr << 'node3_action2_stderr '
-                sleep 1
-              end } },
-              { test_action: { code: proc do |stdout, stderr|
-                stdout << 'node3_action3_stdout'
-                stderr << 'node3_action3_stderr'
-              end } }
-            ]
-          }, log_to_dir: logs_repository)).to eq(
+          expect(
+            test_actions_executor.execute_actions(
+              {
+                'node1' => [
+                  { test_action: { code: proc do |stdout, stderr|
+                    stdout << 'node1_action1_stdout '
+                    stderr << 'node1_action1_stderr '
+                    sleep 1
+                  end } },
+                  { test_action: { code: proc do |stdout, stderr|
+                    stdout << 'node1_action2_stdout '
+                    stderr << 'node1_action2_stderr '
+                    sleep 1
+                  end } },
+                  { test_action: { code: proc do |stdout, stderr|
+                    stdout << 'node1_action3_stdout'
+                    stderr << 'node1_action3_stderr'
+                  end } }
+                ],
+                'node2' => [
+                  { test_action: { code: proc do |stdout, stderr|
+                    stdout << 'node2_action1_stdout '
+                    stderr << 'node2_action1_stderr '
+                    sleep 1
+                  end } },
+                  { test_action: { code: proc do |stdout, stderr|
+                    stdout << 'node2_action2_stdout '
+                    stderr << 'node2_action2_stderr '
+                    sleep 1
+                  end } },
+                  { test_action: { code: proc do |stdout, stderr|
+                    stdout << 'node2_action3_stdout'
+                    stderr << 'node2_action3_stderr'
+                  end } }
+                ],
+                'node3' => [
+                  { test_action: { code: proc do |stdout, stderr|
+                    stdout << 'node3_action1_stdout '
+                    stderr << 'node3_action1_stderr '
+                    sleep 1
+                  end } },
+                  { test_action: { code: proc do |stdout, stderr|
+                    stdout << 'node3_action2_stdout '
+                    stderr << 'node3_action2_stderr '
+                    sleep 1
+                  end } },
+                  { test_action: { code: proc do |stdout, stderr|
+                    stdout << 'node3_action3_stdout'
+                    stderr << 'node3_action3_stderr'
+                  end } }
+                ]
+              },
+              log_to_dir: logs_repository
+            )
+          ).to eq(
             'node1' => [0, 'node1_action1_stdout node1_action2_stdout node1_action3_stdout', 'node1_action1_stderr node1_action2_stderr node1_action3_stderr'],
             'node2' => [0, 'node2_action1_stdout node2_action2_stdout node2_action3_stdout', 'node2_action1_stderr node2_action2_stderr node2_action3_stderr'],
             'node3' => [0, 'node3_action1_stdout node3_action2_stdout node3_action3_stdout', 'node3_action1_stderr node3_action2_stderr node3_action3_stderr']

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

@@ -1,39 +1,43 @@
 describe HybridPlatformsConductor::ActionsExecutor do
 
-  context 'checking parallel runs' do
+  context 'when checking parallel runs' do
 
     # Get a test platform to test parallel runs
     #
     # Parameters::
-    # * Proc: Code called with platform setup
-    def with_test_platform_for_parallel_tests
-      with_test_platform_for_executor(nodes: {
-        'node1' => {},
-        'node2' => {},
-        'node3' => {},
-        'node4' => {}
-      }) do
-        yield
-      end
+    # * *block* (Proc): Code called with platform setup
+    def with_test_platform_for_parallel_tests(&block)
+      with_test_platform_for_executor(
+        nodes: {
+          'node1' => {},
+          'node2' => {},
+          'node3' => {},
+          'node4' => {}
+        },
+        &block
+      )
     end
 
     it 'executes a simple command on several nodes in parallel' do
       with_test_platform_for_parallel_tests do
         nodes_executed = []
-        test_actions_executor.execute_actions({
-          'node1' => { test_action: { code: proc do
-            sleep 2
-            nodes_executed << 'node1'
-          end } },
-          'node2' => { test_action: { code: proc do
-            sleep 3
-            nodes_executed << 'node2'
-          end } },
-          'node3' => { test_action: { code: proc do
-            sleep 1
-            nodes_executed << 'node3'
-          end } }
-        }, concurrent: true)
+        test_actions_executor.execute_actions(
+          {
+            'node1' => { test_action: { code: proc do
+              sleep 2
+              nodes_executed << 'node1'
+            end } },
+            'node2' => { test_action: { code: proc do
+              sleep 3
+              nodes_executed << 'node2'
+            end } },
+            'node3' => { test_action: { code: proc do
+              sleep 1
+              nodes_executed << 'node3'
+            end } }
+          },
+          concurrent: true
+        )
         expect(nodes_executed).to eq %w[node3 node1 node2]
       end
     end
@@ -46,76 +50,81 @@ describe HybridPlatformsConductor::ActionsExecutor do
         # * node2: 1---2-----3
         # * node3: ------1-2-----3
         # * Time : 0 1 2 3 4 5 6 7 8
-        expect(test_actions_executor.execute_actions({
-          'node1' => [
-            {
-              test_action: { code: proc do |stdout, stderr|
-                sleep 1
-                stdout << 'node1_action1 '
-                actions_executed << 'node1_action1'
-              end }
-            },
-            {
-              test_action: { code: proc do |stdout, stderr|
-                sleep 5
-                stdout << 'node1_action2 '
-                actions_executed << 'node1_action2'
-              end }
-            },
-            {
-              test_action: { code: proc do |stdout, stderr|
-                sleep 2
-                stdout << 'node1_action3'
-                actions_executed << 'node1_action3'
-              end }
-            }
-          ],
-          'node2' => [
-            {
-              test_action: { code: proc do |stdout, stderr|
-                stdout << 'node2_action1 '
-                actions_executed << 'node2_action1'
-              end }
-            },
-            {
-              test_action: { code: proc do |stdout, stderr|
-                sleep 2
-                stdout << 'node2_action2 '
-                actions_executed << 'node2_action2'
-              end }
-            },
-            {
-              test_action: { code: proc do |stdout, stderr|
-                sleep 3
-                stdout << 'node2_action3'
-                actions_executed << 'node2_action3'
-              end }
-            }
-          ],
-          'node3' => [
-            {
-              test_action: { code: proc do |stdout, stderr|
-                sleep 3
-                stdout << 'node3_action1 '
-                actions_executed << 'node3_action1'
-              end }
-            },
+        expect(
+          test_actions_executor.execute_actions(
             {
-              test_action: { code: proc do |stdout, stderr|
-                sleep 1
-                stdout << 'node3_action2 '
-                actions_executed << 'node3_action2'
-              end }
+              'node1' => [
+                {
+                  test_action: { code: proc do |stdout|
+                    sleep 1
+                    stdout << 'node1_action1 '
+                    actions_executed << 'node1_action1'
+                  end }
+                },
+                {
+                  test_action: { code: proc do |stdout|
+                    sleep 5
+                    stdout << 'node1_action2 '
+                    actions_executed << 'node1_action2'
+                  end }
+                },
+                {
+                  test_action: { code: proc do |stdout|
+                    sleep 2
+                    stdout << 'node1_action3'
+                    actions_executed << 'node1_action3'
+                  end }
+                }
+              ],
+              'node2' => [
+                {
+                  test_action: { code: proc do |stdout|
+                    stdout << 'node2_action1 '
+                    actions_executed << 'node2_action1'
+                  end }
+                },
+                {
+                  test_action: { code: proc do |stdout|
+                    sleep 2
+                    stdout << 'node2_action2 '
+                    actions_executed << 'node2_action2'
+                  end }
+                },
+                {
+                  test_action: { code: proc do |stdout|
+                    sleep 3
+                    stdout << 'node2_action3'
+                    actions_executed << 'node2_action3'
+                  end }
+                }
+              ],
+              'node3' => [
+                {
+                  test_action: { code: proc do |stdout|
+                    sleep 3
+                    stdout << 'node3_action1 '
+                    actions_executed << 'node3_action1'
+                  end }
+                },
+                {
+                  test_action: { code: proc do |stdout|
+                    sleep 1
+                    stdout << 'node3_action2 '
+                    actions_executed << 'node3_action2'
+                  end }
+                },
+                {
+                  test_action: { code: proc do |stdout|
+                    sleep 3
+                    stdout << 'node3_action3'
+                    actions_executed << 'node3_action3'
+                  end }
+                }
+              ]
             },
-            {
-              test_action: { code: proc do |stdout, stderr|
-                sleep 3
-                stdout << 'node3_action3'
-                actions_executed << 'node3_action3'
-              end }
-            }
-          ]
-        }, concurrent: true)).to eq(
+            concurrent: true
+          )
+        ).to eq(
           'node1' => [0, 'node1_action1 node1_action2 node1_action3', ''],
           'node2' => [0, 'node2_action1 node2_action2 node2_action3', ''],
           'node3' => [0, 'node3_action1 node3_action2 node3_action3', '']
@@ -136,82 +145,89 @@ describe HybridPlatformsConductor::ActionsExecutor do
 
     it 'executes several commands on several nodes with timeout on different actions depending on the node, in parallel' do
       with_test_platform_for_parallel_tests do
-        expect(test_actions_executor.execute_actions(
-          {
-            'node1' => [
-              { bash: 'sleep 1 && echo Node11' },
-              { bash: 'sleep 5 && echo Node12' }
-            ],
-            'node2' => [
-              { bash: 'echo Node21' },
-              { bash: 'sleep 1 && echo Node22' }
-            ],
-            'node3' => [
-              { bash: 'sleep 1 && echo Node31' },
-              { bash: 'sleep 1 && echo Node32' },
-              { bash: 'sleep 5 && echo Node33' }
-            ],
-            'node4' => [
-              { bash: 'sleep 5 && echo Node41' }
-            ]
-          },
-          timeout: 3,
-          concurrent: true
-        )).to eq(
-         'node1' => [:timeout, "Node11\n", ''],
-         'node2' => [0, "Node21\nNode22\n", ''],
-         'node3' => [:timeout, "Node31\nNode32\n", ''],
-         'node4' => [:timeout, '', '']
+        expect(
+          test_actions_executor.execute_actions(
+            {
+              'node1' => [
+                { bash: 'sleep 1 && echo Node11' },
+                { bash: 'sleep 5 && echo Node12' }
+              ],
+              'node2' => [
+                { bash: 'echo Node21' },
+                { bash: 'sleep 1 && echo Node22' }
+              ],
+              'node3' => [
+                { bash: 'sleep 1 && echo Node31' },
+                { bash: 'sleep 1 && echo Node32' },
+                { bash: 'sleep 5 && echo Node33' }
+              ],
+              'node4' => [
+                { bash: 'sleep 5 && echo Node41' }
+              ]
+            },
+            timeout: 3,
+            concurrent: true
+          )
+        ).to eq(
+          'node1' => [:timeout, "Node11\n", ''],
+          'node2' => [0, "Node21\nNode22\n", ''],
+          'node3' => [:timeout, "Node31\nNode32\n", ''],
+          'node4' => [:timeout, '', '']
         )
       end
     end
 
     it 'executes several actions on several nodes and returns the corresponding stdout and stderr correctly in parallel' do
       with_test_platform_for_parallel_tests do
-        expect(test_actions_executor.execute_actions({
-          'node1' => [
-            { test_action: { code: proc do |stdout, stderr|
-              stdout << 'node1_action1_stdout '
-              stderr << 'node1_action1_stderr '
-            end } },
-            { test_action: { code: proc do |stdout, stderr|
-              stdout << 'node1_action2_stdout '
-              stderr << 'node1_action2_stderr '
-            end } },
-            { test_action: { code: proc do |stdout, stderr|
-              stdout << 'node1_action3_stdout'
-              stderr << 'node1_action3_stderr'
-            end } }
-          ],
-          'node2' => [
-            { test_action: { code: proc do |stdout, stderr|
-              stdout << 'node2_action1_stdout '
-              stderr << 'node2_action1_stderr '
-            end } },
-            { test_action: { code: proc do |stdout, stderr|
-              stdout << 'node2_action2_stdout '
-              stderr << 'node2_action2_stderr '
-            end } },
-            { test_action: { code: proc do |stdout, stderr|
-              stdout << 'node2_action3_stdout'
-              stderr << 'node2_action3_stderr'
-            end } }
-          ],
-          'node3' => [
-            { test_action: { code: proc do |stdout, stderr|
-              stdout << 'node3_action1_stdout '
-              stderr << 'node3_action1_stderr '
-            end } },
-            { test_action: { code: proc do |stdout, stderr|
-              stdout << 'node3_action2_stdout '
-              stderr << 'node3_action2_stderr '
-            end } },
-            { test_action: { code: proc do |stdout, stderr|
-              stdout << 'node3_action3_stdout'
-              stderr << 'node3_action3_stderr'
-            end } }
-          ]
-        }, concurrent: true)).to eq(
+        expect(
+          test_actions_executor.execute_actions(
+            {
+              'node1' => [
+                { test_action: { code: proc do |stdout, stderr|
+                  stdout << 'node1_action1_stdout '
+                  stderr << 'node1_action1_stderr '
+                end } },
+                { test_action: { code: proc do |stdout, stderr|
+                  stdout << 'node1_action2_stdout '
+                  stderr << 'node1_action2_stderr '
+                end } },
+                { test_action: { code: proc do |stdout, stderr|
+                  stdout << 'node1_action3_stdout'
+                  stderr << 'node1_action3_stderr'
+                end } }
+              ],
+              'node2' => [
+                { test_action: { code: proc do |stdout, stderr|
+                  stdout << 'node2_action1_stdout '
+                  stderr << 'node2_action1_stderr '
+                end } },
+                { test_action: { code: proc do |stdout, stderr|
+                  stdout << 'node2_action2_stdout '
+                  stderr << 'node2_action2_stderr '
+                end } },
+                { test_action: { code: proc do |stdout, stderr|
+                  stdout << 'node2_action3_stdout'
+                  stderr << 'node2_action3_stderr'
+                end } }
+              ],
+              'node3' => [
+                { test_action: { code: proc do |stdout, stderr|
+                  stdout << 'node3_action1_stdout '
+                  stderr << 'node3_action1_stderr '
+                end } },
+                { test_action: { code: proc do |stdout, stderr|
+                  stdout << 'node3_action2_stdout '
+                  stderr << 'node3_action2_stderr '
+                end } },
+                { test_action: { code: proc do |stdout, stderr|
+                  stdout << 'node3_action3_stdout'
+                  stderr << 'node3_action3_stderr'
+                end } }
+              ]
+            },
+            concurrent: true
+          )
+        ).to eq(
           'node1' => [0, 'node1_action1_stdout node1_action2_stdout node1_action3_stdout', 'node1_action1_stderr node1_action2_stderr node1_action3_stderr'],
           'node2' => [0, 'node2_action1_stdout node2_action2_stdout node2_action3_stdout', 'node2_action1_stderr node2_action2_stderr node2_action3_stderr'],
           'node3' => [0, 'node3_action1_stdout node3_action2_stdout node3_action3_stdout', 'node3_action1_stderr node3_action2_stderr node3_action3_stderr']
@@ -222,65 +238,71 @@ describe HybridPlatformsConductor::ActionsExecutor do
     it 'executes several actions on several nodes and returns the corresponding stdout and stderr correctly in parallel and in files' do
       with_repository do |logs_repository|
         with_test_platform_for_parallel_tests do
-          expect(test_actions_executor.execute_actions({
-            'node1' => [
-              { test_action: { code: proc do |stdout, stderr|
-                stdout << 'node1_action1_stdout '
-                sleep 1
-                stderr << 'node1_action1_stderr '
-                sleep 1
-              end } },
-              { test_action: { code: proc do |stdout, stderr|
-                stdout << 'node1_action2_stdout '
-                sleep 1
-                stderr << 'node1_action2_stderr '
-                sleep 1
-              end } },
-              { test_action: { code: proc do |stdout, stderr|
-                stdout << 'node1_action3_stdout'
-                sleep 1
-                stderr << 'node1_action3_stderr'
-              end } }
-            ],
-            'node2' => [
-              { test_action: { code: proc do |stdout, stderr|
-                stdout << 'node2_action1_stdout '
-                sleep 1
-                stderr << 'node2_action1_stderr '
-                sleep 1
-              end } },
-              { test_action: { code: proc do |stdout, stderr|
-                stdout << 'node2_action2_stdout '
-                sleep 1
-                stderr << 'node2_action2_stderr '
-                sleep 1
-              end } },
-              { test_action: { code: proc do |stdout, stderr|
-                stdout << 'node2_action3_stdout'
-                sleep 1
-                stderr << 'node2_action3_stderr'
-              end } }
-            ],
-            'node3' => [
-              { test_action: { code: proc do |stdout, stderr|
-                stdout << 'node3_action1_stdout '
-                sleep 1
-                stderr << 'node3_action1_stderr '
-                sleep 1
-              end } },
-              { test_action: { code: proc do |stdout, stderr|
-                stdout << 'node3_action2_stdout '
-                sleep 1
-                stderr << 'node3_action2_stderr '
-                sleep 1
-              end } },
-              { test_action: { code: proc do |stdout, stderr|
-                stdout << 'node3_action3_stdout'
-                sleep 1
-                stderr << 'node3_action3_stderr'
-              end } }
-            ]
-          }, concurrent: true, log_to_dir: logs_repository)).to eq(
+          expect(
+            test_actions_executor.execute_actions(
+              {
+                'node1' => [
+                  { test_action: { code: proc do |stdout, stderr|
+                    stdout << 'node1_action1_stdout '
+                    sleep 1
+                    stderr << 'node1_action1_stderr '
+                    sleep 1
+                  end } },
+                  { test_action: { code: proc do |stdout, stderr|
+                    stdout << 'node1_action2_stdout '
+                    sleep 1
+                    stderr << 'node1_action2_stderr '
+                    sleep 1
+                  end } },
+                  { test_action: { code: proc do |stdout, stderr|
+                    stdout << 'node1_action3_stdout'
+                    sleep 1
+                    stderr << 'node1_action3_stderr'
+                  end } }
+                ],
+                'node2' => [
+                  { test_action: { code: proc do |stdout, stderr|
+                    stdout << 'node2_action1_stdout '
+                    sleep 1
+                    stderr << 'node2_action1_stderr '
+                    sleep 1
+                  end } },
+                  { test_action: { code: proc do |stdout, stderr|
+                    stdout << 'node2_action2_stdout '
+                    sleep 1
+                    stderr << 'node2_action2_stderr '
+                    sleep 1
+                  end } },
+                  { test_action: { code: proc do |stdout, stderr|
+                    stdout << 'node2_action3_stdout'
+                    sleep 1
+                    stderr << 'node2_action3_stderr'
+                  end } }
+                ],
+                'node3' => [
+                  { test_action: { code: proc do |stdout, stderr|
+                    stdout << 'node3_action1_stdout '
+                    sleep 1
+                    stderr << 'node3_action1_stderr '
+                    sleep 1
+                  end } },
+                  { test_action: { code: proc do |stdout, stderr|
+                    stdout << 'node3_action2_stdout '
+                    sleep 1
+                    stderr << 'node3_action2_stderr '
+                    sleep 1
+                  end } },
+                  { test_action: { code: proc do |stdout, stderr|
+                    stdout << 'node3_action3_stdout'
+                    sleep 1
+                    stderr << 'node3_action3_stderr'
+                  end } }
+                ]
+              },
+              concurrent: true,
+              log_to_dir: logs_repository
+            )
+          ).to eq(
             'node1' => [0, 'node1_action1_stdout node1_action2_stdout node1_action3_stdout', 'node1_action1_stderr node1_action2_stderr node1_action3_stderr'],
             'node2' => [0, 'node2_action1_stdout node2_action2_stdout node2_action3_stdout', 'node2_action1_stderr node2_action2_stderr node2_action3_stderr'],
             'node3' => [0, 'node3_action1_stdout node3_action2_stdout node3_action3_stdout', 'node3_action1_stderr node3_action2_stderr node3_action3_stderr']
@@ -297,28 +319,34 @@ describe HybridPlatformsConductor::ActionsExecutor do
 
     it 'executes the same actions on several nodes and returns the corresponding stdout and stderr correctly in parallel and in files' do
       with_repository do |logs_repository|
-        with_test_platform(nodes: { 'node1' => {}, 'node2' => {}, 'node3' => {} }) do
-          expect(test_actions_executor.execute_actions({
-            %w[node1 node2 node3] => [
-              { ruby: proc do |stdout, stderr, action|
-                stdout << "#{action.node}_action1_stdout "
-                sleep 1
-                stderr << "#{action.node}_action1_stderr "
-                sleep 1
-              end },
-              { ruby: proc do |stdout, stderr, action|
-                stdout << "#{action.node}_action2_stdout "
-                sleep 1
-                stderr << "#{action.node}_action2_stderr "
-                sleep 1
-              end },
-              { ruby: proc do |stdout, stderr, action|
-                stdout << "#{action.node}_action3_stdout"
-                sleep 1
-                stderr << "#{action.node}_action3_stderr"
-              end }
-            ]
-          }, concurrent: true, log_to_dir: logs_repository)).to eq(
+        with_test_platform({ nodes: { 'node1' => {}, 'node2' => {}, 'node3' => {} } }) do
+          expect(
+            test_actions_executor.execute_actions(
+              {
+                %w[node1 node2 node3] => [
+                  { ruby: proc do |stdout, stderr, action|
+                    stdout << "#{action.node}_action1_stdout "
+                    sleep 1
+                    stderr << "#{action.node}_action1_stderr "
+                    sleep 1
+                  end },
+                  { ruby: proc do |stdout, stderr, action|
+                    stdout << "#{action.node}_action2_stdout "
+                    sleep 1
+                    stderr << "#{action.node}_action2_stderr "
+                    sleep 1
+                  end },
+                  { ruby: proc do |stdout, stderr, action|
+                    stdout << "#{action.node}_action3_stdout"
+                    sleep 1
+                    stderr << "#{action.node}_action3_stderr"
+                  end }
+                ]
+              },
+              concurrent: true,
+              log_to_dir: logs_repository
+            )
+          ).to eq(
             'node1' => [0, 'node1_action1_stdout node1_action2_stdout node1_action3_stdout', 'node1_action1_stderr node1_action2_stderr node1_action3_stderr'],
             'node2' => [0, 'node2_action1_stdout node2_action2_stdout node2_action3_stdout', 'node2_action1_stderr node2_action2_stderr node2_action3_stderr'],
             'node3' => [0, 'node3_action1_stdout node3_action2_stdout node3_action3_stdout', 'node3_action1_stderr node3_action2_stderr node3_action3_stderr']