hybrid_platforms_conductor 33.0.0 → 33.0.1

data/spec/hybrid_platforms_conductor_test/test_connector.rb

@@ -45,7 +45,7 @@ module HybridPlatformsConductorTest
     #
     # Parameters::
     # * *options_parser* (OptionParser): The option parser to complete
-    def options_parse(options_parser)
+    def options_parse(_options_parser)
       @calls << [:options_parse]
     end
 
@@ -102,7 +102,7 @@ module HybridPlatformsConductorTest
     # * *bash_cmds* (String): Bash commands to execute
     def remote_bash(bash_cmds)
       @calls << [:remote_bash, bash_cmds]
-      @remote_bash_code.call(@stdout_io, @stderr_io, self) unless @remote_bash_code.nil?
+      @remote_bash_code&.call(@stdout_io, @stderr_io, self)
     end
 
     # Execute an interactive shell on the remote node
@@ -140,7 +140,7 @@ module HybridPlatformsConductorTest
       extra_opts[:owner] = owner if owner
       extra_opts[:group] = group if group
       @calls << [:remote_copy, from, to] + (extra_opts.empty? ? [] : [extra_opts])
-      @remote_copy_code.call(@stdout_io, @stderr_io, self) unless @remote_copy_code.nil?
+      @remote_copy_code&.call(@stdout_io, @stderr_io, self)
     end
 
     # Integer: The current desired timeout

data/spec/hybrid_platforms_conductor_test/test_log_no_read_plugin.rb

@@ -4,7 +4,9 @@ module HybridPlatformsConductorTest
   class TestLogNoReadPlugin < HybridPlatformsConductor::Log
 
     class << self
+
       attr_accessor :calls
+
     end
 
     # Get actions to save logs
@@ -29,7 +31,7 @@ module HybridPlatformsConductorTest
         node: node,
         services: services,
         # Don't store the date
-        deployment_info: deployment_info.select { |k, _v| k != :date },
+        deployment_info: deployment_info.reject { |k, _v| k == :date },
         exit_status: exit_status,
         stdout: stdout,
         stderr: stderr

data/spec/hybrid_platforms_conductor_test/test_log_plugin.rb

@@ -4,7 +4,9 @@ module HybridPlatformsConductorTest
   class TestLogPlugin < HybridPlatformsConductor::Log
 
     class << self
+
       attr_accessor :calls
+
     end
 
     # Get actions to save logs
@@ -29,7 +31,7 @@ module HybridPlatformsConductorTest
         node: node,
         services: services,
         # Don't store the date
-        deployment_info: deployment_info.select { |k, _v| k != :date },
+        deployment_info: deployment_info.reject { |k, _v| k == :date },
         exit_status: exit_status,
         stdout: stdout,
         stderr: stderr

data/spec/hybrid_platforms_conductor_test/test_plugins/global.rb

@@ -20,6 +20,7 @@ module HybridPlatformsConductorTest
       # Check my_test_plugin.rb.sample documentation for signature details.
       def test
         raise 'Failing test' if Global.fail
+
         Global.nbr_runs += 1
       end
 

data/spec/hybrid_platforms_conductor_test/test_plugins/node.rb

@@ -33,6 +33,7 @@ module HybridPlatformsConductorTest
       # Check my_test_plugin.rb.sample documentation for signature details.
       def test_for_node
         raise "Failing test #{@name} for #{@node}" if Node.fail_for.key?(@name) && Node.fail_for[@name].include?(@node)
+
         sleep_time = Node.sleeps.dig(@name, @node)
         sleep sleep_time unless sleep_time.nil?
         Node.runs << [@name, @node]

data/spec/hybrid_platforms_conductor_test/test_plugins/node_check.rb

@@ -29,6 +29,7 @@ module HybridPlatformsConductorTest
       # Check my_test_plugin.rb.sample documentation for signature details.
       def test_on_check_node(stdout, stderr, exit_status)
         raise 'Failing test' if NodeCheck.fail_for.include? @node
+
         NodeCheck.runs << [@name, @node, stdout, stderr, exit_status]
       end
 

data/spec/hybrid_platforms_conductor_test/test_plugins/platform.rb

@@ -30,6 +30,7 @@ module HybridPlatformsConductorTest
       def test_on_platform
         platform_name = @platform.name
         raise 'Failing test' if Platform.fail_for.include? platform_name
+
         sleep_time = Platform.sleeps.dig(@name, platform_name)
         sleep sleep_time unless sleep_time.nil?
         Platform.runs << [@name, platform_name]

data/spec/hybrid_platforms_conductor_test/test_plugins/several_checks.rb

@@ -32,14 +32,14 @@ module HybridPlatformsConductorTest
       # Check my_test_plugin.rb.sample documentation for signature details.
       def test_on_node
         {
-          "test_#{@node}.sh" => proc do |stdout, stderr, exit_code|
+          "test_#{@node}.sh" => proc do |stdout, stderr|
             SeveralChecks.runs << [@name, @node, "Node SSH test: #{stdout.join("\n")} - #{stderr.join("\n")}"]
           end
         }
       end
 
       # Check my_test_plugin.rb.sample documentation for signature details.
-      def test_on_check_node(stdout, stderr, exit_status)
+      def test_on_check_node(stdout, _stderr, _exit_status)
         SeveralChecks.runs << [@name, @node, "Node check-node test: #{stdout}"]
       end
 

data/spec/hybrid_platforms_conductor_test/test_secrets_reader_plugin.rb

@@ -6,9 +6,9 @@ module HybridPlatformsConductorTest
   class TestSecretsReaderPlugin < HybridPlatformsConductor::SecretsReader
 
     class << self
-      attr_accessor :calls
-      attr_accessor :deployer
-      attr_accessor :mocked_secrets
+
+      attr_accessor :calls, :deployer, :mocked_secrets
+
     end
 
     # Return secrets for a given service to be deployed on a node.
@@ -25,7 +25,7 @@ module HybridPlatformsConductorTest
     # * Hash: The secrets
     def secrets_for(node, service)
       # Get the name by looking into the plugins' map
-      plugin_name, _plugin = TestSecretsReaderPlugin.deployer.instance_variable_get(:@secrets_readers).find { |plugin_name, plugin| plugin == self }
+      plugin_name, _plugin = TestSecretsReaderPlugin.deployer.instance_variable_get(:@secrets_readers).find { |_plugin_name, plugin| plugin == self }
       TestSecretsReaderPlugin.calls << {
         instance: plugin_name,
         node: node,

data/spec/hybrid_platforms_conductor_test/tests_report_plugin.rb

@@ -4,9 +4,11 @@ module HybridPlatformsConductorTest
   class TestsReportPlugin < HybridPlatformsConductor::TestReport
 
     class << self
+
       # Reports (that can be compared), per report name
       # Array< Hash<Symbol, Object> >
       attr_accessor :reports
+
     end
 
     # Handle tests reports
@@ -15,12 +17,9 @@ module HybridPlatformsConductorTest
         global_tests: report_from(global_tests),
         platform_tests: report_from(platform_tests),
         node_tests: report_from(node_tests),
-        errors_per_test: Hash[group_errors(node_tests, :test_name).map do |test_name, errors|
-          [
-            test_name,
-            errors.map { |error| error.split("\n").first }
-          ]
-        end],
+        errors_per_test: group_errors(node_tests, :test_name).transform_values do |errors|
+          errors.map { |error| error.split("\n").first }
+        end,
         nodes_by_nodes_list: nodes_by_nodes_list
       }
     end

data/spec/spec_helper.rb

@@ -32,7 +32,7 @@ RSpec.configure do |config|
     expectations.include_chain_clauses_in_custom_matcher_descriptions = true
 
     # Increase the diffs length being output, as some tests are dealing with a few lines of stdout
-    expectations.max_formatted_output_length = 65536
+    expectations.max_formatted_output_length = 65_536
 
   end
 
@@ -55,57 +55,56 @@ RSpec.configure do |config|
   # triggering implicit auto-inclusion in groups with matching metadata.
   config.shared_context_metadata_behavior = :apply_to_host_groups
 
-# The settings below are suggested to provide a good initial experience
-# with RSpec, but feel free to customize to your heart's content.
-=begin
+  # The settings below are suggested to provide a good initial experience
+  # with RSpec, but feel free to customize to your heart's content.
+
   # This allows you to limit a spec run to individual examples or groups
   # you care about by tagging them with `:focus` metadata. When nothing
   # is tagged with `:focus`, all examples get run. RSpec also provides
   # aliases for `it`, `describe`, and `context` that include `:focus`
   # metadata: `fit`, `fdescribe` and `fcontext`, respectively.
-  config.filter_run_when_matching :focus
+  # config.filter_run_when_matching :focus
 
   # Allows RSpec to persist some state between runs in order to support
   # the `--only-failures` and `--next-failure` CLI options. We recommend
   # you configure your source control system to ignore this file.
-  config.example_status_persistence_file_path = "spec/examples.txt"
+  # config.example_status_persistence_file_path = "spec/examples.txt"
 
   # Limits the available syntax to the non-monkey patched syntax that is
   # recommended. For more details, see:
   #   - http://rspec.info/blog/2012/06/rspecs-new-expectation-syntax/
   #   - http://www.teaisaweso.me/blog/2013/05/27/rspecs-new-message-expectation-syntax/
   #   - http://rspec.info/blog/2014/05/notable-changes-in-rspec-3/#zero-monkey-patching-mode
-  config.disable_monkey_patching!
+  # config.disable_monkey_patching!
 
   # This setting enables warnings. It's recommended, but in some cases may
   # be too noisy due to issues in dependencies.
-  config.warnings = true
+  # config.warnings = true
 
   # Many RSpec users commonly either run the entire suite or an individual
   # file, and it's useful to allow more verbose output when running an
   # individual spec file.
-  if config.files_to_run.one?
-    # Use the documentation formatter for detailed output,
-    # unless a formatter has already been configured
-    # (e.g. via a command-line flag).
-    config.default_formatter = "doc"
-  end
+  # if config.files_to_run.one?
+  #   # Use the documentation formatter for detailed output,
+  #   # unless a formatter has already been configured
+  #   # (e.g. via a command-line flag).
+  #   config.default_formatter = "doc"
+  # end
 
   # Print the 10 slowest examples and example groups at the
   # end of the spec run, to help surface which specs are running
   # particularly slow.
-  config.profile_examples = 10
+  # config.profile_examples = 10
 
   # Run specs in random order to surface order dependencies. If you find an
   # order dependency and want to debug it, you can fix the order by providing
   # the seed, which is printed after each run.
   #     --seed 1234
-  config.order = :random
+  # config.order = :random
 
   # Seed global randomization in this process using the `--seed` CLI option.
   # Setting this allows you to use `--seed` to deterministically reproduce
   # test failures related to randomization by passing the same `--seed` value
   # as the one that triggered the failure.
-  Kernel.srand config.seed
-=end
+  # Kernel.srand config.seed
 end

data/tools/check_md

@@ -18,7 +18,7 @@ def link_to_file(link, file)
   if link_file == ''
     file
   elsif link_file.start_with?('/')
-    link_file[1..-1]
+    link_file[1..]
   else
     File.expand_path("#{File.dirname(file)}/#{link_file}").gsub("#{Dir.pwd}/", '')
   end
@@ -29,22 +29,20 @@ end
 # Parameters::
 # * *file* (String): Markdown file to be parsed
 def parse_md(file)
-  unless @info.key?(file)
-    puts "Parsing #{file}..."
-    content = File.read(file)
-    @info[file] = {
-      anchors: content.scan(/<a name="([^"]*)"><\/a>/).map { |(anchor)| anchor },
-      links: content.scan(/\[[^\]]*\]\(([^\)]*)\)/).map { |(link)| link }
-    }
-    # Parse linked files
-    @info[file][:links].each do |link|
-      puts "Found #{file} => #{link}"
-    end
-    @info[file][:links].each do |link|
-      unless link.start_with?('http')
-        linked_file = link_to_file(link, file)
-        parse_md(linked_file) if File.exist?(linked_file) && linked_file.end_with?('.md')
-      end
+  return if @info.key?(file)
+
+  puts "Parsing #{file}..."
+  content = File.read(file)
+  @info[file] = {
+    anchors: content.scan(%r{<a name="([^"]*)"></a>}).map { |(anchor)| anchor },
+    links: content.scan(/\[[^\]]*\]\(([^)]*)\)/).map { |(link)| link }
+  }
+  # Parse linked files
+  @info[file][:links].each do |link|
+    puts "Found #{file} => #{link}"
+    unless link.start_with?('http')
+      linked_file = link_to_file(link, file)
+      parse_md(linked_file) if File.exist?(linked_file) && linked_file.end_with?('.md')
     end
   end
 end
@@ -63,9 +61,7 @@ errors = []
         anchor = link.split('#')[1]
         linked_file = link_to_file(link, file)
         if @info.key?(linked_file)
-          if anchor.nil?
-            '*'
-          elsif @info[linked_file][:anchors].include?(anchor)
+          if anchor.nil? || @info[linked_file][:anchors].include?(anchor)
             '*'
           else
             errors << "[#{file} -> #{link}] - Destination has no anchor named #{anchor}"

data/tools/generate_mermaid

@@ -64,7 +64,7 @@ def gen_mermaid(file)
       markdown << line
     end
   end
-  File.write(file, markdown.map { |line| "#{line}\n" }.join) if idx_mermaid > 0
+  File.write(file, markdown.map { |line| "#{line}\n" }.join) if idx_mermaid.positive?
 end
 
 Tempfile.create('puppeteer') do |puppeteer_file|

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hybrid_platforms_conductor
 version: !ruby/object:Gem::Version
-  version: 33.0.0
+  version: 33.0.1
 platform: ruby
 authors:
 - Muriel Salvan
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-06-15 00:00:00.000000000 Z
+date: 2021-06-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: range_operators
@@ -290,6 +290,34 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.0'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.16'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.16'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.4'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.4'
 description: Provides a complete toolset to help DevOps maintain, deploy, monitor
   and test multiple platforms using various technologies
 email:
@@ -667,6 +695,7 @@ files:
 - lib/hybrid_platforms_conductor/config.rb
 - lib/hybrid_platforms_conductor/confluence.rb
 - lib/hybrid_platforms_conductor/connector.rb
+- lib/hybrid_platforms_conductor/core_extensions/symbol/zero.rb
 - lib/hybrid_platforms_conductor/credentials.rb
 - lib/hybrid_platforms_conductor/current_dir_monitor.rb
 - lib/hybrid_platforms_conductor/deployer.rb
@@ -857,11 +886,11 @@ files:
 - spec/hybrid_platforms_conductor_test/api/tests_runner/test_plugins/github_ci_spec.rb
 - spec/hybrid_platforms_conductor_test/api/tests_runner/test_reports_plugins/confluence_spec.rb
 - spec/hybrid_platforms_conductor_test/cmdb_plugins/test_cmdb.rb
-- spec/hybrid_platforms_conductor_test/cmdb_plugins/test_cmdb2.rb
+- spec/hybrid_platforms_conductor_test/cmdb_plugins/test_cmdb_2.rb
 - spec/hybrid_platforms_conductor_test/cmdb_plugins/test_cmdb_others.rb
-- spec/hybrid_platforms_conductor_test/cmdb_plugins/test_cmdb_others2.rb
+- spec/hybrid_platforms_conductor_test/cmdb_plugins/test_cmdb_others_2.rb
 - spec/hybrid_platforms_conductor_test/docs_spec.rb
-- spec/hybrid_platforms_conductor_test/executables/check-node_spec.rb
+- spec/hybrid_platforms_conductor_test/executables/check_node_spec.rb
 - spec/hybrid_platforms_conductor_test/executables/deploy_spec.rb
 - spec/hybrid_platforms_conductor_test/executables/get_impacted_nodes_spec.rb
 - spec/hybrid_platforms_conductor_test/executables/last_deploys_spec.rb
@@ -884,7 +913,6 @@ files:
 - spec/hybrid_platforms_conductor_test/helpers/config_helpers.rb
 - spec/hybrid_platforms_conductor_test/helpers/connector_ssh_helpers.rb
 - spec/hybrid_platforms_conductor_test/helpers/deployer_helpers.rb
-- spec/hybrid_platforms_conductor_test/helpers/deployer_test_helpers.rb
 - spec/hybrid_platforms_conductor_test/helpers/executables_helpers.rb
 - spec/hybrid_platforms_conductor_test/helpers/nodes_handler_helpers.rb
 - spec/hybrid_platforms_conductor_test/helpers/platform_handler_helpers.rb
@@ -895,13 +923,14 @@ files:
 - spec/hybrid_platforms_conductor_test/helpers/serverless_chef_helpers.rb
 - spec/hybrid_platforms_conductor_test/helpers/services_handler_helpers.rb
 - spec/hybrid_platforms_conductor_test/helpers/tests_runner_helpers.rb
-- spec/hybrid_platforms_conductor_test/mocked_lib/my_test_gem/hpc_plugins/test_plugin_type/test_plugin_id1.rb
-- spec/hybrid_platforms_conductor_test/mocked_lib/my_test_gem/hpc_plugins/test_plugin_type/test_plugin_id2.rb
-- spec/hybrid_platforms_conductor_test/mocked_lib/my_test_gem2/sub_dir/hpc_plugins/test_plugin_type/test_plugin_id3.rb
-- spec/hybrid_platforms_conductor_test/mocked_lib/my_test_gem2/sub_dir/hpc_plugins/test_plugin_type2/test_plugin_id4.rb
+- spec/hybrid_platforms_conductor_test/mocked_lib/my_test_gem/hpc_plugins/test_plugin_type/test_plugin_id_1.rb
+- spec/hybrid_platforms_conductor_test/mocked_lib/my_test_gem/hpc_plugins/test_plugin_type/test_plugin_id_2.rb
+- spec/hybrid_platforms_conductor_test/mocked_lib/my_test_gem2/sub_dir/hpc_plugins/test_plugin_type/test_plugin_id_3.rb
+- spec/hybrid_platforms_conductor_test/mocked_lib/my_test_gem2/sub_dir/hpc_plugins/test_plugin_type_2/test_plugin_id_4.rb
 - spec/hybrid_platforms_conductor_test/platform_handler_plugins/test.rb
-- spec/hybrid_platforms_conductor_test/platform_handler_plugins/test2.rb
+- spec/hybrid_platforms_conductor_test/platform_handler_plugins/test_2.rb
 - spec/hybrid_platforms_conductor_test/report_plugin.rb
+- spec/hybrid_platforms_conductor_test/rubocop_spec.rb
 - spec/hybrid_platforms_conductor_test/serverless_chef_repositories/1_node/chef_versions.yml
 - spec/hybrid_platforms_conductor_test/serverless_chef_repositories/1_node/nodes/node.json
 - spec/hybrid_platforms_conductor_test/serverless_chef_repositories/1_node/policyfiles/test_policy.rb
@@ -939,6 +968,7 @@ files:
 - spec/hybrid_platforms_conductor_test/serverless_chef_repositories/several_nodes/nodes/node2.json
 - spec/hybrid_platforms_conductor_test/serverless_chef_repositories/several_nodes/policyfiles/test_policy_1.rb
 - spec/hybrid_platforms_conductor_test/serverless_chef_repositories/several_nodes/policyfiles/test_policy_2.rb
+- spec/hybrid_platforms_conductor_test/shared_examples/deployer.rb
 - spec/hybrid_platforms_conductor_test/test_action.rb
 - spec/hybrid_platforms_conductor_test/test_connector.rb
 - spec/hybrid_platforms_conductor_test/test_log_no_read_plugin.rb
@@ -965,9 +995,9 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - "~>"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.6'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="

data/spec/hybrid_platforms_conductor_test/helpers/deployer_test_helpers.rb

@@ -1,1152 +0,0 @@
-require 'timeout'
-
-module HybridPlatformsConductorTest
-
-  module Helpers
-
-    module DeployerTestHelpers
-
-      # Define deployment specs that are common for check mode and real deployment
-      #
-      # Parameters::
-      # * *check_mode* (Boolean): Are we in check-mode? [default: true]
-      def deploy_specs_for(check_mode: true)
-        expected_deploy_result = [0, "#{check_mode ? 'Check' : 'Deploy'} successful", '']
-        platform_name = check_mode ? 'platform' : 'my_remote_platform'
-
-        context "testing deployment#{check_mode ? ' in why-run mode' : ''}" do
-
-          # Get expected actions for a deployment
-          #
-          # Parameters::
-          # * *services* (Hash<String, Array<String> >): Expected nodes that should be deployed, with their corresponding services [default: { 'node' => %w[service] }]
-          # * *sudo* (String or nil): sudo supposed to be used, or nil if none [default: 'sudo -u root']
-          # * *check_mode* (Boolean): Are we testing in check mode? [default: @check_mode]
-          # * *mocked_deploy_result* (Hash or nil): Mocked result of the deployment actions, or nil to use the helper's default [default: nil]
-          # * *additional_expected_actions* (Array): Additional expected actions [default: []]
-          # * *expect_concurrent_actions* (Boolean): Are actions expected to be run in parallel? [default: false]
-          # * *expect_actions_timeout* (Integer or nil): Expected timeout in actions, or nil for none [default: nil]
-          def expected_actions_for_deploy_on(
-            services: { 'node' => %w[service] },
-            sudo: 'sudo -u root',
-            check_mode: @check_mode,
-            mocked_deploy_result: nil,
-            additional_expected_actions: [],
-            expect_concurrent_actions: false,
-            expect_actions_timeout: nil
-          )
-            actions = [
-              # First run, we expect the mutex to be setup, and the deployment actions to be run
-              proc do |actions_per_nodes, timeout: nil, concurrent: false, log_to_dir: 'run_logs', log_to_stdout: true|
-                expect(timeout).to eq expect_actions_timeout
-                expect(concurrent).to eq expect_concurrent_actions
-                expect(log_to_dir).to eq 'run_logs'
-                expect_actions_to_deploy_on(
-                  actions_per_nodes,
-                  services.keys,
-                  check: check_mode,
-                  sudo: sudo,
-                  mocked_result: mocked_deploy_result,
-                  expected_actions: additional_expected_actions
-                )
-              end,
-              # Second run, we expect the mutex to be released
-              proc { |actions_per_nodes| expect_actions_to_unlock(actions_per_nodes, services.keys, sudo: sudo) }
-            ]
-            services.each do |node, node_services|
-              expect(test_services_handler).to receive(:actions_to_deploy_on).with(node, node_services, check_mode) do
-                [{ bash: "echo \"#{check_mode ? 'Checking' : 'Deploying'} on #{node}\"" }]
-              end
-            end
-            # Third run, we expect logs to be uploaded on the node (only if not check mode)
-            unless check_mode
-              services.each do |node, node_services|
-                expect(test_services_handler).to receive(:log_info_for).with(node, node_services) do
-                  {
-                    repo_name_0: 'platform',
-                    commit_id_0: '123456',
-                    commit_message_0: 'Test commit'
-                  }
-                end
-              end
-              actions << proc { |actions_per_nodes| expect_actions_to_upload_logs(actions_per_nodes, services.keys) }
-            end
-            actions
-          end
-
-          # Prepare a platform ready to test deployments on.
-          #
-          # Parameters::
-          # * *nodes_info* (Hash): Node info to give the platform [default: 1 node having 1 service]
-          # * *expect_services_to_deploy* (Hash<String,Array<String>>): Expected services to be deployed [default: all services from nodes_info]
-          # * *expect_deploy_allowed* (Boolean): Should we expect the call to deploy_allowed? [default: true]
-          # * *expect_package* (Boolean): Should we expect packaging? [default: true]
-          # * *expect_prepare_for_deploy* (Boolean): Should we expect calls to prepare for deploy? [default: true]
-          # * *expect_connections_to_nodes* (Boolean): Should we expect connections to nodes? [default: true]
-          # * *expect_default_actions* (Boolean): Should we expect default actions? [default: true]
-          # * *expect_sudo* (String or nil): Expected sudo command, or nil if none [default: 'sudo -u root']
-          # * *expect_secrets* (Hash): Secrets to be expected during deployment [default: {}]
-          # * *expect_local_environment* (Boolean): Expected local environment flag [default: false]
-          # * *expect_additional_actions* (Array): Additional expected actions [default: []]
-          # * *expect_concurrent_actions* (Boolean): Are actions expected to be run in parallel? [default: false]
-          # * *expect_actions_timeout* (Integer or nil): Expected timeout in actions, or nil for none [default: nil]
-          # * *check_mode* (Boolean): Are we testing in check mode? [default: @check_mode]
-          # * *additional_config* (String): Additional configuration to set [default: '']
-          # * Proc: Code called once the platform is ready for testing the deployer
-          #   * Parameters::
-          #     * *repository* (String): Path to the repository
-          def with_platform_to_deploy(
-            nodes_info: { nodes: { 'node' => { services: %w[service] } } },
-            expect_services_to_deploy: Hash[nodes_info[:nodes].map { |node, node_info| [node, node_info[:services]] }],
-            expect_deploy_allowed: true,
-            expect_package: true,
-            expect_prepare_for_deploy: true,
-            expect_connections_to_nodes: true,
-            expect_default_actions: true,
-            expect_sudo: 'sudo -u root',
-            expect_secrets: {},
-            expect_local_environment: false,
-            expect_additional_actions: [],
-            expect_concurrent_actions: false,
-            expect_actions_timeout: nil,
-            check_mode: @check_mode,
-            additional_config: ''
-          )
-            platform_name = check_mode ? 'platform' : 'my_remote_platform'
-            with_test_platform(nodes_info, !check_mode, additional_config + "\nsend_logs_to :test_log") do |repository|
-              # Mock the ServicesHandler accesses
-              if !check_mode && expect_deploy_allowed
-                expect(test_services_handler).to receive(:deploy_allowed?).with(
-                  services: expect_services_to_deploy,
-                  secrets: expect_secrets,
-                  local_environment: expect_local_environment
-                ) do
-                  nil
-                end
-              end
-              if expect_package
-                expect(test_services_handler).to receive(:package).with(
-                  services: expect_services_to_deploy,
-                  secrets: expect_secrets,
-                  local_environment: expect_local_environment
-                )
-              else
-                expect(test_services_handler).not_to receive(:package)
-              end
-              if expect_prepare_for_deploy
-                expect(test_services_handler).to receive(:prepare_for_deploy).with(
-                  services: expect_services_to_deploy,
-                  secrets: expect_secrets,
-                  local_environment: expect_local_environment,
-                  why_run: check_mode
-                )
-              else
-                expect(test_services_handler).not_to receive(:prepare_for_deploy)
-              end
-              test_deployer.use_why_run = true if check_mode
-              if expect_connections_to_nodes
-                with_connections_mocked_on(expect_services_to_deploy.keys) do
-                  expect_actions_executor_runs(expected_actions_for_deploy_on(
-                    services: expect_services_to_deploy,
-                    check_mode: check_mode,
-                    sudo: expect_sudo,
-                    additional_expected_actions: expect_additional_actions,
-                    expect_concurrent_actions: expect_concurrent_actions,
-                    expect_actions_timeout: expect_actions_timeout
-                  )) if expect_default_actions
-                  yield repository
-                end
-              else
-                yield repository
-              end
-            end
-          end
-
-          # Prepare a directory with certificates
-          #
-          # Parameters::
-          # * Proc: Code called with the directory created with a mocked certificate
-          #   * Parameters::
-          #     * *certs_dir* (String): Directory containing certificates
-          def with_certs_dir
-            with_repository do |repository|
-              certs_dir = "#{repository}/certificates"
-              FileUtils.mkdir_p certs_dir
-              File.write("#{certs_dir}/test_cert.crt", 'Hello')
-              yield certs_dir
-            end
-          end
-
-          before :each do
-            @check_mode = check_mode
-          end
-
-          it 'deploys on 1 node' do
-            with_platform_to_deploy do
-              expect(test_deployer.deploy_on('node')).to eq('node' => expected_deploy_result)
-            end
-          end
-
-          it 'deploys on 1 node having several services' do
-            with_platform_to_deploy(nodes_info: { nodes: { 'node' => { services: %w[service1 service2 service3] } } }) do
-              expect(test_deployer.deploy_on('node')).to eq('node' => expected_deploy_result)
-            end
-          end
-
-          it 'deploys on 1 node in a local environment' do
-            with_platform_to_deploy(expect_local_environment: true) do
-              test_deployer.local_environment = true
-              expect(test_deployer.deploy_on('node')).to eq('node' => expected_deploy_result)
-              expect(test_deployer.local_environment).to eq true
-            end
-          end
-
-          it 'deploys on 1 node using root' do
-            with_platform_to_deploy(expect_sudo: nil) do
-              test_actions_executor.connector(:ssh).ssh_user = 'root'
-              expect(test_deployer.deploy_on('node')).to eq('node' => expected_deploy_result)
-            end
-          end
-
-          it 'deploys on 1 node using an alternate sudo' do
-            with_platform_to_deploy(
-              expect_sudo: 'other_sudo --user root',
-              additional_config: 'sudo_for { |user| "other_sudo --user #{user}" }'
-            ) do
-              expect(test_deployer.deploy_on('node')).to eq('node' => expected_deploy_result)
-            end
-          end
-
-          it 'deploys on 1 node using 1 secret' do
-            with_platform_to_deploy(expect_secrets: { 'secret1' => 'password1' }) do
-              test_deployer.override_secrets('secret1' => 'password1')
-              expect(test_deployer.deploy_on('node')).to eq('node' => expected_deploy_result)
-            end
-          end
-
-          it 'deploys on 1 node in local environment with certificates to install using hpc_certificates on Debian' do
-            with_certs_dir do |certs_dir|
-              with_platform_to_deploy(
-                nodes_info: { nodes: { 'node' => { meta: { image: 'debian_9' }, services: %w[service] } } },
-                expect_local_environment: true,
-                expect_additional_actions: [
-                  { remote_bash: 'sudo -u root apt update && sudo -u root apt install -y ca-certificates' },
-                  {
-                    remote_bash: 'sudo -u root update-ca-certificates',
-                    scp:  {
-                      certs_dir => '/usr/local/share/ca-certificates',
-                      :sudo => true
-                    }
-                  }
-                ]
-              ) do
-                ENV['hpc_certificates'] = certs_dir
-                test_deployer.local_environment = true
-                expect(test_deployer.deploy_on('node')).to eq('node' => expected_deploy_result)
-              end
-            end
-          end
-
-          it 'deploys on 1 node in local environment with certificates to install using hpc_certificates on Debian and an alternate sudo' do
-            with_certs_dir do |certs_dir|
-              with_platform_to_deploy(
-                nodes_info: { nodes: { 'node' => { meta: { image: 'debian_9' }, services: %w[service] } } },
-                expect_sudo: 'other_sudo --user root',
-                expect_local_environment: true,
-                expect_additional_actions: [
-                  { remote_bash: 'other_sudo --user root apt update && other_sudo --user root apt install -y ca-certificates' },
-                  {
-                    remote_bash: 'other_sudo --user root update-ca-certificates',
-                    scp:  {
-                      certs_dir => '/usr/local/share/ca-certificates',
-                      :sudo => true
-                    }
-                  }
-                ],
-                additional_config: 'sudo_for { |user| "other_sudo --user #{user}" }'
-              ) do
-                ENV['hpc_certificates'] = certs_dir
-                test_deployer.local_environment = true
-                expect(test_deployer.deploy_on('node')).to eq('node' => expected_deploy_result)
-              end
-            end
-          end
-
-          it 'deploys on 1 node with certificates to install using hpc_certificates on Debian but ignores them in non-local environment' do
-            with_certs_dir do |certs_dir|
-              with_platform_to_deploy(nodes_info: { nodes: { 'node' => { meta: { image: 'debian_9' }, services: %w[service] } } }) do
-                ENV['hpc_certificates'] = certs_dir
-                expect(test_deployer.deploy_on('node')).to eq('node' => expected_deploy_result)
-              end
-            end
-          end
-
-          it 'deploys on 1 node with certificates to install using hpc_certificates on Debian using root' do
-            with_certs_dir do |certs_dir|
-              with_platform_to_deploy(
-                nodes_info: { nodes: { 'node' => { meta: { image: 'debian_9' }, services: %w[service] } } },
-                expect_sudo: nil,
-                expect_local_environment: true,
-                expect_additional_actions: [
-                  { remote_bash: 'apt update && apt install -y ca-certificates' },
-                  {
-                    remote_bash: 'update-ca-certificates',
-                    scp:  {
-                      certs_dir => '/usr/local/share/ca-certificates',
-                      :sudo => false
-                    }
-                  }
-                ]
-              ) do
-                ENV['hpc_certificates'] = certs_dir
-                test_actions_executor.connector(:ssh).ssh_user = 'root'
-                test_deployer.local_environment = true
-                expect(test_deployer.deploy_on('node')).to eq('node' => expected_deploy_result)
-              end
-            end
-          end
-
-          it 'deploys on 1 node with certificates to install using hpc_certificates on CentOS' do
-            with_certs_dir do |certs_dir|
-              with_platform_to_deploy(
-                nodes_info: { nodes: { 'node' => { meta: { image: 'centos_7' }, services: %w[service] } } },
-                expect_local_environment: true,
-                expect_additional_actions: [
-                  { remote_bash: 'sudo -u root yum install -y ca-certificates' },
-                  {
-                    remote_bash: ['sudo -u root update-ca-trust enable', 'sudo -u root update-ca-trust extract'],
-                    scp: {
-                      "#{certs_dir}/test_cert.crt" => '/etc/pki/ca-trust/source/anchors',
-                      :sudo => true
-                    }
-                  }
-                ]
-              ) do
-                ENV['hpc_certificates'] = certs_dir
-                test_deployer.local_environment = true
-                expect(test_deployer.deploy_on('node')).to eq('node' => expected_deploy_result)
-              end
-            end
-          end
-
-          it 'deploys on 1 node with certificates to install using hpc_certificates on CentOS and an alternate sudo' do
-            with_certs_dir do |certs_dir|
-              with_platform_to_deploy(
-                nodes_info: { nodes: { 'node' => { meta: { image: 'centos_7' }, services: %w[service] } } },
-                expect_sudo: 'other_sudo --user root',
-                expect_local_environment: true,
-                expect_additional_actions: [
-                  { remote_bash: 'other_sudo --user root yum install -y ca-certificates' },
-                  {
-                    remote_bash: ['other_sudo --user root update-ca-trust enable', 'other_sudo --user root update-ca-trust extract'],
-                    scp: {
-                      "#{certs_dir}/test_cert.crt" => '/etc/pki/ca-trust/source/anchors',
-                      :sudo => true
-                    }
-                  }
-                ],
-                additional_config: 'sudo_for { |user| "other_sudo --user #{user}" }'
-              ) do
-                ENV['hpc_certificates'] = certs_dir
-                test_deployer.local_environment = true
-                expect(test_deployer.deploy_on('node')).to eq('node' => expected_deploy_result)
-              end
-            end
-          end
-
-          it 'deploys on 1 node with certificates to install using hpc_certificates on CentOS using root' do
-            with_certs_dir do |certs_dir|
-              with_platform_to_deploy(
-                nodes_info: { nodes: { 'node' => { meta: { image: 'centos_7' }, services: %w[service] } } },
-                expect_sudo: nil,
-                expect_local_environment: true,
-                expect_additional_actions: [
-                  { remote_bash: 'yum install -y ca-certificates' },
-                  {
-                    remote_bash: ['update-ca-trust enable', 'update-ca-trust extract'],
-                    scp: {
-                      "#{certs_dir}/test_cert.crt" => '/etc/pki/ca-trust/source/anchors',
-                      :sudo => false
-                    }
-                  }
-                ]
-              ) do
-                ENV['hpc_certificates'] = certs_dir
-                test_actions_executor.connector(:ssh).ssh_user = 'root'
-                test_deployer.local_environment = true
-                expect(test_deployer.deploy_on('node')).to eq('node' => expected_deploy_result)
-              end
-            end
-          end
-
-          it 'deploys on several nodes' do
-            with_platform_to_deploy(nodes_info: { nodes: {
-              'node1' => { services: %w[service1] },
-              'node2' => { services: %w[service2] },
-              'node3' => { services: %w[service3] }
-            } }) do
-              expect(test_deployer.deploy_on(%w[node1 node2 node3])).to eq(
-                'node1' => expected_deploy_result,
-                'node2' => expected_deploy_result,
-                'node3' => expected_deploy_result
-              )
-            end
-          end
-
-          it 'deploys on several nodes in parallel' do
-            with_platform_to_deploy(
-              nodes_info: {
-                nodes: {
-                  'node1' => { services: %w[service1] },
-                  'node2' => { services: %w[service2] },
-                  'node3' => { services: %w[service3] }
-                }
-              },
-              expect_concurrent_actions: true
-            ) do
-              test_deployer.concurrent_execution = true
-              expect(test_deployer.deploy_on(%w[node1 node2 node3])).to eq(
-                'node1' => expected_deploy_result,
-                'node2' => expected_deploy_result,
-                'node3' => expected_deploy_result
-              )
-            end
-          end
-
-          it 'deploys on several nodes with timeout' do
-            with_platform_to_deploy(
-              nodes_info: {
-                nodes: {
-                  'node1' => { services: %w[service1] },
-                  'node2' => { services: %w[service2] },
-                  'node3' => { services: %w[service3] }
-                }
-              },
-              expect_actions_timeout: 5
-            ) do
-              test_deployer.timeout = 5
-              expect(test_deployer.deploy_on(%w[node1 node2 node3])).to eq(
-                'node1' => expected_deploy_result,
-                'node2' => expected_deploy_result,
-                'node3' => expected_deploy_result
-              )
-            end
-          end
-
-          it 'fails when packaging timeout has been reached while taking the futex' do
-            with_platform_to_deploy(
-                additional_config: 'packaging_timeout 1',
-                expect_package: false,
-                expect_prepare_for_deploy: false,
-                expect_connections_to_nodes: false
-            ) do
-              # Simulate another process taking the packaging futex
-              futex_file = HybridPlatformsConductor::Deployer.const_get(:PACKAGING_FUTEX_FILE)
-              Futex.new(futex_file).open do
-                # Expect the error to be raised within 2 seconds (as it should timeout after 1 second)
-                begin
-                  Timeout::timeout(2) {
-                    expect { test_deployer.deploy_on('node') }.to raise_error(
-                      Futex::CantLock,
-                      /can't get exclusive access to the file #{Regexp.escape(futex_file)} because of the lock at #{Regexp.escape(futex_file)}\.lock, after 1\.\d+s of waiting/
-                    )
-                  }
-                rescue Timeout::Error
-                  raise 'The packaging timeout (set to 1 seconds) did not fire within 2 seconds. Looks like it is not working properly.'
-                end
-              end
-            end
-          end
-
-          context 'checking deployment retries' do
-
-            # Prepare a platform ready to test deployments' retries on.
-            #
-            # Parameters::
-            # * *nodes_info* (Hash): Node info to give the platform [default: { nodes: { 'node' => {} } }]
-            # * Proc: Code called once the platform is ready for testing the deployer
-            #   * Parameters::
-            #     * *repository* (String): Path to the repository
-            def with_platform_to_retry_deploy(nodes_info: { nodes: { 'node' => { services: %w[service] } } })
-              with_platform_to_deploy(
-                nodes_info: nodes_info,
-                expect_default_actions: false,
-                additional_config: "
-                for_nodes([#{nodes_info[:nodes].keys.map { |node| "'#{node}'" }.join(', ')}]) do
-                  retry_deploy_for_errors_on_stdout [
-                    'stdout non-deterministic error'
-                  ]
-                  retry_deploy_for_errors_on_stderr [
-                    'stderr non-deterministic error',
-                    /stderr regexp error \\d+/
-                  ]
-                end
-                for_nodes([#{nodes_info[:nodes].keys.map { |node| "'#{node}'" }.join(', ')}]) do
-                  retry_deploy_for_errors_on_stdout [
-                    /stdout regexp error \\d+/
-                  ]
-                end
-                "
-              ) do |repository|
-                yield repository
-              end
-            end
-
-            # Mock a sequential list of deployments
-            #
-            # Parameters::
-            # * *statuses* (Array<Hash<String,Status> or Status>)>): List of mocked deployment statuses per node name, or just the status for the default node.
-            #   A status is a triplet [Integer or Symbol, String, String]: exit status, stdout and stderr.
-            def mock_deploys_with(statuses)
-              expect_actions_executor_runs(statuses.map do |status|
-                status = { 'node' => status } if status.is_a?(Array)
-                expected_actions_for_deploy_on(
-                  services: Hash[status.keys.map { |node| [node, %w[service]] }],
-                  mocked_deploy_result: status
-                )
-              end.flatten)
-            end
-
-            it 'restarts deployment for a non-deterministic error' do
-              with_platform_to_retry_deploy do
-                test_deployer.nbr_retries_on_error = 1
-                mock_deploys_with [
-                  [1, "Error: This is a stdout non-deterministic error\nDeploy failed\n", ''],
-                  [0, 'Deploy ok', '']
-                ]
-                expect(test_deployer.deploy_on('node')).to eq('node' => [
-                  0,
-                  <<~EOS,
-                    Error: This is a stdout non-deterministic error
-                    Deploy failed
-
-                    Deployment exit status code: 1
-                    !!! Retry deployment due to non-deterministic error (0 remaining attempts)...
-                    Deploy ok
-                  EOS
-                  <<~EOS
-                    !!! 1 retriable errors detected in this deployment:
-                    * stdout non-deterministic error
-
-                    !!! Retry deployment due to non-deterministic error (0 remaining attempts)...
-
-                  EOS
-                ])
-              end
-            end
-
-            it 'restarts deployment for a non-deterministic error matched with a Regexp' do
-              with_platform_to_retry_deploy do
-                test_deployer.nbr_retries_on_error = 1
-                mock_deploys_with [
-                  [1, "Error: This is a stdout regexp error 42\nDeploy failed\n", ''],
-                  [0, 'Deploy ok', '']
-                ]
-                expect(test_deployer.deploy_on('node')).to eq('node' => [
-                  0,
-                  <<~EOS,
-                    Error: This is a stdout regexp error 42
-                    Deploy failed
-
-                    Deployment exit status code: 1
-                    !!! Retry deployment due to non-deterministic error (0 remaining attempts)...
-                    Deploy ok
-                  EOS
-                  <<~EOS
-                    !!! 1 retriable errors detected in this deployment:
-                    * /stdout regexp error \\d+/ matched 'stdout regexp error 42'
-
-                    !!! Retry deployment due to non-deterministic error (0 remaining attempts)...
-
-                  EOS
-                ])
-              end
-            end
-
-            it 'restarts deployment for a non-deterministic error on stderr' do
-              with_platform_to_retry_deploy do
-                test_deployer.nbr_retries_on_error = 1
-                mock_deploys_with [
-                  [1, '', "Error: This is a stderr non-deterministic error\nDeploy failed\n"],
-                  [0, 'Deploy ok', '']
-                ]
-                expect(test_deployer.deploy_on('node')).to eq('node' => [
-                  0,
-                  <<~EOS,
-
-                    Deployment exit status code: 1
-                    !!! Retry deployment due to non-deterministic error (0 remaining attempts)...
-                    Deploy ok
-                  EOS
-                  <<~EOS
-                    Error: This is a stderr non-deterministic error
-                    Deploy failed
-                    !!! 1 retriable errors detected in this deployment:
-                    * stderr non-deterministic error
-
-                    !!! Retry deployment due to non-deterministic error (0 remaining attempts)...
-
-                  EOS
-                ])
-              end
-            end
-
-            it 'restarts deployment for a non-deterministic error on stderr matched with a Regexp' do
-              with_platform_to_retry_deploy do
-                test_deployer.nbr_retries_on_error = 1
-                mock_deploys_with [
-                  [1, '', "Error: This is a stderr regexp error 42\nDeploy failed\n"],
-                  [0, 'Deploy ok', '']
-                ]
-                expect(test_deployer.deploy_on('node')).to eq('node' => [
-                  0,
-                  <<~EOS,
-
-                    Deployment exit status code: 1
-                    !!! Retry deployment due to non-deterministic error (0 remaining attempts)...
-                    Deploy ok
-                  EOS
-                  <<~EOS
-                    Error: This is a stderr regexp error 42
-                    Deploy failed
-                    !!! 1 retriable errors detected in this deployment:
-                    * /stderr regexp error \\d+/ matched 'stderr regexp error 42'
-
-                    !!! Retry deployment due to non-deterministic error (0 remaining attempts)...
-
-                  EOS
-                ])
-              end
-            end
-
-            it 'stops restarting deployments for a non-deterministic error when errors has disappeared, even if retries were remaining' do
-              with_platform_to_retry_deploy do
-                test_deployer.nbr_retries_on_error = 5
-                mock_deploys_with [
-                  [1, "Error: This is a stdout non-deterministic error 1\nDeploy failed", ''],
-                  [1, "Error: This is a stdout non-deterministic error 2\nDeploy failed", ''],
-                  [0, 'Deploy ok', '']
-                ]
-                expect(test_deployer.deploy_on('node')).to eq('node' => [
-                  0,
-                  <<~EOS,
-                    Error: This is a stdout non-deterministic error 1
-                    Deploy failed
-                    Deployment exit status code: 1
-                    !!! Retry deployment due to non-deterministic error (4 remaining attempts)...
-                    Error: This is a stdout non-deterministic error 2
-                    Deploy failed
-
-                    Deployment exit status code: 1
-                    !!! Retry deployment due to non-deterministic error (3 remaining attempts)...
-                    Deploy ok
-                  EOS
-                  <<~EOS
-                    !!! 1 retriable errors detected in this deployment:
-                    * stdout non-deterministic error
-
-                    !!! Retry deployment due to non-deterministic error (4 remaining attempts)...
-                    !!! 1 retriable errors detected in this deployment:
-                    * stdout non-deterministic error
-
-
-                    !!! Retry deployment due to non-deterministic error (3 remaining attempts)...
-
-                  EOS
-                ])
-              end
-            end
-
-            it 'stops restarting deployments for a non-deterministic error that became deterministic, even if retries were remaining' do
-              with_platform_to_retry_deploy do
-                test_deployer.nbr_retries_on_error = 5
-                mock_deploys_with [
-                  [1, "Error: This is a stdout non-deterministic error 1\nDeploy failed", ''],
-                  [1, "Error: This is a stdout non-deterministic error 2\nDeploy failed", ''],
-                  [1, "Error: This is a stdout deterministic error 3\nDeploy failed", '']
-                ]
-                expect(test_deployer.deploy_on('node')).to eq('node' => [
-                  1,
-                  <<~EOS,
-                    Error: This is a stdout non-deterministic error 1
-                    Deploy failed
-                    Deployment exit status code: 1
-                    !!! Retry deployment due to non-deterministic error (4 remaining attempts)...
-                    Error: This is a stdout non-deterministic error 2
-                    Deploy failed
-
-                    Deployment exit status code: 1
-                    !!! Retry deployment due to non-deterministic error (3 remaining attempts)...
-                    Error: This is a stdout deterministic error 3
-                    Deploy failed
-                  EOS
-                  <<~EOS
-                    !!! 1 retriable errors detected in this deployment:
-                    * stdout non-deterministic error
-
-                    !!! Retry deployment due to non-deterministic error (4 remaining attempts)...
-                    !!! 1 retriable errors detected in this deployment:
-                    * stdout non-deterministic error
-
-
-                    !!! Retry deployment due to non-deterministic error (3 remaining attempts)...
-
-                  EOS
-                ])
-              end
-            end
-
-            it 'does not restart deployment for a deterministic error' do
-              with_platform_to_retry_deploy do
-                test_deployer.nbr_retries_on_error = 5
-                mock_deploys_with [
-                  [1, "Error: This is a stdout deterministic error\nDeploy failed\n", '']
-                ]
-                expect(test_deployer.deploy_on('node')).to eq('node' => [
-                  1,
-                  <<~EOS,
-                    Error: This is a stdout deterministic error
-                    Deploy failed
-                  EOS
-                  ''
-                ])
-              end
-            end
-
-            it 'does not restart deployment for a non-deterministic error logged during a successful deploy' do
-              with_platform_to_retry_deploy do
-                test_deployer.nbr_retries_on_error = 5
-                mock_deploys_with [
-                  [0, "Error: This is a stdout non-deterministic error\nDeploy failed\n", '']
-                ]
-                expect(test_deployer.deploy_on('node')).to eq('node' => [
-                  0,
-                  <<~EOS,
-                    Error: This is a stdout non-deterministic error
-                    Deploy failed
-                  EOS
-                  ''
-                ])
-              end
-            end
-
-            it 'does not restart deployment for a non-deterministic error if retries are 0' do
-              with_platform_to_retry_deploy do
-                test_deployer.nbr_retries_on_error = 0
-                mock_deploys_with [
-                  [1, "Error: This is a stdout non-deterministic error\nDeploy failed\n", '']
-                ]
-                expect(test_deployer.deploy_on('node')).to eq('node' => [
-                  1,
-                  <<~EOS,
-                    Error: This is a stdout non-deterministic error
-                    Deploy failed
-                  EOS
-                  ''
-                ])
-              end
-            end
-
-            it 'restarts deployment for non-deterministic errors with a limited amount of retries' do
-              with_platform_to_retry_deploy do
-                test_deployer.nbr_retries_on_error = 2
-                mock_deploys_with [
-                  [1, "Error: This is a stdout non-deterministic error 1\nDeploy failed", ''],
-                  [1, "Error: This is a stdout non-deterministic error 2\nDeploy failed", ''],
-                  [1, "Error: This is a stdout non-deterministic error 3\nDeploy failed", '']
-                ]
-                expect(test_deployer.deploy_on('node')).to eq('node' => [
-                  1,
-                  <<~EOS,
-                    Error: This is a stdout non-deterministic error 1
-                    Deploy failed
-                    Deployment exit status code: 1
-                    !!! Retry deployment due to non-deterministic error (1 remaining attempts)...
-                    Error: This is a stdout non-deterministic error 2
-                    Deploy failed
-
-                    Deployment exit status code: 1
-                    !!! Retry deployment due to non-deterministic error (0 remaining attempts)...
-                    Error: This is a stdout non-deterministic error 3
-                    Deploy failed
-                  EOS
-                  <<~EOS
-                    !!! 1 retriable errors detected in this deployment:
-                    * stdout non-deterministic error
-
-                    !!! Retry deployment due to non-deterministic error (1 remaining attempts)...
-                    !!! 1 retriable errors detected in this deployment:
-                    * stdout non-deterministic error
-
-
-                    !!! Retry deployment due to non-deterministic error (0 remaining attempts)...
-
-                  EOS
-                ])
-              end
-            end
-
-            it 'restarts deployment for non-deterministic errors only on nodes needing it' do
-              with_platform_to_retry_deploy(nodes_info: { nodes: {
-                'node1' => { services: %w[service] },
-                'node2' => { services: %w[service] },
-                'node3' => { services: %w[service] },
-                'node4' => { services: %w[service] }
-              } }) do
-                test_deployer.nbr_retries_on_error = 2
-                # Some nodes deploy successfully,
-                # others have deterministic errors,
-                # others have non-deterministic errors being corrected
-                # others have non-deterministic errors not being corrected
-                mock_deploys_with [
-                  {
-                    'node1' => [1, "Error: This is a stdout non-deterministic error\n[node1] Deploy failed\n", ''],
-                    'node2' => [0, '[node2] Deploy ok', ''],
-                    'node3' => [1, "Error: This is a stdout non-deterministic error\n[node3] Deploy failed\n", ''],
-                    'node4' => [1, "Error: This is a stdout non-deterministic error\n[node4] Deploy failed\n", '']
-                  },
-                  {
-                    'node1' => [0, '[node1] Deploy ok', ''],
-                    'node3' => [1, "Error: This is a stdout deterministic error\n[node3] Deploy failed\n", ''],
-                    'node4' => [1, "Error: This is a stdout non-deterministic error\n[node4] Deploy failed\n", '']
-                  },
-                  {
-                    'node4' => [1, "Error: This is a stdout non-deterministic error\n[node4] Deploy failed\n", '']
-                  }
-                ]
-                expect(test_deployer.deploy_on(%w[node1 node2 node3 node4])).to eq(
-                  'node1' => [
-                    0,
-                    <<~EOS,
-                      Error: This is a stdout non-deterministic error
-                      [node1] Deploy failed
-
-                      Deployment exit status code: 1
-                      !!! Retry deployment due to non-deterministic error (1 remaining attempts)...
-                      [node1] Deploy ok
-                    EOS
-                    <<~EOS
-                      !!! 1 retriable errors detected in this deployment:
-                      * stdout non-deterministic error
-
-                      !!! Retry deployment due to non-deterministic error (1 remaining attempts)...
-
-                    EOS
-                  ],
-                  'node2' => [
-                    0,
-                    '[node2] Deploy ok',
-                    ''
-                  ],
-                  'node3' => [
-                    1,
-                    <<~EOS,
-                      Error: This is a stdout non-deterministic error
-                      [node3] Deploy failed
-
-                      Deployment exit status code: 1
-                      !!! Retry deployment due to non-deterministic error (1 remaining attempts)...
-                      Error: This is a stdout deterministic error
-                      [node3] Deploy failed
-
-                    EOS
-                    <<~EOS
-                      !!! 1 retriable errors detected in this deployment:
-                      * stdout non-deterministic error
-
-                      !!! Retry deployment due to non-deterministic error (1 remaining attempts)...
-
-                    EOS
-                  ],
-                  'node4' => [
-                    1,
-                    <<~EOS,
-                      Error: This is a stdout non-deterministic error
-                      [node4] Deploy failed
-
-                      Deployment exit status code: 1
-                      !!! Retry deployment due to non-deterministic error (1 remaining attempts)...
-                      Error: This is a stdout non-deterministic error
-                      [node4] Deploy failed
-
-
-                      Deployment exit status code: 1
-                      !!! Retry deployment due to non-deterministic error (0 remaining attempts)...
-                      Error: This is a stdout non-deterministic error
-                      [node4] Deploy failed
-
-                    EOS
-                    <<~EOS
-                      !!! 1 retriable errors detected in this deployment:
-                      * stdout non-deterministic error
-
-                      !!! Retry deployment due to non-deterministic error (1 remaining attempts)...
-                      !!! 1 retriable errors detected in this deployment:
-                      * stdout non-deterministic error
-
-
-                      !!! Retry deployment due to non-deterministic error (0 remaining attempts)...
-
-                    EOS
-                  ]
-                )
-              end
-            end
-
-          end
-
-          context 'checking secrets handling' do
-
-            it 'calls secrets readers only for nodes and services to be deployed and merges their secrets' do
-              register_plugins(
-                :secrets_reader,
-                {
-                  secrets_reader1: HybridPlatformsConductorTest::TestSecretsReaderPlugin,
-                  secrets_reader2: HybridPlatformsConductorTest::TestSecretsReaderPlugin,
-                  secrets_reader3: HybridPlatformsConductorTest::TestSecretsReaderPlugin
-                }
-              )
-              with_platform_to_deploy(
-                nodes_info: {
-                  nodes: {
-                    'node1' => { services: %w[service1 service2] },
-                    'node2' => { services: %w[service2 service3] },
-                    'node3' => { services: %w[service3] },
-                    'node4' => { services: %w[service1 service3] }
-                  }
-                },
-                expect_services_to_deploy: {
-                  'node1' => %w[service1 service2],
-                  'node2' => %w[service2 service3],
-                  'node3' => %w[service3]
-                },
-                expect_secrets: {
-                  'node1' => {
-                    'service1' => {
-                      'secrets_reader1' => 'Secret value',
-                      'secrets_reader2' => 'Secret value'
-                    },
-                    'service2' => {
-                      'secrets_reader1' => 'Secret value',
-                      'secrets_reader2' => 'Secret value'
-                    }
-                  },
-                  'node2' => {
-                    'service2' => {
-                      'secrets_reader1' => 'Secret value',
-                      'secrets_reader2' => 'Secret value',
-                      'secrets_reader3' => 'Secret value'
-                    },
-                    'service3' => {
-                      'secrets_reader1' => 'Secret value',
-                      'secrets_reader2' => 'Secret value',
-                      'secrets_reader3' => 'Secret value'
-                    }
-                  },
-                  'node3' => {
-                    'service3' => {
-                      'secrets_reader1' => 'Secret value',
-                      'secrets_reader2' => 'Secret value'
-                    }
-                  }
-                },
-                additional_config: <<~EOS
-                  read_secrets_from %i[secrets_reader1 secrets_reader2]
-                  for_nodes('node2') { read_secrets_from :secrets_reader3 }
-                EOS
-              ) do
-                TestSecretsReaderPlugin.deployer = test_deployer
-                expect(test_deployer.deploy_on(%w[node1 node2 node3])).to eq(
-                  'node1' => expected_deploy_result,
-                  'node2' => expected_deploy_result,
-                  'node3' => expected_deploy_result
-                )
-                expect(HybridPlatformsConductorTest::TestSecretsReaderPlugin.calls).to eq [
-                  { instance: :secrets_reader1, node: 'node1', service: 'service1' },
-                  { instance: :secrets_reader1, node: 'node1', service: 'service2' },
-                  { instance: :secrets_reader2, node: 'node1', service: 'service1' },
-                  { instance: :secrets_reader2, node: 'node1', service: 'service2' },
-                  { instance: :secrets_reader1, node: 'node2', service: 'service2' },
-                  { instance: :secrets_reader1, node: 'node2', service: 'service3' },
-                  { instance: :secrets_reader2, node: 'node2', service: 'service2' },
-                  { instance: :secrets_reader2, node: 'node2', service: 'service3' },
-                  { instance: :secrets_reader3, node: 'node2', service: 'service2' },
-                  { instance: :secrets_reader3, node: 'node2', service: 'service3' },
-                  { instance: :secrets_reader1, node: 'node3', service: 'service3' },
-                  { instance: :secrets_reader2, node: 'node3', service: 'service3' }
-                ]
-              end
-            end
-
-            it 'merges secrets having same values' do
-              register_plugins(
-                :secrets_reader,
-                {
-                  secrets_reader1: HybridPlatformsConductorTest::TestSecretsReaderPlugin,
-                  secrets_reader2: HybridPlatformsConductorTest::TestSecretsReaderPlugin
-                }
-              )
-              with_platform_to_deploy(
-                nodes_info: {
-                  nodes: {
-                    'node1' => { services: %w[service1] },
-                    'node2' => { services: %w[service2] }
-                  }
-                },
-                expect_secrets: {
-                  'global1' => 'value1',
-                  'global2' => 'value2',
-                  'global3' => 'value3',
-                  'global4' => 'value4'
-                },
-                additional_config: <<~EOS
-                  read_secrets_from :secrets_reader1
-                  for_nodes('node2') { read_secrets_from :secrets_reader2 }
-                EOS
-              ) do
-                TestSecretsReaderPlugin.deployer = test_deployer
-                TestSecretsReaderPlugin.mocked_secrets = {
-                  'node1' => {
-                    'service1' => {
-                      secrets_reader1: {
-                        'global1' => 'value1',
-                        'global2' => 'value2'
-                      }
-                    }
-                  },
-                  'node2' => {
-                    'service2' => {
-                      secrets_reader1: {
-                        'global2' => 'value2',
-                        'global3' => 'value3'
-                      },
-                      secrets_reader2: {
-                        'global3' => 'value3',
-                        'global4' => 'value4'
-                      }
-                    }
-                  }
-                }
-                expect(test_deployer.deploy_on(%w[node1 node2])).to eq(
-                  'node1' => expected_deploy_result,
-                  'node2' => expected_deploy_result
-                )
-                expect(HybridPlatformsConductorTest::TestSecretsReaderPlugin.calls).to eq [
-                  { instance: :secrets_reader1, node: 'node1', service: 'service1' },
-                  { instance: :secrets_reader1, node: 'node2', service: 'service2' },
-                  { instance: :secrets_reader2, node: 'node2', service: 'service2' }
-                ]
-              end
-            end
-
-            it 'fails when merging secrets having different values' do
-              register_plugins(
-                :secrets_reader,
-                {
-                  secrets_reader1: HybridPlatformsConductorTest::TestSecretsReaderPlugin,
-                  secrets_reader2: HybridPlatformsConductorTest::TestSecretsReaderPlugin
-                }
-              )
-              with_platform_to_deploy(
-                nodes_info: {
-                  nodes: {
-                    'node1' => { services: %w[service1] },
-                    'node2' => { services: %w[service2] }
-                  }
-                },
-                expect_deploy_allowed: false,
-                expect_package: false,
-                expect_prepare_for_deploy: false,
-                expect_connections_to_nodes: false,
-                additional_config: <<~EOS
-                  read_secrets_from :secrets_reader1
-                  for_nodes('node2') { read_secrets_from :secrets_reader2 }
-                EOS
-              ) do
-                TestSecretsReaderPlugin.deployer = test_deployer
-                TestSecretsReaderPlugin.mocked_secrets = {
-                  'node1' => {
-                    'service1' => {
-                      secrets_reader1: {
-                        'global1' => 'value1',
-                        'global2' => 'value2'
-                      }
-                    }
-                  },
-                  'node2' => {
-                    'service2' => {
-                      secrets_reader1: {
-                        'global2' => 'value2',
-                        'global3' => {
-                          'sub_key' => 'value3'
-                        }
-                      },
-                      secrets_reader2: {
-                        'global3' => {
-                          'sub_key' => 'Other value'
-                        },
-                        'global4' => 'value4'
-                      }
-                    }
-                  }
-                }
-                expect { test_deployer.deploy_on(%w[node1 node2]) }.to raise_error 'Secret set at path global3->sub_key by secrets_reader2 for service service2 on node node2 has conflicting values (set debug for value details).'
-                expect(HybridPlatformsConductorTest::TestSecretsReaderPlugin.calls).to eq [
-                  { instance: :secrets_reader1, node: 'node1', service: 'service1' },
-                  { instance: :secrets_reader1, node: 'node2', service: 'service2' },
-                  { instance: :secrets_reader2, node: 'node2', service: 'service2' }
-                ]
-              end
-            end
-
-            it 'does not call secrets readers when secrets are overridden' do
-              register_plugins(
-                :secrets_reader,
-                {
-                  secrets_reader1: HybridPlatformsConductorTest::TestSecretsReaderPlugin,
-                  secrets_reader2: HybridPlatformsConductorTest::TestSecretsReaderPlugin,
-                  secrets_reader3: HybridPlatformsConductorTest::TestSecretsReaderPlugin
-                }
-              )
-              with_platform_to_deploy(
-                nodes_info: {
-                  nodes: {
-                    'node1' => { services: %w[service1] },
-                    'node2' => { services: %w[service2] },
-                    'node3' => { services: %w[service3] }
-                  }
-                },
-                expect_secrets: {
-                  'overridden_secrets' => 'value'
-                },
-                additional_config: <<~EOS
-                  read_secrets_from %i[secrets_reader1 secrets_reader2]
-                  for_nodes('node2') { read_secrets_from :secrets_reader3 }
-                EOS
-              ) do
-                TestSecretsReaderPlugin.deployer = test_deployer
-                test_deployer.override_secrets('overridden_secrets' => 'value')
-                expect(test_deployer.deploy_on(%w[node1 node2 node3])).to eq(
-                  'node1' => expected_deploy_result,
-                  'node2' => expected_deploy_result,
-                  'node3' => expected_deploy_result
-                )
-                expect(HybridPlatformsConductorTest::TestSecretsReaderPlugin.calls).to eq []
-              end
-            end
-
-          end
-
-        end
-
-      end
-
-    end
-
-  end
-
-end
-
-RSpec.configure do |c|
-  c.extend HybridPlatformsConductorTest::Helpers::DeployerTestHelpers
-end