hybrid_platforms_conductor 32.17.0 → 33.0.2

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']