hybrid_platforms_conductor 32.12.0 → 32.13.0

data/lib/hybrid_platforms_conductor/hpc_plugins/provisioner/proxmox.rb

@@ -15,7 +15,7 @@ module HybridPlatformsConductor
 
         include LoggerHelpers
 
-        attr_accessor *%i[logger logger_stderr]
+        attr_accessor(*%i[logger logger_stderr])
 
         def check_response(response)
           msg = "Response from Proxmox API: #{response} - #{response.net_http_res.message}"
@@ -540,7 +540,10 @@ module HybridPlatformsConductor
         #   * *api_wait_between_retries_secs* (Integer): Number of seconds to wait between API retries
         #   * *sync_node* (String): Node to be used to synchronize Proxmox resources acquisition
         #   * *test_config* (Hash<Symbol,Object>): The test configuration. Check ProxmoxWaiter#initialize (config_file structure) method to get details.
-        #   * *vm_config* (Hash<Symbol,Object>): Extra configuration of a created container. Check #request_lxc_creation_for results to get details.
+        #   * *vm_config* (Hash<Symbol,Object>): Extra configuration of a created container:
+        #     * *vm_dns_servers* (Array<String>): List of DNS servers
+        #     * *vm_search_domain* (String): Default search domain
+        #     * *vm_gateway* (String): Gateway hostname or IP
         #   * *default_timeout* (Integer): The default timeout tobe applied when starting/stopping containers [default: 3600].
         def proxmox_test_info
           @config.proxmox_servers.first

data/lib/hybrid_platforms_conductor/hpc_plugins/test/bitbucket_conf.rb

@@ -60,7 +60,7 @@ module HybridPlatformsConductor
               end
             end
             # Merge checks
-            required_approvers = repo_info.dig *%i[checks pr_settings required_approvers]
+            required_approvers = repo_info.dig(*%i[checks pr_settings required_approvers])
             if required_approvers
               assert_equal(
                 settings_pr.dig('com.atlassian.bitbucket.server.bitbucket-bundled-hooks:requiredApprovers', 'enable'),
@@ -73,7 +73,7 @@ module HybridPlatformsConductor
                 "[#{repo_id}] - Number of required approvers should be #{required_approvers}"
               )
             end
-            required_builds = repo_info.dig *%i[checks pr_settings required_builds]
+            required_builds = repo_info.dig(*%i[checks pr_settings required_builds])
             if required_builds
               assert_equal(
                 settings_pr.dig('com.atlassian.bitbucket.server.bitbucket-build:requiredBuilds', 'enable'),
@@ -87,7 +87,7 @@ module HybridPlatformsConductor
               )
             end
             # Default merge strategy
-            default_merge_strategy = repo_info.dig *%i[checks pr_settings default_merge_strategy]
+            default_merge_strategy = repo_info.dig(*%i[checks pr_settings default_merge_strategy])
             if default_merge_strategy
               assert_equal(
                 settings_pr.dig('mergeConfig', 'defaultStrategy', 'id'),
@@ -104,7 +104,7 @@ module HybridPlatformsConductor
               )
             end
             # Default reviewers should include our team from any branch to any branch
-            mandatory_default_reviewers = repo_info.dig *%i[checks pr_settings mandatory_default_reviewers]
+            mandatory_default_reviewers = repo_info.dig(*%i[checks pr_settings mandatory_default_reviewers])
             if mandatory_default_reviewers
               reviewers_found = default_reviewers.any? do |condition_info|
                 reviewers = condition_info.dig('reviewers')

data/lib/hybrid_platforms_conductor/hpc_plugins/test/deploy_freshness.rb

@@ -22,7 +22,7 @@ module HybridPlatformsConductor
                 error 'Node has never been deployed using deploy (/var/log/deployments does not exist)'
               else
                 # Expecting following file names
-                # 2017-12-01_093418_a_usernme
+                # node-name_2017-12-01_093418_user-name
                 file_match = stdout.first.match(/^#{Regexp.escape(@node)}_(\d{4}-\d{2}-\d{2})_.+$/)
                 if file_match.nil?
                   error "Invalid chef deployment log file found: #{stdout.first}"

data/lib/hybrid_platforms_conductor/hpc_plugins/test/deploy_removes_root_access.rb

@@ -12,22 +12,9 @@ module HybridPlatformsConductor
 
         # Check my_test_plugin.rb.sample documentation for signature details.
         def test_for_node
-          @deployer.with_test_provisioned_instance(@config.tests_provisioner_id, @node, environment: 'deploy_removes_root_access', reuse_instance: log_debug?) do |deployer, instance|
-            # Check that we can connect with root
-            ssh_ok = false
-            begin
-              Net::SSH.start(instance.ip, 'root', password: 'root_pwd', auth_methods: ['password'], verify_host_key: :never) do |ssh|
-                ssh_ok = ssh.exec!('echo Works').strip == 'Works'
-              end
-            rescue
-            end
-            assert_equal ssh_ok, true, 'Root does not have access from the empty image'
-            if ssh_ok
-              deployer.nbr_retries_on_error = 3
-              deployer.deploy_on @node
-              # As sshd is certainly being restarted, start and stop the container to reload it.
-              deployer.restart @node
-              # Check that we can't connect with root
+          unless @nodes_handler.get_root_access_allowed_of(@node) == 'true'
+            @deployer.with_test_provisioned_instance(@config.tests_provisioner_id, @node, environment: 'deploy_removes_root_access', reuse_instance: log_debug?) do |deployer, instance|
+              # Check that we can connect with root
               ssh_ok = false
               begin
                 Net::SSH.start(instance.ip, 'root', password: 'root_pwd', auth_methods: ['password'], verify_host_key: :never) do |ssh|
@@ -35,7 +22,22 @@ module HybridPlatformsConductor
                 end
               rescue
               end
-              assert_equal ssh_ok, false, 'Root can still connect on the image after deployment'
+              assert_equal ssh_ok, true, 'Root does not have access from the empty image'
+              if ssh_ok
+                deployer.nbr_retries_on_error = 3
+                deployer.deploy_on @node
+                # As sshd is certainly being restarted, start and stop the container to reload it.
+                deployer.restart @node
+                # Check that we can't connect with root
+                ssh_ok = false
+                begin
+                  Net::SSH.start(instance.ip, 'root', password: 'root_pwd', auth_methods: ['password'], verify_host_key: :never) do |ssh|
+                    ssh_ok = ssh.exec!('echo Works').strip == 'Works'
+                  end
+                rescue
+                end
+                assert_equal ssh_ok, false, 'Root can still connect on the image after deployment'
+              end
             end
           end
         end

data/lib/hybrid_platforms_conductor/hpc_plugins/test/divergence.rb

@@ -1,4 +1,5 @@
 require 'json'
+require 'hybrid_platforms_conductor/common_config_dsl/idempotence_tests'
 
 module HybridPlatformsConductor
 
@@ -9,6 +10,8 @@ module HybridPlatformsConductor
       # Test that the node has not diverged since last deployment
       class Divergence < HybridPlatformsConductor::Test
 
+        self.extend_config_dsl_with CommonConfigDsl::IdempotenceTests, :init_idempotence_tests
+
         # Check my_test_plugin.rb.sample documentation for signature details.
         def test_on_check_node(stdout, stderr, exit_status)
           # Check that the output of the check-node returns no changes.

data/lib/hybrid_platforms_conductor/hpc_plugins/test/hostname.rb

@@ -10,7 +10,8 @@ module HybridPlatformsConductor
         # Check my_test_plugin.rb.sample documentation for signature details.
         def test_on_node
           {
-            "#{@nodes_handler.sudo_on(@node)} hostname -s" => proc do |stdout|
+            # TODO: Access the user correctly when the user notion will be moved out of the ssh connector
+            "#{@deployer.instance_variable_get(:@actions_executor).connector(:ssh).ssh_user == 'root' ? '' : "#{@nodes_handler.sudo_on(@node)} "}hostname -s" => proc do |stdout|
               assert_equal stdout.first, @node, "Expected hostname to be #{@node}, but got #{stdout.first} instead."
             end
           }

data/lib/hybrid_platforms_conductor/hpc_plugins/test/ip.rb

@@ -10,7 +10,8 @@ module HybridPlatformsConductor
         # Check my_test_plugin.rb.sample documentation for signature details.
         def test_on_node
           {
-            "#{@nodes_handler.sudo_on(@node)} hostname -I" => proc do |stdout|
+            # TODO: Access the user correctly when the user notion will be moved out of the ssh connector
+            "#{@deployer.instance_variable_get(:@actions_executor).connector(:ssh).ssh_user == 'root' ? '' : "#{@nodes_handler.sudo_on(@node)} "}hostname -I" => proc do |stdout|
               if stdout.first.nil?
                 error 'No IP returned by "hostname -I"'
               else

data/lib/hybrid_platforms_conductor/hpc_plugins/test/local_users.rb

@@ -57,7 +57,8 @@ module HybridPlatformsConductor
         # Check my_test_plugin.rb.sample documentation for signature details.
         def test_on_node
           {
-            "#{@nodes_handler.sudo_on(@node)} cat /etc/passwd" => proc do |stdout|
+            # TODO: Access the user correctly when the user notion will be moved out of the ssh connector
+            "#{@deployer.instance_variable_get(:@actions_executor).connector(:ssh).ssh_user == 'root' ? '' : "#{@nodes_handler.sudo_on(@node)} "}cat /etc/passwd" => proc do |stdout|
               passwd_users = stdout.map { |passwd_line| passwd_line.split(':').first }
               missing_users = @nodes_handler.
                 select_confs_for_node(@node, @config.users_that_should_be_present).

data/lib/hybrid_platforms_conductor/hpc_plugins/test/mounts.rb

@@ -32,7 +32,7 @@ module HybridPlatformsConductor
           #
           # Parameters::
           # * *mount_rules* (Hash<String or Regexp, String or Regexp>):
-          #     List of (or single) sets of { source => destination } mounts that should not be present.
+          #     Set of { source => destination } mounts that should not be present.
           #     Each source or destination can be a string for exact match, or a regexp to match a pattern on the mounts done on the node.
           def check_mounts_do_not_include(mount_rules)
             @mount_rules_that_should_be_absent << {
@@ -45,7 +45,7 @@ module HybridPlatformsConductor
           #
           # Parameters::
           # * *mount_rules* (Hash<String or Regexp, String or Regexp>):
-          #     List of (or single) sets of { source => destination } mounts that should be present.
+          #     Set of { source => destination } mounts that should be present.
           #     Each source or destination can be a string for exact match, or a regexp to match a pattern on the mounts done on the node.
           def check_mounts_do_include(mount_rules)
             @mount_rules_that_should_be_present << {
@@ -61,7 +61,8 @@ module HybridPlatformsConductor
         # Check my_test_plugin.rb.sample documentation for signature details.
         def test_on_node
           {
-            "#{@nodes_handler.sudo_on(@node)} mount" => proc do |stdout|
+            # TODO: Access the user correctly when the user notion will be moved out of the ssh connector
+            "#{@deployer.instance_variable_get(:@actions_executor).connector(:ssh).ssh_user == 'root' ? '' : "#{@nodes_handler.sudo_on(@node)} "}mount" => proc do |stdout|
               mounts_info = stdout.map do |line|
                 fields = line.split
                 {

data/lib/hybrid_platforms_conductor/hpc_plugins/test/orphan_files.rb

@@ -50,7 +50,8 @@ module HybridPlatformsConductor
         # Check my_test_plugin.rb.sample documentation for signature details.
         def test_on_node
           {
-            "#{@nodes_handler.sudo_on(@node)} /usr/bin/find / \\( #{@nodes_handler.
+            # TODO: Access the user correctly when the user notion will be moved out of the ssh connector
+            "#{@deployer.instance_variable_get(:@actions_executor).connector(:ssh).ssh_user == 'root' ? '' : "#{@nodes_handler.sudo_on(@node)} "}/usr/bin/find / \\( #{@nodes_handler.
               select_confs_for_node(@node, @config.ignored_orphan_files_paths).
               inject(DIRECTORIES_TO_ALWAYS_IGNORE) { |merged_paths, paths_to_ignore_info| merged_paths + paths_to_ignore_info[:ignored_paths] }.
               uniq.

data/lib/hybrid_platforms_conductor/hpc_plugins/test/spectre.rb

@@ -16,7 +16,7 @@ module HybridPlatformsConductor
         # Check my_test_plugin.rb.sample documentation for signature details.
         def test_on_node
           spectre_cmd = <<~EOS
-            #{@nodes_handler.sudo_on(@node)} /bin/bash <<'EOAction'
+            #{@deployer.instance_variable_get(:@actions_executor).connector(:ssh).ssh_user == 'root' ? '' : "#{@nodes_handler.sudo_on(@node)} "}/bin/bash <<'EOAction'
             #{File.read("#{__dir__}/spectre-meltdown-checker.sh")}
             EOAction
           EOS

data/lib/hybrid_platforms_conductor/hpc_plugins/test/vulnerabilities.rb

@@ -54,7 +54,8 @@ module HybridPlatformsConductor
                   current_url
                 end
               )
-              sudo = @nodes_handler.sudo_on(@node)
+              # TODO: Access the user correctly when the user notion will be moved out of the ssh connector
+              sudo = @deployer.instance_variable_get(:@actions_executor).connector(:ssh).ssh_user == 'root' ? '' : "#{@nodes_handler.sudo_on(@node)} "
               Hash[urls.map do |url|
                 # 1. Get the OVAL file on the node to be tested (uncompress it if needed)
                 # 2. Make sure oscap is installed
@@ -75,9 +76,9 @@ module HybridPlatformsConductor
                   #{
                     case image
                     when :centos_7
-                      "#{sudo} yum install -y wget openscap-scanner #{packages_to_install.join(' ')}"
+                      "#{sudo}yum install -y wget openscap-scanner #{packages_to_install.join(' ')}"
                     when :debian_9
-                      "#{sudo} apt install -y wget libopenscap8 #{packages_to_install.join(' ')}"
+                      "#{sudo}apt install -y wget libopenscap8 #{packages_to_install.join(' ')}"
                     when :debian_10
                       # On Debian 10 we have to compile it from sources, as the packaged official version has core dumps.
                       # cf https://www.mail-archive.com/debian-bugs-dist@lists.debian.org/msg1688223.html
@@ -87,13 +88,13 @@ module HybridPlatformsConductor
                           rm -rf openscap
                           git clone --recurse-submodules https://github.com/OpenSCAP/openscap.git
                           cd openscap
-                          #{sudo} apt install -y cmake libdbus-1-dev libdbus-glib-1-dev libcurl4-openssl-dev libgcrypt20-dev libselinux1-dev libxslt1-dev libgconf2-dev libacl1-dev libblkid-dev libcap-dev libxml2-dev libldap2-dev libpcre3-dev python-dev swig libxml-parser-perl libxml-xpath-perl libperl-dev libbz2-dev librpm-dev g++ libapt-pkg-dev libyaml-dev
+                          #{sudo}apt install -y cmake libdbus-1-dev libdbus-glib-1-dev libcurl4-openssl-dev libgcrypt20-dev libselinux1-dev libxslt1-dev libgconf2-dev libacl1-dev libblkid-dev libcap-dev libxml2-dev libldap2-dev libpcre3-dev python-dev swig libxml-parser-perl libxml-xpath-perl libperl-dev libbz2-dev librpm-dev g++ libapt-pkg-dev libyaml-dev
                           cd build
                           cmake ../
                           make
-                          #{sudo} make install
+                          #{sudo}make install
                         fi
-                        #{sudo} apt install -y wget #{packages_to_install.join(' ')}
+                        #{sudo}apt install -y wget #{packages_to_install.join(' ')}
                       EOS2
                     else
                       raise "Non supported image: #{image}. Please adapt this test's code."
@@ -104,7 +105,7 @@ module HybridPlatformsConductor
                   cd hpc_vulnerabilities_test
                   wget -N #{url}
                   #{uncompress_cmds.join("\n")}
-                  #{sudo} oscap oval eval --skip-valid --results "#{local_oval_file}.results.xml" "#{local_oval_file}"
+                  #{sudo}oscap oval eval --skip-valid --results "#{local_oval_file}.results.xml" "#{local_oval_file}"
                   echo "===== RESULTS ====="
                   cat "#{local_oval_file}.results.xml"
                   cd ..

data/lib/hybrid_platforms_conductor/hpc_plugins/test_report/confluence.rb

@@ -53,7 +53,7 @@ module HybridPlatformsConductor
 
         private
 
-        TEMPLATES_PATH = File.expand_path("#{File.dirname(__FILE__)}/templates")
+        TEMPLATES_PATH = File.expand_path("#{__dir__}/templates")
 
         # Render a given ERB template into a String
         #

data/lib/hybrid_platforms_conductor/json_dumper.rb

@@ -56,7 +56,7 @@ module HybridPlatformsConductor
       nodes = @nodes_handler.select_nodes(nodes_selectors)
       unless @skip_run
         nodes.map { |node| @nodes_handler.platform_for(node) }.uniq.each.each do |platform_handler|
-          platform_handler.prepare_why_run_deploy_for_json_dump
+          platform_handler.prepare_why_run_deploy_for_json_dump if platform_handler.respond_to?(:prepare_why_run_deploy_for_json_dump)
         end
         @deployer.concurrent_execution = true
         @deployer.use_why_run = true

data/lib/hybrid_platforms_conductor/platform_handler.rb

@@ -142,7 +142,7 @@ module HybridPlatformsConductor
             git_status = git.status
             git_commit = git.log.first
             {
-              repo_name: File.basename(git.remotes.first.url).gsub(/\.git$/, ''),
+              repo_name: git.remotes.empty? ? File.basename(@repository_path) : File.basename(git.remotes.first.url).gsub(/\.git$/, ''),
               commit: {
                 id: git_commit.sha,
                 ref: git_commit.name,

data/lib/hybrid_platforms_conductor/services_handler.rb

@@ -108,23 +108,25 @@ module HybridPlatformsConductor
       local_environment:
     )
       platforms_for(services).each do |platform, platform_services|
-        platform_name = platform.name
-        # Compute the package ID that is unique to this packaging, so that we don't mix it with others if needed.
-        package_id = {
-          platform_name: platform_name,
-          services: Hash[platform_services.map { |node, node_services| [node, node_services.sort] }].sort,
-          secrets: secrets.sort,
-          local_environment: local_environment
-        }
-        if ServicesHandler.packaged_deployments.include?(package_id)
-          log_debug "Platform #{platform_name} has already been packaged for this deployment (package ID #{package_id}). Won't package it another time."
-        else
-          platform.package(
-            services: platform_services,
-            secrets: secrets,
+        if platform.respond_to?(:package)
+          platform_name = platform.name
+          # Compute the package ID that is unique to this packaging, so that we don't mix it with others if needed.
+          package_id = {
+            platform_name: platform_name,
+            services: Hash[platform_services.map { |node, node_services| [node, node_services.sort] }].sort,
+            secrets: secrets.sort,
             local_environment: local_environment
-          )
-          ServicesHandler.packaged_deployments << package_id
+          }
+          if ServicesHandler.packaged_deployments.include?(package_id)
+            log_debug "Platform #{platform_name} has already been packaged for this deployment (package ID #{package_id}). Won't package it another time."
+          else
+            platform.package(
+              services: platform_services,
+              secrets: secrets,
+              local_environment: local_environment
+            )
+            ServicesHandler.packaged_deployments << package_id
+          end
         end
       end
     end

data/lib/hybrid_platforms_conductor/tests_runner.rb

@@ -452,7 +452,6 @@ module HybridPlatformsConductor
           end]
           section "Run test commands on #{@test_cmds.keys.size} connected nodes (timeout to #{timeout} secs)" do
             start_time = Time.now
-            nbr_secs = nil
             @actions_executor.max_threads = @max_threads_connection_on_nodes
             @actions_result = @actions_executor.execute_actions(
               @test_cmds,

data/lib/hybrid_platforms_conductor/topographer.rb

@@ -110,7 +110,6 @@ module HybridPlatformsConductor
       @skip_run = false
 
       # Parse plugins
-      root_path = File.expand_path("#{File.dirname(__FILE__)}/..")
       @plugins = Hash[Dir.
         glob("#{File.dirname(__FILE__)}/topographer/plugins/*.rb").
         map do |file_name|

data/lib/hybrid_platforms_conductor/version.rb

@@ -1,5 +1,5 @@
 module HybridPlatformsConductor
 
-  VERSION = '32.12.0'
+  VERSION = '32.13.0'
 
 end

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

@@ -79,6 +79,22 @@ describe HybridPlatformsConductor::ActionsExecutor do
       end
     end
 
+    it 'executes remote Bash code both from commands and a file in sequence' do
+      with_test_platform_for_action_plugins do |repository|
+        File.write("#{repository}/commands.txt", "bash_cmd3.bash\nbash_cmd4.bash")
+        test_actions_executor.execute_actions('node' => { remote_bash: [
+          'bash_cmd1.bash',
+          'bash_cmd2.bash',
+          { file: "#{repository}/commands.txt" }
+        ] })
+        expect(test_actions_executor.connector(:test_connector).calls).to eq [
+          [:connectable_nodes_from, ['node']],
+          [:with_connection_to, ['node'], { no_exception: true }],
+          [:remote_bash, "bash_cmd1.bash\nbash_cmd2.bash\nbash_cmd3.bash\nbash_cmd4.bash"]
+        ]
+      end
+    end
+
     it 'executes remote Bash code with environment variables set' do
       with_test_platform_for_action_plugins do
         test_actions_executor.execute_actions('node' => { remote_bash: {

data/spec/hybrid_platforms_conductor_test/api/actions_executor/connectors/local/connectable_nodes_spec.rb

@@ -0,0 +1,30 @@
+describe HybridPlatformsConductor::ActionsExecutor do
+
+  context 'checking connector plugin local' do
+
+    context 'checking connectable nodes selection' do
+
+      # Return the connector to be tested
+      #
+      # Result::
+      # * Connector: Connector to be tested
+      def test_connector
+        test_actions_executor.connector(:local)
+      end
+
+      it 'selects connectable nodes correctly' do
+        with_test_platform(nodes: {
+          'node1' => { meta: { host_ip: '192.168.42.42' } },
+          'node2' => {},
+          'node3' => { meta: { host_ip: '127.0.0.1', local_node: true } },
+          'node4' => { meta: { local_node: true } }
+        }) do
+          expect(test_connector.connectable_nodes_from(%w[node1 node2 node3 node4]).sort).to eq %w[node3 node4].sort
+        end
+      end
+
+    end
+
+  end
+
+end

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

@@ -0,0 +1,113 @@
+describe HybridPlatformsConductor::ActionsExecutor do
+
+  context 'checking connector plugin local' do
+
+    context 'checking remote actions' do
+
+      # Return the connector to be tested
+      #
+      # Result::
+      # * Connector: Connector to be tested
+      def test_connector
+        test_actions_executor.connector(:local)
+      end
+
+      # Get a test platform and the connector prepared the same way Actions Executor does before calling remote_* methods
+      #
+      # Parameters::
+      # * *expected_cmds* (Array< [String or Regexp, Proc] >): The expected commands that should be used, and their corresponding mocked code [default: []]
+      # * *expected_stdout* (String): Expected stdout after client code execution [default: '']
+      # * *expected_stderr* (String): Expected stderr after client code execution [default: '']
+      # * *timeout* (Integer or nil): Timeout to prepare the connector for [default: nil]
+      # * *password* (String or nil): Password to set for the node, or nil for none [default: nil]
+      # * *additional_config* (String): Additional config [default: '']
+      # * Proc: Client code to execute testing
+      def with_test_platform_for_remote_testing(
+        expected_cmds: [],
+        expected_stdout: '',
+        expected_stderr: '',
+        timeout: nil,
+        password: nil,
+        additional_config: ''
+      )
+        with_test_platform(
+          { nodes: { 'node' => { meta: { local_node: true } } } },
+          false,
+          additional_config
+        ) do
+          with_cmd_runner_mocked(expected_cmds) do
+            test_connector.with_connection_to(['node']) do
+              stdout = ''
+              stderr = ''
+              test_connector.prepare_for('node', timeout, stdout, stderr)
+              yield
+              expect(stdout).to eq expected_stdout
+              expect(stderr).to eq expected_stderr
+            end
+          end
+        end
+      end
+
+      it 'executes bash commands remotely' do
+        with_test_platform_for_remote_testing(
+          expected_cmds: [['cd /tmp/hpc_local_workspaces/node ; bash_cmd.bash', proc { [0, 'Bash commands executed on node', ''] }]],
+          expected_stdout: 'Bash commands executed on node'
+        ) do
+          test_connector.remote_bash('bash_cmd.bash')
+        end
+      end
+
+      it 'executes bash commands remotely with timeout' do
+        with_test_platform_for_remote_testing(
+          expected_cmds: [
+            [
+              'cd /tmp/hpc_local_workspaces/node ; bash_cmd.bash',
+              proc do |cmd, log_to_file: nil, log_to_stdout: true, log_stdout_to_io: nil, log_stderr_to_io: nil, expected_code: 0, timeout: nil, no_exception: false|
+                expect(timeout).to eq 5
+                [0, '', '']
+              end
+            ]
+          ],
+          timeout: 5
+        ) do
+          test_connector.remote_bash('bash_cmd.bash')
+        end
+      end
+
+      it 'executes interactive commands remotely' do
+        with_test_platform_for_remote_testing do
+          expect(test_connector).to receive(:system) do |cmd|
+            expect(cmd).to eq 'cd /tmp/hpc_local_workspaces/node ; /bin/bash'
+          end
+          test_connector.remote_interactive
+        end
+      end
+
+      it 'copies files remotely' do
+        with_test_platform_for_remote_testing do
+          expect(FileUtils).to receive(:cp_r).with('/path/to/src.file', '/remote_path/to/dst.dir')
+          test_connector.remote_copy('/path/to/src.file', '/remote_path/to/dst.dir')
+        end
+      end
+
+      it 'copies files remotely with timeout' do
+        with_test_platform_for_remote_testing(
+          timeout: 5
+        ) do
+          expect(FileUtils).to receive(:cp_r).with('/path/to/src.file', '/remote_path/to/dst.dir')
+          test_connector.remote_copy('/path/to/src.file', '/remote_path/to/dst.dir')
+        end
+      end
+
+      it 'copies relative files remotely' do
+        with_test_platform_for_remote_testing do
+          expect(FileUtils).to receive(:cp_r).with('/path/to/src.file', '/tmp/hpc_local_workspaces/node/to/dst.dir')
+          test_connector.remote_copy('/path/to/src.file', 'to/dst.dir')
+        end
+      end
+
+    end
+
+  end
+
+end