hybrid_platforms_conductor 32.17.0 → 33.0.2

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,32 @@
 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 +34,19 @@ 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
@@ -65,7 +75,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
     it 'captures stdout and stderr from action correctly when using run_cmd' 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' } })
+          test_actions_executor.execute_actions({ 'node1' => { test_action: { run_cmd: 'echo action_stdout && >&2 echo action_stderr' } } })
         ).to eq('node1' => [0, "action_stdout\n", "action_stderr\n"])
       end
     end
@@ -73,35 +83,41 @@ describe HybridPlatformsConductor::ActionsExecutor do
     it 'captures stdout and stderr from action correctly when using run_cmd but with a failing command' 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"
-        ])
+          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"
+          ]
+        )
       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
@@ -110,24 +126,26 @@ describe HybridPlatformsConductor::ActionsExecutor 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
-            } },
-            'node2' => { test_action: {
-              code: proc do |stdout, stderr|
-                stdout << 'action2_stdout'
-                stderr << 'action2_stderr'
-              end
-            } },
-            'node3' => { test_action: {
-              code: proc do |stdout, stderr|
-                stdout << 'action3_stdout'
-                stderr << 'action3_stderr'
-              end
-            } }
+            {
+              'node1' => { test_action: {
+                code: proc do |stdout, stderr|
+                  stdout << 'action1_stdout'
+                  stderr << 'action1_stderr'
+                end
+              } },
+              'node2' => { test_action: {
+                code: proc do |stdout, stderr|
+                  stdout << 'action2_stdout'
+                  stderr << 'action2_stderr'
+                end
+              } },
+              'node3' => { test_action: {
+                code: proc do |stdout, stderr|
+                  stdout << 'action3_stdout'
+                  stderr << 'action3_stderr'
+                end
+              } }
+            }
           )
         ).to eq(
           'node1' => [0, 'action1_stdout', 'action1_stderr'],
@@ -141,40 +159,42 @@ describe HybridPlatformsConductor::ActionsExecutor 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
-            } },
-            %w[node1 node2] => [
-              { test_action: {
-                code: proc do |stdout, stderr|
-                  stdout << 'action2_stdout '
-                  stderr << 'action2_stderr '
-                end
-              } },
-              { test_action: {
-                code: proc do |stdout, stderr|
-                  stdout << 'action3_stdout '
-                  stderr << 'action3_stderr '
-                end
-              } }
-            ],
-            %w[node2 node3] => [
-              { test_action: {
+            {
+              'node1' => { test_action: {
                 code: proc do |stdout, stderr|
-                  stdout << 'action4_stdout '
-                  stderr << 'action4_stderr '
+                  stdout << 'action1_stdout '
+                  stderr << 'action1_stderr '
                 end
               } },
-              { test_action: {
-                code: proc do |stdout, stderr|
-                  stdout << 'action5_stdout '
-                  stderr << 'action5_stderr '
-                end
-              } }
-            ]
+              %w[node1 node2] => [
+                { test_action: {
+                  code: proc do |stdout, stderr|
+                    stdout << 'action2_stdout '
+                    stderr << 'action2_stderr '
+                  end
+                } },
+                { test_action: {
+                  code: proc do |stdout, stderr|
+                    stdout << 'action3_stdout '
+                    stderr << 'action3_stderr '
+                  end
+                } }
+              ],
+              %w[node2 node3] => [
+                { test_action: {
+                  code: proc do |stdout, stderr|
+                    stdout << 'action4_stdout '
+                    stderr << 'action4_stderr '
+                  end
+                } },
+                { test_action: {
+                  code: proc do |stdout, stderr|
+                    stdout << 'action5_stdout '
+                    stderr << 'action5_stderr '
+                  end
+                } }
+              ]
+            }
           )
         ).to eq(
           'node1' => [0, 'action1_stdout action2_stdout action3_stdout ', 'action1_stderr action2_stderr action3_stderr '],
@@ -187,56 +207,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']