File.exists? had been deprecated since Ruby 2.1 and has been deleted …

…in Ruby 3.2
carrierwaveuploader · Jan 8, 2023 · bdb0be0 · bdb0be0
1 parent ed8c518
commit bdb0be0
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/spec/storage/fog_helper.rb b/spec/storage/fog_helper.rb
@@ -224,7 +224,7 @@ class FogSpec#{fog_credentials[:provider]}Uploader < CarrierWave::Uploader::Base
       it "should create local file for processing" do
         @uploader.class_eval do
           def check_file
-            raise unless File.exists?(file.path)
+            raise unless File.exist?(file.path)
           end
           process :check_file
         end

diff --git a/spec/uploader/versions_spec.rb b/spec/uploader/versions_spec.rb
@@ -660,7 +660,7 @@ def upcase
           @uploader.store!(bork_file)
           FileUtils.rm([@uploader.small_thumb.path, @uploader.thumb.path])
           @uploader.recreate_versions!(:small_thumb)
-          expect(File.exists?(public_path(@uploader.thumb.to_s))).to be true
+          expect(File.exist?(public_path(@uploader.thumb.to_s))).to be true
           expect(small_thumb_contents).to eq(thumb_contents)
           expect(small_thumb_contents).not_to eq(original_contents)
         end
@@ -675,22 +675,22 @@ def upcase
           @uploader.store!(bork_file)
           FileUtils.rm([@uploader.small_thumb.path, @uploader.thumb.path])
           @uploader.recreate_versions!(:small_thumb, :thumb)
-          expect(File.exists?(public_path(@uploader.small_thumb.to_s))).to be true
-          expect(File.exists?(public_path(@uploader.thumb.to_s))).to be true
+          expect(File.exist?(public_path(@uploader.small_thumb.to_s))).to be true
+          expect(File.exist?(public_path(@uploader.thumb.to_s))).to be true
 
           # doesn't depend on arguments order
           FileUtils.rm([@uploader.small_thumb.path, @uploader.thumb.path])
           @uploader.recreate_versions!(:thumb, :small_thumb)
-          expect(File.exists?(public_path(@uploader.small_thumb.to_s))).to be true
-          expect(File.exists?(public_path(@uploader.thumb.to_s))).to be true
+          expect(File.exist?(public_path(@uploader.small_thumb.to_s))).to be true
+          expect(File.exist?(public_path(@uploader.thumb.to_s))).to be true
         end
 
         it "doesn't touch other versions" do
           @uploader_class.version(:another)
           @uploader.store!(bork_file)
           FileUtils.rm(@uploader.another.path)
           @uploader.recreate_versions!(:small_thumb)
-          expect(File.exists?(public_path(@uploader.another.to_s))).to be false
+          expect(File.exist?(public_path(@uploader.another.to_s))).to be false
         end
       end
     end