spec/cli/setup/service_spec.rb in travis-1.11.1 vs spec/cli/setup/service_spec.rb in travis-1.12.0

- old
+ new

@@ -1,53 +1,52 @@ +# frozen_string_literal: true + require 'spec_helper' describe Travis::CLI::Setup::Service do subject :service do - repository = double('repository',{:slug => 'test_slug'}) - setup = double('Setup', :repository => repository) + repository = double('repository', { slug: 'test_slug' }) + setup = double('Setup', repository:) described_class.new(setup) end - + describe '#deploy' do - subject :deploy_config do - {'provider' => 'dummy'} + { 'provider' => 'dummy' } end - + before do allow(service).to receive(:configure).and_yield(deploy_config) allow(service).to receive(:on).and_return(nil) end - + context 'with no existing deploy section' do - it 'creates section contents without optional items' do service.send(:deploy, 'dummy') { |_| } expect(deploy_config).to include( - 'provider' => 'dummy', - 'skip_cleanup' => 'true' + 'provider' => 'dummy', + 'skip_cleanup' => 'true' ) end end - + context 'with existing deploy section' do it 'doesn\'t overwrite existing skip_cleanup' do - deploy_config['skip_cleanup']='false' + deploy_config['skip_cleanup'] = 'false' service.send(:deploy, 'dummy') { |_| } - expect(deploy_config).to include('skip_cleanup' => 'false') + expect(deploy_config).to include('skip_cleanup' => 'false') end it 'doesn\'t set skip_cleanup for v2' do - deploy_config['edge']='true' + deploy_config['edge'] = 'true' service.send(:deploy, 'dummy') { |_| } expect(deploy_config).not_to include('skip_cleanup') end - + it 'sets skip_cleanup for explicit v1' do - deploy_config['edge']='false' + deploy_config['edge'] = 'false' service.send(:deploy, 'dummy') { |_| } - expect(deploy_config).to include('skip_cleanup' => 'true') + expect(deploy_config).to include('skip_cleanup' => 'true') end end - end end