spec/fileuploads_spec.rb in rails-uploader-0.5.8 vs spec/fileuploads_spec.rb in rails-uploader-0.5.9

@@ -1,54 +1,52 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
+require 'rails_helper'
 
-describe Uploader::Fileuploads do
-  before(:all) do
-    @picture = FactoryGirl.create(:picture, assetable_type: 'Article')
-  end
+RSpec.describe Uploader::Fileuploads do
+  let(:picture) { create(:picture, assetable_type: 'Article') }
 
   it 'should be a Module' do
-    Uploader::Fileuploads.should be_a(Module)
+    # raise FactoryBot.find_definitions.inspect
+    require "/Users/mario/www/gems/rails-uploader/spec/factories/factories.rb"
+    expect(Uploader::Fileuploads).to be_a(Module)
   end
 
   context 'instance methods' do
-    before(:each) do
-      @article = FactoryGirl.build(:article)
-    end
+    let(:article) { build(:article) }
 
     it 'should return asset class' do
-      @article.fileupload_klass('picture').should == Picture
+      expect(article.fileupload_klass('picture')).to eq Picture
     end
 
     it 'should find asset by guid' do
-      @picture.update_column(:guid, @article.fileupload_guid)
+      picture.update_column(:guid, article.fileupload_guid)
 
-      asset = @article.fileupload_asset('picture')
-      asset.should == @picture
+      asset = article.fileupload_asset('picture')
+      expect(asset).to eq picture
     end
 
     it 'should generate guid' do
-      @article.fileupload_guid.should_not be_blank
+      expect(article.fileupload_guid).not_to be_blank
     end
 
     it 'should change guid' do
-      @article.fileupload_guid = 'other guid'
-      @article.fileupload_changed?.should be_truthy
-      @article.fileupload_guid.should == 'other guid'
+      article.fileupload_guid = 'other guid'
+      expect(article.fileupload_changed?).to be_truthy
+      expect(article.fileupload_guid).to eq 'other guid'
     end
 
     it 'should not multiplay upload' do
-      @article.fileupload_multiple?('picture').should be_falsey
+      expect(article.fileupload_multiple?('picture')).to be_falsey
     end
 
     it 'should find uploaded asset or build new record' do
-      picture = @article.fileupload_asset(:picture)
-      picture.should_not be_nil
-      picture.should be_new_record
+      picture = article.fileupload_asset(:picture)
+      expect(picture).not_to be_nil
+      expect(picture).to be_new_record
     end
 
     it 'must get fileupload params' do
-      @article.fileupload_params(:picture).should_not be nil
+      expect(article.fileupload_params(:picture)).not_to be nil
     end
   end
 end