summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--lib/puppet/provider/file_line/ruby.rb4
-rw-r--r--lib/puppet/type/file_line.rb4
-rwxr-xr-xspec/unit/puppet/provider/file_line/ruby_spec.rb2
-rwxr-xr-xspec/unit/puppet/type/file_line_spec.rb2
4 files changed, 6 insertions, 6 deletions
diff --git a/lib/puppet/provider/file_line/ruby.rb b/lib/puppet/provider/file_line/ruby.rb
index ea1d44d..d4cdfec 100644
--- a/lib/puppet/provider/file_line/ruby.rb
+++ b/lib/puppet/provider/file_line/ruby.rb
@@ -1,6 +1,6 @@
Puppet::Type.type(:file_line).provide(:ruby) do
def exists?
- if !resource[:replace] and count_matches(match_regex) > 0
+ if resource[:replace].to_s != 'true' and count_matches(match_regex) > 0
true
else
lines.find do |line|
@@ -10,7 +10,7 @@ Puppet::Type.type(:file_line).provide(:ruby) do
end
def create
- unless !resource[:replace] and count_matches(match_regex) > 0
+ unless resource[:replace].to_s != 'true' and count_matches(match_regex) > 0
if resource[:match]
handle_create_with_match
elsif resource[:after]
diff --git a/lib/puppet/type/file_line.rb b/lib/puppet/type/file_line.rb
index 190105c..4a96ba7 100644
--- a/lib/puppet/type/file_line.rb
+++ b/lib/puppet/type/file_line.rb
@@ -1,4 +1,3 @@
-require 'puppet/parameter/boolean'
Puppet::Type.newtype(:file_line) do
desc <<-EOT
@@ -79,8 +78,9 @@ Puppet::Type.newtype(:file_line) do
end
end
- newparam(:replace, :boolean => true, :parent => Puppet::Parameter::Boolean) do
+ newparam(:replace) do
desc 'If true, replace line that matches. If false, do not write line if a match is found'
+ newvalues(true, false)
defaultto true
end
diff --git a/spec/unit/puppet/provider/file_line/ruby_spec.rb b/spec/unit/puppet/provider/file_line/ruby_spec.rb
index 5eff09a..792391a 100755
--- a/spec/unit/puppet/provider/file_line/ruby_spec.rb
+++ b/spec/unit/puppet/provider/file_line/ruby_spec.rb
@@ -85,7 +85,7 @@ describe provider_class do
:replace => 'asgadga',
}
)
- }.to raise_error(Puppet::Error, /Invalid value "asgadga"\. Valid values are true, false, yes, no\./)
+ }.to raise_error(Puppet::Error, /Invalid value "asgadga"\. Valid values are true, false\./)
end
end
context "when matching" do
diff --git a/spec/unit/puppet/type/file_line_spec.rb b/spec/unit/puppet/type/file_line_spec.rb
index 58c88e3..f1430f2 100755
--- a/spec/unit/puppet/type/file_line_spec.rb
+++ b/spec/unit/puppet/type/file_line_spec.rb
@@ -50,7 +50,7 @@ describe Puppet::Type.type(:file_line) do
expect(file_line[:ensure]).to eq :present
end
it 'should default to replace => true' do
- expect(file_line[:replace]).to eq true
+ expect(file_line[:replace]).to eq :true
end
it "should autorequire the file it manages" do