Merge pull request #2701 from mhashizume/maint/main/revert-confine

Revert "Merge pull request #2699 from mhashizume/FACT-3156/main/confi…

lib/facter/custom_facts/util/confine.rb

@@ -35,7 +35,7 @@ def to_s
       end
 
       # Evaluate the fact, returning true or false.
-      # if we have a block parameter then we only evaluate that instead
+      # if we have a block paramter then we only evaluate that instead
       def true?
         if @block && !@fact
           begin
@@ -54,11 +54,9 @@ def true?
 
         return false if value.nil?
 
-        # We call the block with both the downcased and raw fact value for
-        # backwards-compatibility.
         if @block
           begin
-            return !@block.call(value).nil? || !@block.call(fact.value).nil?
+            return !!@block.call(value)
           rescue StandardError => e
             log.debug "Confine raised #{e.class} #{e}"
             return false

spec/custom_facts/util/confine_spec.rb

@@ -126,12 +126,6 @@ def confined(fact_value, *confines)
       expect(confine.true?).to be true
     end
 
-    it 'accepts and evaluate a block argument against the fact while respecting case' do
-      allow(fact).to receive(:value).and_return 'Foo'
-      confine = LegacyFacter::Util::Confine.new(:yay) { |f| f == 'Foo' }
-      expect(confine.true?).to be true
-    end
-
     it 'returns false if the block raises a StandardError when checking a fact' do
       allow(fact).to receive(:value).and_return 'foo'
       confine = LegacyFacter::Util::Confine.new(:yay) { |_f| raise StandardError }