Use :skip instead of :if/:unless

Author: pirj

spec/rspec/mocks/matchers/receive_spec.rb

@@ -54,13 +54,13 @@ module Mocks
           expect(receiver.foo).to eq(:curly)
         end
 
-        it 'does not support other matchers', :unless => options.include?(:allow_other_matchers) do
+        it 'does not support other matchers', :skip => options.include?(:allow_other_matchers) do
           expect {
             wrapped.to eq(3)
           }.to raise_error(UnsupportedMatcherError)
         end
 
-        it 'does support inherited matchers', :unless => options.include?(:allow_other_matchers) do
+        it 'does support inherited matchers', :skip => options.include?(:allow_other_matchers) do
           receive_foo = Class.new(RSpec::Mocks::Matchers::Receive).new(:foo, nil)
           wrapped.to receive_foo
           receiver.foo

spec/rspec/mocks/mutate_const_spec.rb

@@ -170,7 +170,7 @@ def change_const_value_to(value)
             expect(::Hash).to equal(top_level_hash)
           end
 
-          it 'does not affect the ability to access the top-level constant from nested contexts', :silence_warnings, :if => RUBY_VERSION < '2.5' do
+          it 'does not affect the ability to access the top-level constant from nested contexts', :silence_warnings, :skip => RUBY_VERSION >= '2.5' do
             top_level_hash = ::Hash
 
             hide_const("TestClass::Hash")