hybrid_platforms_conductor 32.17.1 → 32.18.0

data/spec/hybrid_platforms_conductor_test/executables/nodes_to_deploy_spec.rb

@@ -8,23 +8,43 @@ describe 'nodes_to_deploy executable' do
   #   * Parameters::
   #     * *repository* (String): Platform's repository
   def with_test_platform_for_nodes_to_deploy(additional_platforms_content: '')
-    with_test_platform({ nodes: { 'node1' => {}, 'node2' => {} } }, false, additional_platforms_content) do |repository|
+    with_test_platform(
+      { nodes: { 'node1' => {}, 'node2' => {} } },
+      false,
+      additional_platforms_content + "\nsend_logs_to :test_log"
+    ) do |repository|
       yield repository
     end
   end
 
   it 'returns all nodes by default' do
     with_test_platform_for_nodes_to_deploy do
-      expect_actions_executor_runs([proc do |actions|
-        expect(actions).to eq(
-          'node1' => { remote_bash: "cd /var/log/deployments && ls -t | head -1 | xargs sed '/===== STDOUT =====/q'" },
-          'node2' => { remote_bash: "cd /var/log/deployments && ls -t | head -1 | xargs sed '/===== STDOUT =====/q'" }
-        )
+      expect(test_deployer).to receive(:deployment_info_from).with(%w[node1 node2]) do
         {
-          'node1' => [0, "repo_name_0: platform\nexit_status: 0\ncommit_id_0: abcdef1", ''],
-          'node2' => [0, "repo_name_0: platform\nexit_status: 0\ncommit_id_0: abcdef2", '']
+          'node1' => {
+            services: %w[service1],
+            deployment_info: {
+              repo_name_0: 'platform',
+              commit_id_0: 'abcdef1',
+              exit_status: 0
+            },
+            exit_status: 0,
+            stdout: '',
+            stderr: ''
+          },
+          'node2' => {
+            services: %w[service2],
+            deployment_info: {
+              repo_name_0: 'platform',
+              commit_id_0: 'abcdef2',
+              exit_status: 0
+            },
+            exit_status: 0,
+            stdout: '',
+            stderr: ''
+          }
         }
-      end])
+      end
       expect(test_nodes_handler).to receive(:impacted_nodes_from_git_diff).with('platform', from_commit: 'abcdef1', to_commit: 'master') { [%w[node1 node2], [], [], false] }
       expect(test_nodes_handler).to receive(:impacted_nodes_from_git_diff).with('platform', from_commit: 'abcdef2', to_commit: 'master') { [%w[node1 node2], [], [], false] }
       exit_code, stdout, stderr = run 'nodes_to_deploy'
@@ -41,14 +61,21 @@ describe 'nodes_to_deploy executable' do
 
   it 'can filter nodes' do
     with_test_platform_for_nodes_to_deploy do
-      expect_actions_executor_runs([proc do |actions|
-        expect(actions).to eq(
-          'node2' => { remote_bash: "cd /var/log/deployments && ls -t | head -1 | xargs sed '/===== STDOUT =====/q'" }
-        )
+      expect(test_deployer).to receive(:deployment_info_from).with(%w[node2]) do
         {
-          'node2' => [0, "repo_name_0: platform\nexit_status: 0\ncommit_id_0: abcdef2", '']
+          'node2' => {
+            services: %w[service2],
+            deployment_info: {
+              repo_name_0: 'platform',
+              commit_id_0: 'abcdef2',
+              exit_status: 0
+            },
+            exit_status: 0,
+            stdout: '',
+            stderr: ''
+          }
         }
-      end])
+      end
       expect(test_nodes_handler).to receive(:impacted_nodes_from_git_diff).with('platform', from_commit: 'abcdef2', to_commit: 'master') { [%w[node1 node2], [], [], false] }
       exit_code, stdout, stderr = run 'nodes_to_deploy', '--node', 'node2'
       expect(exit_code).to eq 0
@@ -63,16 +90,32 @@ describe 'nodes_to_deploy executable' do
 
   it 'does not return nodes that have no impact' do
     with_test_platform_for_nodes_to_deploy do
-      expect_actions_executor_runs([proc do |actions|
-        expect(actions).to eq(
-          'node1' => { remote_bash: "cd /var/log/deployments && ls -t | head -1 | xargs sed '/===== STDOUT =====/q'" },
-          'node2' => { remote_bash: "cd /var/log/deployments && ls -t | head -1 | xargs sed '/===== STDOUT =====/q'" }
-        )
+      expect(test_deployer).to receive(:deployment_info_from).with(%w[node1 node2]) do
         {
-          'node1' => [0, "repo_name_0: platform\nexit_status: 0\ncommit_id_0: abcdef1", ''],
-          'node2' => [0, "repo_name_0: platform\nexit_status: 0\ncommit_id_0: abcdef2", '']
+          'node1' => {
+            services: %w[service1],
+            deployment_info: {
+              repo_name_0: 'platform',
+              commit_id_0: 'abcdef1',
+              exit_status: 0
+            },
+            exit_status: 0,
+            stdout: '',
+            stderr: ''
+          },
+          'node2' => {
+            services: %w[service2],
+            deployment_info: {
+              repo_name_0: 'platform',
+              commit_id_0: 'abcdef2',
+              exit_status: 0
+            },
+            exit_status: 0,
+            stdout: '',
+            stderr: ''
+          }
         }
-      end])
+      end
       expect(test_nodes_handler).to receive(:impacted_nodes_from_git_diff).with('platform', from_commit: 'abcdef1', to_commit: 'master') { [%w[node1], [], [], false] }
       expect(test_nodes_handler).to receive(:impacted_nodes_from_git_diff).with('platform', from_commit: 'abcdef2', to_commit: 'master') { [%w[], [], [], false] }
       exit_code, stdout, stderr = run 'nodes_to_deploy'
@@ -86,16 +129,28 @@ describe 'nodes_to_deploy executable' do
 
   it 'considers nodes having no repository info in their logs to be deployed' do
     with_test_platform_for_nodes_to_deploy do
-      expect_actions_executor_runs([proc do |actions|
-        expect(actions).to eq(
-          'node1' => { remote_bash: "cd /var/log/deployments && ls -t | head -1 | xargs sed '/===== STDOUT =====/q'" },
-          'node2' => { remote_bash: "cd /var/log/deployments && ls -t | head -1 | xargs sed '/===== STDOUT =====/q'" }
-        )
+      expect(test_deployer).to receive(:deployment_info_from).with(%w[node1 node2]) do
         {
-          'node1' => [0, "exit_status: 0\n", ''],
-          'node2' => [0, "repo_name_0: platform\nexit_status: 0\ncommit_id_0: abcdef2", '']
+          'node1' => {
+            services: %w[service1],
+            deployment_info: {},
+            exit_status: 0,
+            stdout: '',
+            stderr: ''
+          },
+          'node2' => {
+            services: %w[service2],
+            deployment_info: {
+              repo_name_0: 'platform',
+              commit_id_0: 'abcdef2',
+              exit_status: 0
+            },
+            exit_status: 0,
+            stdout: '',
+            stderr: ''
+          }
         }
-      end])
+      end
       expect(test_nodes_handler).to receive(:impacted_nodes_from_git_diff).with('platform', from_commit: 'abcdef2', to_commit: 'master') { [%w[], [], [], false] }
       exit_code, stdout, stderr = run 'nodes_to_deploy'
       expect(exit_code).to eq 0
@@ -108,16 +163,32 @@ describe 'nodes_to_deploy executable' do
 
   it 'considers nodes having invalid commit ids in their logs to be deployed' do
     with_test_platform_for_nodes_to_deploy do
-      expect_actions_executor_runs([proc do |actions|
-        expect(actions).to eq(
-          'node1' => { remote_bash: "cd /var/log/deployments && ls -t | head -1 | xargs sed '/===== STDOUT =====/q'" },
-          'node2' => { remote_bash: "cd /var/log/deployments && ls -t | head -1 | xargs sed '/===== STDOUT =====/q'" }
-        )
+      expect(test_deployer).to receive(:deployment_info_from).with(%w[node1 node2]) do
         {
-          'node1' => [0, "repo_name_0: platform\nexit_status: 0\ncommit_id_0: abcdef1", ''],
-          'node2' => [0, "repo_name_0: platform\nexit_status: 0\ncommit_id_0: abcdef2", '']
+          'node1' => {
+            services: %w[service1],
+            deployment_info: {
+              repo_name_0: 'platform',
+              commit_id_0: 'abcdef1',
+              exit_status: 0
+            },
+            exit_status: 0,
+            stdout: '',
+            stderr: ''
+          },
+          'node2' => {
+            services: %w[service2],
+            deployment_info: {
+              repo_name_0: 'platform',
+              commit_id_0: 'abcdef2',
+              exit_status: 0
+            },
+            exit_status: 0,
+            stdout: '',
+            stderr: ''
+          }
         }
-      end])
+      end
       expect(test_nodes_handler).to receive(:impacted_nodes_from_git_diff).with('platform', from_commit: 'abcdef1', to_commit: 'master') do
         raise HybridPlatformsConductor::NodesHandler::GitError, 'Mocked git error due to an invalid commit id'
       end
@@ -148,14 +219,21 @@ describe 'nodes_to_deploy executable' do
       for_nodes(\'node1\') { deployment_schedule(IceCube::Schedule.new(Time.now.utc - 120, duration: 60)) }
       for_nodes(\'node2\') { deployment_schedule(IceCube::Schedule.new(Time.now.utc - 60, duration: 120)) }
     ') do
-      expect_actions_executor_runs([proc do |actions|
-        expect(actions).to eq(
-          'node2' => { remote_bash: "cd /var/log/deployments && ls -t | head -1 | xargs sed '/===== STDOUT =====/q'" }
-        )
+      expect(test_deployer).to receive(:deployment_info_from).with(%w[node2]) do
         {
-          'node2' => [0, "repo_name_0: platform\nexit_status: 0\ncommit_id_0: abcdef2", '']
+          'node2' => {
+            services: %w[service2],
+            deployment_info: {
+              repo_name_0: 'platform',
+              commit_id_0: 'abcdef2',
+              exit_status: 0
+            },
+            exit_status: 0,
+            stdout: '',
+            stderr: ''
+          }
         }
-      end])
+      end
       expect(test_nodes_handler).to receive(:impacted_nodes_from_git_diff).with('platform', from_commit: 'abcdef2', to_commit: 'master') { [%w[node1 node2], [], [], false] }
       exit_code, stdout, stderr = run 'nodes_to_deploy'
       expect(exit_code).to eq 0
@@ -172,14 +250,21 @@ describe 'nodes_to_deploy executable' do
       for_nodes(\'node1\') { deployment_schedule(IceCube::Schedule.new(Time.now.utc - 120, duration: 60)) }
       for_nodes(\'node2\') { deployment_schedule(IceCube::Schedule.new(Time.now.utc - 60, duration: 120)) }
     ') do
-      expect_actions_executor_runs([proc do |actions|
-        expect(actions).to eq(
-          'node1' => { remote_bash: "cd /var/log/deployments && ls -t | head -1 | xargs sed '/===== STDOUT =====/q'" }
-        )
+      expect(test_deployer).to receive(:deployment_info_from).with(%w[node1]) do
         {
-          'node1' => [0, "repo_name_0: platform\nexit_status: 0\ncommit_id_0: abcdef1", '']
+          'node1' => {
+            services: %w[service1],
+            deployment_info: {
+              repo_name_0: 'platform',
+              commit_id_0: 'abcdef1',
+              exit_status: 0
+            },
+            exit_status: 0,
+            stdout: '',
+            stderr: ''
+          }
         }
-      end])
+      end
       expect(test_nodes_handler).to receive(:impacted_nodes_from_git_diff).with('platform', from_commit: 'abcdef1', to_commit: 'master') { [%w[node1 node2], [], [], false] }
       # 90 seconds before now, the schedule should match only node1
       exit_code, stdout, stderr = run 'nodes_to_deploy', '--deployment-time', (Time.now.utc - 90).strftime('%F %T')
@@ -197,16 +282,32 @@ describe 'nodes_to_deploy executable' do
       for_nodes(\'node1\') { deployment_schedule(IceCube::Schedule.new(Time.now.utc - 120, duration: 60)) }
       for_nodes(\'node2\') { deployment_schedule(IceCube::Schedule.new(Time.now.utc - 60, duration: 120)) }
     ') do
-      expect_actions_executor_runs([proc do |actions|
-        expect(actions).to eq(
-          'node1' => { remote_bash: "cd /var/log/deployments && ls -t | head -1 | xargs sed '/===== STDOUT =====/q'" },
-          'node2' => { remote_bash: "cd /var/log/deployments && ls -t | head -1 | xargs sed '/===== STDOUT =====/q'" }
-        )
+      expect(test_deployer).to receive(:deployment_info_from).with(%w[node1 node2]) do
         {
-          'node1' => [0, "repo_name_0: platform\nexit_status: 0\ncommit_id_0: abcdef1", ''],
-          'node2' => [0, "repo_name_0: platform\nexit_status: 0\ncommit_id_0: abcdef2", '']
+          'node1' => {
+            services: %w[service1],
+            deployment_info: {
+              repo_name_0: 'platform',
+              commit_id_0: 'abcdef1',
+              exit_status: 0
+            },
+            exit_status: 0,
+            stdout: '',
+            stderr: ''
+          },
+          'node2' => {
+            services: %w[service2],
+            deployment_info: {
+              repo_name_0: 'platform',
+              commit_id_0: 'abcdef2',
+              exit_status: 0
+            },
+            exit_status: 0,
+            stdout: '',
+            stderr: ''
+          }
         }
-      end])
+      end
       expect(test_nodes_handler).to receive(:impacted_nodes_from_git_diff).with('platform', from_commit: 'abcdef1', to_commit: 'master') { [%w[node1 node2], [], [], false] }
       expect(test_nodes_handler).to receive(:impacted_nodes_from_git_diff).with('platform', from_commit: 'abcdef2', to_commit: 'master') { [%w[node1 node2], [], [], false] }
       exit_code, stdout, stderr = run 'nodes_to_deploy', '--ignore-schedule'
@@ -225,16 +326,32 @@ describe 'nodes_to_deploy executable' do
       'platform1' => { nodes: { 'node1' => {}, 'node2' => {} } },
       'platform2' => { nodes: {} }
     ) do
-      expect_actions_executor_runs([proc do |actions|
-        expect(actions).to eq(
-          'node1' => { remote_bash: "cd /var/log/deployments && ls -t | head -1 | xargs sed '/===== STDOUT =====/q'" },
-          'node2' => { remote_bash: "cd /var/log/deployments && ls -t | head -1 | xargs sed '/===== STDOUT =====/q'" }
-        )
+      expect(test_deployer).to receive(:deployment_info_from).with(%w[node1 node2]) do
         {
-          'node1' => [0, "repo_name_0: platform1\nexit_status: 0\ncommit_id_0: abcdef1", ''],
-          'node2' => [0, "repo_name_0: platform2\nexit_status: 0\ncommit_id_0: abcdef2", '']
+          'node1' => {
+            services: %w[service1],
+            deployment_info: {
+              repo_name_0: 'platform1',
+              commit_id_0: 'abcdef1',
+              exit_status: 0
+            },
+            exit_status: 0,
+            stdout: '',
+            stderr: ''
+          },
+          'node2' => {
+            services: %w[service2],
+            deployment_info: {
+              repo_name_0: 'platform2',
+              commit_id_0: 'abcdef2',
+              exit_status: 0
+            },
+            exit_status: 0,
+            stdout: '',
+            stderr: ''
+          }
         }
-      end])
+      end
       expect(test_nodes_handler).to receive(:impacted_nodes_from_git_diff).with('platform1', from_commit: 'abcdef1', to_commit: 'master') { [%w[node1], [], [], false] }
       expect(test_nodes_handler).to receive(:impacted_nodes_from_git_diff).with('platform2', from_commit: 'abcdef2', to_commit: 'master') { [%w[node2], [], [], false] }
       exit_code, stdout, stderr = run 'nodes_to_deploy'
@@ -253,16 +370,36 @@ describe 'nodes_to_deploy executable' do
       'platform2' => { nodes: {} },
       'platform3' => { nodes: {} }
     ) do
-      expect_actions_executor_runs([proc do |actions|
-        expect(actions).to eq(
-          'node1' => { remote_bash: "cd /var/log/deployments && ls -t | head -1 | xargs sed '/===== STDOUT =====/q'" },
-          'node2' => { remote_bash: "cd /var/log/deployments && ls -t | head -1 | xargs sed '/===== STDOUT =====/q'" }
-        )
+      expect(test_deployer).to receive(:deployment_info_from).with(%w[node1 node2]) do
         {
-          'node1' => [0, "exit_status: 0\nrepo_name_0: platform1\ncommit_id_0: abcdef1\nrepo_name_1: platform2\ncommit_id_1: 1234567", ''],
-          'node2' => [0, "exit_status: 0\nrepo_name_0: platform2\ncommit_id_0: abcdef2\nrepo_name_1: platform3\ncommit_id_1: 2345678", '']
+          'node1' => {
+            services: %w[service1],
+            deployment_info: {
+              repo_name_0: 'platform1',
+              commit_id_0: 'abcdef1',
+              repo_name_1: 'platform2',
+              commit_id_1: '1234567',
+              exit_status: 0
+            },
+            exit_status: 0,
+            stdout: '',
+            stderr: ''
+          },
+          'node2' => {
+            services: %w[service2],
+            deployment_info: {
+              repo_name_0: 'platform2',
+              commit_id_0: 'abcdef2',
+              repo_name_1: 'platform3',
+              commit_id_1: '2345678',
+              exit_status: 0
+            },
+            exit_status: 0,
+            stdout: '',
+            stderr: ''
+          }
         }
-      end])
+      end
       expect(test_nodes_handler).to receive(:impacted_nodes_from_git_diff).with('platform1', from_commit: 'abcdef1', to_commit: 'master') { [%w[], [], [], false] }
       expect(test_nodes_handler).to receive(:impacted_nodes_from_git_diff).with('platform2', from_commit: '1234567', to_commit: 'master') { [%w[node1], [], [], false] }
       expect(test_nodes_handler).to receive(:impacted_nodes_from_git_diff).with('platform2', from_commit: 'abcdef2', to_commit: 'master') { [%w[], [], [], false] }
@@ -283,16 +420,36 @@ describe 'nodes_to_deploy executable' do
       'platform2' => { nodes: {} },
       'platform3' => { nodes: {} }
     ) do
-      expect_actions_executor_runs([proc do |actions|
-        expect(actions).to eq(
-          'node1' => { remote_bash: "cd /var/log/deployments && ls -t | head -1 | xargs sed '/===== STDOUT =====/q'" },
-          'node2' => { remote_bash: "cd /var/log/deployments && ls -t | head -1 | xargs sed '/===== STDOUT =====/q'" }
-        )
+      expect(test_deployer).to receive(:deployment_info_from).with(%w[node1 node2]) do
         {
-          'node1' => [0, "exit_status: 0\nrepo_name_0: platform1\ncommit_id_0: abcdef1\nrepo_name_1: platform2\ncommit_id_1: 1234567", ''],
-          'node2' => [0, "exit_status: 0\nrepo_name_0: platform2\ncommit_id_0: abcdef2\nrepo_name_1: platform3\ncommit_id_1: 2345678", '']
+          'node1' => {
+            services: %w[service1],
+            deployment_info: {
+              repo_name_0: 'platform1',
+              commit_id_0: 'abcdef1',
+              repo_name_1: 'platform2',
+              commit_id_1: '1234567',
+              exit_status: 0
+            },
+            exit_status: 0,
+            stdout: '',
+            stderr: ''
+          },
+          'node2' => {
+            services: %w[service2],
+            deployment_info: {
+              repo_name_0: 'platform2',
+              commit_id_0: 'abcdef2',
+              repo_name_1: 'platform3',
+              commit_id_1: '2345678',
+              exit_status: 0
+            },
+            exit_status: 0,
+            stdout: '',
+            stderr: ''
+          }
         }
-      end])
+      end
       expect(test_nodes_handler).to receive(:impacted_nodes_from_git_diff).with('platform1', from_commit: 'abcdef1', to_commit: 'master') { [%w[node1], [], [], false] }
       expect(test_nodes_handler).to receive(:impacted_nodes_from_git_diff).with('platform2', from_commit: 'abcdef2', to_commit: 'master') { [%w[node2], [], [], false] }
       exit_code, stdout, stderr = run 'nodes_to_deploy'

data/spec/hybrid_platforms_conductor_test/executables/options/common_spec.rb

@@ -12,7 +12,8 @@ describe 'executables\' common options' do
         nodes: { 'node1' => { meta: { host_ip: '192.168.42.42' }, services: ['node1_service'] } },
         deployable_services: %w[node1_service]
       },
-      true
+      true,
+      'send_logs_to :test_log'
     ) do |repository|
       yield repository
     end

data/spec/hybrid_platforms_conductor_test/helpers/deployer_helpers.rb

@@ -68,71 +68,21 @@ module HybridPlatformsConductorTest
         Hash[nodes.map { |node| [node, [0, 'Release mutex successful', '']] }]
       end
 
-      # Expect a given set of actions to upload log files on a list of nodes
+      # Expect a given set of actions to upload log files on a list of nodes (using the test_log log plugin)
       #
       # Parameters::
       # * *actions* (Object): Actions
       # * *nodes* (String or Array<String>): Node (or list of nodes) that should be checked
-      # * *sudo* (String or nil): sudo supposed to be used, or nil if none [default: 'sudo -u root']
-      def expect_actions_to_upload_logs(actions, nodes, sudo: 'sudo -u root')
+      def expect_actions_to_upload_logs(actions, nodes)
         nodes = [nodes] if nodes.is_a?(String)
         expect(actions.size).to eq nodes.size
         nodes.each do |node|
           expect(actions.key?(node)).to eq true
-          expect(actions[node][:remote_bash]).to eq "#{sudo ? "#{sudo} " : ''}mkdir -p /var/log/deployments"
-          expect(actions[node][:scp].first[1]).to eq '/var/log/deployments'
-          expect(actions[node][:scp][:group]).to eq 'root'
-          expect(actions[node][:scp][:owner]).to eq 'root'
-          expect(actions[node][:scp][:sudo]).to eq (!sudo.nil?)
+          expect(actions[node]).to eq [{ bash: "echo Save test logs to #{node}" }]
         end
         Hash[nodes.map { |node| [node, [0, 'Logs uploaded', '']] }]
       end
 
-      # Expect some logs to have the following information.
-      # Expected logs format:
-      #
-      # date: 2019-08-14 17:02:57
-      # user: muriel
-      # debug: Yes
-      # repo_name: my_remote_platform
-      # commit_id: c0d16b1b7ae286ae4a059185957e08f0ddc95517
-      # commit_message: Test commit
-      # diff_files: 
-      # ===== STDOUT =====
-      # Deploy successful
-      # ===== STDERR =====
-      #
-      # Parameters::
-      # * *logs* (String): The logs content
-      # * *stdout* (String): Expected STDOUT
-      # * *stderr* (String): Expected STDERR
-      # * *properties* (Hash<Symbol, String or Regexp>): Expected properties values, per name. Values can be exact strings or regexps.
-      def expect_logs_to_be(logs, stdout, stderr, properties)
-        lines = logs.split("\n")
-        idx_stdout = lines.index('===== STDOUT =====')
-        expect(idx_stdout).not_to eq nil
-        idx_stderr = lines.index('===== STDERR =====')
-        expect(idx_stderr).not_to eq nil
-        logs_properties = Hash[lines[0..idx_stdout - 1].map do |property_line|
-          property_fields = property_line.split(': ')
-          [
-            property_fields.first.to_sym,
-            property_fields[1..-1].join(': ')
-          ]
-        end]
-        expect(logs_properties.size).to eq properties.size
-        properties.each do |expected_property, expected_property_value|
-          expect(logs_properties.key?(expected_property)).to eq true
-          if expected_property_value.is_a?(String)
-            expect(logs_properties[expected_property]).to eq expected_property_value
-          else
-            expect(logs_properties[expected_property]).to match expected_property_value
-          end
-        end
-        expect(lines[idx_stdout + 1..idx_stderr - 1].join("\n")).to eq stdout
-        expect(lines[idx_stderr + 1..-1].join("\n")).to eq stderr
-      end
-
       # Get a test Deployer
       #
       # Result::
@@ -142,6 +92,43 @@ module HybridPlatformsConductorTest
         @deployer
       end
 
+      # Expect the test services handler to be called to deploy a given list of services
+      #
+      # Parameters::
+      # * *services* (Hash<String, Array<String> >): List of services to be expected, per node name
+      def expect_services_handler_to_deploy(services)
+        expect(test_services_handler).to receive(:deploy_allowed?).with(
+          services: services,
+          secrets: {},
+          local_environment: false
+        ) do
+          nil
+        end
+        expect(test_services_handler).to receive(:package).with(
+          services: services,
+          secrets: {},
+          local_environment: false
+        )
+        expect(test_services_handler).to receive(:prepare_for_deploy).with(
+          services: services,
+          secrets: {},
+          local_environment: false,
+          why_run: false
+        )
+        services.each do |node, services|
+          expect(test_services_handler).to receive(:actions_to_deploy_on).with(node, services, false) do
+            [{ bash: "echo \"Deploying on #{node}\"" }]
+          end
+          expect(test_services_handler).to receive(:log_info_for).with(node, services) do
+            {
+              repo_name_0: 'platform',
+              commit_id_0: '123456',
+              commit_message_0: "Test commit for #{node}: #{services.join(', ')}"
+            }
+          end
+        end
+      end
+
     end
 
   end