hybrid_platforms_conductor 32.18.0 → 33.0.4

data/spec/hybrid_platforms_conductor_test/api/actions_executor/actions/scp_spec.rb

@@ -1,10 +1,10 @@
 describe HybridPlatformsConductor::ActionsExecutor do
 
-  context 'checking actions\' plugin scp' do
+  context 'when checking actions\' plugin scp' do
 
     it 'executes remote SCP' do
       with_test_platform_for_action_plugins do
-        test_actions_executor.execute_actions('node' => { scp: { 'from' => 'to' } })
+        test_actions_executor.execute_actions({ 'node' => { scp: { 'from' => 'to' } } })
         expect(test_actions_executor.connector(:test_connector).calls).to eq [
           [:connectable_nodes_from, ['node']],
           [:with_connection_to, ['node'], { no_exception: true }],
@@ -32,10 +32,14 @@ describe HybridPlatformsConductor::ActionsExecutor do
 
     it 'executes remote SCP on several files' do
       with_test_platform_for_action_plugins do
-        test_actions_executor.execute_actions('node' => { scp: {
-          'from1' => 'to1',
-          'from2' => 'to2'
-        } })
+        test_actions_executor.execute_actions(
+          {
+            'node' => { scp: {
+              'from1' => 'to1',
+              'from2' => 'to2'
+            } }
+          }
+        )
         expect(test_actions_executor.connector(:test_connector).calls).to eq [
           [:connectable_nodes_from, ['node']],
           [:with_connection_to, ['node'], { no_exception: true }],
@@ -47,10 +51,14 @@ describe HybridPlatformsConductor::ActionsExecutor do
 
     it 'executes remote SCP with sudo' do
       with_test_platform_for_action_plugins do
-        test_actions_executor.execute_actions('node' => { scp: {
-          'from' => 'to',
-          sudo: true
-        } })
+        test_actions_executor.execute_actions(
+          {
+            'node' => { scp: {
+              'from' => 'to',
+              sudo: true
+            } }
+          }
+        )
         expect(test_actions_executor.connector(:test_connector).calls).to eq [
           [:connectable_nodes_from, ['node']],
           [:with_connection_to, ['node'], { no_exception: true }],
@@ -61,11 +69,15 @@ describe HybridPlatformsConductor::ActionsExecutor do
 
     it 'executes remote SCP with different owner and group' do
       with_test_platform_for_action_plugins do
-        test_actions_executor.execute_actions('node' => { scp: {
-          'from' => 'to',
-          owner: 'new_owner',
-          group: 'new_group'
-        } })
+        test_actions_executor.execute_actions(
+          {
+            'node' => { scp: {
+              'from' => 'to',
+              owner: 'new_owner',
+              group: 'new_group'
+            } }
+          }
+        )
         expect(test_actions_executor.connector(:test_connector).calls).to eq [
           [:connectable_nodes_from, ['node']],
           [:with_connection_to, ['node'], { no_exception: true }],

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

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

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

@@ -1,39 +1,39 @@
 describe HybridPlatformsConductor::ActionsExecutor do
 
-  context 'checking connections handling' do
+  context 'when checking connections handling' do
 
     # Get a test platform to test connection, using the test_connector
     #
     # Parameters::
-    # * Proc: Code called with platform setup
+    # * *block* (Proc): Code called with platform setup
     #   * Parameters::
     #     * *repository* (String): Repository where the platform has been setup
-    def with_test_platform_for_connections
-      with_test_platform_for_executor(nodes: {
-        'node1' => {},
-        'node2' => {},
-        'node3' => {},
-        'node4' => {}
-      }) do |repository|
-        yield repository
-      end
+    def with_test_platform_for_connections(&block)
+      with_test_platform_for_executor(
+        nodes: {
+          'node1' => {},
+          'node2' => {},
+          'node3' => {},
+          'node4' => {}
+        }, &block
+      )
     end
 
     it 'connects on a node before executing actions needing connection' do
       with_test_platform_for_connections do
         test_actions_executor.connector(:test_connector).accept_nodes = ['node1']
-        test_actions_executor.execute_actions('node1' => { test_action: { need_connector: true } })
+        test_actions_executor.execute_actions({ 'node1' => { test_action: { need_connector: true } } })
         expect(action_executions).to eq [{ node: 'node1', message: 'Action executed' }]
         expect(test_actions_executor.connector(:test_connector).calls).to eq [
-         [:connectable_nodes_from, ['node1']],
-         [:with_connection_to, ['node1'], { no_exception: true }]
+          [:connectable_nodes_from, ['node1']],
+          [:with_connection_to, ['node1'], { no_exception: true }]
         ]
       end
     end
 
     it 'returns an error when no connector can connect to the needed node' do
       with_test_platform_for_connections do
-        expect(test_actions_executor.execute_actions('node1' => { test_action: { need_connector: true } })).to eq(
+        expect(test_actions_executor.execute_actions({ 'node1' => { test_action: { need_connector: true } } })).to eq(
           'node1' => [:no_connector, '', 'Unable to get a connector to node1']
         )
       end
@@ -42,7 +42,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
     it 'connects on several nodes before executing actions needing connection' do
       with_test_platform_for_connections do
         test_actions_executor.connector(:test_connector).accept_nodes = %w[node1 node2 node3 node4]
-        test_actions_executor.execute_actions(%w[node1 node2 node3 node4] => { test_action: { need_connector: true } })
+        test_actions_executor.execute_actions({ %w[node1 node2 node3 node4] => { test_action: { need_connector: true } } })
         expect(action_executions).to eq [
           { node: 'node1', message: 'Action executed' },
           { node: 'node2', message: 'Action executed' },
@@ -50,8 +50,8 @@ describe HybridPlatformsConductor::ActionsExecutor do
           { node: 'node4', message: 'Action executed' }
         ]
         expect(test_actions_executor.connector(:test_connector).calls).to eq [
-         [:connectable_nodes_from, %w[node1 node2 node3 node4]],
-         [:with_connection_to, %w[node1 node2 node3 node4], { no_exception: true }]
+          [:connectable_nodes_from, %w[node1 node2 node3 node4]],
+          [:with_connection_to, %w[node1 node2 node3 node4], { no_exception: true }]
         ]
       end
     end
@@ -60,15 +60,15 @@ describe HybridPlatformsConductor::ActionsExecutor do
       with_test_platform_for_connections do
         test_actions_executor.connector(:test_connector).accept_nodes = %w[node1 node2 node3 node4]
         test_actions_executor.connector(:test_connector).connected_nodes = %w[node1 node2 node4]
-        test_actions_executor.execute_actions(%w[node1 node2 node3 node4] => { test_action: { need_connector: true } })
+        test_actions_executor.execute_actions({ %w[node1 node2 node3 node4] => { test_action: { need_connector: true } } })
         expect(action_executions).to eq [
           { node: 'node1', message: 'Action executed' },
           { node: 'node2', message: 'Action executed' },
           { node: 'node4', message: 'Action executed' }
         ]
         expect(test_actions_executor.connector(:test_connector).calls).to eq [
-         [:connectable_nodes_from, %w[node1 node2 node3 node4]],
-         [:with_connection_to, %w[node1 node2 node3 node4], { no_exception: true }]
+          [:connectable_nodes_from, %w[node1 node2 node3 node4]],
+          [:with_connection_to, %w[node1 node2 node3 node4], { no_exception: true }]
         ]
       end
     end
@@ -77,8 +77,10 @@ describe HybridPlatformsConductor::ActionsExecutor do
       with_test_platform_for_connections do
         test_actions_executor.connector(:test_connector).accept_nodes = %w[node1 node2 node3 node4]
         test_actions_executor.execute_actions(
-          %w[node1 node3] => { test_action: { need_connector: true } },
-          %w[node2 node4] => { test_action: { need_connector: false } }
+          {
+            %w[node1 node3] => { test_action: { need_connector: true } },
+            %w[node2 node4] => { test_action: { need_connector: false } }
+          }
         )
         expect(action_executions).to eq [
           { node: 'node1', message: 'Action executed' },
@@ -87,8 +89,8 @@ describe HybridPlatformsConductor::ActionsExecutor do
           { node: 'node4', message: 'Action executed' }
         ]
         expect(test_actions_executor.connector(:test_connector).calls).to eq [
-         [:connectable_nodes_from, %w[node1 node3]],
-         [:with_connection_to, %w[node1 node3], { no_exception: true }]
+          [:connectable_nodes_from, %w[node1 node3]],
+          [:with_connection_to, %w[node1 node3], { no_exception: true }]
         ]
       end
     end
@@ -96,7 +98,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
     it 'does not ask for any connection if actions don\'t need remote' do
       with_test_platform_for_connections do
         test_actions_executor.connector(:test_connector).accept_nodes = %w[node1 node2 node3 node4]
-        test_actions_executor.execute_actions(%w[node1 node2 node3 node4] => { test_action: { need_connector: false } })
+        test_actions_executor.execute_actions({ %w[node1 node2 node3 node4] => { test_action: { need_connector: false } } })
         expect(action_executions).to eq [
           { node: 'node1', message: 'Action executed' },
           { node: 'node2', message: 'Action executed' },
@@ -111,7 +113,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
       with_test_platform_for_connections do
         test_actions_executor.connector(:test_connector).accept_nodes = %w[node1 node3]
         test_actions_executor.connector(:test_connector_2).accept_nodes = %w[node2 node4]
-        test_actions_executor.execute_actions(%w[node1 node2 node3 node4] => { test_action: { need_connector: true } })
+        test_actions_executor.execute_actions({ %w[node1 node2 node3 node4] => { test_action: { need_connector: true } } })
         expect(action_executions).to eq [
           { node: 'node1', message: 'Action executed' },
           { node: 'node2', message: 'Action executed' },
@@ -119,12 +121,12 @@ describe HybridPlatformsConductor::ActionsExecutor do
           { node: 'node4', message: 'Action executed' }
         ]
         expect(test_actions_executor.connector(:test_connector).calls).to eq [
-         [:connectable_nodes_from, %w[node1 node2 node3 node4]],
-         [:with_connection_to, %w[node1 node3], { no_exception: true }]
+          [:connectable_nodes_from, %w[node1 node2 node3 node4]],
+          [:with_connection_to, %w[node1 node3], { no_exception: true }]
         ]
         expect(test_actions_executor.connector(:test_connector_2).calls).to eq [
-         [:connectable_nodes_from, %w[node2 node4]],
-         [:with_connection_to, %w[node2 node4], { no_exception: true }]
+          [:connectable_nodes_from, %w[node2 node4]],
+          [:with_connection_to, %w[node2 node4], { no_exception: true }]
         ]
       end
     end
@@ -133,7 +135,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
       with_test_platform_for_connections do
         test_actions_executor.connector(:test_connector).accept_nodes = %w[node1 node2 node3]
         test_actions_executor.connector(:test_connector_2).accept_nodes = %w[node2 node4]
-        test_actions_executor.execute_actions(%w[node1 node2 node3 node4] => { test_action: { need_connector: true } })
+        test_actions_executor.execute_actions({ %w[node1 node2 node3 node4] => { test_action: { need_connector: true } } })
         expect(action_executions).to eq [
           { node: 'node1', message: 'Action executed' },
           { node: 'node2', message: 'Action executed' },
@@ -141,12 +143,12 @@ describe HybridPlatformsConductor::ActionsExecutor do
           { node: 'node4', message: 'Action executed' }
         ]
         expect(test_actions_executor.connector(:test_connector).calls).to eq [
-         [:connectable_nodes_from, %w[node1 node2 node3 node4]],
-         [:with_connection_to, %w[node1 node2 node3], { no_exception: true }]
+          [:connectable_nodes_from, %w[node1 node2 node3 node4]],
+          [:with_connection_to, %w[node1 node2 node3], { no_exception: true }]
         ]
         expect(test_actions_executor.connector(:test_connector_2).calls).to eq [
-         [:connectable_nodes_from, %w[node4]],
-         [:with_connection_to, %w[node4], { no_exception: true }]
+          [:connectable_nodes_from, %w[node4]],
+          [:with_connection_to, %w[node4], { no_exception: true }]
         ]
       end
     end
@@ -157,12 +159,12 @@ describe HybridPlatformsConductor::ActionsExecutor do
         test_actions_executor.connector(:test_connector_2).accept_nodes = %w[node2 node4]
         test_actions_executor.with_connections_prepared_to(%w[node1 node2 node3 node4]) do |connected_nodes|
           expect(test_actions_executor.connector(:test_connector).calls).to eq [
-           [:connectable_nodes_from, %w[node1 node2 node3 node4]],
-           [:with_connection_to, %w[node1 node3], { no_exception: false }]
+            [:connectable_nodes_from, %w[node1 node2 node3 node4]],
+            [:with_connection_to, %w[node1 node3], { no_exception: false }]
           ]
           expect(test_actions_executor.connector(:test_connector_2).calls).to eq [
-           [:connectable_nodes_from, %w[node2 node4]],
-           [:with_connection_to, %w[node2 node4], { no_exception: false }]
+            [:connectable_nodes_from, %w[node2 node4]],
+            [:with_connection_to, %w[node2 node4], { no_exception: false }]
           ]
           expect(connected_nodes).to eq(
             'node1' => test_actions_executor.connector(:test_connector),
@@ -190,12 +192,12 @@ describe HybridPlatformsConductor::ActionsExecutor do
         test_actions_executor.connector(:test_connector_2).accept_nodes = %w[node2]
         test_actions_executor.with_connections_prepared_to(%w[node1 node2 node3 node4], no_exception: true) do |connected_nodes|
           expect(test_actions_executor.connector(:test_connector).calls).to eq [
-           [:connectable_nodes_from, %w[node1 node2 node3 node4]],
-           [:with_connection_to, %w[node1 node3], { no_exception: true }]
+            [:connectable_nodes_from, %w[node1 node2 node3 node4]],
+            [:with_connection_to, %w[node1 node3], { no_exception: true }]
           ]
           expect(test_actions_executor.connector(:test_connector_2).calls).to eq [
-           [:connectable_nodes_from, %w[node2 node4]],
-           [:with_connection_to, %w[node2], { no_exception: true }]
+            [:connectable_nodes_from, %w[node2 node4]],
+            [:with_connection_to, %w[node2], { no_exception: true }]
           ]
           expect(connected_nodes).to eq(
             'node1' => test_actions_executor.connector(:test_connector),