Merge pull request #719 from tas50/virt_cleanup

Linux virtualization plugin style cleanup

4 files changed, 76 insertions, 76 deletions

diff --git a/lib/ohai/mixin/dmi_decode.rb b/lib/ohai/mixin/dmi_decode.rb
index 9f39b910..755afda0 100644
--- a/lib/ohai/mixin/dmi_decode.rb
+++ b/lib/ohai/mixin/dmi_decode.rb
@@ -17,7 +17,7 @@
 
 # http://www.dmo.ca/blog/detecting-virtualization-on-linux
 module ::Ohai::Mixin::DmiDecode
-  def determine_guest(dmi_data)
+  def guest_from_dmi(dmi_data)
     dmi_data.each_line do |line|
       case line
       when /Manufacturer: Microsoft/
diff --git a/lib/ohai/plugins/bsd/virtualization.rb b/lib/ohai/plugins/bsd/virtualization.rb
index 5317fa9e..518d664d 100644
--- a/lib/ohai/plugins/bsd/virtualization.rb
+++ b/lib/ohai/plugins/bsd/virtualization.rb
@@ -69,7 +69,7 @@ Ohai.plugin(:Virtualization) do
 
     # parse dmidecode to discover various virtualization guests
     if File.exist?('/usr/local/sbin/dmidecode') || File.exist?('/usr/pkg/sbin/dmidecode')
-      guest = determine_guest(shell_out('dmidecode').stdout)
+      guest = guest_from_dmi(shell_out('dmidecode').stdout)
       if guest
         virtualization[:system] = guest
         virtualization[:role] = 'guest'
diff --git a/lib/ohai/plugins/linux/virtualization.rb b/lib/ohai/plugins/linux/virtualization.rb
index 66e51ccf..f3182974 100644
--- a/lib/ohai/plugins/linux/virtualization.rb
+++ b/lib/ohai/plugins/linux/virtualization.rb
@@ -41,15 +41,15 @@ Ohai.plugin(:Virtualization) do
     # virtualization[:mechanism]
 
     ## Xen
-    # /proc/xen is an empty dir for EL6 + Linode Guests
-    if File.exists?("/proc/xen")
+    # /proc/xen is an empty dir for EL6 + Linode Guests + Paravirt EC2 instances
+    if File.exist?("/proc/xen")
       virtualization[:system] = "xen"
       # Assume guest
       virtualization[:role] = "guest"
       virtualization[:systems][:xen] = "guest"
 
       # This file should exist on most Xen systems, normally empty for guests
-      if File.exists?("/proc/xen/capabilities")
+      if File.exist?("/proc/xen/capabilities")
         if File.read("/proc/xen/capabilities") =~ /control_d/i
           virtualization[:role] = "host"
           virtualization[:systems][:xen] = "host"
@@ -65,7 +65,7 @@ Ohai.plugin(:Virtualization) do
     #   but rather be additive - btm
 
     # Detect from kernel module
-    if File.exists?("/proc/modules")
+    if File.exist?("/proc/modules")
       modules = File.read("/proc/modules")
       if modules =~ /^kvm/
         virtualization[:system] = "kvm"
@@ -87,7 +87,7 @@ Ohai.plugin(:Virtualization) do
     # 2.6.27-9-server (intrepid) has this / 2.6.18-6-amd64 (etch) does not
     # It would be great if we could read pv_info in the kernel
     # Wait for reply to: http://article.gmane.org/gmane.comp.emulators.kvm.devel/27885
-    if File.exists?("/proc/cpuinfo")
+    if File.exist?("/proc/cpuinfo")
       if File.read("/proc/cpuinfo") =~ /QEMU Virtual CPU|Common KVM processor|Common 32-bit KVM processor/
         virtualization[:system] = "kvm"
         virtualization[:role] = "guest"
@@ -97,18 +97,18 @@ Ohai.plugin(:Virtualization) do
 
     # Detect OpenVZ / Virtuozzo.
     # http://wiki.openvz.org/BC_proc_entries
-    if File.exists?("/proc/bc/0")
+    if File.exist?("/proc/bc/0")
       virtualization[:system] = "openvz"
       virtualization[:role] = "host"
       virtualization[:systems][:openvz] = "host"
-    elsif File.exists?("/proc/vz")
+    elsif File.exist?("/proc/vz")
       virtualization[:system] = "openvz"
       virtualization[:role] = "guest"
       virtualization[:systems][:openvz] = "guest"
     end
 
     # Detect Parallels virtual machine from pci devices
-    if File.exists?("/proc/bus/pci/devices")
+    if File.exist?("/proc/bus/pci/devices")
       if File.read("/proc/bus/pci/devices") =~ /1ab84000/
         virtualization[:system] = "parallels"
         virtualization[:role] = "guest"
@@ -117,8 +117,8 @@ Ohai.plugin(:Virtualization) do
     end
 
     # parse dmidecode to discover various virtualization guests
-    if File.exists?("/usr/sbin/dmidecode")
-      guest = determine_guest(shell_out('dmidecode').stdout)
+    if File.exist?("/usr/sbin/dmidecode")
+      guest = guest_from_dmi(shell_out('dmidecode').stdout)
       if guest
         virtualization[:system] = guest
         virtualization[:role] = 'guest'
@@ -127,10 +127,10 @@ Ohai.plugin(:Virtualization) do
     end
 
     # Detect Linux-VServer
-    if File.exists?("/proc/self/status")
+    if File.exist?("/proc/self/status")
       proc_self_status = File.read("/proc/self/status")
       vxid = proc_self_status.match(/^(s_context|VxID):\s*(\d+)$/)
-      if vxid and vxid[2]
+      if vxid && vxid[2]
         virtualization[:system] = "linux-vserver"
         if vxid[2] == "0"
           virtualization[:role] = "host"
@@ -160,7 +160,7 @@ Ohai.plugin(:Virtualization) do
     #
     # Full notes, https://tickets.opscode.com/browse/OHAI-551
     # Kernel docs, https://www.kernel.org/doc/Documentation/cgroups
-    if File.exists?("/proc/self/cgroup")
+    if File.exist?("/proc/self/cgroup")
       cgroup_content = File.read("/proc/self/cgroup")
       if cgroup_content =~ %r{^\d+:[^:]+:/(lxc|docker)/.+$} ||
          cgroup_content =~ %r{^\d+:[^:]+:/[^/]+/(lxc|docker)-.+$}
@@ -180,10 +180,10 @@ Ohai.plugin(:Virtualization) do
         # If so, we may need to look further for a differentiator (OHAI-573)
         virtualization[:systems][:lxc] = "host"
       end
-    elsif File.exists?("/.dockerenv") || File.exists?("/.dockerinit")
-        virtualization[:system] = "docker"
-        virtualization[:role] = "guest"
-        virtualization[:systems][:docker] = "guest"
+    elsif File.exist?("/.dockerenv") || File.exist?("/.dockerinit")
+      virtualization[:system] = "docker"
+      virtualization[:role] = "guest"
+      virtualization[:systems][:docker] = "guest"
     end
   end
 end
diff --git a/spec/unit/plugins/linux/virtualization_spec.rb b/spec/unit/plugins/linux/virtualization_spec.rb
index 165b65ed..1ed15659 100644
--- a/spec/unit/plugins/linux/virtualization_spec.rb
+++ b/spec/unit/plugins/linux/virtualization_spec.rb
@@ -25,24 +25,24 @@ describe Ohai::System, "Linux virtualization platform" do
     allow(plugin).to receive(:collect_os).and_return(:linux)
 
     # default to all requested Files not existing
-    allow(File).to receive(:exists?).with("/proc/xen").and_return(false)
-    allow(File).to receive(:exists?).with("/proc/xen/capabilities").and_return(false)
-    allow(File).to receive(:exists?).with("/proc/modules").and_return(false)
-    allow(File).to receive(:exists?).with("/proc/cpuinfo").and_return(false)
-    allow(File).to receive(:exists?).with("/usr/sbin/dmidecode").and_return(false)
-    allow(File).to receive(:exists?).with("/proc/self/status").and_return(false)
-    allow(File).to receive(:exists?).with("/proc/bc/0").and_return(false)
-    allow(File).to receive(:exists?).with("/proc/vz").and_return(false)
-    allow(File).to receive(:exists?).with("/proc/self/cgroup").and_return(false)
-    allow(File).to receive(:exists?).with("/.dockerenv").and_return(false)
-    allow(File).to receive(:exists?).with("/.dockerinit").and_return(false)
-    allow(File).to receive(:exists?).with("/proc/bus/pci/devices").and_return(false)
+    allow(File).to receive(:exist?).with("/proc/xen").and_return(false)
+    allow(File).to receive(:exist?).with("/proc/xen/capabilities").and_return(false)
+    allow(File).to receive(:exist?).with("/proc/modules").and_return(false)
+    allow(File).to receive(:exist?).with("/proc/cpuinfo").and_return(false)
+    allow(File).to receive(:exist?).with("/usr/sbin/dmidecode").and_return(false)
+    allow(File).to receive(:exist?).with("/proc/self/status").and_return(false)
+    allow(File).to receive(:exist?).with("/proc/bc/0").and_return(false)
+    allow(File).to receive(:exist?).with("/proc/vz").and_return(false)
+    allow(File).to receive(:exist?).with("/proc/self/cgroup").and_return(false)
+    allow(File).to receive(:exist?).with("/.dockerenv").and_return(false)
+    allow(File).to receive(:exist?).with("/.dockerinit").and_return(false)
+    allow(File).to receive(:exist?).with("/proc/bus/pci/devices").and_return(false)
   end
 
   describe "when we are checking for xen" do
     it "sets xen guest if /proc/xen exists but /proc/xen/capabilities does not" do
-      expect(File).to receive(:exists?).with("/proc/xen").and_return(true)
-      expect(File).to receive(:exists?).with("/proc/xen/capabilities").and_return(false)
+      expect(File).to receive(:exist?).with("/proc/xen").and_return(true)
+      expect(File).to receive(:exist?).with("/proc/xen/capabilities").and_return(false)
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("xen")
       expect(plugin[:virtualization][:role]).to eq("guest")
@@ -50,8 +50,8 @@ describe Ohai::System, "Linux virtualization platform" do
     end
 
     it "sets xen host if /proc/xen/capabilities contains control_d " do
-      expect(File).to receive(:exists?).with("/proc/xen").and_return(true)
-      expect(File).to receive(:exists?).with("/proc/xen/capabilities").and_return(true)
+      expect(File).to receive(:exist?).with("/proc/xen").and_return(true)
+      expect(File).to receive(:exist?).with("/proc/xen/capabilities").and_return(true)
       allow(File).to receive(:read).with("/proc/xen/capabilities").and_return("control_d")
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("xen")
@@ -60,8 +60,8 @@ describe Ohai::System, "Linux virtualization platform" do
     end
 
     it "sets xen guest if /proc/xen/capabilities exists but is empty" do
-      expect(File).to receive(:exists?).with("/proc/xen").and_return(true)
-      expect(File).to receive(:exists?).with("/proc/xen/capabilities").and_return(true)
+      expect(File).to receive(:exist?).with("/proc/xen").and_return(true)
+      expect(File).to receive(:exist?).with("/proc/xen/capabilities").and_return(true)
       allow(File).to receive(:read).with("/proc/xen/capabilities").and_return("")
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("xen")
@@ -70,7 +70,7 @@ describe Ohai::System, "Linux virtualization platform" do
     end
 
     it "does not set virtualization if xen isn't there" do
-      expect(File).to receive(:exists?).at_least(:once).and_return(false)
+      expect(File).to receive(:exist?).at_least(:once).and_return(false)
       plugin.run
       expect(plugin[:virtualization]).to eq({'systems' => {}})
     end
@@ -78,7 +78,7 @@ describe Ohai::System, "Linux virtualization platform" do
 
   describe "when we are checking for kvm" do
     it "sets kvm host if /proc/modules contains kvm" do
-      expect(File).to receive(:exists?).with("/proc/modules").and_return(true)
+      expect(File).to receive(:exist?).with("/proc/modules").and_return(true)
       allow(File).to receive(:read).with("/proc/modules").and_return("kvm 165872  1 kvm_intel")
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("kvm")
@@ -87,7 +87,7 @@ describe Ohai::System, "Linux virtualization platform" do
     end
 
     it "sets kvm guest if /proc/cpuinfo contains QEMU Virtual CPU" do
-      expect(File).to receive(:exists?).with("/proc/cpuinfo").and_return(true)
+      expect(File).to receive(:exist?).with("/proc/cpuinfo").and_return(true)
       allow(File).to receive(:read).with("/proc/cpuinfo").and_return("QEMU Virtual CPU")
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("kvm")
@@ -96,7 +96,7 @@ describe Ohai::System, "Linux virtualization platform" do
     end
 
     it "sets kvm guest if /proc/cpuinfo contains Common KVM processor" do
-      expect(File).to receive(:exists?).with("/proc/cpuinfo").and_return(true)
+      expect(File).to receive(:exist?).with("/proc/cpuinfo").and_return(true)
       allow(File).to receive(:read).with("/proc/cpuinfo").and_return("Common KVM processor")
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("kvm")
@@ -105,7 +105,7 @@ describe Ohai::System, "Linux virtualization platform" do
     end
 
     it "sets kvm guest if /proc/cpuinfo contains Common 32-bit KVM processor" do
-      expect(File).to receive(:exists?).with("/proc/cpuinfo").and_return(true)
+      expect(File).to receive(:exist?).with("/proc/cpuinfo").and_return(true)
       allow(File).to receive(:read).with("/proc/cpuinfo").and_return("Common 32-bit KVM processor")
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("kvm")
@@ -114,7 +114,7 @@ describe Ohai::System, "Linux virtualization platform" do
     end
 
     it "does not set virtualization if kvm isn't there" do
-      expect(File).to receive(:exists?).at_least(:once).and_return(false)
+      expect(File).to receive(:exist?).at_least(:once).and_return(false)
       plugin.run
       expect(plugin[:virtualization]).to eq({'systems' => {}})
     end
@@ -122,7 +122,7 @@ describe Ohai::System, "Linux virtualization platform" do
 
   describe "when we are checking for VirtualBox" do
     it "sets vbox host if /proc/modules contains vboxdrv" do
-      expect(File).to receive(:exists?).with("/proc/modules").and_return(true)
+      expect(File).to receive(:exist?).with("/proc/modules").and_return(true)
       allow(File).to receive(:read).with("/proc/modules").and_return("vboxdrv 268268 3 vboxnetadp,vboxnetflt")
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("vbox")
@@ -131,7 +131,7 @@ describe Ohai::System, "Linux virtualization platform" do
     end
 
     it "sets vbox gues if /proc/modules contains vboxguest" do
-      expect(File).to receive(:exists?).with("/proc/modules").and_return(true)
+      expect(File).to receive(:exist?).with("/proc/modules").and_return(true)
       allow(File).to receive(:read).with("/proc/modules").and_return("vboxguest 214901 2 vboxsf, Live 0xffffffffa00db000 (OF)")
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("vbox")
@@ -140,7 +140,7 @@ describe Ohai::System, "Linux virtualization platform" do
     end
 
     it "does not set virtualization if vbox isn't there" do
-      expect(File).to receive(:exists?).at_least(:once).and_return(false)
+      expect(File).to receive(:exist?).at_least(:once).and_return(false)
       plugin.run
       expect(plugin[:virtualization]).to eq({'systems' => {}})
     end
@@ -148,7 +148,7 @@ describe Ohai::System, "Linux virtualization platform" do
 
   describe "when we are parsing dmidecode" do
     before(:each) do
-      expect(File).to receive(:exists?).with("/usr/sbin/dmidecode").and_return(true)
+      expect(File).to receive(:exist?).with("/usr/sbin/dmidecode").and_return(true)
     end
 
     it "sets virtualpc guest if dmidecode detects Microsoft Virtual Machine" do
@@ -308,7 +308,7 @@ RHEV
 
   describe "when we are checking for Linux-VServer" do
     it "sets Linux-VServer host if /proc/self/status contains s_context: 0" do
-      expect(File).to receive(:exists?).with("/proc/self/status").and_return(true)
+      expect(File).to receive(:exist?).with("/proc/self/status").and_return(true)
       allow(File).to receive(:read).with("/proc/self/status").and_return("s_context: 0")
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("linux-vserver")
@@ -317,7 +317,7 @@ RHEV
     end
 
     it "sets Linux-VServer host if /proc/self/status contains VxID: 0" do
-      expect(File).to receive(:exists?).with("/proc/self/status").and_return(true)
+      expect(File).to receive(:exist?).with("/proc/self/status").and_return(true)
       allow(File).to receive(:read).with("/proc/self/status").and_return("VxID: 0")
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("linux-vserver")
@@ -326,7 +326,7 @@ RHEV
     end
 
     it "sets Linux-VServer host if /proc/self/status contains multiple space VxID:   0" do
-      expect(File).to receive(:exists?).with("/proc/self/status").and_return(true)
+      expect(File).to receive(:exist?).with("/proc/self/status").and_return(true)
       allow(File).to receive(:read).with("/proc/self/status").and_return("VxID:   0")
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("linux-vserver")
@@ -335,7 +335,7 @@ RHEV
     end
 
     it "sets Linux-VServer host if /proc/self/status contains tabbed VxID:\t0" do
-      expect(File).to receive(:exists?).with("/proc/self/status").and_return(true)
+      expect(File).to receive(:exist?).with("/proc/self/status").and_return(true)
       allow(File).to receive(:read).with("/proc/self/status").and_return("VxID:\t0")
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("linux-vserver")
@@ -344,7 +344,7 @@ RHEV
     end
 
     it "sets Linux-VServer guest if /proc/self/status contains s_context > 0" do
-      expect(File).to receive(:exists?).with("/proc/self/status").and_return(true)
+      expect(File).to receive(:exist?).with("/proc/self/status").and_return(true)
       allow(File).to receive(:read).with("/proc/self/status").and_return("s_context: 2")
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("linux-vserver")
@@ -353,7 +353,7 @@ RHEV
     end
 
     it "sets Linux-VServer guest if /proc/self/status contains VxID > 0" do
-      expect(File).to receive(:exists?).with("/proc/self/status").and_return(true)
+      expect(File).to receive(:exist?).with("/proc/self/status").and_return(true)
       allow(File).to receive(:read).with("/proc/self/status").and_return("VxID: 2")
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("linux-vserver")
@@ -362,7 +362,7 @@ RHEV
     end
 
     it "does not set virtualization if Linux-VServer isn't there" do
-      expect(File).to receive(:exists?).at_least(:once).and_return(false)
+      expect(File).to receive(:exist?).at_least(:once).and_return(false)
       plugin.run
       expect(plugin[:virtualization]).to eq({'systems' => {}})
     end
@@ -370,7 +370,7 @@ RHEV
 
   describe "when we are checking for openvz" do
     it "sets openvz host if /proc/bc/0 exists" do
-      expect(File).to receive(:exists?).with("/proc/bc/0").and_return(true)
+      expect(File).to receive(:exist?).with("/proc/bc/0").and_return(true)
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("openvz")
       expect(plugin[:virtualization][:role]).to eq("host")
@@ -378,8 +378,8 @@ RHEV
     end
 
     it "sets openvz guest if /proc/bc/0 does not exist and /proc/vz exists" do
-      expect(File).to receive(:exists?).with("/proc/bc/0").and_return(false)
-      expect(File).to receive(:exists?).with("/proc/vz").and_return(true)
+      expect(File).to receive(:exist?).with("/proc/bc/0").and_return(false)
+      expect(File).to receive(:exist?).with("/proc/vz").and_return(true)
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("openvz")
       expect(plugin[:virtualization][:role]).to eq("guest")
@@ -387,8 +387,8 @@ RHEV
     end
 
     it "does not set virtualization if openvz isn't there" do
-      expect(File).to receive(:exists?).with("/proc/bc/0").and_return(false)
-      expect(File).to receive(:exists?).with("/proc/vz").and_return(false)
+      expect(File).to receive(:exist?).with("/proc/bc/0").and_return(false)
+      expect(File).to receive(:exist?).with("/proc/vz").and_return(false)
       plugin.run
       expect(plugin[:virtualization]).to eq({'systems' => {}})
     end
@@ -400,7 +400,7 @@ RHEV
 0018	1ab84000	1f	            8001	               0	               0	               0	               0	               0	               0	              20	               0	               0	               0	               0	               0	               0	prl_tg
 0028	1af41000	17	            8201	        ee000000	               0	               0	               0	               0	               0	              40	            1000	               0	               0	               0	               0	               0	virtio-pci
       DEVICES
-      expect(File).to receive(:exists?).with("/proc/bus/pci/devices").and_return(true)
+      expect(File).to receive(:exist?).with("/proc/bus/pci/devices").and_return(true)
       allow(File).to receive(:read).with("/proc/bus/pci/devices").and_return(devices)
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("parallels")
@@ -413,7 +413,7 @@ RHEV
 0030	1af41000	a	            8401	        ee040000	               0	               0	               0	               0	               0	              40	            1000	               0	               0	               0	               0	               0	virtio-pci
 0050	10110022	0	               0	               0	               0	               0	               0	               0	               0	               0	               0	               0	               0	               0	               0	               0
       DEVICES
-      expect(File).to receive(:exists?).with("/proc/bus/pci/devices").and_return(true)
+      expect(File).to receive(:exist?).with("/proc/bus/pci/devices").and_return(true)
       allow(File).to receive(:read).with("/proc/bus/pci/devices").and_return(devices)
       plugin.run
       expect(plugin[:virtualization]).to eq({'systems' => {}})
@@ -432,7 +432,7 @@ RHEV
 2:cpu:/lxc/baa660ed81bc81d262ac6e19486142aeec5fce2043e2a173eb2505c6fbed89bc
 1:cpuset:/
 CGROUP
-      expect(File).to receive(:exists?).with("/proc/self/cgroup").and_return(true)
+      expect(File).to receive(:exist?).with("/proc/self/cgroup").and_return(true)
       allow(File).to receive(:read).with("/proc/self/cgroup").and_return(self_cgroup)
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("lxc")
@@ -451,7 +451,7 @@ CGROUP
 2:cpu:/lxc/vanilla
 1:cpuset:/lxc/vanilla
 CGROUP
-      expect(File).to receive(:exists?).with("/proc/self/cgroup").and_return(true)
+      expect(File).to receive(:exist?).with("/proc/self/cgroup").and_return(true)
       allow(File).to receive(:read).with("/proc/self/cgroup").and_return(self_cgroup)
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("lxc")
@@ -470,7 +470,7 @@ CGROUP
 2:cpu:/Charlie
 1:cpuset:/Charlie
 CGROUP
-      expect(File).to receive(:exists?).with("/proc/self/cgroup").and_return(true)
+      expect(File).to receive(:exist?).with("/proc/self/cgroup").and_return(true)
       allow(File).to receive(:read).with("/proc/self/cgroup").and_return(self_cgroup)
       plugin.run
       expect(plugin[:virtualization]).to eq({'systems' => {}})
@@ -488,7 +488,7 @@ CGROUP
 2:cpu:/
 1:cpuset:/
 CGROUP
-        expect(File).to receive(:exists?).with("/proc/self/cgroup").and_return(true)
+        expect(File).to receive(:exist?).with("/proc/self/cgroup").and_return(true)
         allow(File).to receive(:read).with("/proc/self/cgroup").and_return(self_cgroup)
       end
 
@@ -521,7 +521,7 @@ CGROUP
     end
 
     it "does not set virtualization if /proc/self/cgroup isn't there" do
-      expect(File).to receive(:exists?).with("/proc/self/cgroup").and_return(false)
+      expect(File).to receive(:exist?).with("/proc/self/cgroup").and_return(false)
       plugin.run
       expect(plugin[:virtualization]).to eq({'systems' => {}})
     end
@@ -539,7 +539,7 @@ CGROUP
 2:cpu:/docker/baa660ed81bc81d262ac6e19486142aeec5fce2043e2a173eb2505c6fbed89bc
 1:cpuset:/
 CGROUP
-      allow(File).to receive(:exists?).with("/proc/self/cgroup").and_return(true)
+      allow(File).to receive(:exist?).with("/proc/self/cgroup").and_return(true)
       allow(File).to receive(:read).with("/proc/self/cgroup").and_return(self_cgroup)
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("docker")
@@ -558,7 +558,7 @@ CGROUP
 2:cpu:/docker/vanilla
 1:cpuset:/docker/vanilla
 CGROUP
-      allow(File).to receive(:exists?).with("/proc/self/cgroup").and_return(true)
+      allow(File).to receive(:exist?).with("/proc/self/cgroup").and_return(true)
       allow(File).to receive(:read).with("/proc/self/cgroup").and_return(self_cgroup)
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("docker")
@@ -579,7 +579,7 @@ CGROUP
 2:cpu,cpuacct:/system.slice/docker-47341c91be8d491cb3b8a475ad5b4aef6e79bf728cbb351c384e4a6c410f172f.scope
 1:name=systemd:/system.slice/docker-47341c91be8d491cb3b8a475ad5b4aef6e79bf728cbb351c384e4a6c410f172f.scope
 CGROUP
-      allow(File).to receive(:exists?).with("/proc/self/cgroup").and_return(true)
+      allow(File).to receive(:exist?).with("/proc/self/cgroup").and_return(true)
       allow(File).to receive(:read).with("/proc/self/cgroup").and_return(self_cgroup)
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("docker")
@@ -598,7 +598,7 @@ CGROUP
 2:cpu:/Charlie
 1:cpuset:/Charlie
 CGROUP
-      allow(File).to receive(:exists?).with("/proc/self/cgroup").and_return(true)
+      allow(File).to receive(:exist?).with("/proc/self/cgroup").and_return(true)
       allow(File).to receive(:read).with("/proc/self/cgroup").and_return(self_cgroup)
       plugin.run
       expect(plugin[:virtualization]).to eq({'systems' => {}})
@@ -616,7 +616,7 @@ CGROUP
 2:cpu:/
 1:cpuset:/
 CGROUP
-        allow(File).to receive(:exists?).with("/proc/self/cgroup").and_return(true)
+        allow(File).to receive(:exist?).with("/proc/self/cgroup").and_return(true)
         allow(File).to receive(:read).with("/proc/self/cgroup").and_return(self_cgroup)
         plugin.run
         expect(plugin[:virtualization]).to eq({'systems' => {}})
@@ -643,13 +643,13 @@ CGROUP
     end
 
     it "does not set virtualization if /proc/self/cgroup isn't there" do
-      allow(File).to receive(:exists?).with("/proc/self/cgroup").and_return(false)
+      allow(File).to receive(:exist?).with("/proc/self/cgroup").and_return(false)
       plugin.run
       expect(plugin[:virtualization]).to eq({'systems' => {}})
     end
 
     it "sets virtualization if /.dockerenv exists" do
-      allow(File).to receive(:exists?).with("/.dockerenv").and_return(true)
+      allow(File).to receive(:exist?).with("/.dockerenv").and_return(true)
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("docker")
       expect(plugin[:virtualization][:role]).to eq("guest")
@@ -657,7 +657,7 @@ CGROUP
     end
 
     it "sets virtualization if /.dockerinit exists" do
-      allow(File).to receive(:exists?).with("/.dockerinit").and_return(true)
+      allow(File).to receive(:exist?).with("/.dockerinit").and_return(true)
       plugin.run
       expect(plugin[:virtualization][:system]).to eq("docker")
       expect(plugin[:virtualization][:role]).to eq("guest")
@@ -665,8 +665,8 @@ CGROUP
     end
 
     it "does not set virtualization if /.dockerenv or /.dockerinit does not exists" do
-      allow(File).to receive(:exists?).with("/.dockerenv").and_return(false)
-      allow(File).to receive(:exists?).with("/.dockerinit").and_return(false)
+      allow(File).to receive(:exist?).with("/.dockerenv").and_return(false)
+      allow(File).to receive(:exist?).with("/.dockerinit").and_return(false)
       plugin.run
       expect(plugin[:virtualization]).to eq({'systems' => {}})
     end