This change creates a parser function that can be used to check if a
value is set to the '<SERVICE DEFAULT>' string. The is_service_default
function will return true if the parameter passed in is '<SERVICE
DEFAULT>' otherwise it returns false.
Checks like:
if ($our_param == '<SERVICE DEFAULT>') { ... }
Should be replaced with:
if is_service_default($our_param) { ... }
This change will also be useful if we ever refactor the default value
string or have different values as this function could be updated to
support multiple values without having to adjust the calling code.
Change-Id: I07b8b9b54ed1e88891f74da9b930e4f39876a607
41 lines
1.0 KiB
Ruby
41 lines
1.0 KiB
Ruby
require 'spec_helper'
|
|
|
|
describe 'is_service_default' do
|
|
|
|
it 'refuses without at least one argument' do
|
|
is_expected.to run.with_params().\
|
|
and_raise_error(Puppet::ParseError, /Wrong number of arguments/)
|
|
end
|
|
|
|
it 'refuses too many arguments' do
|
|
is_expected.to run.with_params('foo', 'bar').\
|
|
and_raise_error(Puppet::ParseError, /Wrong number of arguments/)
|
|
end
|
|
|
|
context 'is_service_default' do
|
|
it 'with <SERVICE DEFAULT>' do
|
|
is_expected.to run.with_params('<SERVICE DEFAULT>').and_return(true)
|
|
end
|
|
|
|
it 'with string != <SERVICE DEFAULT>' do
|
|
is_expected.to run.with_params('a value').and_return(false)
|
|
end
|
|
|
|
it 'with array' do
|
|
is_expected.to run.with_params([1,2,3]).and_return(false)
|
|
end
|
|
|
|
it 'with hash' do
|
|
is_expected.to run.with_params({'foo' => 'bar'}).and_return(false)
|
|
end
|
|
|
|
it 'with integer' do
|
|
is_expected.to run.with_params(1234).and_return(false)
|
|
end
|
|
|
|
it 'with boolean' do
|
|
is_expected.to run.with_params(false).and_return(false)
|
|
end
|
|
end
|
|
end
|