hybrid_platforms_conductor 33.0.0 → 33.0.1

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

@@ -1,13 +1,13 @@
 describe HybridPlatformsConductor::ActionsExecutor do
 
-  context 'checking connector plugin ssh' do
+  context 'when checking connector plugin ssh' do
 
-    context 'checking Config DSL extensions' do
+    context 'when checking Config DSL extensions' do
 
       it 'returns 1 defined gateway' do
-        with_repository do |repository|
-          with_platforms 'gateway :gateway1, \'\'' do
-            expect(test_config.known_gateways).to eq [:gateway1]
+        with_repository do
+          with_platforms 'gateway :gateway_1, \'\'' do
+            expect(test_config.known_gateways).to eq [:gateway_1]
           end
         end
       end
@@ -17,28 +17,28 @@ describe HybridPlatformsConductor::ActionsExecutor do
           Host gateway
             Hostname mygateway.com
         '
-        with_repository do |repository|
-          with_platforms "gateway :gateway1, '#{ssh_gateway}'" do
-            expect(test_config.ssh_for_gateway(:gateway1)).to eq ssh_gateway
+        with_repository do
+          with_platforms "gateway :gateway_1, '#{ssh_gateway}'" do
+            expect(test_config.ssh_for_gateway(:gateway_1)).to eq ssh_gateway
           end
         end
       end
 
       it 'returns 1 defined gateway with its content and replacing ERB template correctly' do
-        with_repository do |repository|
-          with_platforms 'gateway :gateway1, \'Host gateway_<%= @user %>\'' do
-            expect(test_config.ssh_for_gateway(:gateway1, user: 'test_user')).to eq 'Host gateway_test_user'
+        with_repository do
+          with_platforms 'gateway :gateway_1, \'Host gateway_<%= @user %>\'' do
+            expect(test_config.ssh_for_gateway(:gateway_1, user: 'test_user')).to eq 'Host gateway_test_user'
           end
         end
       end
 
       it 'returns several defined gateways' do
-        with_repository do |repository|
+        with_repository do
           with_platforms '
-            gateway :gateway1, \'\'
-            gateway :gateway2, \'\'
+            gateway :gateway_1, \'\'
+            gateway :gateway_2, \'\'
           ' do
-            expect(test_config.known_gateways.sort).to eq %i[gateway1 gateway2].sort
+            expect(test_config.known_gateways.sort).to eq %i[gateway_1 gateway_2].sort
           end
         end
       end
@@ -50,30 +50,29 @@ describe HybridPlatformsConductor::ActionsExecutor do
               'node1' => {},
               'node2' => {},
               'node3' => {}
-            },
+            }
           },
-          false,
-          '
+          additional_config: <<~'EO_CONFIG'
             for_nodes(%w[node1 node3]) do
               transform_ssh_connection do |node, connection, connection_user, gateway, gateway_user|
                 ["#{connection}_#{node}_13", "#{connection_user}_#{node}_13", "#{gateway}_#{node}_13", "#{gateway_user}_#{node}_13"]
               end
             end
-            for_nodes(\'node1\') do
+            for_nodes('node1') do
               transform_ssh_connection do |node, connection, connection_user, gateway, gateway_user|
                 ["#{connection}_#{node}_1", "#{connection_user}_#{node}_1", "#{gateway}_#{node}_1", "#{gateway_user}_#{node}_1"]
               end
             end
-          '
+          EO_CONFIG
         ) do
           expect(test_config.ssh_connection_transforms.size).to eq 2
           expect(test_config.ssh_connection_transforms[0][:nodes_selectors_stack]).to eq [%w[node1 node3]]
-          expect(test_config.ssh_connection_transforms[0][:transform].call('node1', 'test_host', 'test_user', 'test_gateway', 'test_gateway_user')).to eq [
-            'test_host_node1_13', 'test_user_node1_13', 'test_gateway_node1_13', 'test_gateway_user_node1_13'
+          expect(test_config.ssh_connection_transforms[0][:transform].call('node1', 'test_host', 'test_user', 'test_gateway', 'test_gateway_user')).to eq %w[
+            test_host_node1_13 test_user_node1_13 test_gateway_node1_13 test_gateway_user_node1_13
           ]
           expect(test_config.ssh_connection_transforms[1][:nodes_selectors_stack]).to eq ['node1']
-          expect(test_config.ssh_connection_transforms[1][:transform].call('node1', 'test_host', 'test_user', 'test_gateway', 'test_gateway_user')).to eq [
-            'test_host_node1_1', 'test_user_node1_1', 'test_gateway_node1_1', 'test_gateway_user_node1_1'
+          expect(test_config.ssh_connection_transforms[1][:transform].call('node1', 'test_host', 'test_user', 'test_gateway', 'test_gateway_user')).to eq %w[
+            test_host_node1_1 test_user_node1_1 test_gateway_node1_1 test_gateway_user_node1_1
           ]
         end
       end

data/spec/hybrid_platforms_conductor_test/api/actions_executor/connectors/ssh/connectable_nodes_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 connectable nodes selection' do
+    context 'when checking connectable nodes selection' do
 
       # Return the connector to be tested
       #
@@ -13,10 +13,14 @@ describe HybridPlatformsConductor::ActionsExecutor do
       end
 
       it 'selects connectable nodes correctly' do
-        with_test_platform(nodes: {
-          'node1' => { meta: { host_ip: '192.168.42.42' } },
-          'node2' => {}
-        }) do
+        with_test_platform(
+          {
+            nodes: {
+              'node1' => { meta: { host_ip: '192.168.42.42' } },
+              'node2' => {}
+            }
+          }
+        ) do
           expect(test_connector.connectable_nodes_from(%w[node1 node2])).to eq ['node1']
         end
       end

data/spec/hybrid_platforms_conductor_test/api/actions_executor/connectors/ssh/connections_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 connections preparations' do
+    context 'when checking connections preparations' do
 
       # Return the connector to be tested
       #
@@ -13,7 +13,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
       end
 
       it 'creates an SSH master to 1 node' do
-        with_test_platform(nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } }) do
+        with_test_platform({ nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } } }) do
           with_cmd_runner_mocked(
             [
               ['which env', proc { [0, "/usr/bin/env\n", ''] }],
@@ -29,7 +29,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
       end
 
       it 'creates an SSH master to 1 node not having Session Exec capabilities' do
-        with_test_platform(nodes: { 'node' => { meta: { host_ip: '192.168.42.42', ssh_session_exec: false } } }) do
+        with_test_platform({ nodes: { 'node' => { meta: { host_ip: '192.168.42.42', ssh_session_exec: false } } } }) do
           with_cmd_runner_mocked(
             [
               ['which env', proc { [0, "/usr/bin/env\n", ''] }],
@@ -45,7 +45,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
       end
 
       it 'can\'t create an SSH master to 1 node not having Session Exec capabilities when hpc_interactive is false' do
-        with_test_platform(nodes: { 'node' => { meta: { host_ip: '192.168.42.42', ssh_session_exec: false } } }) do
+        with_test_platform({ nodes: { 'node' => { meta: { host_ip: '192.168.42.42', ssh_session_exec: false } } } }) do
           ENV['hpc_interactive'] = 'false'
           with_cmd_runner_mocked(
             [
@@ -67,11 +67,15 @@ describe HybridPlatformsConductor::ActionsExecutor do
       end
 
       it 'fails without creating exception when creating an SSH master to 1 node not having Session Exec capabilities when hpc_interactive is false and we use no_exception' do
-        with_test_platform(nodes: {
-          'node1' => { meta: { host_ip: '192.168.42.1' } },
-          'node2' => { meta: { host_ip: '192.168.42.2', ssh_session_exec: false } },
-          'node3' => { meta: { host_ip: '192.168.42.3' } }
-        }) do
+        with_test_platform(
+          {
+            nodes: {
+              'node1' => { meta: { host_ip: '192.168.42.1' } },
+              'node2' => { meta: { host_ip: '192.168.42.2', ssh_session_exec: false } },
+              'node3' => { meta: { host_ip: '192.168.42.3' } }
+            }
+          }
+        ) do
           ENV['hpc_interactive'] = 'false'
           with_cmd_runner_mocked(
             [
@@ -97,11 +101,15 @@ describe HybridPlatformsConductor::ActionsExecutor do
       end
 
       it 'creates SSH master to several nodes' do
-        with_test_platform(nodes: {
-          'node1' => { meta: { host_ip: '192.168.42.1' } },
-          'node2' => { meta: { host_ip: '192.168.42.2' } },
-          'node3' => { meta: { host_ip: '192.168.42.3' } }
-        }) do
+        with_test_platform(
+          {
+            nodes: {
+              'node1' => { meta: { host_ip: '192.168.42.1' } },
+              'node2' => { meta: { host_ip: '192.168.42.2' } },
+              'node3' => { meta: { host_ip: '192.168.42.3' } }
+            }
+          }
+        ) do
           with_cmd_runner_mocked(
             [
               ['which env', proc { [0, "/usr/bin/env\n", ''] }],
@@ -123,23 +131,22 @@ describe HybridPlatformsConductor::ActionsExecutor do
       it 'creates SSH master to several nodes with ssh connections transformed' do
         with_test_platform(
           { nodes: {
-          'node1' => { meta: { host_ip: '192.168.42.1' } },
-          'node2' => { meta: { host_ip: '192.168.42.2' } },
-          'node3' => { meta: { host_ip: '192.168.42.3' } }
+            'node1' => { meta: { host_ip: '192.168.42.1' } },
+            'node2' => { meta: { host_ip: '192.168.42.2' } },
+            'node3' => { meta: { host_ip: '192.168.42.3' } }
           } },
-          false,
-          '
+          additional_config: <<~'EO_CONFIG'
             for_nodes(%w[node1 node3]) do
               transform_ssh_connection do |node, connection, connection_user, gateway, gateway_user|
                 ["#{connection}_#{node}_13", "#{connection_user}_#{node}_13", "#{gateway}_#{node}_13", "#{gateway_user}_#{node}_13"]
               end
             end
-            for_nodes(\'node1\') do
+            for_nodes('node1') do
               transform_ssh_connection do |node, connection, connection_user, gateway, gateway_user|
                 ["#{connection}_#{node}_1", "#{connection_user}_#{node}_1", "#{gateway}_#{node}_1", "#{gateway_user}_#{node}_1"]
               end
             end
-          '
+          EO_CONFIG
         ) do
           with_cmd_runner_mocked(
             [
@@ -160,11 +167,15 @@ describe HybridPlatformsConductor::ActionsExecutor do
       end
 
       it 'fails when an SSH master can\'t be created' do
-        with_test_platform(nodes: {
-          'node1' => { meta: { host_ip: '192.168.42.1' } },
-          'node2' => { meta: { host_ip: '192.168.42.2' } },
-          'node3' => { meta: { host_ip: '192.168.42.3' } }
-        }) do
+        with_test_platform(
+          {
+            nodes: {
+              'node1' => { meta: { host_ip: '192.168.42.1' } },
+              'node2' => { meta: { host_ip: '192.168.42.2' } },
+              'node3' => { meta: { host_ip: '192.168.42.3' } }
+            }
+          }
+        ) do
           with_cmd_runner_mocked(
             [
               ['which env', proc { [0, "/usr/bin/env\n", ''] }],
@@ -186,17 +197,21 @@ describe HybridPlatformsConductor::ActionsExecutor do
             )
           ) do
             test_connector.ssh_user = 'test_user'
-            expect { test_connector.with_connection_to(%w[node1 node2 node3]) }.to raise_error(/^Error while starting SSH Control Master with .+\/ssh -o BatchMode=yes -o ControlMaster=yes -o ControlPersist=yes hpc.node2 true: Can't connect to 192.168.42.2$/)
+            expect { test_connector.with_connection_to(%w[node1 node2 node3]) }.to raise_error(%r{^Error while starting SSH Control Master with .+/ssh -o BatchMode=yes -o ControlMaster=yes -o ControlPersist=yes hpc.node2 true: Can't connect to 192.168.42.2$})
           end
         end
       end
 
       it 'fails without throwing exception when an SSH master can\'t be created and we use no_exception' do
-        with_test_platform(nodes: {
-          'node1' => { meta: { host_ip: '192.168.42.1' } },
-          'node2' => { meta: { host_ip: '192.168.42.2' } },
-          'node3' => { meta: { host_ip: '192.168.42.3' } }
-        }) do
+        with_test_platform(
+          {
+            nodes: {
+              'node1' => { meta: { host_ip: '192.168.42.1' } },
+              'node2' => { meta: { host_ip: '192.168.42.2' } },
+              'node3' => { meta: { host_ip: '192.168.42.3' } }
+            }
+          }
+        ) do
           with_cmd_runner_mocked(
             [
               ['which env', proc { [0, "/usr/bin/env\n", ''] }],
@@ -220,7 +235,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
       end
 
       it 'reuses SSH master already created to 1 node' do
-        with_test_platform(nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } }) do
+        with_test_platform({ nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } } }) do
           with_cmd_runner_mocked(
             [
               ['which env', proc { [0, "/usr/bin/env\n", ''] }],
@@ -245,12 +260,16 @@ describe HybridPlatformsConductor::ActionsExecutor do
       end
 
       it 'reuses SSH masters already created to some nodes and create new ones if needed' do
-        with_test_platform(nodes: {
-          'node1' => { meta: { host_ip: '192.168.42.1' } },
-          'node2' => { meta: { host_ip: '192.168.42.2' } },
-          'node3' => { meta: { host_ip: '192.168.42.3' } },
-          'node4' => { meta: { host_ip: '192.168.42.4' } }
-        }) do
+        with_test_platform(
+          {
+            nodes: {
+              'node1' => { meta: { host_ip: '192.168.42.1' } },
+              'node2' => { meta: { host_ip: '192.168.42.2' } },
+              'node3' => { meta: { host_ip: '192.168.42.3' } },
+              'node4' => { meta: { host_ip: '192.168.42.4' } }
+            }
+          }
+        ) do
           with_cmd_runner_mocked(
             [
               ['which env', proc { [0, "/usr/bin/env\n", ''] }],
@@ -285,14 +304,14 @@ describe HybridPlatformsConductor::ActionsExecutor do
       end
 
       it 'makes sure the last client using ControlMaster destroys it, even using a different environment' do
-        with_test_platform(nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } }) do
+        with_test_platform({ nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } } }) do
           # 1. Current thread creates the ControlMaster.
           # 2. Second thread connects to it.
           # 3. Current thread releases it.
           # 4. Second thread releases it, hence destroying it.
           init_commands = [
             ['which env', proc { [0, "/usr/bin/env\n", ''] }],
-            ['ssh -V 2>&1', proc { [0, "OpenSSH_7.4p1 Debian-10+deb9u7, OpenSSL 1.0.2u  20 Dec 2019\n", ''] }],
+            ['ssh -V 2>&1', proc { [0, "OpenSSH_7.4p1 Debian-10+deb9u7, OpenSSL 1.0.2u  20 Dec 2019\n", ''] }]
           ]
           nodes_connections_to_mock = { 'node' => { connection: '192.168.42.42', user: 'test_user' } }
           step = 0
@@ -311,7 +330,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
               second_config = HybridPlatformsConductor::Config.new logger: logger, logger_stderr: logger
               second_platforms_handler = HybridPlatformsConductor::PlatformsHandler.new logger: logger, logger_stderr: logger, config: second_config, cmd_runner: second_cmd_runner
               second_nodes_handler = HybridPlatformsConductor::NodesHandler.new logger: logger, logger_stderr: logger, config: second_config, cmd_runner: second_cmd_runner, platforms_handler: second_platforms_handler
-              second_actions_executor = HybridPlatformsConductor::ActionsExecutor.new logger: logger, logger_stderr: logger, config: second_config, cmd_runner: second_cmd_runner, nodes_handler: second_nodes_handler
+              second_actions_executor = described_class.new logger: logger, logger_stderr: logger, config: second_config, cmd_runner: second_cmd_runner, nodes_handler: second_nodes_handler
               second_actions_executor.connector(:ssh).ssh_user = 'test_user'
               # Wait for the first thread to create ControlMaster
               sleep 0.1 while step == 0
@@ -339,7 +358,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
       end
 
       it 'does not create SSH master if asked' do
-        with_test_platform(nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } }) do
+        with_test_platform({ nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } } }) do
           with_cmd_runner_mocked(
             [
               ['which env', proc { [0, "/usr/bin/env\n", ''] }],
@@ -360,7 +379,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
       end
 
       it 'does not check host keys if asked' do
-        with_test_platform(nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } }) do
+        with_test_platform({ nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } } }) do
           with_cmd_runner_mocked(
             [
               ['which env', proc { [0, "/usr/bin/env\n", ''] }],
@@ -379,7 +398,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
       end
 
       it 'does not use batch mode when passwords are to be expected' do
-        with_test_platform(nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } }) do
+        with_test_platform({ nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } } }) do
           with_cmd_runner_mocked(
             [
               ['which env', proc { [0, "/usr/bin/env\n", ''] }],
@@ -398,7 +417,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
       end
 
       it 'uses sshpass to prepare connections needing passwords' do
-        with_test_platform(nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } }) do
+        with_test_platform({ nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } } }) do
           with_cmd_runner_mocked(
             [
               ['sshpass -V', proc { [0, "sshpass 1.06\n", ''] }],
@@ -418,7 +437,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
       end
 
       it 'does not reuse provided SSH executables and configs' do
-        with_test_platform(nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } }) do
+        with_test_platform({ nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } } }) do
           with_cmd_runner_mocked(
             [
               ['which env', proc { [0, "/usr/bin/env\n", ''] }],
@@ -449,7 +468,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
       end
 
       it 'cleans provided SSH executables and configs after use' do
-        with_test_platform(nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } }) do
+        with_test_platform({ nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } } }) do
           with_cmd_runner_mocked(
             [
               ['which env', proc { [0, "/usr/bin/env\n", ''] }],
@@ -488,7 +507,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
       end
 
       it 'creates an SSH master to 1 node even when there is a stalled ControlMaster file' do
-        with_test_platform(nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } }) do
+        with_test_platform({ nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } } }) do
           with_cmd_runner_mocked(
             [
               ['which env', proc { [0, "/usr/bin/env\n", ''] }],
@@ -505,7 +524,7 @@ describe HybridPlatformsConductor::ActionsExecutor do
       end
 
       it 'creates an SSH master to 1 node even when there is a left-over user for the ControlMaster file that has not been unregistered' do
-        with_test_platform(nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } }) do
+        with_test_platform({ nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } } }) do
           with_cmd_runner_mocked(
             [
               ['which env', proc { [0, "/usr/bin/env\n", ''] }],
@@ -522,15 +541,15 @@ describe HybridPlatformsConductor::ActionsExecutor do
       end
 
       it 'retries when the remote node is booting up' do
-        with_test_platform(nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } }) do
+        with_test_platform({ nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } } }) do
           nbr_boot_messages = 0
           with_cmd_runner_mocked(
             [
               ['which env', proc { [0, "/usr/bin/env\n", ''] }],
-              ['ssh -V 2>&1', proc { [0, "OpenSSH_7.4p1 Debian-10+deb9u7, OpenSSL 1.0.2u  20 Dec 2019\n", ''] }],
+              ['ssh -V 2>&1', proc { [0, "OpenSSH_7.4p1 Debian-10+deb9u7, OpenSSL 1.0.2u  20 Dec 2019\n", ''] }]
             ] +
               [[
-                /^.+\/ssh -o BatchMode=yes -o ControlMaster=yes -o ControlPersist=yes hpc\.node true$/,
+                %r{^.+/ssh -o BatchMode=yes -o ControlMaster=yes -o ControlPersist=yes hpc\.node true$},
                 proc do
                   nbr_boot_messages += 1
                   [255, '', "System is booting up. See pam_nologin(8)\nAuthentication failed.\n"]

data/spec/hybrid_platforms_conductor_test/api/actions_executor/connectors/ssh/global_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' do
+    context 'when checking additional helpers' do
 
       # Return the connector to be tested
       #
@@ -35,44 +35,47 @@ describe HybridPlatformsConductor::ActionsExecutor do
         begin_marker = node.nil? ? /^Host \*$/ : /^# #{Regexp.escape(node)} - .+$/
         start_idx = ssh_config_lines.index { |line| line =~ begin_marker }
         return nil if start_idx.nil?
+
         end_markers = [
           /^\# \w+ - .+$/,
           /^\#+$/
         ]
-        end_idx = ssh_config_lines[start_idx + 1..-1].index { |line| end_markers.any? { |end_marker| line =~ end_marker } }
+        end_idx = ssh_config_lines[start_idx + 1..].index { |line| end_markers.any? { |end_marker| line =~ end_marker } }
         end_idx = end_idx.nil? ? -1 : start_idx + end_idx
-        ssh_config_lines[start_idx..end_idx].select do |line|
-          stripped_line = line.strip
-          !stripped_line.empty? && stripped_line[0] != '#'
-        end.join("\n") + "\n"
+        "#{
+          ssh_config_lines[start_idx..end_idx].select do |line|
+            stripped_line = line.strip
+            !stripped_line.empty? && stripped_line[0] != '#'
+          end.join("\n")
+        }\n"
       end
 
       it 'generates a global configuration with user from hpc_ssh_user environment variable' do
-        with_test_platform do
+        with_test_platform({}) do
           ENV['hpc_ssh_user'] = 'test_user'
-          expect(ssh_config_for(nil)).to eq <<~EOS
+          expect(ssh_config_for(nil)).to eq <<~EO_SSH_CONFIG
             Host *
               User test_user
               ControlPath #{Dir.tmpdir}/hpc_ssh/hpc_ssh_mux_%h_%p_%r
               PubkeyAcceptedKeyTypes +ssh-dss
-          EOS
+          EO_SSH_CONFIG
         end
       end
 
       it 'generates a global configuration with user from USER environment variable' do
-        with_test_platform do
+        with_test_platform({}) do
           ENV['USER'] = 'test_user'
-          expect(ssh_config_for(nil)).to eq <<~EOS
+          expect(ssh_config_for(nil)).to eq <<~EO_SSH_CONFIG
             Host *
               User test_user
               ControlPath #{Dir.tmpdir}/hpc_ssh/hpc_ssh_mux_%h_%p_%r
               PubkeyAcceptedKeyTypes +ssh-dss
-          EOS
+          EO_SSH_CONFIG
         end
       end
 
       it 'generates a global configuration with user taken from whoami when no env variable is set' do
-        with_test_platform do
+        with_test_platform({}) do
           original_user = ENV['USER']
           begin
             ENV.delete 'USER'
@@ -83,12 +86,12 @@ describe HybridPlatformsConductor::ActionsExecutor do
                 ['whoami', proc { [0, 'test_whoami_user', ''] }]
               ]
             ) do
-              expect(ssh_config_for(nil)).to eq <<~EOS
+              expect(ssh_config_for(nil)).to eq <<~EO_SSH_CONFIG
                 Host *
                   User test_whoami_user
                   ControlPath #{Dir.tmpdir}/hpc_ssh/hpc_ssh_mux_%h_%p_%r
                   PubkeyAcceptedKeyTypes +ssh-dss
-              EOS
+              EO_SSH_CONFIG
             end
           ensure
             ENV['USER'] = original_user
@@ -97,309 +100,325 @@ describe HybridPlatformsConductor::ActionsExecutor do
       end
 
       it 'generates a global configuration with user from setting' do
-        with_test_platform do
+        with_test_platform({}) do
           test_connector.ssh_user = 'test_user'
-          expect(ssh_config_for(nil)).to eq <<~EOS
+          expect(ssh_config_for(nil)).to eq <<~EO_SSH_CONFIG
             Host *
               User test_user
               ControlPath #{Dir.tmpdir}/hpc_ssh/hpc_ssh_mux_%h_%p_%r
               PubkeyAcceptedKeyTypes +ssh-dss
-          EOS
+          EO_SSH_CONFIG
         end
       end
 
       it 'generates a global configuration with known hosts file' do
-        with_test_platform do
+        with_test_platform({}) do
           test_connector.ssh_user = 'test_user'
-          expect(ssh_config_for(nil, known_hosts_file: '/path/to/known_hosts')).to eq <<~EOS
+          expect(ssh_config_for(nil, known_hosts_file: '/path/to/known_hosts')).to eq <<~EO_SSH_CONFIG
             Host *
               User test_user
               ControlPath #{Dir.tmpdir}/hpc_ssh/hpc_ssh_mux_%h_%p_%r
               PubkeyAcceptedKeyTypes +ssh-dss
               UserKnownHostsFile /path/to/known_hosts
-          EOS
+          EO_SSH_CONFIG
         end
       end
 
       it 'generates a global configuration without strict host key checking' do
-        with_test_platform do
+        with_test_platform({}) do
           test_connector.ssh_user = 'test_user'
           test_connector.ssh_strict_host_key_checking = false
-          expect(ssh_config_for(nil)).to eq <<~EOS
+          expect(ssh_config_for(nil)).to eq <<~EO_SSH_CONFIG
             Host *
               User test_user
               ControlPath #{Dir.tmpdir}/hpc_ssh/hpc_ssh_mux_%h_%p_%r
               PubkeyAcceptedKeyTypes +ssh-dss
               StrictHostKeyChecking no
-          EOS
+          EO_SSH_CONFIG
         end
       end
 
       it 'includes the gateway definition from environment' do
-        with_test_platform({}, false, 'gateway :gateway1, \'Host my_gateway\'') do
-          ENV['hpc_ssh_gateways_conf'] = 'gateway1'
-          expect(test_connector.ssh_config).to match /^Host my_gateway$/
+        with_test_platform({}, additional_config: 'gateway :gateway_1, \'Host my_gateway\'') do
+          ENV['hpc_ssh_gateways_conf'] = 'gateway_1'
+          expect(test_connector.ssh_config).to match(/^Host my_gateway$/)
         end
       end
 
       it 'includes the gateway definition from setting' do
-        with_test_platform({}, false, 'gateway :gateway1, \'Host my_gateway\'') do
-          test_connector.ssh_gateways_conf = :gateway1
-          expect(test_connector.ssh_config).to match /^Host my_gateway$/
+        with_test_platform({}, additional_config: 'gateway :gateway_1, \'Host my_gateway\'') do
+          test_connector.ssh_gateways_conf = :gateway_1
+          expect(test_connector.ssh_config).to match(/^Host my_gateway$/)
         end
       end
 
       it 'includes the gateway definition with a different ssh executable' do
-        with_test_platform({}, false, 'gateway :gateway1, \'Host my_gateway_<%= @ssh_exec %>\'') do
-          test_connector.ssh_gateways_conf = :gateway1
-          expect(test_connector.ssh_config(ssh_exec: 'new_ssh')).to match /^Host my_gateway_new_ssh$/
+        with_test_platform({}, additional_config: 'gateway :gateway_1, \'Host my_gateway_<%= @ssh_exec %>\'') do
+          test_connector.ssh_gateways_conf = :gateway_1
+          expect(test_connector.ssh_config(ssh_exec: 'new_ssh')).to match(/^Host my_gateway_new_ssh$/)
         end
       end
 
       it 'does not include the gateway definition if it is not selected' do
-        with_test_platform({}, false, 'gateway :gateway2, \'Host my_gateway\'') do
-          test_connector.ssh_gateways_conf = :gateway1
-          expect(test_connector.ssh_config).not_to match /^Host my_gateway$/
+        with_test_platform({}, additional_config: 'gateway :gateway_2, \'Host my_gateway\'') do
+          test_connector.ssh_gateways_conf = :gateway_1
+          expect(test_connector.ssh_config).not_to match(/^Host my_gateway$/)
         end
       end
 
       it 'generates a simple config for a node with host_ip' do
-        with_test_platform(nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } }) do
-          expect(ssh_config_for('node')).to eq <<~EOS
+        with_test_platform({ nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } } }) do
+          expect(ssh_config_for('node')).to eq <<~EO_SSH_CONFIG
             Host hpc.node
               Hostname 192.168.42.42
-          EOS
+          EO_SSH_CONFIG
         end
       end
 
       it 'generates a simple config for several nodes' do
-        with_test_platform(nodes: {
-          'node1' => { meta: { host_ip: '192.168.42.1' } },
-          'node2' => { meta: { host_ip: '192.168.42.2' } },
-          'node3' => { meta: { host_ip: '192.168.42.3' } }
-        }) do
-          expect(ssh_config_for('node1')).to eq <<~EOS
+        with_test_platform(
+          {
+            nodes: {
+              'node1' => { meta: { host_ip: '192.168.42.1' } },
+              'node2' => { meta: { host_ip: '192.168.42.2' } },
+              'node3' => { meta: { host_ip: '192.168.42.3' } }
+            }
+          }
+        ) do
+          expect(ssh_config_for('node1')).to eq <<~EO_SSH_CONFIG
             Host hpc.node1
               Hostname 192.168.42.1
-          EOS
-          expect(ssh_config_for('node2')).to eq <<~EOS
+          EO_SSH_CONFIG
+          expect(ssh_config_for('node2')).to eq <<~EO_SSH_CONFIG
             Host hpc.node2
               Hostname 192.168.42.2
-          EOS
-          expect(ssh_config_for('node3')).to eq <<~EOS
+          EO_SSH_CONFIG
+          expect(ssh_config_for('node3')).to eq <<~EO_SSH_CONFIG
             Host hpc.node3
               Hostname 192.168.42.3
-          EOS
+          EO_SSH_CONFIG
         end
       end
 
       it 'generates a simple config for several nodes even when some of them can\'t be connected' do
-        with_test_platform(nodes: {
-          'node1' => { meta: { host_ip: '192.168.42.1' } },
-          'node2' => { meta: {} },
-          'node3' => { meta: { host_ip: '192.168.42.3' } }
-        }) do
-          expect(ssh_config_for('node1')).to eq <<~EOS
+        with_test_platform(
+          {
+            nodes: {
+              'node1' => { meta: { host_ip: '192.168.42.1' } },
+              'node2' => { meta: {} },
+              'node3' => { meta: { host_ip: '192.168.42.3' } }
+            }
+          }
+        ) do
+          expect(ssh_config_for('node1')).to eq <<~EO_SSH_CONFIG
             Host hpc.node1
               Hostname 192.168.42.1
-          EOS
+          EO_SSH_CONFIG
           expect(ssh_config_for('node2')).to eq "\n"
-          expect(ssh_config_for('node3')).to eq <<~EOS
+          expect(ssh_config_for('node3')).to eq <<~EO_SSH_CONFIG
             Host hpc.node3
               Hostname 192.168.42.3
-          EOS
+          EO_SSH_CONFIG
         end
       end
 
       it 'selects nodes when generating the config' do
-        with_test_platform(nodes: {
-          'node1' => { meta: { host_ip: '192.168.42.1' } },
-          'node2' => { meta: { host_ip: '192.168.42.2' } },
-          'node3' => { meta: { host_ip: '192.168.42.3' } }
-        }) do
-          expect(ssh_config_for('node1', nodes: %w[node1 node3])).to eq <<~EOS
+        with_test_platform(
+          {
+            nodes: {
+              'node1' => { meta: { host_ip: '192.168.42.1' } },
+              'node2' => { meta: { host_ip: '192.168.42.2' } },
+              'node3' => { meta: { host_ip: '192.168.42.3' } }
+            }
+          }
+        ) do
+          expect(ssh_config_for('node1', nodes: %w[node1 node3])).to eq <<~EO_SSH_CONFIG
             Host hpc.node1
               Hostname 192.168.42.1
-          EOS
+          EO_SSH_CONFIG
           expect(ssh_config_for('node2', nodes: %w[node1 node3])).to eq nil
         end
       end
 
       it 'generates an alias if the node has a hostname' do
-        with_test_platform(nodes: { 'node' => { meta: { host_ip: '192.168.42.42', hostname: 'my_hostname.my_domain' } } }) do
-          expect(ssh_config_for('node')).to eq <<~EOS
+        with_test_platform({ nodes: { 'node' => { meta: { host_ip: '192.168.42.42', hostname: 'my_hostname.my_domain' } } } }) do
+          expect(ssh_config_for('node')).to eq <<~EO_SSH_CONFIG
             Host hpc.node my_hostname.my_domain
               Hostname 192.168.42.42
-          EOS
+          EO_SSH_CONFIG
         end
       end
 
       it 'generates aliases if the node has private ips' do
-        with_test_platform(nodes: { 'node' => { meta: { host_ip: '192.168.42.42', private_ips: ['192.168.42.1', '192.168.42.2'] } } }) do
-          expect(ssh_config_for('node')).to eq <<~EOS
+        with_test_platform({ nodes: { 'node' => { meta: { host_ip: '192.168.42.42', private_ips: ['192.168.42.1', '192.168.42.2'] } } } }) do
+          expect(ssh_config_for('node')).to eq <<~EO_SSH_CONFIG
             Host hpc.node hpc.192.168.42.1 hpc.192.168.42.2
               Hostname 192.168.42.42
-          EOS
+          EO_SSH_CONFIG
         end
       end
 
       it 'generates a simple config for a node with hostname' do
-        with_test_platform(nodes: { 'node' => { meta: { hostname: 'my_hostname.my_domain' } } }) do
-          expect(ssh_config_for('node')).to eq <<~EOS
+        with_test_platform({ nodes: { 'node' => { meta: { hostname: 'my_hostname.my_domain' } } } }) do
+          expect(ssh_config_for('node')).to eq <<~EO_SSH_CONFIG
             Host hpc.node my_hostname.my_domain
               Hostname my_hostname.my_domain
-          EOS
+          EO_SSH_CONFIG
         end
       end
 
       it 'generates a simple config for a node with private_ips' do
-        with_test_platform(nodes: { 'node' => { meta: { private_ips: ['192.168.42.1', '192.168.42.2'] } } }) do
-          expect(ssh_config_for('node')).to eq <<~EOS
+        with_test_platform({ nodes: { 'node' => { meta: { private_ips: ['192.168.42.1', '192.168.42.2'] } } } }) do
+          expect(ssh_config_for('node')).to eq <<~EO_SSH_CONFIG
             Host hpc.node hpc.192.168.42.1 hpc.192.168.42.2
               Hostname 192.168.42.1
-          EOS
+          EO_SSH_CONFIG
         end
       end
 
       it 'uses node forced gateway information' do
-        with_test_platform(nodes: { 'node' => { meta: { host_ip: '192.168.42.42', gateway: 'test_gateway', gateway_user: 'test_gateway_user' } } }) do
-          expect(ssh_config_for('node')).to eq <<~EOS
+        with_test_platform({ nodes: { 'node' => { meta: { host_ip: '192.168.42.42', gateway: 'test_gateway', gateway_user: 'test_gateway_user' } } } }) do
+          expect(ssh_config_for('node')).to eq <<~EO_SSH_CONFIG
             Host hpc.node
               Hostname 192.168.42.42
               ProxyCommand ssh -q -W %h:%p test_gateway_user@test_gateway
-          EOS
+          EO_SSH_CONFIG
         end
       end
 
       it 'uses node default gateway information and user from environment' do
-        with_test_platform(nodes: { 'node' => { meta: { host_ip: '192.168.42.42', gateway: 'test_gateway' } } }) do
+        with_test_platform({ nodes: { 'node' => { meta: { host_ip: '192.168.42.42', gateway: 'test_gateway' } } } }) do
           ENV['hpc_ssh_gateway_user'] = 'test_gateway_user'
-          expect(ssh_config_for('node')).to eq <<~EOS
+          expect(ssh_config_for('node')).to eq <<~EO_SSH_CONFIG
             Host hpc.node
               Hostname 192.168.42.42
               ProxyCommand ssh -q -W %h:%p test_gateway_user@test_gateway
-          EOS
+          EO_SSH_CONFIG
         end
       end
 
       it 'uses node default gateway information and user from setting' do
-        with_test_platform(nodes: { 'node' => { meta: { host_ip: '192.168.42.42', gateway: 'test_gateway' } } }) do
+        with_test_platform({ nodes: { 'node' => { meta: { host_ip: '192.168.42.42', gateway: 'test_gateway' } } } }) do
           test_connector.ssh_gateway_user = 'test_gateway_user'
-          expect(ssh_config_for('node')).to eq <<~EOS
+          expect(ssh_config_for('node')).to eq <<~EO_SSH_CONFIG
             Host hpc.node
               Hostname 192.168.42.42
               ProxyCommand ssh -q -W %h:%p test_gateway_user@test_gateway
-          EOS
+          EO_SSH_CONFIG
         end
       end
 
       it 'uses node forced gateway information with a different ssh executable' do
-        with_test_platform(nodes: { 'node' => { meta: { host_ip: '192.168.42.42', gateway: 'test_gateway', gateway_user: 'test_gateway_user' } } }) do
-          expect(ssh_config_for('node', ssh_exec: 'new_ssh')).to eq <<~EOS
+        with_test_platform({ nodes: { 'node' => { meta: { host_ip: '192.168.42.42', gateway: 'test_gateway', gateway_user: 'test_gateway_user' } } } }) do
+          expect(ssh_config_for('node', ssh_exec: 'new_ssh')).to eq <<~EO_SSH_CONFIG
             Host hpc.node
               Hostname 192.168.42.42
               ProxyCommand new_ssh -q -W %h:%p test_gateway_user@test_gateway
-          EOS
+          EO_SSH_CONFIG
         end
       end
 
       it 'uses node default gateway information with a different ssh executable' do
-        with_test_platform(nodes: { 'node' => { meta: { host_ip: '192.168.42.42', gateway: 'test_gateway' } } }) do
+        with_test_platform({ nodes: { 'node' => { meta: { host_ip: '192.168.42.42', gateway: 'test_gateway' } } } }) do
           test_connector.ssh_gateway_user = 'test_gateway_user'
-          expect(ssh_config_for('node', ssh_exec: 'new_ssh')).to eq <<~EOS
+          expect(ssh_config_for('node', ssh_exec: 'new_ssh')).to eq <<~EO_SSH_CONFIG
             Host hpc.node
               Hostname 192.168.42.42
               ProxyCommand new_ssh -q -W %h:%p test_gateway_user@test_gateway
-          EOS
+          EO_SSH_CONFIG
         end
       end
 
       it 'uses node transformed SSH connection' do
         with_test_platform(
           { nodes: {
-            'node1' => { meta: { host_ip: '192.168.42.1', gateway: 'test_gateway1', gateway_user: 'test_gateway1_user' } },
-            'node2' => { meta: { host_ip: '192.168.42.2', gateway: 'test_gateway2', gateway_user: 'test_gateway2_user' } },
+            'node1' => { meta: { host_ip: '192.168.42.1', gateway: 'test_gateway_1', gateway_user: 'test_gateway_1_user' } },
+            'node2' => { meta: { host_ip: '192.168.42.2', gateway: 'test_gateway_2', gateway_user: 'test_gateway_2_user' } },
             'node3' => { meta: { host_ip: '192.168.42.3', gateway: 'test_gateway3', gateway_user: 'test_gateway3_user' } }
           } },
-          false,
-          '
+          additional_config: <<~'EO_CONFIG'
             for_nodes(%w[node1 node3]) do
               transform_ssh_connection do |node, connection, connection_user, gateway, gateway_user|
                 ["#{connection}_#{node}_13", "#{connection_user}_#{node}_13", "#{gateway}_#{node}_13", "#{gateway_user}_#{node}_13"]
               end
             end
-            for_nodes(\'node1\') do
+            for_nodes('node1') do
               transform_ssh_connection do |node, connection, connection_user, gateway, gateway_user|
                 ["#{connection}_#{node}_1", "#{connection_user}_#{node}_1", "#{gateway}_#{node}_1", "#{gateway_user}_#{node}_1"]
               end
             end
-          ') do
+          EO_CONFIG
+        ) do
           test_connector.ssh_user = 'test_user'
-          expect(ssh_config_for('node1')).to eq <<~EOS
+          expect(ssh_config_for('node1')).to eq <<~EO_SSH_CONFIG
             Host hpc.node1
               Hostname 192.168.42.1_node1_13_node1_1
               User "test_user_node1_13_node1_1"
-              ProxyCommand ssh -q -W %h:%p test_gateway1_user_node1_13_node1_1@test_gateway1_node1_13_node1_1
-          EOS
-          expect(ssh_config_for('node2')).to eq <<~EOS
+              ProxyCommand ssh -q -W %h:%p test_gateway_1_user_node1_13_node1_1@test_gateway_1_node1_13_node1_1
+          EO_SSH_CONFIG
+          expect(ssh_config_for('node2')).to eq <<~EO_SSH_CONFIG
             Host hpc.node2
               Hostname 192.168.42.2
-              ProxyCommand ssh -q -W %h:%p test_gateway2_user@test_gateway2
-          EOS
-          expect(ssh_config_for('node3')).to eq <<~EOS
+              ProxyCommand ssh -q -W %h:%p test_gateway_2_user@test_gateway_2
+          EO_SSH_CONFIG
+          expect(ssh_config_for('node3')).to eq <<~EO_SSH_CONFIG
             Host hpc.node3
               Hostname 192.168.42.3_node3_13
               User "test_user_node3_13"
               ProxyCommand ssh -q -W %h:%p test_gateway3_user_node3_13@test_gateway3_node3_13
-          EOS
+          EO_SSH_CONFIG
         end
       end
 
       it 'generates a config compatible for passwords authentication' do
-        with_test_platform(nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } }) do
+        with_test_platform({ nodes: { 'node' => { meta: { host_ip: '192.168.42.42' } } } }) do
           test_connector.passwords['node'] = 'PaSsWoRd'
-          expect(ssh_config_for('node')).to eq <<~EOS
+          expect(ssh_config_for('node')).to eq <<~EO_SSH_CONFIG
             Host hpc.node
               Hostname 192.168.42.42
               PreferredAuthentications password
               PubkeyAuthentication no
-          EOS
+          EO_SSH_CONFIG
         end
       end
 
       it 'generates a config compatible for passwords authentication only for marked nodes' do
-        with_test_platform(nodes: {
-          'node1' => { meta: { host_ip: '192.168.42.1' } },
-          'node2' => { meta: { host_ip: '192.168.42.2' } },
-          'node3' => { meta: { host_ip: '192.168.42.3' } },
-          'node4' => { meta: { host_ip: '192.168.42.4' } }
-        }) do
+        with_test_platform(
+          {
+            nodes: {
+              'node1' => { meta: { host_ip: '192.168.42.1' } },
+              'node2' => { meta: { host_ip: '192.168.42.2' } },
+              'node3' => { meta: { host_ip: '192.168.42.3' } },
+              'node4' => { meta: { host_ip: '192.168.42.4' } }
+            }
+          }
+        ) do
           test_connector.passwords['node1'] = 'PaSsWoRd1'
           test_connector.passwords['node3'] = 'PaSsWoRd3'
-          expect(ssh_config_for('node1')).to eq <<~EOS
+          expect(ssh_config_for('node1')).to eq <<~EO_SSH_CONFIG
             Host hpc.node1
               Hostname 192.168.42.1
               PreferredAuthentications password
               PubkeyAuthentication no
-          EOS
-          expect(ssh_config_for('node2')).to eq <<~EOS
+          EO_SSH_CONFIG
+          expect(ssh_config_for('node2')).to eq <<~EO_SSH_CONFIG
             Host hpc.node2
               Hostname 192.168.42.2
-          EOS
-          expect(ssh_config_for('node3')).to eq <<~EOS
+          EO_SSH_CONFIG
+          expect(ssh_config_for('node3')).to eq <<~EO_SSH_CONFIG
             Host hpc.node3
               Hostname 192.168.42.3
               PreferredAuthentications password
               PubkeyAuthentication no
-          EOS
-          expect(ssh_config_for('node4')).to eq <<~EOS
+          EO_SSH_CONFIG
+          expect(ssh_config_for('node4')).to eq <<~EO_SSH_CONFIG
             Host hpc.node4
               Hostname 192.168.42.4
-          EOS
+          EO_SSH_CONFIG
         end
       end
 
-   end
+    end
 
   end