opuppet/module-nginx
5
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Merge pull request #762 from alexharv074/Issue_758

Browse source 
nginx version fact compatibility with Facter 1.7
This commit is contained in:
Matthew Haughton 2016-02-06 01:18:51 -05:00
commit 332d4e5665
2 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
lib/facter/nginx_version.rb
View file

@ -1,7 +1,7 @@
Facter.add(:nginx_version) do Facter.add(:nginx_version) do
setcode do setcode do
if Facter::Core::Execution.which('nginx') if Facter::Util::Resolution.which('nginx')
nginx_version = Facter::Core::Execution.execute('nginx -v 2>&1') nginx_version = Facter::Util::Resolution.exec('nginx -v 2>&1')
%r{^nginx version: nginx\/([\w\.]+)}.match(nginx_version)[1] %r{^nginx version: nginx\/([\w\.]+)}.match(nginx_version)[1]
end end
end end

4
spec/unit/nginx_version_spec.rb
View file

@ -8,8 +8,8 @@ describe Facter::Util::Fact do
describe "nginx_version" do describe "nginx_version" do
context 'with value' do context 'with value' do
before :each do before :each do
Facter::Core::Execution.stubs(:which).with('nginx').returns(true) Facter::Util::Resolution.stubs(:which).with('nginx').returns(true)
Facter::Core::Execution.stubs(:execute).with('nginx -v 2>&1').returns('nginx version: nginx/1.8.1') Facter::Util::Resolution.stubs(:exec).with('nginx -v 2>&1').returns('nginx version: nginx/1.8.1')
end end
it { it {
expect(Facter.fact(:nginx_version).value).to eq('1.8.1') expect(Facter.fact(:nginx_version).value).to eq('1.8.1')