Skip to content

Commit 3100e58

Browse files
kyoshidajpkyoshidajp
committedDec 6, 2022
Replace File.exists? with File.exist?
·
v1.5.0v1.4.0
1 parent cf305ed commit 3100e58

File tree

1 file changed

+46
-46
lines changed

1 file changed

+46
-46
lines changed
 

‎spec/mongoid_spec.rb

+46-46
Original file line numberDiff line numberDiff line change
@@ -420,7 +420,7 @@ def extension_white_list
420420
@doc = @class.new
421421
@doc.image = stub_file('old.jpeg')
422422
expect(@doc.save).to be_truthy
423-
expect(File.exists?(public_path('uploads/old.jpeg'))).to be_truthy
423+
expect(File.exist?(public_path('uploads/old.jpeg'))).to be_truthy
424424
end
425425

426426
after do
@@ -432,29 +432,29 @@ def extension_white_list
432432
it "should remove old file if old file had a different path" do
433433
@doc.image = stub_file('new.jpeg')
434434
expect(@doc.save).to be_truthy
435-
expect(File.exists?(public_path('uploads/new.jpeg'))).to be_truthy
436-
expect(File.exists?(public_path('uploads/old.jpeg'))).to be_falsey
435+
expect(File.exist?(public_path('uploads/new.jpeg'))).to be_truthy
436+
expect(File.exist?(public_path('uploads/old.jpeg'))).to be_falsey
437437
end
438438

439439
it "should not remove old file if old file had a different path but config is false" do
440440
@doc.image.class.remove_previously_stored_files_after_update = false
441441
@doc.image = stub_file('new.jpeg')
442442
expect(@doc.save).to be_truthy
443-
expect(File.exists?(public_path('uploads/new.jpeg'))).to be_truthy
444-
expect(File.exists?(public_path('uploads/old.jpeg'))).to be_truthy
443+
expect(File.exist?(public_path('uploads/new.jpeg'))).to be_truthy
444+
expect(File.exist?(public_path('uploads/old.jpeg'))).to be_truthy
445445
end
446446

447447
it "should not remove file if old file had the same path" do
448448
@doc.image = stub_file('old.jpeg')
449449
expect(@doc.save).to be_truthy
450-
expect(File.exists?(public_path('uploads/old.jpeg'))).to be_truthy
450+
expect(File.exist?(public_path('uploads/old.jpeg'))).to be_truthy
451451
end
452452

453453
it "should not remove file if validations fail on save" do
454454
@class.validate { |r| r.errors.add :textfile, "FAIL!" }
455455
@doc.image = stub_file('new.jpeg')
456456
expect(@doc.save).to be_falsey
457-
expect(File.exists?(public_path('uploads/old.jpeg'))).to be_truthy
457+
expect(File.exist?(public_path('uploads/old.jpeg'))).to be_truthy
458458
end
459459
end
460460

@@ -469,52 +469,52 @@ def filename
469469
@doc.image = stub_file('old.jpeg')
470470
@doc.foo = "test"
471471
expect(@doc.save).to be_truthy
472-
expect(File.exists?(public_path('uploads/test.jpeg'))).to be_truthy
472+
expect(File.exist?(public_path('uploads/test.jpeg'))).to be_truthy
473473
expect(@doc.image.read).to eq "this is stuff"
474474
end
475475

476476
it "should not remove file if old file had the same dynamic path" do
477477
@doc.image = stub_file('test.jpeg')
478478
expect(@doc.save).to be_truthy
479-
expect(File.exists?(public_path('uploads/test.jpeg'))).to be_truthy
479+
expect(File.exist?(public_path('uploads/test.jpeg'))).to be_truthy
480480
end
481481

482482
it "should remove old file if old file had a different dynamic path" do
483483
@doc.foo = "new"
484484
@doc.image = stub_file('test.jpeg')
485485
expect(@doc.save).to be_truthy
486-
expect(File.exists?(public_path('uploads/new.jpeg'))).to be_truthy
487-
expect(File.exists?(public_path('uploads/test.jpeg'))).to be_falsey
486+
expect(File.exist?(public_path('uploads/new.jpeg'))).to be_truthy
487+
expect(File.exist?(public_path('uploads/test.jpeg'))).to be_falsey
488488
end
489489
end
490490

491491
shared_examples "embedded documents" do
492492
it "should remove old file if old file had a different path" do
493493
@embedded_doc.image = stub_file('new.jpeg')
494494
expect(@embedded_doc.save).to be_truthy
495-
expect(File.exists?(public_path('uploads/new.jpeg'))).to be_truthy
496-
expect(File.exists?(public_path('uploads/old.jpeg'))).to be_falsey
495+
expect(File.exist?(public_path('uploads/new.jpeg'))).to be_truthy
496+
expect(File.exist?(public_path('uploads/old.jpeg'))).to be_falsey
497497
end
498498

499499
it "should not remove old file if old file had a different path but config is false" do
500500
@embedded_doc.image.class.remove_previously_stored_files_after_update = false
501501
@embedded_doc.image = stub_file('new.jpeg')
502502
expect(@embedded_doc.save).to be_truthy
503-
expect(File.exists?(public_path('uploads/new.jpeg'))).to be_truthy
504-
expect(File.exists?(public_path('uploads/old.jpeg'))).to be_truthy
503+
expect(File.exist?(public_path('uploads/new.jpeg'))).to be_truthy
504+
expect(File.exist?(public_path('uploads/old.jpeg'))).to be_truthy
505505
end
506506

507507
it "should not remove file if old file had the same path" do
508508
@embedded_doc.image = stub_file('old.jpeg')
509509
expect(@embedded_doc.save).to be_truthy
510-
expect(File.exists?(public_path('uploads/old.jpeg'))).to be_truthy
510+
expect(File.exist?(public_path('uploads/old.jpeg'))).to be_truthy
511511
end
512512

513513
it "should not remove file if validations fail on save" do
514514
@embedded_doc_class.validate { |r| r.errors.add :textfile, "FAIL!" }
515515
@embedded_doc.image = stub_file('new.jpeg')
516516
expect(@embedded_doc.save).to be_falsey
517-
expect(File.exists?(public_path('uploads/old.jpeg'))).to be_truthy
517+
expect(File.exist?(public_path('uploads/old.jpeg'))).to be_truthy
518518
end
519519

520520
it "should not touch parent's dirty attributes" do
@@ -530,29 +530,29 @@ def filename
530530
it "should remove old file if old file had a different path" do
531531
@double_embedded_doc.image = stub_file('new.jpeg')
532532
expect(@double_embedded_doc.save).to be_truthy
533-
expect(File.exists?(public_path('uploads/new.jpeg'))).to be_truthy
534-
expect(File.exists?(public_path('uploads/old.jpeg'))).to be_falsey
533+
expect(File.exist?(public_path('uploads/new.jpeg'))).to be_truthy
534+
expect(File.exist?(public_path('uploads/old.jpeg'))).to be_falsey
535535
end
536536

537537
it "should not remove old file if old file had a different path but config is false" do
538538
@double_embedded_doc.image.class.remove_previously_stored_files_after_update = false
539539
@double_embedded_doc.image = stub_file('new.jpeg')
540540
expect(@double_embedded_doc.save).to be_truthy
541-
expect(File.exists?(public_path('uploads/new.jpeg'))).to be_truthy
542-
expect(File.exists?(public_path('uploads/old.jpeg'))).to be_truthy
541+
expect(File.exist?(public_path('uploads/new.jpeg'))).to be_truthy
542+
expect(File.exist?(public_path('uploads/old.jpeg'))).to be_truthy
543543
end
544544

545545
it "should not remove file if old file had the same path" do
546546
@double_embedded_doc.image = stub_file('old.jpeg')
547547
expect(@double_embedded_doc.save).to be_truthy
548-
expect(File.exists?(public_path('uploads/old.jpeg'))).to be_truthy
548+
expect(File.exist?(public_path('uploads/old.jpeg'))).to be_truthy
549549
end
550550

551551
it "should not remove file if validations fail on save" do
552552
@double_embedded_doc_class.validate { |r| r.errors.add :textfile, "FAIL!" }
553553
@double_embedded_doc.image = stub_file('new.jpeg')
554554
expect(@double_embedded_doc.save).to be_falsey
555-
expect(File.exists?(public_path('uploads/old.jpeg'))).to be_truthy
555+
expect(File.exist?(public_path('uploads/old.jpeg'))).to be_truthy
556556
end
557557

558558
end
@@ -763,8 +763,8 @@ def filename
763763
@doc = @class.new
764764
@doc.image = stub_file('old.jpeg')
765765
expect(@doc.save).to be_truthy
766-
expect(File.exists?(public_path('uploads/old.jpeg'))).to be_truthy
767-
expect(File.exists?(public_path('uploads/thumb_old.jpeg'))).to be_truthy
766+
expect(File.exist?(public_path('uploads/old.jpeg'))).to be_truthy
767+
expect(File.exist?(public_path('uploads/thumb_old.jpeg'))).to be_truthy
768768
end
769769

770770
after do
@@ -774,17 +774,17 @@ def filename
774774
it "should remove old file if old file had a different path" do
775775
@doc.image = stub_file('new.jpeg')
776776
expect(@doc.save).to be_truthy
777-
expect(File.exists?(public_path('uploads/new.jpeg'))).to be_truthy
778-
expect(File.exists?(public_path('uploads/thumb_new.jpeg'))).to be_truthy
779-
expect(File.exists?(public_path('uploads/old.jpeg'))).to be_falsey
780-
expect(File.exists?(public_path('uploads/thumb_old.jpeg'))).to be_falsey
777+
expect(File.exist?(public_path('uploads/new.jpeg'))).to be_truthy
778+
expect(File.exist?(public_path('uploads/thumb_new.jpeg'))).to be_truthy
779+
expect(File.exist?(public_path('uploads/old.jpeg'))).to be_falsey
780+
expect(File.exist?(public_path('uploads/thumb_old.jpeg'))).to be_falsey
781781
end
782782

783783
it "should not remove file if old file had the same path" do
784784
@doc.image = stub_file('old.jpeg')
785785
expect(@doc.save).to be_truthy
786-
expect(File.exists?(public_path('uploads/old.jpeg'))).to be_truthy
787-
expect(File.exists?(public_path('uploads/thumb_old.jpeg'))).to be_truthy
786+
expect(File.exist?(public_path('uploads/old.jpeg'))).to be_truthy
787+
expect(File.exist?(public_path('uploads/thumb_old.jpeg'))).to be_truthy
788788
end
789789
end
790790

@@ -799,8 +799,8 @@ def filename
799799
@doc.image = stub_file('old.jpeg')
800800
@doc.textfile = stub_file('old.txt')
801801
expect(@doc.save).to be_truthy
802-
expect(File.exists?(public_path('uploads/old.jpeg'))).to be_truthy
803-
expect(File.exists?(public_path('uploads/old.txt'))).to be_truthy
802+
expect(File.exist?(public_path('uploads/old.jpeg'))).to be_truthy
803+
expect(File.exist?(public_path('uploads/old.txt'))).to be_truthy
804804
end
805805

806806
after do
@@ -811,27 +811,27 @@ def filename
811811
@doc.image = stub_file('new.jpeg')
812812
@doc.textfile = stub_file('new.txt')
813813
expect(@doc.save).to be_truthy
814-
expect(File.exists?(public_path('uploads/new.jpeg'))).to be_truthy
815-
expect(File.exists?(public_path('uploads/old.jpeg'))).to be_falsey
816-
expect(File.exists?(public_path('uploads/new.txt'))).to be_truthy
817-
expect(File.exists?(public_path('uploads/old.txt'))).to be_falsey
814+
expect(File.exist?(public_path('uploads/new.jpeg'))).to be_truthy
815+
expect(File.exist?(public_path('uploads/old.jpeg'))).to be_falsey
816+
expect(File.exist?(public_path('uploads/new.txt'))).to be_truthy
817+
expect(File.exist?(public_path('uploads/old.txt'))).to be_falsey
818818
end
819819

820820
it "should remove old file1 but not file2 if old file1 had a different path but old file2 has the same path" do
821821
@doc.image = stub_file('new.jpeg')
822822
@doc.textfile = stub_file('old.txt')
823823
expect(@doc.save).to be_truthy
824-
expect(File.exists?(public_path('uploads/new.jpeg'))).to be_truthy
825-
expect(File.exists?(public_path('uploads/old.jpeg'))).to be_falsey
826-
expect(File.exists?(public_path('uploads/old.txt'))).to be_truthy
824+
expect(File.exist?(public_path('uploads/new.jpeg'))).to be_truthy
825+
expect(File.exist?(public_path('uploads/old.jpeg'))).to be_falsey
826+
expect(File.exist?(public_path('uploads/old.txt'))).to be_truthy
827827
end
828828

829829
it "should not remove file1 or file2 if file1 and file2 have the same paths" do
830830
@doc.image = stub_file('old.jpeg')
831831
@doc.textfile = stub_file('old.txt')
832832
expect(@doc.save).to be_truthy
833-
expect(File.exists?(public_path('uploads/old.jpeg'))).to be_truthy
834-
expect(File.exists?(public_path('uploads/old.txt'))).to be_truthy
833+
expect(File.exist?(public_path('uploads/old.jpeg'))).to be_truthy
834+
expect(File.exist?(public_path('uploads/old.txt'))).to be_truthy
835835
end
836836
end
837837

@@ -844,7 +844,7 @@ def filename
844844
@doc = @class.new
845845
@doc.avatar = stub_file('old.jpeg')
846846
expect(@doc.save).to be_truthy
847-
expect(File.exists?(public_path('uploads/old.jpeg'))).to be_truthy
847+
expect(File.exist?(public_path('uploads/old.jpeg'))).to be_truthy
848848
end
849849

850850
after do
@@ -854,14 +854,14 @@ def filename
854854
it "should remove old file if old file had a different path" do
855855
@doc.avatar = stub_file('new.jpeg')
856856
expect(@doc.save).to be_truthy
857-
expect(File.exists?(public_path('uploads/new.jpeg'))).to be_truthy
858-
expect(File.exists?(public_path('uploads/old.jpeg'))).to be_falsey
857+
expect(File.exist?(public_path('uploads/new.jpeg'))).to be_truthy
858+
expect(File.exist?(public_path('uploads/old.jpeg'))).to be_falsey
859859
end
860860

861861
it "should not remove file if old file had the same path" do
862862
@doc.avatar = stub_file('old.jpeg')
863863
expect(@doc.save).to be_truthy
864-
expect(File.exists?(public_path('uploads/old.jpeg'))).to be_truthy
864+
expect(File.exist?(public_path('uploads/old.jpeg'))).to be_truthy
865865
end
866866
end
867867

0 commit comments

Comments
 (0)
Please sign in to comment.