enju_biblio 0.1.0.pre63 → 0.1.0.pre64

data/spec/controllers/identifier_types_controller_spec.rb

@@ -84,15 +84,15 @@ describe IdentifierTypesController do
       it "assigns a newly created but unsaved identifier_type as @identifier_type" do
         # Trigger the behavior that occurs when invalid params are submitted
         IdentifierType.any_instance.stub(:save).and_return(false)
-        post :create, :identifier_type => {}
+        post :create, :identifier_type => {name: "test"}
         expect(assigns(:identifier_type)).to be_a_new(IdentifierType)
       end
 
       it "re-renders the 'new' template" do
         # Trigger the behavior that occurs when invalid params are submitted
         IdentifierType.any_instance.stub(:save).and_return(false)
-        post :create, :identifier_type => {}
-        #expect(response).to render_template("new")
+        post :create, :identifier_type => {name: "test"}
+        expect(response).to render_template("new")
       end
     end
   end
@@ -105,8 +105,8 @@ describe IdentifierTypesController do
         # specifies that the IdentifierType created on the previous line
         # receives the :update_attributes message with whatever params are
         # submitted in the request.
-        IdentifierType.any_instance.should_receive(:update_attributes).with({'these' => 'params'})
-        put :update, :id => identifier_type.id, :identifier_type => {'these' => 'params'}
+        IdentifierType.any_instance.should_receive(:update_attributes).with({'name' => 'test'})
+        put :update, :id => identifier_type.id, :identifier_type => {'name' => 'test'}
       end
 
       it "assigns the requested identifier_type as @identifier_type" do
@@ -135,7 +135,7 @@ describe IdentifierTypesController do
         identifier_type = IdentifierType.create! valid_attributes
         # Trigger the behavior that occurs when invalid params are submitted
         IdentifierType.any_instance.stub(:save).and_return(false)
-        put :update, :id => identifier_type.id, :identifier_type => {}
+        put :update, :id => identifier_type.id, :identifier_type => {name: "test"}
         expect(assigns(:identifier_type)).to eq(identifier_type)
       end
 
@@ -143,8 +143,8 @@ describe IdentifierTypesController do
         identifier_type = IdentifierType.create! valid_attributes
         # Trigger the behavior that occurs when invalid params are submitted
         IdentifierType.any_instance.stub(:save).and_return(false)
-        put :update, :id => identifier_type.id, :identifier_type => {}
-        #expect(response).to render_template("edit")
+        put :update, :id => identifier_type.id, :identifier_type => {name: "test"}
+        expect(response).to render_template("edit")
       end
     end
   end

data/spec/controllers/licenses_controller_spec.rb

@@ -84,14 +84,14 @@ describe LicensesController do
       it "assigns a newly created but unsaved license as @license" do
         # Trigger the behavior that occurs when invalid params are submitted
         License.any_instance.stub(:save).and_return(false)
-        post :create, :license => {}
+        post :create, :license => {name: "test"}
         expect(assigns(:license)).to be_a_new(License)
       end
 
       it "re-renders the 'new' template" do
         # Trigger the behavior that occurs when invalid params are submitted
         License.any_instance.stub(:save).and_return(false)
-        post :create, :license => {}
+        post :create, :license => {name: "test"}
         #expect(response).to render_template("new")
       end
     end
@@ -105,8 +105,8 @@ describe LicensesController do
         # specifies that the License created on the previous line
         # receives the :update_attributes message with whatever params are
         # submitted in the request.
-        License.any_instance.should_receive(:update_attributes).with({'these' => 'params'})
-        put :update, :id => license.id, :license => {'these' => 'params'}
+        License.any_instance.should_receive(:update_attributes).with({'name' => 'test'})
+        put :update, :id => license.id, :license => {'name' => 'test'}
       end
 
       it "assigns the requested license as @license" do
@@ -135,7 +135,7 @@ describe LicensesController do
         license = License.create! valid_attributes
         # Trigger the behavior that occurs when invalid params are submitted
         License.any_instance.stub(:save).and_return(false)
-        put :update, :id => license.id, :license => {}
+        put :update, :id => license.id, :license => {name: "test"}
         expect(assigns(:license)).to eq(license)
       end
 
@@ -143,7 +143,7 @@ describe LicensesController do
         license = License.create! valid_attributes
         # Trigger the behavior that occurs when invalid params are submitted
         License.any_instance.stub(:save).and_return(false)
-        put :update, :id => license.id, :license => {}
+        put :update, :id => license.id, :license => {name: "test"}
         #expect(response).to render_template("edit")
       end
     end

data/spec/controllers/manifestations_controller_spec.rb

@@ -88,12 +88,24 @@ describe ManifestationsController do
         expect(assigns(:manifestations)).to_not be_nil
       end
 
+      it "assigns all manifestations as @manifestations when old pub_date_from and pub_date_to are specified" do
+        get :index, :pub_date_from => '200', :pub_date_to => '207'
+        assigns(:query).should eq "date_of_publication_d:[#{Time.zone.parse('200-01-01').utc.iso8601} TO #{Time.zone.parse('207-12-31').end_of_year.utc.iso8601}]"
+        expect(assigns(:manifestations)).to_not be_nil
+      end
+
       it "assigns all manifestations as @manifestations when acquired_from and pub_date_to are specified" do
         get :index, :acquired_from => '2000', :acquired_to => '2007'
         assigns(:query).should eq "acquired_at_d:[#{Time.zone.parse('2000-01-01').utc.iso8601} TO #{Time.zone.parse('2007-12-31').end_of_day.utc.iso8601}]"
         expect(assigns(:manifestations)).to_not be_nil
       end
 
+      it "assigns all manifestations as @manifestations when old acquired_from and pub_date_to are specified" do
+        get :index, :acquired_from => '200', :acquired_to => '207'
+        assigns(:query).should eq "acquired_at_d:[#{Time.zone.parse('200-01-01').utc.iso8601} TO #{Time.zone.parse('207-12-31').end_of_day.utc.iso8601}]"
+        expect(assigns(:manifestations)).to_not be_nil
+      end
+
       it "assigns all manifestations as @manifestations when number_of_pages_at_least and number_of_pages_at_most are specified" do
         get :index, :number_of_pages_at_least => '100', :number_of_pages_at_least => '200'
         expect(assigns(:manifestations)).to_not be_nil
@@ -381,7 +393,7 @@ describe ManifestationsController do
         end
 
         it "assigns a series_statement" do
-          post :create, :manifestation => @attrs.merge(:series_statements_attributes => {[0] => {:original_title => SeriesStatement.find(1).original_title}})
+          post :create, :manifestation => @attrs.merge(:series_statements_attributes => {"0" => {:original_title => SeriesStatement.find(1).original_title}})
           assigns(:manifestation).reload
           assigns(:manifestation).series_statements.pluck(:original_title).include?(series_statements(:one).original_title).should be_truthy
         end
@@ -510,7 +522,7 @@ describe ManifestationsController do
         end
 
         it "assigns a series_statement" do
-          put :update, :id => @manifestation.id, :manifestation => @attrs.merge(:series_statements_attributes => {[0] => {:original_title => series_statements(:two).original_title, "_destroy"=>"false"}})
+          put :update, :id => @manifestation.id, :manifestation => @attrs.merge(:series_statements_attributes => {"0" => {:original_title => series_statements(:two).original_title, "_destroy"=>"false"}})
           assigns(:manifestation).reload
           assigns(:manifestation).series_statements.pluck(:original_title).include?(series_statements(:two).original_title).should be_truthy
         end
@@ -627,6 +639,14 @@ describe ManifestationsController do
         delete :destroy, :id => 1
         expect(response).to be_forbidden
       end
+
+      it "should not destroy manifestation of series master with children" do
+        @series_statement = FactoryGirl.create(:series_statement_serial)
+	@manifestation = FactoryGirl.create(:manifestation_serial)
+        delete :destroy, :id => @manifestation.id
+        expect(response).to redirect_to(manifestations_url)
+      end
+
     end
 
     describe "When logged in as Librarian" do

data/spec/controllers/medium_of_performances_controller_spec.rb

@@ -84,14 +84,14 @@ describe MediumOfPerformancesController do
       it "assigns a newly created but unsaved medium_of_performance as @medium_of_performance" do
         # Trigger the behavior that occurs when invalid params are submitted
         MediumOfPerformance.any_instance.stub(:save).and_return(false)
-        post :create, :medium_of_performance => {}
+        post :create, :medium_of_performance => {name: "test"}
         expect(assigns(:medium_of_performance)).to be_a_new(MediumOfPerformance)
       end
 
       it "re-renders the 'new' template" do
         # Trigger the behavior that occurs when invalid params are submitted
         MediumOfPerformance.any_instance.stub(:save).and_return(false)
-        post :create, :medium_of_performance => {}
+        post :create, :medium_of_performance => {name: "test"}
         #expect(response).to render_template("new")
       end
     end
@@ -105,8 +105,8 @@ describe MediumOfPerformancesController do
         # specifies that the MediumOfPerformance created on the previous line
         # receives the :update_attributes message with whatever params are
         # submitted in the request.
-        MediumOfPerformance.any_instance.should_receive(:update_attributes).with({'these' => 'params'})
-        put :update, :id => medium_of_performance.id, :medium_of_performance => {'these' => 'params'}
+        MediumOfPerformance.any_instance.should_receive(:update_attributes).with({'name' => 'test'})
+        put :update, :id => medium_of_performance.id, :medium_of_performance => {'name' => 'test'}
       end
 
       it "assigns the requested medium_of_performance as @medium_of_performance" do
@@ -135,7 +135,7 @@ describe MediumOfPerformancesController do
         medium_of_performance = MediumOfPerformance.create! valid_attributes
         # Trigger the behavior that occurs when invalid params are submitted
         MediumOfPerformance.any_instance.stub(:save).and_return(false)
-        put :update, :id => medium_of_performance.id, :medium_of_performance => {}
+        put :update, :id => medium_of_performance.id, :medium_of_performance => {name: "test"}
         expect(assigns(:medium_of_performance)).to eq(medium_of_performance)
       end
 
@@ -143,7 +143,7 @@ describe MediumOfPerformancesController do
         medium_of_performance = MediumOfPerformance.create! valid_attributes
         # Trigger the behavior that occurs when invalid params are submitted
         MediumOfPerformance.any_instance.stub(:save).and_return(false)
-        put :update, :id => medium_of_performance.id, :medium_of_performance => {}
+        put :update, :id => medium_of_performance.id, :medium_of_performance => {name: "test"}
         #expect(response).to render_template("edit")
       end
     end

data/spec/controllers/produce_types_controller_spec.rb

@@ -84,14 +84,14 @@ describe ProduceTypesController do
       it "assigns a newly created but unsaved produce_type as @produce_type" do
         # Trigger the behavior that occurs when invalid params are submitted
         ProduceType.any_instance.stub(:save).and_return(false)
-        post :create, :produce_type => {}
+        post :create, :produce_type => {name: "test"}
         expect(assigns(:produce_type)).to be_a_new(ProduceType)
       end
 
       it "re-renders the 'new' template" do
         # Trigger the behavior that occurs when invalid params are submitted
         ProduceType.any_instance.stub(:save).and_return(false)
-        post :create, :produce_type => {}
+        post :create, :produce_type => {name: "test"}
         #expect(response).to render_template("new")
       end
     end
@@ -105,8 +105,8 @@ describe ProduceTypesController do
         # specifies that the ProduceType created on the previous line
         # receives the :update_attributes message with whatever params are
         # submitted in the request.
-        ProduceType.any_instance.should_receive(:update_attributes).with({'these' => 'params'})
-        put :update, :id => produce_type.id, :produce_type => {'these' => 'params'}
+        ProduceType.any_instance.should_receive(:update_attributes).with({'name' => 'test'})
+        put :update, :id => produce_type.id, :produce_type => {'name' => 'test'}
       end
 
       it "assigns the requested produce_type as @produce_type" do
@@ -135,7 +135,7 @@ describe ProduceTypesController do
         produce_type = ProduceType.create! valid_attributes
         # Trigger the behavior that occurs when invalid params are submitted
         ProduceType.any_instance.stub(:save).and_return(false)
-        put :update, :id => produce_type.id, :produce_type => {}
+        put :update, :id => produce_type.id, :produce_type => {name: "test"}
         expect(assigns(:produce_type)).to eq(produce_type)
       end
 
@@ -143,7 +143,7 @@ describe ProduceTypesController do
         produce_type = ProduceType.create! valid_attributes
         # Trigger the behavior that occurs when invalid params are submitted
         ProduceType.any_instance.stub(:save).and_return(false)
-        put :update, :id => produce_type.id, :produce_type => {}
+        put :update, :id => produce_type.id, :produce_type => {name: "test"}
         #expect(response).to render_template("edit")
       end
     end

data/spec/controllers/realize_types_controller_spec.rb

@@ -84,14 +84,14 @@ describe RealizeTypesController do
       it "assigns a newly created but unsaved realize_type as @realize_type" do
         # Trigger the behavior that occurs when invalid params are submitted
         RealizeType.any_instance.stub(:save).and_return(false)
-        post :create, :realize_type => {}
+        post :create, :realize_type => {name: "test"}
         expect(assigns(:realize_type)).to be_a_new(RealizeType)
       end
 
       it "re-renders the 'new' template" do
         # Trigger the behavior that occurs when invalid params are submitted
         RealizeType.any_instance.stub(:save).and_return(false)
-        post :create, :realize_type => {}
+        post :create, :realize_type => {name: "test"}
         #expect(response).to render_template("new")
       end
     end
@@ -105,8 +105,8 @@ describe RealizeTypesController do
         # specifies that the RealizeType created on the previous line
         # receives the :update_attributes message with whatever params are
         # submitted in the request.
-        RealizeType.any_instance.should_receive(:update_attributes).with({'these' => 'params'})
-        put :update, :id => realize_type.id, :realize_type => {'these' => 'params'}
+        RealizeType.any_instance.should_receive(:update_attributes).with({'name' => 'test'})
+        put :update, :id => realize_type.id, :realize_type => {'name' => 'test'}
       end
 
       it "assigns the requested realize_type as @realize_type" do
@@ -135,7 +135,7 @@ describe RealizeTypesController do
         realize_type = RealizeType.create! valid_attributes
         # Trigger the behavior that occurs when invalid params are submitted
         RealizeType.any_instance.stub(:save).and_return(false)
-        put :update, :id => realize_type.id, :realize_type => {}
+        put :update, :id => realize_type.id, :realize_type => {name: "test"}
         expect(assigns(:realize_type)).to eq(realize_type)
       end
 
@@ -143,7 +143,7 @@ describe RealizeTypesController do
         realize_type = RealizeType.create! valid_attributes
         # Trigger the behavior that occurs when invalid params are submitted
         RealizeType.any_instance.stub(:save).and_return(false)
-        put :update, :id => realize_type.id, :realize_type => {}
+        put :update, :id => realize_type.id, :realize_type => {name: "test"}
         #expect(response).to render_template("edit")
       end
     end

data/spec/controllers/resource_export_files_controller_spec.rb

@@ -46,7 +46,7 @@ describe ResourceExportFilesController do
       login_fixture_admin
 
       it "assigns the requested resource_export_file as @resource_export_file" do
-        get :show, :id => resource_export_files(:resource_export_file_00003).id
+        get :show, id: resource_export_files(:resource_export_file_00003).id
         expect(assigns(:resource_export_file)).to eq(resource_export_files(:resource_export_file_00003))
         expect(response).to be_success
       end
@@ -56,7 +56,7 @@ describe ResourceExportFilesController do
       login_fixture_librarian
 
       it "assigns the requested resource_export_file as @resource_export_file" do
-        get :show, :id => resource_export_files(:resource_export_file_00003).id
+        get :show, id: resource_export_files(:resource_export_file_00003).id
         expect(assigns(:resource_export_file)).to eq(resource_export_files(:resource_export_file_00003))
         expect(response).to be_success
       end
@@ -66,7 +66,7 @@ describe ResourceExportFilesController do
       login_fixture_user
 
       it "assigns the requested resource_export_file as @resource_export_file" do
-        get :show, :id => resource_export_files(:resource_export_file_00003).id
+        get :show, id: resource_export_files(:resource_export_file_00003).id
         expect(assigns(:resource_export_file)).to eq(resource_export_files(:resource_export_file_00003))
         expect(response).to be_forbidden
       end
@@ -74,7 +74,7 @@ describe ResourceExportFilesController do
 
     describe "When not logged in" do
       it "assigns the requested resource_export_file as @resource_export_file" do
-        get :show, :id => resource_export_files(:resource_export_file_00003).id
+        get :show, id: resource_export_files(:resource_export_file_00003).id
         expect(assigns(:resource_export_file)).to eq(resource_export_files(:resource_export_file_00003))
         expect(response).to redirect_to(new_user_session_url)
       end
@@ -129,7 +129,7 @@ describe ResourceExportFilesController do
       end
 
       it "should create agent_export_file" do
-        post :create, :resource_export_file => { }
+        post :create, resource_export_file: {mode: 'export'}
         expect(assigns(:resource_export_file)).to be_valid
         assigns(:resource_export_file).user.username.should eq @user.username
         expect(response).to redirect_to resource_export_file_url(assigns(:resource_export_file))
@@ -143,7 +143,7 @@ describe ResourceExportFilesController do
       end
 
       it "should be forbidden" do
-        post :create, :resource_export_file => { }
+        post :create, resource_export_file: {mode: 'export'}
         assigns(:resource_export_file).user.should be_nil
         expect(response).to be_forbidden
       end
@@ -151,7 +151,7 @@ describe ResourceExportFilesController do
 
     describe "When not logged in" do
       it "should be redirected to new session url" do
-        post :create, :resource_export_file => { }
+        post :create, resource_export_file: {mode: 'export'}
         assigns(:resource_export_file).user.should be_nil
         expect(response).to redirect_to new_user_session_url
       end
@@ -164,7 +164,7 @@ describe ResourceExportFilesController do
 
       it "assigns the requested resource_export_file as @resource_export_file" do
         resource_export_file = resource_export_files(:resource_export_file_00001)
-        get :edit, :id => resource_export_file.id
+        get :edit, id: resource_export_file.id
         expect(assigns(:resource_export_file)).to eq(resource_export_file)
       end
     end
@@ -174,7 +174,7 @@ describe ResourceExportFilesController do
 
       it "assigns the requested resource_export_file as @resource_export_file" do
         resource_export_file = resource_export_files(:resource_export_file_00001)
-        get :edit, :id => resource_export_file.id
+        get :edit, id: resource_export_file.id
         expect(assigns(:resource_export_file)).to eq(resource_export_file)
       end
     end
@@ -184,7 +184,7 @@ describe ResourceExportFilesController do
 
       it "assigns the requested resource_export_file as @resource_export_file" do
         resource_export_file = resource_export_files(:resource_export_file_00001)
-        get :edit, :id => resource_export_file.id
+        get :edit, id: resource_export_file.id
         expect(response).to be_forbidden
       end
     end
@@ -192,7 +192,7 @@ describe ResourceExportFilesController do
     describe "When not logged in" do
       it "should not assign the requested resource_export_file as @resource_export_file" do
         resource_export_file = resource_export_files(:resource_export_file_00001)
-        get :edit, :id => resource_export_file.id
+        get :edit, id: resource_export_file.id
         expect(response).to redirect_to(new_user_session_url)
       end
     end
@@ -203,7 +203,7 @@ describe ResourceExportFilesController do
       login_fixture_admin
 
       it "should update resource_export_file" do
-        put :update, :id => resource_export_files(:resource_export_file_00003).id, :resource_export_file => { }
+        put :update, id: resource_export_files(:resource_export_file_00003).id, resource_export_file: {mode: 'export'}
         expect(response).to redirect_to resource_export_file_url(assigns(:resource_export_file))
       end
     end
@@ -212,7 +212,7 @@ describe ResourceExportFilesController do
       login_fixture_librarian
 
       it "should update resource_export_file" do
-        put :update, :id => resource_export_files(:resource_export_file_00003).id, :resource_export_file => { }
+        put :update, id: resource_export_files(:resource_export_file_00003).id, resource_export_file: {mode: 'export'}
         expect(response).to redirect_to resource_export_file_url(assigns(:resource_export_file))
       end
     end
@@ -221,14 +221,14 @@ describe ResourceExportFilesController do
       login_fixture_user
 
       it "should not update resource_export_file" do
-        put :update, :id => resource_export_files(:resource_export_file_00003).id, :resource_export_file => { }
+        put :update, id: resource_export_files(:resource_export_file_00003).id, resource_export_file: {mode: 'export'}
         expect(response).to be_forbidden
       end
     end
 
     describe "When not logged in" do
       it "should not update resource_export_file" do
-        put :update, :id => resource_export_files(:resource_export_file_00003).id, :resource_export_file => { }
+        put :update, id: resource_export_files(:resource_export_file_00003).id, resource_export_file: {mode: 'export'}
         expect(response).to redirect_to new_user_session_url
       end
     end
@@ -243,11 +243,11 @@ describe ResourceExportFilesController do
       login_fixture_admin
 
       it "destroys the requested resource_export_file" do
-        delete :destroy, :id => @resource_export_file.id
+        delete :destroy, id: @resource_export_file.id
       end
 
       it "redirects to the resource_export_files list" do
-        delete :destroy, :id => @resource_export_file.id
+        delete :destroy, id: @resource_export_file.id
         expect(response).to redirect_to(resource_export_files_url)
       end
     end
@@ -256,11 +256,11 @@ describe ResourceExportFilesController do
       login_fixture_librarian
 
       it "destroys the requested resource_export_file" do
-        delete :destroy, :id => @resource_export_file.id
+        delete :destroy, id: @resource_export_file.id
       end
 
       it "redirects to the resource_export_files list" do
-        delete :destroy, :id => @resource_export_file.id
+        delete :destroy, id: @resource_export_file.id
         expect(response).to redirect_to(resource_export_files_url)
       end
     end
@@ -269,22 +269,22 @@ describe ResourceExportFilesController do
       login_fixture_user
 
       it "destroys the requested resource_export_file" do
-        delete :destroy, :id => @resource_export_file.id
+        delete :destroy, id: @resource_export_file.id
       end
 
       it "should be forbidden" do
-        delete :destroy, :id => @resource_export_file.id
+        delete :destroy, id: @resource_export_file.id
         expect(response).to be_forbidden
       end
     end
 
     describe "When not logged in" do
       it "destroys the requested resource_export_file" do
-        delete :destroy, :id => @resource_export_file.id
+        delete :destroy, id: @resource_export_file.id
       end
 
       it "should be forbidden" do
-        delete :destroy, :id => @resource_export_file.id
+        delete :destroy, id: @resource_export_file.id
         expect(response).to redirect_to(new_user_session_url)
       end
     end