enju_biblio 0.2.3 → 0.2.4

data/spec/controllers/form_of_works_controller_spec.rb

@@ -26,7 +26,7 @@ describe FormOfWorksController do
   # FormOfWork. As you add validations to FormOfWork, be sure to
   # update the return value of this method accordingly.
   def valid_attributes
-    FactoryGirl.attributes_for(:form_of_work)
+    FactoryBot.attributes_for(:form_of_work)
   end
 
   describe 'GET index' do

data/spec/controllers/frequencies_controller_spec.rb

@@ -26,7 +26,7 @@ describe FrequenciesController do
   # Frequency. As you add validations to Frequency, be sure to
   # update the return value of this method accordingly.
   def valid_attributes
-    FactoryGirl.attributes_for(:frequency)
+    FactoryBot.attributes_for(:frequency)
   end
 
   describe 'GET index' do
@@ -59,8 +59,8 @@ describe FrequenciesController do
       assigns(:frequency).should eq(frequency)
     end
     it 'assigns the frequency even if it associates manifestation(s)' do
-      frequency = FactoryGirl.create(:frequency)
-      manifestation = FactoryGirl.create(:manifestation, frequency_id: frequency.id)
+      frequency = FactoryBot.create(:frequency)
+      manifestation = FactoryBot.create(:manifestation, frequency_id: frequency.id)
       get :edit, id: frequency.id
       expect(assigns(:frequency)).to eq frequency
       expect(response).to be_success

data/spec/controllers/identifier_types_controller_spec.rb

@@ -26,7 +26,7 @@ describe IdentifierTypesController do
   # IdentifierType. As you add validations to IdentifierType, be sure to
   # update the return value of this method accordingly.
   def valid_attributes
-    FactoryGirl.attributes_for(:identifier_type)
+    FactoryBot.attributes_for(:identifier_type)
   end
 
   describe 'GET index' do

data/spec/controllers/import_requests_controller_spec.rb

@@ -124,7 +124,7 @@ describe ImportRequestsController do
       login_fixture_admin
 
       it 'assigns the requested import_request as @import_request' do
-        import_request = FactoryGirl.create(:import_request, isbn: '9784797350999')
+        import_request = FactoryBot.create(:import_request, isbn: '9784797350999')
         get :edit, id: import_request.id
         expect(assigns(:import_request)).to eq(import_request)
       end
@@ -134,7 +134,7 @@ describe ImportRequestsController do
       login_fixture_librarian
 
       it 'assigns the requested import_request as @import_request' do
-        import_request = FactoryGirl.create(:import_request, isbn: '9784797350999')
+        import_request = FactoryBot.create(:import_request, isbn: '9784797350999')
         get :edit, id: import_request.id
         expect(assigns(:import_request)).to eq(import_request)
       end
@@ -144,7 +144,7 @@ describe ImportRequestsController do
       login_fixture_user
 
       it 'assigns the requested import_request as @import_request' do
-        import_request = FactoryGirl.create(:import_request, isbn: '9784797350999')
+        import_request = FactoryBot.create(:import_request, isbn: '9784797350999')
         get :edit, id: import_request.id
         expect(response).to be_forbidden
       end
@@ -152,7 +152,7 @@ describe ImportRequestsController do
 
     describe 'When not logged in' do
       it 'should not assign the requested import_request as @import_request' do
-        import_request = FactoryGirl.create(:import_request, isbn: '9784797350999')
+        import_request = FactoryBot.create(:import_request, isbn: '9784797350999')
         get :edit, id: import_request.id
         expect(response).to redirect_to(new_user_session_url)
       end
@@ -192,7 +192,7 @@ describe ImportRequestsController do
         end
       end
 
-      describe 'with isbn which is already imported' do
+      describe 'with isbn which is already imported', vcr: true do
         it 'assigns a newly created import_request as @import_request' do
           post :create, import_request: { isbn: manifestations(:manifestation_00001).identifier_contents(:isbn).first }
           expect(assigns(:import_request)).to be_valid

data/spec/controllers/items_controller_spec.rb

@@ -5,7 +5,7 @@ describe ItemsController do
   fixtures :all
 
   def valid_attributes
-    FactoryGirl.attributes_for(:item)
+    FactoryBot.attributes_for(:item)
   end
 
   describe 'GET index', solr: true do
@@ -107,7 +107,7 @@ describe ItemsController do
 
   describe 'GET show' do
     before(:each) do
-      @item = FactoryGirl.create(:item)
+      @item = FactoryBot.create(:item)
     end
 
     describe 'When logged in as Administrator' do
@@ -154,7 +154,7 @@ describe ItemsController do
 
   describe 'GET new' do
     before(:each) do
-      @manifestation = FactoryGirl.create(:manifestation)
+      @manifestation = FactoryBot.create(:manifestation)
     end
 
     describe 'When logged in as Administrator' do
@@ -181,7 +181,7 @@ describe ItemsController do
       end
 
       it 'should not get new item for series_master' do
-        manifestation_serial = FactoryGirl.create(:manifestation_serial)
+        manifestation_serial = FactoryBot.create(:manifestation_serial)
         get :new, manifestation_id: manifestation_serial.id
         expect(response).to redirect_to(manifestations_url(parent_id: manifestation_serial.id))
       end
@@ -221,7 +221,7 @@ describe ItemsController do
       login_fixture_admin
 
       it 'assigns the requested item as @item' do
-        item = FactoryGirl.create(:item)
+        item = FactoryBot.create(:item)
         get :edit, id: item.id
         expect(assigns(:item)).to eq(item)
       end
@@ -238,7 +238,7 @@ describe ItemsController do
       login_fixture_librarian
 
       it 'assigns the requested item as @item' do
-        item = FactoryGirl.create(:item)
+        item = FactoryBot.create(:item)
         get :edit, id: item.id
         expect(assigns(:item)).to eq(item)
       end
@@ -248,7 +248,7 @@ describe ItemsController do
       login_fixture_user
 
       it 'assigns the requested item as @item' do
-        item = FactoryGirl.create(:item)
+        item = FactoryBot.create(:item)
         get :edit, id: item.id
         expect(response).to be_forbidden
       end
@@ -256,7 +256,7 @@ describe ItemsController do
 
     describe 'When not logged in' do
       it 'should not assign the requested item as @item' do
-        item = FactoryGirl.create(:item)
+        item = FactoryBot.create(:item)
         get :edit, id: item.id
         expect(response).to redirect_to(new_user_session_url)
       end
@@ -265,8 +265,8 @@ describe ItemsController do
 
   describe 'POST create' do
     before(:each) do
-      manifestation = FactoryGirl.create(:manifestation)
-      @attrs = FactoryGirl.attributes_for(:item, manifestation_id: manifestation.id)
+      manifestation = FactoryBot.create(:manifestation)
+      @attrs = FactoryBot.attributes_for(:item, manifestation_id: manifestation.id)
       @invalid_attrs = { item_identifier: '無効なID', manifestation_id: manifestation.id }
     end
 
@@ -357,12 +357,12 @@ describe ItemsController do
       end
 
       it "should create another item with already retained" do
-        reserve = FactoryGirl.create(:reserve)
+        reserve = FactoryBot.create(:reserve)
         reserve.transition_to!(:requested)
-        post :create, :item => FactoryGirl.attributes_for(:item, manifestation_id: reserve.manifestation.id)
+        post :create, :item => FactoryBot.attributes_for(:item, manifestation_id: reserve.manifestation.id)
         expect(assigns(:item)).to be_valid
         expect(response).to redirect_to item_url(assigns(:item))
-        post :create, :item => FactoryGirl.attributes_for(:item, manifestation_id: reserve.manifestation.id)
+        post :create, :item => FactoryBot.attributes_for(:item, manifestation_id: reserve.manifestation.id)
         expect(assigns(:item)).to be_valid
         expect(response).to redirect_to item_url(assigns(:item))
       end
@@ -425,8 +425,8 @@ describe ItemsController do
 
   describe 'PUT update' do
     before(:each) do
-      @item = FactoryGirl.create(:item)
-      @attrs = FactoryGirl.attributes_for(:item)
+      @item = FactoryBot.create(:item)
+      @attrs = FactoryBot.attributes_for(:item)
       @invalid_attrs = { item_identifier: '無効なID' }
     end
 

data/spec/controllers/languages_controller_spec.rb

@@ -6,12 +6,12 @@ describe LanguagesController do
   fixtures :all
 
   def valid_attributes
-    FactoryGirl.attributes_for(:language)
+    FactoryBot.attributes_for(:language)
   end
 
   describe 'GET index' do
     before(:each) do
-      FactoryGirl.create(:language)
+      FactoryBot.create(:language)
     end
 
     describe 'When logged in as Administrator' do
@@ -51,7 +51,7 @@ describe LanguagesController do
 
   describe 'GET show' do
     before(:each) do
-      @language = FactoryGirl.create(:language)
+      @language = FactoryBot.create(:language)
     end
 
     describe 'When logged in as Administrator' do
@@ -92,7 +92,7 @@ describe LanguagesController do
 
   describe 'GET edit' do
     before(:each) do
-      @language = FactoryGirl.create(:language)
+      @language = FactoryBot.create(:language)
     end
 
     describe 'When logged in as Administrator' do
@@ -231,7 +231,7 @@ describe LanguagesController do
 
   describe 'PUT update' do
     before(:each) do
-      @language = FactoryGirl.create(:language)
+      @language = FactoryBot.create(:language)
       @attrs = valid_attributes
       @invalid_attrs = { name: '' }
     end
@@ -286,7 +286,7 @@ describe LanguagesController do
 
   describe 'DELETE destroy' do
     before(:each) do
-      @language = FactoryGirl.create(:language)
+      @language = FactoryBot.create(:language)
     end
 
     describe 'When logged in as Administrator' do

data/spec/controllers/licenses_controller_spec.rb

@@ -26,7 +26,7 @@ describe LicensesController do
   # License. As you add validations to License, be sure to
   # update the return value of this method accordingly.
   def valid_attributes
-    FactoryGirl.attributes_for(:license)
+    FactoryBot.attributes_for(:license)
   end
 
   describe 'GET index' do

data/spec/controllers/manifestation_relationship_types_controller_spec.rb

@@ -6,12 +6,12 @@ describe ManifestationRelationshipTypesController do
   disconnect_sunspot
 
   def valid_attributes
-    FactoryGirl.attributes_for(:manifestation_relationship_type)
+    FactoryBot.attributes_for(:manifestation_relationship_type)
   end
 
   describe 'GET index' do
     before(:each) do
-      FactoryGirl.create(:manifestation_relationship_type)
+      FactoryBot.create(:manifestation_relationship_type)
     end
 
     describe 'When logged in as Administrator' do
@@ -54,7 +54,7 @@ describe ManifestationRelationshipTypesController do
       login_fixture_admin
 
       it 'assigns the requested manifestation_relationship_type as @manifestation_relationship_type' do
-        manifestation_relationship_type = FactoryGirl.create(:manifestation_relationship_type)
+        manifestation_relationship_type = FactoryBot.create(:manifestation_relationship_type)
         get :show, id: manifestation_relationship_type.id
         expect(assigns(:manifestation_relationship_type)).to eq(manifestation_relationship_type)
       end
@@ -64,7 +64,7 @@ describe ManifestationRelationshipTypesController do
       login_fixture_librarian
 
       it 'assigns the requested manifestation_relationship_type as @manifestation_relationship_type' do
-        manifestation_relationship_type = FactoryGirl.create(:manifestation_relationship_type)
+        manifestation_relationship_type = FactoryBot.create(:manifestation_relationship_type)
         get :show, id: manifestation_relationship_type.id
         expect(assigns(:manifestation_relationship_type)).to eq(manifestation_relationship_type)
       end
@@ -74,7 +74,7 @@ describe ManifestationRelationshipTypesController do
       login_fixture_user
 
       it 'assigns the requested manifestation_relationship_type as @manifestation_relationship_type' do
-        manifestation_relationship_type = FactoryGirl.create(:manifestation_relationship_type)
+        manifestation_relationship_type = FactoryBot.create(:manifestation_relationship_type)
         get :show, id: manifestation_relationship_type.id
         expect(assigns(:manifestation_relationship_type)).to eq(manifestation_relationship_type)
       end
@@ -82,7 +82,7 @@ describe ManifestationRelationshipTypesController do
 
     describe 'When not logged in' do
       it 'assigns the requested manifestation_relationship_type as @manifestation_relationship_type' do
-        manifestation_relationship_type = FactoryGirl.create(:manifestation_relationship_type)
+        manifestation_relationship_type = FactoryBot.create(:manifestation_relationship_type)
         get :show, id: manifestation_relationship_type.id
         expect(assigns(:manifestation_relationship_type)).to eq(manifestation_relationship_type)
       end
@@ -134,7 +134,7 @@ describe ManifestationRelationshipTypesController do
       login_fixture_admin
 
       it 'assigns the requested manifestation_relationship_type as @manifestation_relationship_type' do
-        manifestation_relationship_type = FactoryGirl.create(:manifestation_relationship_type)
+        manifestation_relationship_type = FactoryBot.create(:manifestation_relationship_type)
         get :edit, id: manifestation_relationship_type.id
         expect(assigns(:manifestation_relationship_type)).to eq(manifestation_relationship_type)
       end
@@ -144,7 +144,7 @@ describe ManifestationRelationshipTypesController do
       login_fixture_librarian
 
       it 'assigns the requested manifestation_relationship_type as @manifestation_relationship_type' do
-        manifestation_relationship_type = FactoryGirl.create(:manifestation_relationship_type)
+        manifestation_relationship_type = FactoryBot.create(:manifestation_relationship_type)
         get :edit, id: manifestation_relationship_type.id
         expect(response).to be_forbidden
       end
@@ -154,7 +154,7 @@ describe ManifestationRelationshipTypesController do
       login_fixture_user
 
       it 'assigns the requested manifestation_relationship_type as @manifestation_relationship_type' do
-        manifestation_relationship_type = FactoryGirl.create(:manifestation_relationship_type)
+        manifestation_relationship_type = FactoryBot.create(:manifestation_relationship_type)
         get :edit, id: manifestation_relationship_type.id
         expect(response).to be_forbidden
       end
@@ -162,7 +162,7 @@ describe ManifestationRelationshipTypesController do
 
     describe 'When not logged in' do
       it 'should not assign the requested manifestation_relationship_type as @manifestation_relationship_type' do
-        manifestation_relationship_type = FactoryGirl.create(:manifestation_relationship_type)
+        manifestation_relationship_type = FactoryBot.create(:manifestation_relationship_type)
         get :edit, id: manifestation_relationship_type.id
         expect(response).to redirect_to(new_user_session_url)
       end
@@ -288,7 +288,7 @@ describe ManifestationRelationshipTypesController do
 
   describe 'PUT update' do
     before(:each) do
-      @manifestation_relationship_type = FactoryGirl.create(:manifestation_relationship_type)
+      @manifestation_relationship_type = FactoryBot.create(:manifestation_relationship_type)
       @attrs = valid_attributes
       @invalid_attrs = { name: '' }
     end
@@ -389,7 +389,7 @@ describe ManifestationRelationshipTypesController do
 
   describe 'DELETE destroy' do
     before(:each) do
-      @manifestation_relationship_type = FactoryGirl.create(:manifestation_relationship_type)
+      @manifestation_relationship_type = FactoryBot.create(:manifestation_relationship_type)
     end
 
     describe 'When logged in as Administrator' do

data/spec/controllers/manifestation_relationships_controller_spec.rb

@@ -6,12 +6,12 @@ describe ManifestationRelationshipsController do
   disconnect_sunspot
 
   def valid_attributes
-    @attrs = FactoryGirl.attributes_for(:manifestation_relationship)
+    @attrs = FactoryBot.attributes_for(:manifestation_relationship)
   end
 
   describe 'GET index' do
     before(:each) do
-      FactoryGirl.create(:manifestation_relationship)
+      FactoryBot.create(:manifestation_relationship)
     end
 
     describe 'When logged in as Administrator' do
@@ -54,7 +54,7 @@ describe ManifestationRelationshipsController do
       login_fixture_admin
 
       it 'assigns the requested manifestation_relationship as @manifestation_relationship' do
-        manifestation_relationship = FactoryGirl.create(:manifestation_relationship)
+        manifestation_relationship = FactoryBot.create(:manifestation_relationship)
         get :show, id: manifestation_relationship.id
         expect(assigns(:manifestation_relationship)).to eq(manifestation_relationship)
       end
@@ -64,7 +64,7 @@ describe ManifestationRelationshipsController do
       login_fixture_librarian
 
       it 'assigns the requested manifestation_relationship as @manifestation_relationship' do
-        manifestation_relationship = FactoryGirl.create(:manifestation_relationship)
+        manifestation_relationship = FactoryBot.create(:manifestation_relationship)
         get :show, id: manifestation_relationship.id
         expect(assigns(:manifestation_relationship)).to eq(manifestation_relationship)
       end
@@ -74,7 +74,7 @@ describe ManifestationRelationshipsController do
       login_fixture_user
 
       it 'assigns the requested manifestation_relationship as @manifestation_relationship' do
-        manifestation_relationship = FactoryGirl.create(:manifestation_relationship)
+        manifestation_relationship = FactoryBot.create(:manifestation_relationship)
         get :show, id: manifestation_relationship.id
         expect(assigns(:manifestation_relationship)).to eq(manifestation_relationship)
       end
@@ -82,7 +82,7 @@ describe ManifestationRelationshipsController do
 
     describe 'When not logged in' do
       it 'assigns the requested manifestation_relationship as @manifestation_relationship' do
-        manifestation_relationship = FactoryGirl.create(:manifestation_relationship)
+        manifestation_relationship = FactoryBot.create(:manifestation_relationship)
         get :show, id: manifestation_relationship.id
         expect(assigns(:manifestation_relationship)).to eq(manifestation_relationship)
       end
@@ -134,7 +134,7 @@ describe ManifestationRelationshipsController do
       login_fixture_admin
 
       it 'assigns the requested manifestation_relationship as @manifestation_relationship' do
-        manifestation_relationship = FactoryGirl.create(:manifestation_relationship)
+        manifestation_relationship = FactoryBot.create(:manifestation_relationship)
         get :edit, id: manifestation_relationship.id
         expect(assigns(:manifestation_relationship)).to eq(manifestation_relationship)
       end
@@ -144,7 +144,7 @@ describe ManifestationRelationshipsController do
       login_fixture_librarian
 
       it 'assigns the requested manifestation_relationship as @manifestation_relationship' do
-        manifestation_relationship = FactoryGirl.create(:manifestation_relationship)
+        manifestation_relationship = FactoryBot.create(:manifestation_relationship)
         get :edit, id: manifestation_relationship.id
         expect(assigns(:manifestation_relationship)).to eq(manifestation_relationship)
       end
@@ -154,7 +154,7 @@ describe ManifestationRelationshipsController do
       login_fixture_user
 
       it 'assigns the requested manifestation_relationship as @manifestation_relationship' do
-        manifestation_relationship = FactoryGirl.create(:manifestation_relationship)
+        manifestation_relationship = FactoryBot.create(:manifestation_relationship)
         get :edit, id: manifestation_relationship.id
         expect(response).to be_forbidden
       end
@@ -162,7 +162,7 @@ describe ManifestationRelationshipsController do
 
     describe 'When not logged in' do
       it 'should not assign the requested manifestation_relationship as @manifestation_relationship' do
-        manifestation_relationship = FactoryGirl.create(:manifestation_relationship)
+        manifestation_relationship = FactoryBot.create(:manifestation_relationship)
         get :edit, id: manifestation_relationship.id
         expect(response).to redirect_to(new_user_session_url)
       end
@@ -288,7 +288,7 @@ describe ManifestationRelationshipsController do
 
   describe 'PUT update' do
     before(:each) do
-      @manifestation_relationship = FactoryGirl.create(:manifestation_relationship)
+      @manifestation_relationship = FactoryBot.create(:manifestation_relationship)
       @attrs = valid_attributes
       @invalid_attrs = { parent_id: '' }
     end
@@ -390,7 +390,7 @@ describe ManifestationRelationshipsController do
 
   describe 'DELETE destroy' do
     before(:each) do
-      @manifestation_relationship = FactoryGirl.create(:manifestation_relationship)
+      @manifestation_relationship = FactoryBot.create(:manifestation_relationship)
     end
 
     describe 'When logged in as Administrator' do

data/spec/controllers/manifestations_controller_spec.rb

@@ -4,7 +4,7 @@ describe ManifestationsController do
   fixtures :all
 
   def valid_attributes
-    FactoryGirl.attributes_for(:manifestation)
+    FactoryBot.attributes_for(:manifestation)
   end
 
   describe 'GET index', solr: true do
@@ -187,9 +187,9 @@ describe ManifestationsController do
       end
 
       it "should show manifestation with shelf", solr: true do
-        shelf = FactoryGirl.create(:shelf)
+        shelf = FactoryBot.create(:shelf)
         library = shelf.library
-        item = FactoryGirl.create(:item, shelf: shelf)
+        item = FactoryBot.create(:item, shelf: shelf)
         get :index, :"#{library.name}_shelf" => [shelf.name]
         expect(response).to be_success
         expect(assigns(:manifestations).size).to eq 1
@@ -203,7 +203,7 @@ describe ManifestationsController do
       end
 
       it 'should show manifestation with NDC', solr: true do
-        classification = FactoryGirl.create(:classification, category: '007.3', classification_type_id: 1)
+        classification = FactoryBot.create(:classification, category: '007.3', classification_type_id: 1)
         Manifestation.first.classifications << classification
         get :index, classification: '007', classification_type: 1
         expect(response).to be_success
@@ -217,8 +217,8 @@ describe ManifestationsController do
       end
 
       it 'should show manifestation with subject', solr: true do
-        subject = FactoryGirl.create(:subject, term: 'test_subject')
-        manifestation = FactoryGirl.create(:manifestation)
+        subject = FactoryBot.create(:subject, term: 'test_subject')
+        manifestation = FactoryBot.create(:manifestation)
         manifestation.subjects << subject
         get :index, subject_text: 'test_subject'
         expect(response).to be_success
@@ -252,8 +252,8 @@ describe ManifestationsController do
       end
 
       it "should get manifestations with series for its children's information" do
-        periodical = FactoryGirl.create(:manifestation_serial)
-        manifestation = FactoryGirl.create(:manifestation, description: "foo")
+        periodical = FactoryBot.create(:manifestation_serial)
+        manifestation = FactoryBot.create(:manifestation, description: "foo")
         periodical.derived_manifestations << manifestation
         periodical.save!
         get :index, query: "foo"
@@ -303,7 +303,7 @@ describe ManifestationsController do
       end
 
       it 'should not show manifestation with required_role of admin' do
-        manifestation = FactoryGirl.create(:manifestation, required_role_id: 4)
+        manifestation = FactoryBot.create(:manifestation, required_role_id: 4)
         get :show, id: manifestation.id
         expect(response).not_to be_success
       end
@@ -421,7 +421,7 @@ describe ManifestationsController do
       end
 
       it 'should get new template with parent_id' do
-        serial = FactoryGirl.create(:manifestation_serial,
+        serial = FactoryBot.create(:manifestation_serial,
                                     statement_of_responsibility: 'statement_of_responsibility1',
                                     title_alternative: 'title_alternative1',
                                     publication_place: 'publication_place1',
@@ -430,14 +430,14 @@ describe ManifestationsController do
                                     depth: 123,
                                     price: 'price1',
                                     access_address: 'http://example.jp',
-                                    language_id: FactoryGirl.create(:language).id,
-                                    frequency_id: FactoryGirl.create(:frequency).id,
-                                    required_role_id: FactoryGirl.create(:role).id)
-        serial.creators << FactoryGirl.create(:agent)
-        serial.contributors << FactoryGirl.create(:agent)
-        serial.publishers << FactoryGirl.create(:agent)
-        serial.subjects << FactoryGirl.create(:subject)
-        serial.classifications << FactoryGirl.create(:classification)
+                                    language_id: FactoryBot.create(:language).id,
+                                    frequency_id: FactoryBot.create(:frequency).id,
+                                    required_role_id: FactoryBot.create(:role).id)
+        serial.creators << FactoryBot.create(:agent)
+        serial.contributors << FactoryBot.create(:agent)
+        serial.publishers << FactoryBot.create(:agent)
+        serial.subjects << FactoryBot.create(:subject)
+        serial.classifications << FactoryBot.create(:classification)
         serial.save!
         get :new, parent_id: serial.id
         expect(response).to be_success
@@ -488,7 +488,7 @@ describe ManifestationsController do
       login_fixture_admin
 
       it 'assigns the requested manifestation as @manifestation' do
-        manifestation = FactoryGirl.create(:manifestation)
+        manifestation = FactoryBot.create(:manifestation)
         get :edit, id: manifestation.id
         expect(assigns(:manifestation)).to eq(manifestation)
       end
@@ -498,15 +498,15 @@ describe ManifestationsController do
       login_fixture_librarian
 
       it 'assigns the requested manifestation as @manifestation' do
-        manifestation = FactoryGirl.create(:manifestation)
+        manifestation = FactoryBot.create(:manifestation)
         get :edit, id: manifestation.id
         expect(assigns(:manifestation)).to eq(manifestation)
       end
 
       render_views
       it 'assigns the identifiers to @manifestation' do
-        manifestation = FactoryGirl.create(:manifestation)
-        identifier = FactoryGirl.create(:identifier)
+        manifestation = FactoryBot.create(:manifestation)
+        identifier = FactoryBot.create(:identifier)
         manifestation.identifiers << identifier
         get :edit, id: manifestation.id
         expect(assigns(:manifestation)).to eq manifestation
@@ -519,7 +519,7 @@ describe ManifestationsController do
       login_fixture_user
 
       it 'assigns the requested manifestation as @manifestation' do
-        manifestation = FactoryGirl.create(:manifestation)
+        manifestation = FactoryBot.create(:manifestation)
         get :edit, id: manifestation.id
         expect(response).to be_forbidden
       end
@@ -532,7 +532,7 @@ describe ManifestationsController do
 
     describe 'When not logged in' do
       it 'should not assign the requested manifestation as @manifestation' do
-        manifestation = FactoryGirl.create(:manifestation)
+        manifestation = FactoryBot.create(:manifestation)
         get :edit, id: manifestation.id
         expect(response).to redirect_to(new_user_session_url)
       end
@@ -669,7 +669,7 @@ describe ManifestationsController do
 
   describe 'PUT update' do
     before(:each) do
-      @manifestation = FactoryGirl.create(:manifestation)
+      @manifestation = FactoryBot.create(:manifestation)
       @manifestation.series_statements = [SeriesStatement.find(1)]
       @attrs = valid_attributes
       @invalid_attrs = { original_title: '' }
@@ -719,7 +719,7 @@ describe ManifestationsController do
 
         it 'assigns identifiers to @manifestation' do
           identifiers_attrs = {
-            identifier_attributes: [FactoryGirl.create(:identifier)]
+            identifier_attributes: [FactoryBot.create(:identifier)]
           }
           put :update, id: @manifestation.id, manifestation: @attrs.merge(identifiers_attrs)
           expect(assigns(:manifestation)).to eq @manifestation
@@ -785,7 +785,7 @@ describe ManifestationsController do
 
   describe 'DELETE destroy' do
     before(:each) do
-      @manifestation = FactoryGirl.create(:manifestation)
+      @manifestation = FactoryBot.create(:manifestation)
     end
 
     describe 'When logged in as Administrator' do
@@ -811,8 +811,8 @@ describe ManifestationsController do
       end
 
       it 'should not destroy manifestation of series master with children' do
-        @manifestation = FactoryGirl.create(:manifestation_serial)
-        child = FactoryGirl.create(:manifestation)
+        @manifestation = FactoryBot.create(:manifestation_serial)
+        child = FactoryBot.create(:manifestation)
         @manifestation.derived_manifestations << child
         delete :destroy, id: @manifestation.id
         expect(response).to be_forbidden

data/spec/controllers/medium_of_performances_controller_spec.rb

@@ -26,7 +26,7 @@ describe MediumOfPerformancesController do
   # MediumOfPerformance. As you add validations to MediumOfPerformance, be sure to
   # update the return value of this method accordingly.
   def valid_attributes
-    FactoryGirl.attributes_for(:medium_of_performance)
+    FactoryBot.attributes_for(:medium_of_performance)
   end
 
   describe 'GET index' do