From 1618034ef0697ac149ede67ff508f6c646fdfb86 Mon Sep 17 00:00:00 2001 From: Anton Chuchkalov Date: Wed, 20 Apr 2016 15:47:20 +0300 Subject: [PATCH] return skip_rollback to true by default --- devops-service/db/mongo/models/stack/stack_base.rb | 3 ++- devops-service/spec/models/stack/stack_ec2_spec.rb | 4 ++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/devops-service/db/mongo/models/stack/stack_base.rb b/devops-service/db/mongo/models/stack/stack_base.rb index c2707be..55e924f 100644 --- a/devops-service/db/mongo/models/stack/stack_base.rb +++ b/devops-service/db/mongo/models/stack/stack_base.rb @@ -122,7 +122,8 @@ module Devops end def skip_rollback? - launch_options['skip_rollback'] || false + result = launch_options['skip_rollback'] + result.nil? ? true : result end class << self diff --git a/devops-service/spec/models/stack/stack_ec2_spec.rb b/devops-service/spec/models/stack/stack_ec2_spec.rb index ee57b44..0105c1b 100644 --- a/devops-service/spec/models/stack/stack_ec2_spec.rb +++ b/devops-service/spec/models/stack/stack_ec2_spec.rb @@ -79,8 +79,8 @@ RSpec.describe Devops::Model::StackEc2, type: :model do expect(stack.skip_rollback?).to be true end - it 'returns false by default' do - expect(described_class.new.skip_rollback?).to be false + it 'returns true by default' do + expect(described_class.new.skip_rollback?).to be true end end