diff --git a/deployment/puppet/l23network/Gemfile b/deployment/puppet/l23network/Gemfile index e9fd5df096..2b54c88769 100644 --- a/deployment/puppet/l23network/Gemfile +++ b/deployment/puppet/l23network/Gemfile @@ -4,7 +4,7 @@ group :development, :test do gem 'rake', :require => false gem 'pry', :require => false gem 'rspec', '~>3.3', :require => false - gem 'rspec-puppet', '~>2.1.0', :require => false + gem 'rspec-puppet', '~>2.2.0', :require => false gem 'puppetlabs_spec_helper', :require => false gem 'puppet-lint', '~> 0.3.2' end diff --git a/deployment/puppet/l23network/spec/functions/get_network_role_property__spec.rb b/deployment/puppet/l23network/spec/functions/get_network_role_property__spec.rb index 40c37a27e4..6317583ed4 100644 --- a/deployment/puppet/l23network/spec/functions/get_network_role_property__spec.rb +++ b/deployment/puppet/l23network/spec/functions/get_network_role_property__spec.rb @@ -90,8 +90,8 @@ describe Puppet::Parser::Functions.function(:get_network_role_property) do it 'should return NIL for "admin" network role' do should run.with_params('admin', 'ipaddr_netmask_pair').and_return(nil) end - it 'should return NIL for "admin" network role' do - should run.with_params('admin', 'phys_dev').and_return(nil) + it 'should return eth0 for "admin" network role phys_dev' do + should run.with_params('admin', 'phys_dev').and_return(["eth0"]) end end diff --git a/deployment/puppet/l23network/spec/functions/get_transformation_property__spec.rb b/deployment/puppet/l23network/spec/functions/get_transformation_property__spec.rb index b05c0655f8..43ad95277d 100644 --- a/deployment/puppet/l23network/spec/functions/get_transformation_property__spec.rb +++ b/deployment/puppet/l23network/spec/functions/get_transformation_property__spec.rb @@ -116,8 +116,12 @@ end should run.with_params('mtu', 'eth0').and_return(2048) end - it 'should return NIL for "eth2" transformation' do - should run.with_params('mtu', 'eth2').and_return(nil) + it 'should return 1024 for "eth2" transformation' do + should run.with_params('mtu', 'eth2').and_return(1024) + end + + it 'should return NIL for "eth3" transformation' do + should run.with_params('mtu', 'eth3').and_return(nil) end end