rubocop: Fix RSpec/BeNil

Author: bastelfreak

spec/custom_facts/core/execution/windows_spec.rb

@@ -44,7 +44,7 @@
       end
 
       it 'disallows other builtin windows commands' do
-        expect(executor.which('dir')).to be nil
+        expect(executor.which('dir')).to be_nil
       end
     end
 

spec/custom_facts/util/config_spec.rb

@@ -113,7 +113,7 @@
     it 'returns nil for windows' do
       allow(LegacyFacter::Util::Config).to receive(:windows?).and_return(true)
       LegacyFacter::Util::Config.setup_default_override_binary_dir
-      expect(LegacyFacter::Util::Config.override_binary_dir).to be nil
+      expect(LegacyFacter::Util::Config.override_binary_dir).to be_nil
     end
 
     it 'outputs new values when explicitly set' do

spec/facter/facter_spec.rb

@@ -230,7 +230,7 @@ def stub_no_fact
       it 'returns the custom fact' do
         stub_one_fact(Facter::ResolvedFact.new('nil_fact', nil, :custom, ''))
 
-        expect(Facter.value('nil_fact')).to be(nil)
+        expect(Facter.value('nil_fact')).to be_nil
       end
     end
   end

spec/facter/resolvers/base_resolver_spec.rb

@@ -77,7 +77,7 @@ def self.post_resolve(fact_name, _options)
       end
 
       it 'sets the fact to nil' do
-        expect(resolver.resolve(fact)).to be(nil)
+        expect(resolver.resolve(fact)).to be_nil
       end
     end
   end

spec/facter/resolvers/linux/containers_spec.rb

@@ -97,8 +97,8 @@
     let(:environ_output) { ['container=zone'] }
 
     it 'return nil' do
-      expect(containers_resolver.resolve(:vm)).to be(nil)
-      expect(containers_resolver.resolve(:hypervisor)).to be(nil)
+      expect(containers_resolver.resolve(:vm)).to be_nil
+      expect(containers_resolver.resolve(:hypervisor)).to be_nil
     end
   end
 
@@ -108,7 +108,7 @@
     let(:result) { nil }
 
     it 'return lxc for vm' do
-      expect(containers_resolver.resolve(:vm)).to be(nil)
+      expect(containers_resolver.resolve(:vm)).to be_nil
     end
 
     it 'return lxc info for hypervisor' do

spec/facter/resolvers/lsb_release_spec.rb

@@ -89,7 +89,7 @@
     it 'returns distro Distributor ID as nil' do
       result = Facter::Resolvers::LsbRelease.resolve(:distributor_id)
 
-      expect(result).to be(nil)
+      expect(result).to be_nil
     end
 
     it 'returns that lsb_release is not installed' do

spec/facter/resolvers/memory_spec.rb

@@ -75,19 +75,19 @@
       end
 
       it 'returns swap total as nil' do
-        expect(resolver.resolve(:swap_total)).to be(nil)
+        expect(resolver.resolve(:swap_total)).to be_nil
       end
 
       it 'returns swap available as nil' do
-        expect(resolver.resolve(:swap_free)).to be(nil)
+        expect(resolver.resolve(:swap_free)).to be_nil
       end
 
       it 'returns swap capacity as nil' do
-        expect(resolver.resolve(:swap_capacity)).to be(nil)
+        expect(resolver.resolve(:swap_capacity)).to be_nil
       end
 
       it 'returns swap usage as nil' do
-        expect(resolver.resolve(:swap_used_bytes)).to be(nil)
+        expect(resolver.resolve(:swap_used_bytes)).to be_nil
       end
 
       it 'returns system capacity' do

spec/facter/resolvers/solaris/dmi_sparc_spec.rb

@@ -46,15 +46,15 @@
     end
 
     it 'does not return manufacturer' do
-      expect(resolver.resolve(:manufacturer)).to be(nil)
+      expect(resolver.resolve(:manufacturer)).to be_nil
     end
 
     it 'does not return product_name' do
-      expect(resolver.resolve(:product_name)).to be(nil)
+      expect(resolver.resolve(:product_name)).to be_nil
     end
 
     it 'does not return serial_number' do
-      expect(resolver.resolve(:serial_number)).to be(nil)
+      expect(resolver.resolve(:serial_number)).to be_nil
     end
   end
 end

spec/facter/resolvers/solaris/solaris_release_spec.rb

@@ -41,15 +41,15 @@
     let(:output) { '' }
 
     it 'returns result nil if file is empty' do
-      expect(solaris_release.resolve(:full)).to be(nil)
+      expect(solaris_release.resolve(:full)).to be_nil
     end
   end
 
   context 'when the result from the os file release has no valid data' do
     let(:output) { 'test test' }
 
     it 'returns nil in case the file returns invalid data' do
-      expect(solaris_release.resolve(:full)).to be(nil)
+      expect(solaris_release.resolve(:full)).to be_nil
     end
   end
 end

spec/facter/resolvers/sw_vers_spec.rb

@@ -51,7 +51,7 @@
     end
 
     it 'does not return os ProductVersionExtra' do
-      expect(sw_vers.resolve(:productversionextra)).to be(nil)
+      expect(sw_vers.resolve(:productversionextra)).to be_nil
     end
   end
 end