ohai 8.10.0 → 8.11.1

data/spec/unit/plugins/passwd_spec.rb

@@ -1,4 +1,4 @@
-require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper.rb')
+require File.expand_path(File.dirname(__FILE__) + "/../../spec_helper.rb")
 
 describe Ohai::System, "plugin etc", :unix_only do
   before(:each) do
@@ -9,42 +9,42 @@ describe Ohai::System, "plugin etc", :unix_only do
   GroupEntry = Struct.new(:name, :gid, :mem)
 
   it "should include a list of all users" do
-    expect(Etc).to receive(:passwd).and_yield(PasswdEntry.new("root", 1, 1, '/root', '/bin/zsh', 'BOFH')).
-      and_yield(PasswdEntry.new('www', 800, 800, '/var/www', '/bin/false', 'Serving the web since 1970'))
+    expect(Etc).to receive(:passwd).and_yield(PasswdEntry.new("root", 1, 1, "/root", "/bin/zsh", "BOFH")).
+      and_yield(PasswdEntry.new("www", 800, 800, "/var/www", "/bin/false", "Serving the web since 1970"))
     @plugin.run
-    expect(@plugin[:etc][:passwd]['root']).to eq(Mash.new(:shell => '/bin/zsh', :gecos => 'BOFH', :gid => 1, :uid => 1, :dir => '/root'))
-    expect(@plugin[:etc][:passwd]['www']).to eq(Mash.new(:shell => '/bin/false', :gecos => 'Serving the web since 1970', :gid => 800, :uid => 800, :dir => '/var/www'))
+    expect(@plugin[:etc][:passwd]["root"]).to eq(Mash.new(:shell => "/bin/zsh", :gecos => "BOFH", :gid => 1, :uid => 1, :dir => "/root"))
+    expect(@plugin[:etc][:passwd]["www"]).to eq(Mash.new(:shell => "/bin/false", :gecos => "Serving the web since 1970", :gid => 800, :uid => 800, :dir => "/var/www"))
   end
 
   it "should ignore duplicate users" do
-    expect(Etc).to receive(:passwd).and_yield(PasswdEntry.new("root", 1, 1, '/root', '/bin/zsh', 'BOFH')).
-      and_yield(PasswdEntry.new('root', 1, 1, '/', '/bin/false', 'I do not belong'))
+    expect(Etc).to receive(:passwd).and_yield(PasswdEntry.new("root", 1, 1, "/root", "/bin/zsh", "BOFH")).
+      and_yield(PasswdEntry.new("root", 1, 1, "/", "/bin/false", "I do not belong"))
     @plugin.run
-    expect(@plugin[:etc][:passwd]['root']).to eq(Mash.new(:shell => '/bin/zsh', :gecos => 'BOFH', :gid => 1, :uid => 1, :dir => '/root'))
+    expect(@plugin[:etc][:passwd]["root"]).to eq(Mash.new(:shell => "/bin/zsh", :gecos => "BOFH", :gid => 1, :uid => 1, :dir => "/root"))
   end
 
   it "should set the current user" do
-    expect(Process).to receive(:euid).and_return('31337')
-    expect(Etc).to receive(:getpwuid).and_return(PasswdEntry.new('chef', 31337, 31337, '/home/chef', '/bin/ksh', 'Julia Child'))
+    expect(Process).to receive(:euid).and_return("31337")
+    expect(Etc).to receive(:getpwuid).and_return(PasswdEntry.new("chef", 31337, 31337, "/home/chef", "/bin/ksh", "Julia Child"))
     @plugin.run
-    expect(@plugin[:current_user]).to eq('chef')
+    expect(@plugin[:current_user]).to eq("chef")
   end
 
   it "should set the available groups" do
-    expect(Etc).to receive(:group).and_yield(GroupEntry.new("admin", 100, ['root', 'chef'])).and_yield(GroupEntry.new('www', 800, ['www', 'deploy']))
+    expect(Etc).to receive(:group).and_yield(GroupEntry.new("admin", 100, %w{root chef})).and_yield(GroupEntry.new("www", 800, %w{www deploy}))
     @plugin.run
-    expect(@plugin[:etc][:group]['admin']).to eq(Mash.new(:gid => 100, :members => ['root', 'chef']))
-    expect(@plugin[:etc][:group]['www']).to eq(Mash.new(:gid => 800, :members => ['www', 'deploy']))
+    expect(@plugin[:etc][:group]["admin"]).to eq(Mash.new(:gid => 100, :members => %w{root chef}))
+    expect(@plugin[:etc][:group]["www"]).to eq(Mash.new(:gid => 800, :members => %w{www deploy}))
   end
 
   if "".respond_to?(:force_encoding)
     it "sets the encoding of strings to the default external encoding" do
-      fields = ["root", 1, 1, '/root', '/bin/zsh', 'BOFH']
-      fields.each {|f| f.force_encoding(Encoding::ASCII_8BIT) if f.respond_to?(:force_encoding) }
+      fields = ["root", 1, 1, "/root", "/bin/zsh", "BOFH"]
+      fields.each { |f| f.force_encoding(Encoding::ASCII_8BIT) if f.respond_to?(:force_encoding) }
       allow(Etc).to receive(:passwd).and_yield(PasswdEntry.new(*fields))
       @plugin.run
-      root = @plugin[:etc][:passwd]['root']
-      expect(root['gecos'].encoding).to eq(Encoding.default_external)
+      root = @plugin[:etc][:passwd]["root"]
+      expect(root["gecos"].encoding).to eq(Encoding.default_external)
     end
   end
 end

data/spec/unit/plugins/perl_spec.rb

@@ -17,7 +17,7 @@
 # limitations under the License.
 #
 
-require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper.rb')
+require File.expand_path(File.dirname(__FILE__) + "/../../spec_helper.rb")
 
 describe Ohai::System, "plugin perl" do
   before(:each) do

data/spec/unit/plugins/php_spec.rb

@@ -18,14 +18,14 @@
 # limitations under the License.
 #
 
-require File.expand_path(File.join(File.dirname(__FILE__), '..', '..', '/spec_helper.rb'))
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "/spec_helper.rb"))
 
 describe Ohai::System, "plugin php" do
 
   before(:each) do
     @plugin = get_plugin("php")
     @plugin[:languages] = Mash.new
-    @stdout =  <<-OUT
+    @stdout = <<-OUT
 PHP 5.1.6 (cli) (built: Jul 16 2008 19:52:52)
 Copyright (c) 1997-2006 The PHP Group
 Zend Engine v2.1.0, Copyright (c) 1998-2006 Zend Technologies
@@ -55,7 +55,7 @@ OUT
   end
 
   it "should parse builddate even if it's suhosin patched" do
-    @stdout =  <<-OUT
+    @stdout = <<-OUT
 PHP 5.3.27 with Suhosin-Patch (cli) (built: Aug 30 2013 04:30:30)
 Copyright (c) 1997-2013 The PHP Group
 Zend Engine v2.3.0, Copyright (c) 1998-2013 Zend Technologies
@@ -66,7 +66,7 @@ OUT
   end
 
   it "should not set zend_optcache_version if not compiled with opcache" do
-    @stdout =  <<-OUT
+    @stdout = <<-OUT
 PHP 5.1.6 (cli) (built: Jul 16 2008 19:52:52)
 Copyright (c) 1997-2006 The PHP Group
 Zend Engine v2.1.0, Copyright (c) 1998-2006 Zend Technologies

data/spec/unit/plugins/platform_spec.rb

@@ -16,15 +16,14 @@
 # limitations under the License.
 #
 
-
-require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper.rb')
+require File.expand_path(File.dirname(__FILE__) + "/../../spec_helper.rb")
 
 describe Ohai::System, "plugin platform" do
   before(:each) do
     @plugin = get_plugin("platform")
     allow(@plugin).to receive(:collect_os).and_return(:default)
-    @plugin[:os] = 'monkey'
-    @plugin[:os_version] = 'poop'
+    @plugin[:os] = "monkey"
+    @plugin[:os_version] = "poop"
   end
 
   it "should set the platform and platform family to the os if it was not set earlier" do
@@ -34,20 +33,20 @@ describe Ohai::System, "plugin platform" do
   end
 
   it "should not set the platform to the os if it was set earlier" do
-    @plugin[:platform] = 'lars'
+    @plugin[:platform] = "lars"
     @plugin.run
     expect(@plugin[:platform]).to eql("lars")
   end
 
   it "should set the platform_family to the platform if platform was set earlier but not platform_family" do
-    @plugin[:platform] = 'lars'
-    @plugin[:platform_family] = 'jack'
+    @plugin[:platform] = "lars"
+    @plugin[:platform_family] = "jack"
     @plugin.run
     expect(@plugin[:platform_family]).to eql("jack")
   end
 
   it "should not set the platform_family if the platform_family was set earlier." do
-    @plugin[:platform] = 'lars'
+    @plugin[:platform] = "lars"
     @plugin.run
     expect(@plugin[:platform]).to eql("lars")
     expect(@plugin[:platform_family]).to eql("lars")
@@ -59,7 +58,7 @@ describe Ohai::System, "plugin platform" do
   end
 
   it "should not set the platform to the os if it was set earlier" do
-    @plugin[:platform_version] = 'ulrich'
+    @plugin[:platform_version] = "ulrich"
     @plugin.run
     expect(@plugin[:platform_version]).to eql("ulrich")
   end

data/spec/unit/plugins/powershell_spec.rb

@@ -15,11 +15,11 @@
 # limitations under the License.
 #
 
-require File.expand_path(File.join(File.dirname(__FILE__), '..', '..', '/spec_helper.rb'))
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "/spec_helper.rb"))
 
 describe Ohai::System, "plugin powershell" do
   before do
-    stub_const('::RbConfig::CONFIG', {'host_os' => 'windows'})
+    stub_const("::RbConfig::CONFIG", { "host_os" => "windows" })
   end
 
   before(:each) do
@@ -50,7 +50,7 @@ END
     expect(@plugin.languages[:powershell][:serialization_version]).to eql("1.1.0.1")
     expect(@plugin.languages[:powershell][:clr_version]).to eql("4.0.30319.34014")
     expect(@plugin.languages[:powershell][:build_version]).to eql("6.3.9600.16394")
-    expect(@plugin.languages[:powershell][:compatible_versions]).to eql(['1.0', '2.0', '3.0', '4.0'])
+    expect(@plugin.languages[:powershell][:compatible_versions]).to eql(["1.0", "2.0", "3.0", "4.0"])
     expect(@plugin.languages[:powershell][:remoting_protocol_version]).to eql("2.2")
   end
 

data/spec/unit/plugins/python_spec.rb

@@ -17,7 +17,7 @@
 # limitations under the License.
 #
 
-require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper.rb')
+require File.expand_path(File.dirname(__FILE__) + "/../../spec_helper.rb")
 
 describe Ohai::System, "plugin python" do
   let(:stdout) { "2.5.2 (r252:60911, Jan  4 2009, 17:40:26)\n[GCC 4.3.2]\n" }

data/spec/unit/plugins/rackspace_spec.rb

@@ -15,53 +15,53 @@
 # limitations under the License.
 #
 
-require 'resolv'
+require "resolv"
 
-require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper.rb')
+require File.expand_path(File.dirname(__FILE__) + "/../../spec_helper.rb")
 
 describe Ohai::System, "plugin rackspace" do
   before(:each) do
     allow(Resolv).to receive(:getname).and_return("1.2.3.4")
     @plugin = get_plugin("rackspace")
     @plugin[:hostname] = "katie"
-    @plugin[:network] = {:interfaces => {:eth0 => {"addresses"=> {
-      "1.2.3.4"=> {
-        "broadcast"=> "67.23.20.255",
-        "netmask"=> "255.255.255.0",
-        "family"=> "inet"
+    @plugin[:network] = { :interfaces => { :eth0 => { "addresses" => {
+      "1.2.3.4" => {
+        "broadcast" => "67.23.20.255",
+        "netmask" => "255.255.255.0",
+        "family" => "inet",
       },
-      "2a00:1a48:7805:111:e875:efaf:ff08:75"=> {
-        "family"=> "inet6",
-        "prefixlen"=> "64",
-        "scope"=> "Global"
+      "2a00:1a48:7805:111:e875:efaf:ff08:75" => {
+        "family" => "inet6",
+        "prefixlen" => "64",
+        "scope" => "Global",
       },
-      "fe80::4240:95ff:fe47:6eed"=> {
-        "scope"=> "Link",
-        "prefixlen"=> "64",
-        "family"=> "inet6"
+      "fe80::4240:95ff:fe47:6eed" => {
+        "scope" => "Link",
+        "prefixlen" => "64",
+        "family" => "inet6",
       },
-      "40:40:95:47:6E:ED"=> {
-        "family"=> "lladdr"
-      }
-      }}
+      "40:40:95:47:6E:ED" => {
+        "family" => "lladdr"
+      },
+      } }
     }
   }
 
-  @plugin[:network][:interfaces][:eth1] = {:addresses => {
-    "fe80::4240:f5ff:feab:2836" => {
-      "scope"=> "Link",
-      "prefixlen"=> "64",
-      "family"=> "inet6"
-    },
-    "5.6.7.8"=> {
-      "broadcast"=> "10.176.191.255",
-      "netmask"=> "255.255.224.0",
-      "family"=> "inet"
-    },
-    "40:40:F5:AB:28:36" => {
-      "family"=> "lladdr"
-    }
-  }}
+    @plugin[:network][:interfaces][:eth1] = { :addresses => {
+      "fe80::4240:f5ff:feab:2836" => {
+        "scope" => "Link",
+        "prefixlen" => "64",
+        "family" => "inet6",
+      },
+      "5.6.7.8" => {
+        "broadcast" => "10.176.191.255",
+        "netmask" => "255.255.224.0",
+        "family" => "inet",
+      },
+      "40:40:F5:AB:28:36" => {
+        "family" => "lladdr"
+      },
+    } }
 
     # In olden days we could detect rackspace by a -rscloud suffix on the kernel
     # This is here to make #has_rackspace_kernel? fail until we remove that check
@@ -72,7 +72,7 @@ describe Ohai::System, "plugin rackspace" do
     allow(@plugin).to receive(:shell_out).and_return(mock_shell_out(1, "", ""))
   end
 
-  shared_examples_for "!rackspace"  do
+  shared_examples_for "!rackspace" do
     it "should NOT create rackspace" do
       @plugin.run
       expect(@plugin[:rackspace]).to be_nil
@@ -119,7 +119,7 @@ describe Ohai::System, "plugin rackspace" do
       expect(@plugin[:rackspace][:public_ipv4]).to eq("1.2.3.4")
       expect(@plugin[:rackspace][:local_ipv4]).to eq("5.6.7.8")
       expect(@plugin[:rackspace][:public_ipv6]).to eq("2a00:1a48:7805:111:e875:efaf:ff08:75")
-      expect(@plugin[:rackspace][:local_hostname]).to eq('katie')
+      expect(@plugin[:rackspace][:local_hostname]).to eq("katie")
       expect(@plugin[:rackspace][:public_hostname]).to eq("1.2.3.4")
     end
 
@@ -149,18 +149,18 @@ OUT
 
     before(:each) do
       allow(Resolv).to receive(:getname).and_raise(Resolv::ResolvError)
-      allow(File).to receive(:exist?).with('/etc/chef/ohai/hints/rackspace.json').and_return(true)
-      allow(File).to receive(:read).with('/etc/chef/ohai/hints/rackspace.json').and_return('')
+      allow(File).to receive(:exist?).with("/etc/chef/ohai/hints/rackspace.json").and_return(true)
+      allow(File).to receive(:read).with("/etc/chef/ohai/hints/rackspace.json").and_return("")
       allow(File).to receive(:exist?).with('C:\chef\ohai\hints/rackspace.json').and_return(true)
-      allow(File).to receive(:read).with('C:\chef\ohai\hints/rackspace.json').and_return('')
-      allow(File).to receive(:exist?).with('/etc/resolv.conf').and_return(true)
-      allow(File).to receive(:read).with('/etc/resolv.conf').and_return('')
+      allow(File).to receive(:read).with('C:\chef\ohai\hints/rackspace.json').and_return("")
+      allow(File).to receive(:exist?).with("/etc/resolv.conf").and_return(true)
+      allow(File).to receive(:read).with("/etc/resolv.conf").and_return("")
     end
 
-    describe 'with no public interfaces (empty eth0)' do
+    describe "with no public interfaces (empty eth0)" do
       before do
         # unset public (eth0) addresses
-        @plugin[:network][:interfaces][:eth0]['addresses'] = {}
+        @plugin[:network][:interfaces][:eth0]["addresses"] = {}
       end
 
       it "should have all required attributes" do
@@ -181,7 +181,7 @@ OUT
         @plugin.run
         expect(@plugin[:rackspace][:private_ip]).to eq("5.6.7.8")
         expect(@plugin[:rackspace][:local_ipv4]).to eq("5.6.7.8")
-        expect(@plugin[:rackspace][:local_hostname]).to eq('katie')
+        expect(@plugin[:rackspace][:local_hostname]).to eq("katie")
       end
     end
   end
@@ -190,7 +190,7 @@ OUT
     it_should_behave_like "!rackspace"
 
     before(:each) do
-      allow(File).to receive(:exist?).with('/etc/chef/ohai/hints/rackspace.json').and_return(false)
+      allow(File).to receive(:exist?).with("/etc/chef/ohai/hints/rackspace.json").and_return(false)
       allow(File).to receive(:exist?).with('C:\chef\ohai\hints/rackspace.json').and_return(false)
     end
   end
@@ -199,12 +199,12 @@ OUT
     it_should_behave_like "!rackspace"
 
     before(:each) do
-      allow(File).to receive(:exist?).with('/etc/chef/ohai/hints/ec2.json').and_return(true)
-      allow(File).to receive(:read).with('/etc/chef/ohai/hints/ec2.json').and_return('')
+      allow(File).to receive(:exist?).with("/etc/chef/ohai/hints/ec2.json").and_return(true)
+      allow(File).to receive(:read).with("/etc/chef/ohai/hints/ec2.json").and_return("")
       allow(File).to receive(:exist?).with('C:\chef\ohai\hints/ec2.json').and_return(true)
-      allow(File).to receive(:read).with('C:\chef\ohai\hints/ec2.json').and_return('')
+      allow(File).to receive(:read).with('C:\chef\ohai\hints/ec2.json').and_return("")
 
-      allow(File).to receive(:exist?).with('/etc/chef/ohai/hints/rackspace.json').and_return(false)
+      allow(File).to receive(:exist?).with("/etc/chef/ohai/hints/rackspace.json").and_return(false)
       allow(File).to receive(:exist?).with('C:\chef\ohai\hints/rackspace.json').and_return(false)
     end
   end
@@ -234,10 +234,10 @@ OUT
       stdout = '{"label": "public", "broadcast": "9.10.11.255", "ips": [{"ip": "9.10.11.12", "netmask": "255.255.255.0", "enabled": "1", "gateway": null}], "mac": "BC:76:4E:20:42:2B", "dns": ["69.20.0.164", "69.20.0.196"], "gateway": null}'
       allow(@plugin).to receive(:shell_out).with("xenstore-read vm-data/networking/BC764E20422B").and_return(mock_shell_out(0, stdout, "" ))
 
-      allow(File).to receive(:exist?).with('/etc/chef/ohai/hints/rackspace.json').and_return(true)
-      allow(File).to receive(:read).with('/etc/chef/ohai/hints/rackspace.json').and_return('')
+      allow(File).to receive(:exist?).with("/etc/chef/ohai/hints/rackspace.json").and_return(true)
+      allow(File).to receive(:read).with("/etc/chef/ohai/hints/rackspace.json").and_return("")
       allow(File).to receive(:exist?).with('C:\chef\ohai\hints/rackspace.json').and_return(true)
-      allow(File).to receive(:read).with('C:\chef\ohai\hints/rackspace.json').and_return('')
+      allow(File).to receive(:read).with('C:\chef\ohai\hints/rackspace.json').and_return("")
     end
 
     it "should not have private_networks object" do
@@ -248,30 +248,30 @@ OUT
 
   describe "has private networks" do
     before do
-      @plugin[:network][:interfaces][:eth2] = {:addresses => {
+      @plugin[:network][:interfaces][:eth2] = { :addresses => {
         "fe80::be76:4eff:fe20:422b" => {
-          "scope"=> "Link",
-          "prefixlen"=> "64",
-          "family"=> "inet6"
+          "scope" => "Link",
+          "prefixlen" => "64",
+          "family" => "inet6",
        },
-        "9.10.11.12"=> {
-          "broadcast"=> "9.10.11.255",
-          "netmask"=> "255.255.255.0",
-          "family"=> "inet"
+        "9.10.11.12" => {
+          "broadcast" => "9.10.11.255",
+          "netmask" => "255.255.255.0",
+          "family" => "inet",
         },
         "BC:76:4E:20:42:2B" => {
-          "family"=> "lladdr"
-        }
-      }}
+          "family" => "lladdr"
+        },
+      } }
       stdout = 'BC764E20422B = "{"label": "private-network"}"\n'
       allow(@plugin).to receive(:shell_out).with("xenstore-ls vm-data/networking").and_return(mock_shell_out(0, stdout, "" ))
       stdout = '{"label": "private-network", "broadcast": "9.10.11.255", "ips": [{"ip": "9.10.11.12", "netmask": "255.255.255.0", "enabled": "1", "gateway": null}], "mac": "BC:76:4E:20:42:2B", "dns": ["69.20.0.164", "69.20.0.196"], "gateway": null}'
       allow(@plugin).to receive(:shell_out).with("xenstore-read vm-data/networking/BC764E20422B").and_return(mock_shell_out(0, stdout, "" ))
 
-      allow(File).to receive(:exist?).with('/etc/chef/ohai/hints/rackspace.json').and_return(true)
-      allow(File).to receive(:read).with('/etc/chef/ohai/hints/rackspace.json').and_return('')
+      allow(File).to receive(:exist?).with("/etc/chef/ohai/hints/rackspace.json").and_return(true)
+      allow(File).to receive(:read).with("/etc/chef/ohai/hints/rackspace.json").and_return("")
       allow(File).to receive(:exist?).with('C:\chef\ohai\hints/rackspace.json').and_return(true)
-      allow(File).to receive(:read).with('C:\chef\ohai\hints/rackspace.json').and_return('')
+      allow(File).to receive(:read).with('C:\chef\ohai\hints/rackspace.json').and_return("")
     end
 
     it "should private_networks object" do
@@ -288,6 +288,4 @@ OUT
 
   end
 
-
-
 end

data/spec/unit/plugins/root_group_spec.rb

@@ -16,73 +16,73 @@
 # limitations under the License.
 #
 
-require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper.rb')
-require File.expand_path(File.dirname(__FILE__) + '/../../../lib/ohai/util/win32/group_helper.rb')
+require File.expand_path(File.dirname(__FILE__) + "/../../spec_helper.rb")
+require File.expand_path(File.dirname(__FILE__) + "/../../../lib/ohai/util/win32/group_helper.rb")
 
-describe Ohai::System, 'root_group' do
+describe Ohai::System, "root_group" do
   before(:each) do
     @plugin = get_plugin("root_group")
   end
 
-  describe 'unix platform', :unix_only do
+  describe "unix platform", :unix_only do
     before(:each) do
       # this is deeply intertwingled. unfortunately, the law of demeter
       # apparently didn't apply to this api. we're just trying to fake
       # Etc.getgrgid(Etc.getpwnam('root').gid).name
       @pwnam = Object.new
       allow(@pwnam).to receive(:gid).and_return(0)
-      allow(Etc).to receive(:getpwnam).with('root').and_return(@pwnam)
+      allow(Etc).to receive(:getpwnam).with("root").and_return(@pwnam)
       @grgid = Object.new
       allow(Etc).to receive(:getgrgid).and_return(@grgid)
     end
 
-    describe 'with wheel group' do
+    describe "with wheel group" do
       before(:each) do
-        allow(@grgid).to receive(:name).and_return('wheel')
+        allow(@grgid).to receive(:name).and_return("wheel")
       end
-      it 'should have a root_group of wheel' do
+      it "should have a root_group of wheel" do
         @plugin.run
-        expect(@plugin[:root_group]).to eq('wheel')
+        expect(@plugin[:root_group]).to eq("wheel")
       end
     end
 
-    describe 'with root group' do
+    describe "with root group" do
       before(:each) do
-        allow(@grgid).to receive(:name).and_return('root')
+        allow(@grgid).to receive(:name).and_return("root")
       end
-      it 'should have a root_group of root' do
+      it "should have a root_group of root" do
         @plugin.run
-        expect(@plugin[:root_group]).to eq('root')
+        expect(@plugin[:root_group]).to eq("root")
       end
     end
 
-    describe 'platform hpux with sys group' do
+    describe "platform hpux with sys group" do
       before(:each) do
         allow(@pwnam).to receive(:gid).and_return(3)
-        allow(@grgid).to receive(:name).and_return('sys')
+        allow(@grgid).to receive(:name).and_return("sys")
       end
-      it 'should have a root_group of sys' do
+      it "should have a root_group of sys" do
         @plugin.run
-        expect(@plugin[:root_group]).to eq('sys')
+        expect(@plugin[:root_group]).to eq("sys")
       end
     end
-    describe 'platform aix with system group' do
+    describe "platform aix with system group" do
       before(:each) do
-        allow(@grgid).to receive(:name).and_return('system')
+        allow(@grgid).to receive(:name).and_return("system")
       end
-      it 'should have a root_group of system' do
+      it "should have a root_group of system" do
         @plugin.run
-        expect(@plugin[:root_group]).to eq('system')
+        expect(@plugin[:root_group]).to eq("system")
       end
     end
   end
 
-  describe 'windows platform' do
-    it 'should return the group administrators' do
-      stub_const('::RbConfig::CONFIG', { 'host_os' => 'windows'} )
-      expect(Ohai::Util::Win32::GroupHelper).to receive(:windows_root_group_name).and_return('administrators')
+  describe "windows platform" do
+    it "should return the group administrators" do
+      stub_const("::RbConfig::CONFIG", { "host_os" => "windows" } )
+      expect(Ohai::Util::Win32::GroupHelper).to receive(:windows_root_group_name).and_return("administrators")
       @plugin.run
-      expect(@plugin[:root_group]).to eq('administrators')
+      expect(@plugin[:root_group]).to eq("administrators")
     end
   end
 end