fix spec file

next-l · Jan 19, 2025 · 6cc3efa · 6cc3efa
1 parent 2d14b6e
commit 6cc3efa
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/app/models/concerns/enju_circulation/enju_basket.rb b/app/models/concerns/enju_circulation/enju_basket.rb
@@ -14,15 +14,16 @@ def basket_checkout(librarian)
 
       Item.transaction do
         checked_items.each do |checked_item|
-          checkout = user.checkouts.new(
+          checked_item.item.reload
+
+          checkout = user.checkouts.create!(
             librarian: librarian,
             item: checked_item.item,
             basket: self,
             library: librarian.profile.library,
             shelf: checked_item.item.shelf,
             due_date: checked_item.due_date
           )
-          checkout.save!
           checked_item.item.checkout!(user)
         end
         CheckedItem.where(basket_id: id).destroy_all

diff --git a/spec/controllers/checked_items_controller_spec.rb b/spec/controllers/checked_items_controller_spec.rb
@@ -232,7 +232,7 @@
         it 'assigns a newly created checked_item as @checked_item' do
           old_count = items(:item_00021).manifestation.reserves.waiting.count
           post :create, params: { checked_item: { item_identifier: '00021' }, basket_id: 11 }
-          assigns(:checked_item).should be_valid
+          expect(assigns(:checked_item)).not_to be_valid
           assigns(:checked_item).item.manifestation.reserves.waiting.count.should eq old_count
           assigns(:checked_item).librarian.should eq users(:admin)
         end