Skip to content

Commit

Permalink
Merge pull request #9067 from mhashizume/maint/7.x/aix-5.3-setting
Browse files Browse the repository at this point in the history
(maint) Removes AIX 5.3-specific setting
  • Loading branch information
joshcooper authored Jun 14, 2023
2 parents b73a071 + b3759a7 commit fd7f09f
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 45 deletions.
6 changes: 1 addition & 5 deletions lib/puppet/defaults.rb
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,7 @@
module Puppet

def self.default_diffargs
if (Puppet.runtime[:facter].value(:kernel) == "AIX" && Puppet.runtime[:facter].value(:kernelmajversion) == "5300")
""
else
"-u"
end
'-u'
end

def self.default_digest_algorithm
Expand Down
42 changes: 2 additions & 40 deletions spec/unit/defaults_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -3,46 +3,8 @@

describe "Defaults" do
describe ".default_diffargs" do
describe "on AIX" do
before(:each) do
allow(Facter).to receive(:value).with(:kernel).and_return("AIX")
end

describe "on 5.3" do
before(:each) do
allow(Facter).to receive(:value).with(:kernelmajversion).and_return("5300")
end

it "should be empty" do
expect(Puppet.default_diffargs).to eq("")
end
end

[ "",
nil,
"6300",
"7300",
].each do |kernel_version|
describe "on kernel version #{kernel_version.inspect}" do
before(:each) do
allow(Facter).to receive(:value).with(:kernelmajversion).and_return(kernel_version)
end

it "should be '-u'" do
expect(Puppet.default_diffargs).to eq("-u")
end
end
end
end

describe "on everything else" do
before(:each) do
allow(Facter).to receive(:value).with(:kernel).and_return("NOT_AIX")
end

it "should be '-u'" do
expect(Puppet.default_diffargs).to eq("-u")
end
it "should be '-u'" do
expect(Puppet.default_diffargs).to eq("-u")
end
end

Expand Down

0 comments on commit fd7f09f

Please sign in to comment.