hybrid_platforms_conductor 33.0.0 → 33.0.1

data/spec/hybrid_platforms_conductor_test/api/services_handler/parse_deploy_output_spec.rb

@@ -1,16 +1,18 @@
 describe HybridPlatformsConductor::ServicesHandler do
 
-  context 'checking parsing deployment outputs' do
+  context 'when checking parsing deployment outputs' do
 
     it 'parses a deployment log for a node' do
       with_test_platform(
-        nodes: { 'node' => { services: %w[service1] } },
-        deployable_services: %w[service1],
-        parse_deploy_output: proc do |stdout, stderr|
-          expect(stdout.strip).to eq 'Task1: ok'
-          expect(stderr.strip).to eq 'Service1 stderr'
-          [{ name: 'Task1', status: :identical }]
-        end
+        {
+          nodes: { 'node' => { services: %w[service1] } },
+          deployable_services: %w[service1],
+          parse_deploy_output: proc do |stdout, stderr|
+            expect(stdout.strip).to eq 'Task1: ok'
+            expect(stderr.strip).to eq 'Service1 stderr'
+            [{ name: 'Task1', status: :identical }]
+          end
+        }
       ) do
         stdout = <<~EOS_STDOUT
           First log lines
@@ -37,13 +39,15 @@ describe HybridPlatformsConductor::ServicesHandler do
 
     it 'parses a deployment log for a node in check mode' do
       with_test_platform(
-        nodes: { 'node' => { services: %w[service1] } },
-        deployable_services: %w[service1],
-        parse_deploy_output: proc do |stdout, stderr|
-          expect(stdout.strip).to eq 'Task1: ok'
-          expect(stderr.strip).to eq 'Service1 stderr'
-          [{ name: 'Task1', status: :identical }]
-        end
+        {
+          nodes: { 'node' => { services: %w[service1] } },
+          deployable_services: %w[service1],
+          parse_deploy_output: proc do |stdout, stderr|
+            expect(stdout.strip).to eq 'Task1: ok'
+            expect(stderr.strip).to eq 'Service1 stderr'
+            [{ name: 'Task1', status: :identical }]
+          end
+        }
       ) do
         stdout = <<~EOS_STDOUT
           First log lines
@@ -70,13 +74,15 @@ describe HybridPlatformsConductor::ServicesHandler do
 
     it 'parses a deployment log for a node even if stderr is empty' do
       with_test_platform(
-        nodes: { 'node' => { services: %w[service1] } },
-        deployable_services: %w[service1],
-        parse_deploy_output: proc do |stdout, stderr|
-          expect(stdout.strip).to eq 'Task1: ok'
-          expect(stderr.strip).to eq ''
-          [{ name: 'Task1', status: :identical }]
-        end
+        {
+          nodes: { 'node' => { services: %w[service1] } },
+          deployable_services: %w[service1],
+          parse_deploy_output: proc do |stdout, stderr|
+            expect(stdout.strip).to eq 'Task1: ok'
+            expect(stderr.strip).to eq ''
+            [{ name: 'Task1', status: :identical }]
+          end
+        }
       ) do
         stdout = <<~EOS_STDOUT
           First log lines
@@ -98,13 +104,15 @@ describe HybridPlatformsConductor::ServicesHandler do
 
     it 'parses a deployment log for a node deploying several services' do
       with_test_platform(
-        nodes: { 'node' => { services: %w[service1 service2] } },
-        deployable_services: %w[service1 service2],
-        parse_deploy_output: proc do |stdout, stderr|
-          task_name, status_str = stdout.match(/^(.+?): (.+)$/)[1..2]
-          expect(stderr.strip).to eq "#{task_name} stderr"
-          [{ name: task_name, status: status_str.to_sym }]
-        end
+        {
+          nodes: { 'node' => { services: %w[service1 service2] } },
+          deployable_services: %w[service1 service2],
+          parse_deploy_output: proc do |stdout, stderr|
+            task_name, status_str = stdout.match(/^(.+?): (.+)$/)[1..2]
+            expect(stderr.strip).to eq "#{task_name} stderr"
+            [{ name: task_name, status: status_str.to_sym }]
+          end
+        }
       ) do
         stdout = <<~EOS_STDOUT
           First log lines
@@ -144,13 +152,15 @@ describe HybridPlatformsConductor::ServicesHandler do
 
     it 'parses a deployment log for several nodes deploying several services' do
       with_test_platform(
-        nodes: { 'node1' => { services: %w[service1 service2] }, 'node2' => { services: %w[service1 service2] } },
-        deployable_services: %w[service1 service2],
-        parse_deploy_output: proc do |stdout, stderr|
-          task_name, status_str = stdout.match(/^(.+?): (.+)$/)[1..2]
-          expect(stderr.strip).to eq "#{task_name} stderr"
-          [{ name: task_name, status: status_str.to_sym }]
-        end
+        {
+          nodes: { 'node1' => { services: %w[service1 service2] }, 'node2' => { services: %w[service1 service2] } },
+          deployable_services: %w[service1 service2],
+          parse_deploy_output: proc do |stdout, stderr|
+            task_name, status_str = stdout.match(/^(.+?): (.+)$/)[1..2]
+            expect(stderr.strip).to eq "#{task_name} stderr"
+            [{ name: task_name, status: status_str.to_sym }]
+          end
+        }
       ) do
         stdout = <<~EOS_STDOUT
           First log lines
@@ -202,24 +212,26 @@ describe HybridPlatformsConductor::ServicesHandler do
 
     it 'parses a deployment log for several nodes deploying several services using different platforms' do
       with_test_platforms(
-        'platform1' => {
-          nodes: { 'node1' => { services: %w[service1 service2] }, 'node2' => { services: %w[service1 service2] } },
-          deployable_services: %w[service1],
-          parse_deploy_output: proc do |stdout, stderr|
-            task_name, status_str = stdout.match(/^(.+?1): (.+)$/)[1..2]
-            expect(stderr.strip).to eq "#{task_name} stderr"
-            [{ name: task_name, status: status_str.to_sym }]
-          end
-        },
-        'platform2' => {
-          nodes: {},
-          deployable_services: %w[service2],
-          parse_deploy_output: proc do |stdout, stderr|
-            task_name, status_str = stdout.match(/^(.+?2): (.+)$/)[1..2]
-            expect(stderr.strip).to eq "#{task_name} stderr"
-            [{ name: task_name, status: status_str.to_sym }]
-          end
-        },
+        {
+          'platform1' => {
+            nodes: { 'node1' => { services: %w[service1 service2] }, 'node2' => { services: %w[service1 service2] } },
+            deployable_services: %w[service1],
+            parse_deploy_output: proc do |stdout, stderr|
+              task_name, status_str = stdout.match(/^(.+?1): (.+)$/)[1..2]
+              expect(stderr.strip).to eq "#{task_name} stderr"
+              [{ name: task_name, status: status_str.to_sym }]
+            end
+          },
+          'platform2' => {
+            nodes: {},
+            deployable_services: %w[service2],
+            parse_deploy_output: proc do |stdout, stderr|
+              task_name, status_str = stdout.match(/^(.+?2): (.+)$/)[1..2]
+              expect(stderr.strip).to eq "#{task_name} stderr"
+              [{ name: task_name, status: status_str.to_sym }]
+            end
+          }
+        }
       ) do
         stdout = <<~EOS_STDOUT
           First log lines

data/spec/hybrid_platforms_conductor_test/api/services_handler/prepare_for_deploy_spec.rb

@@ -1,19 +1,21 @@
 describe HybridPlatformsConductor::ServicesHandler do
 
-  context 'checking preparation for deployment' do
+  context 'when checking preparation for deployment' do
 
     it 'prepares 1 platform' do
       called = false
       with_test_platform(
-        nodes: { 'node1' => { services: %w[service1] }, 'node2' => {}, 'node3' => {} },
-        deployable_services: %w[service1],
-        prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
-          expect(services).to eq('node1' => %w[service1])
-          expect(secrets).to eq({})
-          expect(local_environment).to eq false
-          expect(why_run).to eq false
-          called = true
-        end
+        {
+          nodes: { 'node1' => { services: %w[service1] }, 'node2' => {}, 'node3' => {} },
+          deployable_services: %w[service1],
+          prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
+            expect(services).to eq('node1' => %w[service1])
+            expect(secrets).to eq({})
+            expect(local_environment).to eq false
+            expect(why_run).to eq false
+            called = true
+          end
+        }
       ) do
         test_services_handler.prepare_for_deploy(
           services: { 'node1' => %w[service1] },
@@ -28,15 +30,17 @@ describe HybridPlatformsConductor::ServicesHandler do
     it 'prepares 1 platform with secrets' do
       called = false
       with_test_platform(
-        nodes: { 'node1' => { services: %w[service1] }, 'node2' => {}, 'node3' => {} },
-        deployable_services: %w[service1],
-        prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
-          expect(services).to eq('node1' => %w[service1])
-          expect(secrets).to eq('my_secret' => 'value')
-          expect(local_environment).to eq false
-          expect(why_run).to eq false
-          called = true
-        end
+        {
+          nodes: { 'node1' => { services: %w[service1] }, 'node2' => {}, 'node3' => {} },
+          deployable_services: %w[service1],
+          prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
+            expect(services).to eq('node1' => %w[service1])
+            expect(secrets).to eq('my_secret' => 'value')
+            expect(local_environment).to eq false
+            expect(why_run).to eq false
+            called = true
+          end
+        }
       ) do
         test_services_handler.prepare_for_deploy(
           services: { 'node1' => %w[service1] },
@@ -51,15 +55,17 @@ describe HybridPlatformsConductor::ServicesHandler do
     it 'prepares 1 platform with local environment' do
       called = false
       with_test_platform(
-        nodes: { 'node1' => { services: %w[service1] }, 'node2' => {}, 'node3' => {} },
-        deployable_services: %w[service1],
-        prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
-          expect(services).to eq('node1' => %w[service1])
-          expect(secrets).to eq({})
-          expect(local_environment).to eq true
-          expect(why_run).to eq false
-          called = true
-        end
+        {
+          nodes: { 'node1' => { services: %w[service1] }, 'node2' => {}, 'node3' => {} },
+          deployable_services: %w[service1],
+          prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
+            expect(services).to eq('node1' => %w[service1])
+            expect(secrets).to eq({})
+            expect(local_environment).to eq true
+            expect(why_run).to eq false
+            called = true
+          end
+        }
       ) do
         test_services_handler.prepare_for_deploy(
           services: { 'node1' => %w[service1] },
@@ -74,15 +80,17 @@ describe HybridPlatformsConductor::ServicesHandler do
     it 'prepares 1 platform in why-run mode' do
       called = false
       with_test_platform(
-        nodes: { 'node1' => { services: %w[service1] }, 'node2' => {}, 'node3' => {} },
-        deployable_services: %w[service1],
-        prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
-          expect(services).to eq('node1' => %w[service1])
-          expect(secrets).to eq({})
-          expect(local_environment).to eq false
-          expect(why_run).to eq true
-          called = true
-        end
+        {
+          nodes: { 'node1' => { services: %w[service1] }, 'node2' => {}, 'node3' => {} },
+          deployable_services: %w[service1],
+          prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
+            expect(services).to eq('node1' => %w[service1])
+            expect(secrets).to eq({})
+            expect(local_environment).to eq false
+            expect(why_run).to eq true
+            called = true
+          end
+        }
       ) do
         test_services_handler.prepare_for_deploy(
           services: { 'node1' => %w[service1] },
@@ -101,38 +109,40 @@ describe HybridPlatformsConductor::ServicesHandler do
         'platform3' => false
       }
       with_test_platforms(
-        'platform1' => {
-          nodes: { 'node1' => { services: %w[service1] } },
-          deployable_services: %w[service1],
-          prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
-            expect(services).to eq('node1' => %w[service1])
-            expect(secrets).to eq({})
-            expect(local_environment).to eq false
-            expect(why_run).to eq false
-            called['platform1'] = true
-          end
-        },
-        'platform2' => {
-          nodes: { 'node2' => { services: %w[service2] } },
-          deployable_services: %w[service2],
-          prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
-            expect(services).to eq('node2' => %w[service2])
-            expect(secrets).to eq({})
-            expect(local_environment).to eq false
-            expect(why_run).to eq false
-            called['platform2'] = true
-          end
-        },
-        'platform3' => {
-          nodes: { 'node3' => { services: %w[service3] } },
-          deployable_services: %w[service3],
-          prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
-            expect(services).to eq('node3' => %w[service3])
-            expect(secrets).to eq({})
-            expect(local_environment).to eq false
-            expect(why_run).to eq false
-            called['platform3'] = true
-          end
+        {
+          'platform1' => {
+            nodes: { 'node1' => { services: %w[service1] } },
+            deployable_services: %w[service1],
+            prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
+              expect(services).to eq('node1' => %w[service1])
+              expect(secrets).to eq({})
+              expect(local_environment).to eq false
+              expect(why_run).to eq false
+              called['platform1'] = true
+            end
+          },
+          'platform2' => {
+            nodes: { 'node2' => { services: %w[service2] } },
+            deployable_services: %w[service2],
+            prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
+              expect(services).to eq('node2' => %w[service2])
+              expect(secrets).to eq({})
+              expect(local_environment).to eq false
+              expect(why_run).to eq false
+              called['platform2'] = true
+            end
+          },
+          'platform3' => {
+            nodes: { 'node3' => { services: %w[service3] } },
+            deployable_services: %w[service3],
+            prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
+              expect(services).to eq('node3' => %w[service3])
+              expect(secrets).to eq({})
+              expect(local_environment).to eq false
+              expect(why_run).to eq false
+              called['platform3'] = true
+            end
+          }
         }
       ) do
         test_services_handler.prepare_for_deploy(
@@ -156,38 +166,40 @@ describe HybridPlatformsConductor::ServicesHandler do
         'platform3' => false
       }
       with_test_platforms(
-        'platform1' => {
-          nodes: { 'node1' => { services: %w[service1] } },
-          deployable_services: %w[service1],
-          prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
-            expect(services).to eq('node1' => %w[service1])
-            expect(secrets).to eq({})
-            expect(local_environment).to eq false
-            expect(why_run).to eq false
-            called['platform1'] = true
-          end
-        },
-        'platform2' => {
-          nodes: { 'node2' => { services: %w[service2] } },
-          deployable_services: %w[service2],
-          prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
-            expect(services).to eq('node2' => %w[service2])
-            expect(secrets).to eq({})
-            expect(local_environment).to eq false
-            expect(why_run).to eq false
-            called['platform2'] = true
-          end
-        },
-        'platform3' => {
-          nodes: { 'node3' => { services: %w[service3] } },
-          deployable_services: %w[service3],
-          prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
-            expect(services).to eq('node1' => %w[service3])
-            expect(secrets).to eq({})
-            expect(local_environment).to eq false
-            expect(why_run).to eq false
-            called['platform3'] = true
-          end
+        {
+          'platform1' => {
+            nodes: { 'node1' => { services: %w[service1] } },
+            deployable_services: %w[service1],
+            prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
+              expect(services).to eq('node1' => %w[service1])
+              expect(secrets).to eq({})
+              expect(local_environment).to eq false
+              expect(why_run).to eq false
+              called['platform1'] = true
+            end
+          },
+          'platform2' => {
+            nodes: { 'node2' => { services: %w[service2] } },
+            deployable_services: %w[service2],
+            prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
+              expect(services).to eq('node2' => %w[service2])
+              expect(secrets).to eq({})
+              expect(local_environment).to eq false
+              expect(why_run).to eq false
+              called['platform2'] = true
+            end
+          },
+          'platform3' => {
+            nodes: { 'node3' => { services: %w[service3] } },
+            deployable_services: %w[service3],
+            prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
+              expect(services).to eq('node1' => %w[service3])
+              expect(secrets).to eq({})
+              expect(local_environment).to eq false
+              expect(why_run).to eq false
+              called['platform3'] = true
+            end
+          }
         }
       ) do
         test_services_handler.prepare_for_deploy(
@@ -211,38 +223,40 @@ describe HybridPlatformsConductor::ServicesHandler do
         'platform3' => false
       }
       with_test_platforms(
-        'platform1' => {
-          nodes: { 'node' => { services: %w[service1 service2 service3 service4 service5 service6] } },
-          deployable_services: %w[service1 service2],
-          prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
-            expect(services).to eq('node' => %w[service1 service2])
-            expect(secrets).to eq({})
-            expect(local_environment).to eq false
-            expect(why_run).to eq false
-            called['platform1'] = true
-          end
-        },
-        'platform2' => {
-          nodes: {},
-          deployable_services: %w[service3 service4],
-          prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
-            expect(services).to eq('node' => %w[service3])
-            expect(secrets).to eq({})
-            expect(local_environment).to eq false
-            expect(why_run).to eq false
-            called['platform2'] = true
-          end
-        },
-        'platform3' => {
-          nodes: {},
-          deployable_services: %w[service5 service6],
-          prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
-            expect(services).to eq('node' => %w[service5 service6])
-            expect(secrets).to eq({})
-            expect(local_environment).to eq false
-            expect(why_run).to eq false
-            called['platform3'] = true
-          end
+        {
+          'platform1' => {
+            nodes: { 'node' => { services: %w[service1 service2 service3 service4 service5 service6] } },
+            deployable_services: %w[service1 service2],
+            prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
+              expect(services).to eq('node' => %w[service1 service2])
+              expect(secrets).to eq({})
+              expect(local_environment).to eq false
+              expect(why_run).to eq false
+              called['platform1'] = true
+            end
+          },
+          'platform2' => {
+            nodes: {},
+            deployable_services: %w[service3 service4],
+            prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
+              expect(services).to eq('node' => %w[service3])
+              expect(secrets).to eq({})
+              expect(local_environment).to eq false
+              expect(why_run).to eq false
+              called['platform2'] = true
+            end
+          },
+          'platform3' => {
+            nodes: {},
+            deployable_services: %w[service5 service6],
+            prepare_for_deploy: proc do |services:, secrets:, local_environment:, why_run:|
+              expect(services).to eq('node' => %w[service5 service6])
+              expect(secrets).to eq({})
+              expect(local_environment).to eq false
+              expect(why_run).to eq false
+              called['platform3'] = true
+            end
+          }
         }
       ) do
         test_services_handler.prepare_for_deploy(