Resolve new chefstyle warnings & use safe navigators

lib/ohai/plugins/ec2.rb

@@ -114,7 +114,7 @@ def looks_like_ec2?
   end
 
   collect_data do
-    require "base64"
+    require "base64" unless defined?(Base64)
 
     if looks_like_ec2?
       logger.trace("Plugin EC2: looks_like_ec2? == true")

lib/ohai/plugins/linux/network.rb

@@ -141,11 +141,9 @@ def check_routing_table(family, iface, default_route_table)
   # using a temporary var to hold routes and their interface name
   def parse_routes(family, iface)
     iface.collect do |i, iv|
-      if iv[:routes]
-        iv[:routes].collect do |r|
-          r.merge(dev: i) if r[:family] == family[:name]
-        end.compact
-      end
+      iv[:routes]&.collect do |r|
+        r.merge(dev: i) if r[:family] == family[:name]
+      end&.compact
     end.compact.flatten
   end
 

lib/ohai/plugins/shard.rb

@@ -52,7 +52,7 @@ def default_digest_algorithm
   def digest_algorithm
     case Ohai.config[:plugin][:shard_seed][:digest_algorithm] || default_digest_algorithm
     when "md5"
-      require "digest/md5"
+      require "digest/md5" unless defined?(Digest::MD5)
       Digest::MD5
     when "sha256"
       require "openssl/digest"

lib/ohai/plugins/softlayer.rb

@@ -39,7 +39,7 @@ def looks_like_softlayer?
       logger.trace("Plugin Softlayer: looks_like_softlayer? == true")
       metadata = fetch_metadata
       softlayer Mash.new
-      metadata.each { |k, v| softlayer[k] = v } if metadata
+      metadata&.each { |k, v| softlayer[k] = v }
     else
       logger.trace("Plugin Softlayer: looks_like_softlayer? == false")
     end

lib/ohai/plugins/solaris2/network.rb

@@ -86,7 +86,7 @@ def arpname_to_ifname(iface, arpname)
 
   def full_interface_name(iface, part_name, index)
     iface.each do |name, attrs|
-      next unless attrs && attrs.respond_to?(:[])
+      next unless attrs&.respond_to?(:[])
       return name if /^#{part_name}($|:)/.match(name) && attrs[:index] == index
     end
 
@@ -155,12 +155,10 @@ def full_interface_name(iface, part_name, index)
             break
           end
         end
-        if iface[ifn][:arp]
-          iface[ifn][:arp].each_key do |addr|
-            if addr.eql?(iaddr)
-              iface[ifn][:addresses][iface[ifn][:arp][iaddr]] = { "family" => "lladdr" }
-              break
-            end
+        iface[ifn][:arp]&.each_key do |addr|
+          if addr.eql?(iaddr)
+            iface[ifn][:addresses][iface[ifn][:arp][iaddr]] = { "family" => "lladdr" }
+            break
           end
         end
       end