(bugfix) - Fix ec2 fact issues when on non ec2 systems

  * previously when ec2 facts are executed the check_product_name
    was run which incorrectly returned a hash of values when the product
    name did not exist in the hypervisor hash. This is an edge case
    on linux systems not in ec2.

Author: logicminds

lib/facter/facts/linux/ec2_metadata.rb

@@ -35,7 +35,8 @@ def check_product_name
         product_name = Facter::Resolvers::Linux::DmiBios.resolve(:product_name)
         return unless product_name
 
-        Facter::FactsUtils::HYPERVISORS_HASH.each { |key, value| return value if product_name.include?(key) }
+        _, value = Facter::FactsUtils::HYPERVISORS_HASH.find {|key, value| product_name.include?(key) }
+        value
       end
 
       def check_bios_vendor

lib/facter/facts/linux/ec2_userdata.rb

@@ -32,8 +32,8 @@ def check_xen
       def check_product_name
         product_name = Facter::Resolvers::Linux::DmiBios.resolve(:product_name)
         return unless product_name
-
-        Facter::FactsUtils::HYPERVISORS_HASH.each { |key, value| return value if product_name.include?(key) }
+        _, value = Facter::FactsUtils::HYPERVISORS_HASH.find {|key, value| product_name.include?(key) }
+        value
       end
 
       def check_bios_vendor

spec/facter/facts/linux/ec2_metadata_spec.rb

@@ -13,6 +13,25 @@
       allow(Facter::Resolvers::Linux::DmiBios).to receive(:resolve).with(:bios_vendor).and_return(nil)
     end
 
+    context 'when physical machine with no hypervisor' do
+      let(:hypervisor) { nil }
+      let(:value) { nil }
+
+      before(:each) do
+        allow(Facter::Resolvers::Linux::DmiBios).to receive(:resolve).with(:product_name).and_return('MS-7A71')
+      end
+
+      it 'returns ec2 metadata fact as nil' do
+        expect(fact.call_the_resolver).to be_an_instance_of(Facter::ResolvedFact).and \
+          have_attributes(name: 'ec2_metadata', value: nil)
+      end
+
+      it "doesn't call Ec2 resolver" do
+        fact.call_the_resolver
+        expect(Facter::Resolvers::Ec2).not_to have_received(:resolve).with(:metadata)
+      end
+    end
+
     context 'when hypervisor is not kvm or xen' do
       let(:hypervisor) { nil }
       let(:value) { nil }

spec/facter/facts/linux/ec2_userdata_spec.rb

@@ -13,6 +13,25 @@
       allow(Facter::Resolvers::Linux::DmiBios).to receive(:resolve).with(:bios_vendor).and_return(nil)
     end
 
+    context 'when physical machine with no hypervisor' do
+      let(:hypervisor) { nil }
+      let(:value) { nil }
+
+      before(:each) do
+        allow(Facter::Resolvers::Linux::DmiBios).to receive(:resolve).with(:product_name).and_return('MS-7A71')
+      end
+
+      it 'returns ec2 metadata fact as nil' do
+        expect(fact.call_the_resolver).to be_an_instance_of(Facter::ResolvedFact).and \
+          have_attributes(name: 'ec2_metadata', value: nil)
+      end
+
+      it "doesn't call Ec2 resolver" do
+        fact.call_the_resolver
+        expect(Facter::Resolvers::Ec2).not_to have_received(:resolve).with(:metadata)
+      end
+    end
+
     context 'when hypervisor is not kvm or xen' do
       let(:hypervisor) { nil }
       let(:value) { nil }