diff --git a/lib/facter/java_version.rb b/lib/facter/java_version.rb index 1c427a8..a9d9c18 100644 --- a/lib/facter/java_version.rb +++ b/lib/facter/java_version.rb @@ -23,7 +23,7 @@ Facter.add(:java_version) do # Additionally, facter versions prior to 2.0.1 only support # positive matches, so this needs to be done manually in setcode. setcode do - unless ['darwin'].include? Facter.value(:operatingsystem).downcase + unless ['darwin'].include? Facter.value(:kernel).downcase version = nil if Facter::Util::Resolution.which('java') Facter::Util::Resolution.exec('java -Xmx12m -version 2>&1').lines.each { |line| version = Regexp.last_match(1) if %r{^.+ version \"(.+)\"} =~ line } @@ -34,7 +34,7 @@ Facter.add(:java_version) do end Facter.add(:java_version) do - confine operatingsystem: 'Darwin' + confine kernel: 'Darwin' has_weight 100 setcode do unless Facter::Util::Resolution.exec('/usr/libexec/java_home --failfast 2>&1').include?('Unable to find any JVMs matching version') diff --git a/spec/unit/facter/java_version_spec.rb b/spec/unit/facter/java_version_spec.rb index bb4b167..2f31f6b 100644 --- a/spec/unit/facter/java_version_spec.rb +++ b/spec/unit/facter/java_version_spec.rb @@ -32,9 +32,9 @@ describe 'java_version' do end context 'when on Darwin' do before(:each) do - allow(Facter.fact(:operatingsystem)).to receive(:value).and_return('Darwin') + allow(Facter.fact(:kernel)).to receive(:value).and_return('Darwin') end - let(:facts) { { operatingsystem: 'Darwin' } } + let(:facts) { { kernel: 'Darwin' } } it do expect(Facter::Util::Resolution).to receive(:exec).with('/usr/libexec/java_home --failfast 2>&1').and_return('/Library/Java/JavaVirtualMachines/jdk1.7.0_71.jdk/Contents/Home') @@ -71,9 +71,9 @@ describe 'java_version' do end context 'when on Darwin' do before(:each) do - allow(Facter.fact(:operatingsystem)).to receive(:value).and_return('Darwin') + allow(Facter.fact(:kernel)).to receive(:value).and_return('Darwin') end - let(:facts) { { operatingsystem: 'Darwin' } } + let(:facts) { { kernel: 'Darwin' } } it do expect(Facter::Util::Resolution).to receive(:exec).with('/usr/libexec/java_home --failfast 2>&1').at_least(1).and_return('Unable to find any JVMs matching version "(null)".')