enju_circulation 0.3.7 → 0.3.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ec0186b540696a095318784c23672ed230f8185e2c0d52f365e4695ef4a657da
-  data.tar.gz: a2f541396204e0fedfb66286073ae7692d115139cc2158b60d675e690ec0b64b
+  metadata.gz: 641777c8b977b90b8470a9d766fcd5c6f7e17aa45fddecb32717ee9ef03f5285
+  data.tar.gz: f6341956dc7028dba66c3d9c184a9a94de6f21c8bafb0fc5a2c756be8267a90b
 SHA512:
-  metadata.gz: a6b679339032cac9d2e882dda4a3f07e0a13b4ed14d5dcbca2ce33c4fbeb11ab6f55ceb35cfb5e96f6d937659ad5ddb9df01b733cbad4c22561d1d13d8dbc45c
-  data.tar.gz: 252a3d9f72f818c3923f03b8a7df79c7892afea7453ad173139549a4b83da9991a5fb2063476817a244de68084f93cfe084c5d902b3aa1f3f0e06e667d28fd7f
+  metadata.gz: '08bcdd1b447ac4cd569bc266d888c77dbf9fee65c9d7e34e034dd54e583c812d5e813d3b4e77e29f6b1eb37a02fd72bebb63256718e7c65a599c0b8f07595acb'
+  data.tar.gz: 18ea16bd6e95d5fbfa31780dca80241efef5a80546fddfff7fb9353ef9a1e26edda0035964651f998a8c910c904d1cdca7cc73714dc0f221346b16cc5e5f3566

data/app/controllers/checkins_controller.rb

@@ -77,12 +77,12 @@ class CheckinsController < ApplicationController
         end
         flash[:message] << message if message
         format.html { redirect_to checkins_url(basket_id: @checkin.basket_id) }
-        format.json { render json: @checkin, status: :created, location: @checkin }
+        format.json { render json: {result: @checkin, messages: flash[:message]}, status: :created, location: @checkin }
         format.js { redirect_to checkins_url(basket_id: @basket.id, format: :js) }
       else
         @checkins = @basket.checkins.page(1)
         format.html { render action: "new" }
-        format.json { render json: @checkin.errors, status: :unprocessable_entity }
+        format.json { render json: {messages: @checkin.errors}, status: :unprocessable_entity }
         format.js { render action: "index" }
       end
     end

data/app/controllers/checkouts_controller.rb

@@ -21,7 +21,7 @@ class CheckoutsController < ApplicationController
       end
     end
 
-    if ['txt', 'rss'].include?(params[:format].to_s.downcase)
+    if ['text', 'rss'].include?(params[:format].to_s.downcase)
       per_page = 500
       page = 1
     else
@@ -100,7 +100,7 @@ class CheckoutsController < ApplicationController
       format.json { render json: @checkouts }
       format.rss  { render layout: false }
       format.ics
-      format.txt
+      format.text
       format.atom
     end
   end

data/app/controllers/demands_controller.rb

@@ -19,7 +19,7 @@ class DemandsController < ApplicationController
     respond_to do |format|
       format.html # show.html.erb
       format.json { render json: @demand }
-      format.txt
+      format.text
     end
   end
 

data/app/controllers/manifestation_checkout_stats_controller.rb

@@ -17,7 +17,7 @@ class ManifestationCheckoutStatsController < ApplicationController
   # GET /manifestation_checkout_stats/1
   # GET /manifestation_checkout_stats/1.json
   def show
-    if params[:format] == 'txt'
+    if request.format.text?
       per_page = 65534
     else
       per_page = CheckoutStatHasManifestation.default_per_page
@@ -52,7 +52,7 @@ class ManifestationCheckoutStatsController < ApplicationController
     respond_to do |format|
       format.html # show.html.erb
       format.json { render json: @manifestation_checkout_stat }
-      format.txt
+      format.text
       format.js
     end
   end

data/app/controllers/manifestation_reserve_stats_controller.rb

@@ -18,7 +18,7 @@ class ManifestationReserveStatsController < ApplicationController
   # GET /manifestation_reserve_stats/1
   # GET /manifestation_reserve_stats/1.json
   def show
-    if params[:format] == 'txt'
+    if request.format.text?
       per_page = 65534
     else
       per_page = ReserveStatHasManifestation.default_per_page
@@ -28,7 +28,7 @@ class ManifestationReserveStatsController < ApplicationController
     respond_to do |format|
       format.html # show.html.erb
       format.json { render json: @manifestation_reserve_stat }
-      format.txt
+      format.text
     end
   end
 

data/app/controllers/reserves_controller.rb

@@ -34,7 +34,7 @@ class ReservesController < ApplicationController
       sort_column = :created_at
       order = :desc
     end
-    if ['txt', 'rss'].include?(params[:format].to_s.downcase)
+    if ['text', 'rss'].include?(params[:format].to_s.downcase)
       per_page = 500
       page = 1
     else
@@ -105,7 +105,7 @@ class ReservesController < ApplicationController
       format.json { render json: @reserves }
       format.rss  { render layout: false }
       format.atom
-      format.txt
+      format.text
     end
   end
 

data/app/controllers/user_checkout_stats_controller.rb

@@ -17,7 +17,7 @@ class UserCheckoutStatsController < ApplicationController
   # GET /user_checkout_stats/1
   # GET /user_checkout_stats/1.json
   def show
-    if params[:format] == 'txt'
+    if request.format.text?
       per_page = 65534
     else
       per_page = CheckoutStatHasUser.default_per_page
@@ -27,7 +27,7 @@ class UserCheckoutStatsController < ApplicationController
     respond_to do |format|
       format.html # show.html.erb
       format.json { render json: @user_checkout_stat }
-      format.txt
+      format.text
     end
   end
 

data/app/controllers/user_reserve_stats_controller.rb

@@ -17,7 +17,7 @@ class UserReserveStatsController < ApplicationController
   # GET /user_reserve_stats/1
   # GET /user_reserve_stats/1.json
   def show
-    if params[:format] == 'txt'
+    if request.format.text?
       per_page = 65534
     else
       per_page = ReserveStatHasUser.default_per_page
@@ -27,7 +27,7 @@ class UserReserveStatsController < ApplicationController
     respond_to do |format|
       format.html # show.html.erb
       format.json { render json: @user_reserve_stat }
-      format.txt
+      format.text
     end
   end
 

data/app/models/checkin.rb

@@ -91,5 +91,5 @@ end
 #  basket_id    :integer
 #  created_at   :datetime
 #  updated_at   :datetime
-#  lock_version :integer          default(0), not null
+#  lock_version :integer          default("0"), not null
 #

data/app/models/checkout.rb

@@ -174,8 +174,8 @@ end
 #  librarian_id           :integer
 #  basket_id              :integer
 #  due_date               :datetime
-#  checkout_renewal_count :integer          default(0), not null
-#  lock_version           :integer          default(0), not null
+#  checkout_renewal_count :integer          default("0"), not null
+#  lock_version           :integer          default("0"), not null
 #  created_at             :datetime
 #  updated_at             :datetime
 #  shelf_id               :integer

data/app/models/checkout_stat_has_user.rb

@@ -17,7 +17,7 @@ end
 #  id                    :integer          not null, primary key
 #  user_checkout_stat_id :integer          not null
 #  user_id               :integer          not null
-#  checkouts_count       :integer          default(0), not null
+#  checkouts_count       :integer          default("0"), not null
 #  created_at            :datetime
 #  updated_at            :datetime
 #

data/app/models/concerns/enju_circulation/enju_item.rb

@@ -105,6 +105,8 @@ module EnjuCirculation
         manifestation.next_reservation.transition_to!(:completed)
         manifestation.reload
       end
+
+      reload
       update!(circulation_status: CirculationStatus.find_by(name: 'On Loan'))
     end
 

data/app/models/lending_policy.rb

@@ -19,10 +19,10 @@ end
 #  id             :integer          not null, primary key
 #  item_id        :integer          not null
 #  user_group_id  :integer          not null
-#  loan_period    :integer          default(0), not null
+#  loan_period    :integer          default("0"), not null
 #  fixed_due_date :datetime
-#  renewal        :integer          default(0), not null
-#  fine           :integer          default(0), not null
+#  renewal        :integer          default("0"), not null
+#  fine           :integer          default("0"), not null
 #  note           :text
 #  position       :integer
 #  created_at     :datetime

data/app/models/manifestation_checkout_stat_transition.rb

@@ -12,7 +12,7 @@ end
 #
 #  id                             :integer          not null, primary key
 #  to_state                       :string
-#  metadata                       :text             default({})
+#  metadata                       :text             default("{}")
 #  sort_key                       :integer
 #  manifestation_checkout_stat_id :integer
 #  created_at                     :datetime

data/app/models/manifestation_reserve_stat_transition.rb

@@ -12,7 +12,7 @@ end
 #
 #  id                            :integer          not null, primary key
 #  to_state                      :string
-#  metadata                      :text             default({})
+#  metadata                      :text             default("{}")
 #  sort_key                      :integer
 #  manifestation_reserve_stat_id :integer
 #  created_at                    :datetime

data/app/models/reserve.rb

@@ -2,7 +2,7 @@ class Reserve < ApplicationRecord
   include Statesman::Adapters::ActiveRecordQueries
   scope :hold, -> { where('item_id IS NOT NULL') }
   scope :not_hold, -> { where(item_id: nil) }
-  scope :waiting, -> {not_in_state(:completed, :expired, :retained).where('canceled_at IS NULL AND (expired_at > ? OR expired_at IS NULL)', Time.zone.now).order('reserves.id DESC')}
+  scope :waiting, -> {not_in_state(:completed, :expired).where('canceled_at IS NULL AND (expired_at > ? OR expired_at IS NULL)', Time.zone.now).order('reserves.id DESC')}
   scope :retained, -> {in_state(:retained).where('retained_at IS NOT NULL')}
   scope :completed, -> {in_state(:completed).where('checked_out_at IS NOT NULL')}
   scope :canceled, -> {in_state(:canceled).where('canceled_at IS NOT NULL')}
@@ -404,10 +404,10 @@ end
 #  canceled_at                  :datetime
 #  expired_at                   :datetime
 #  deleted_at                   :datetime
-#  expiration_notice_to_patron  :boolean          default(FALSE)
-#  expiration_notice_to_library :boolean          default(FALSE)
+#  expiration_notice_to_patron  :boolean          default("0")
+#  expiration_notice_to_library :boolean          default("0")
 #  pickup_location_id           :integer
 #  retained_at                  :datetime
 #  postponed_at                 :datetime
-#  lock_version                 :integer          default(0), not null
+#  lock_version                 :integer          default("0"), not null
 #

data/app/models/reserve_state_machine.rb

@@ -14,16 +14,15 @@ class ReserveStateMachine
   transition from: :requested, to: [:retained, :canceled, :expired, :completed]
 
   after_transition(to: :requested) do |reserve|
-    reserve.update(request_status_type: RequestStatusType.where(name: 'In Process').first, item_id: nil, retained_at: nil)
+    reserve.update(request_status_type: RequestStatusType.find_by(name: 'In Process'), item_id: nil, retained_at: nil)
   end
 
   after_transition(to: :retained) do |reserve|
     # TODO: 「取り置き中」の状態を正しく表す
-    reserve.update(request_status_type: RequestStatusType.where(name: 'In Process').first, retained_at: Time.zone.now)
+    reserve.update(request_status_type: RequestStatusType.find_by(name: 'In Process'), retained_at: Time.zone.now)
     Reserve.transaction do
       if reserve.item
         other_reserves = reserve.item.reserves.waiting
-        other_reserves += reserve.item.reserves.in_state(:retained)
         other_reserves.each{|r|
           if r != reserve
             r.transition_to!(:postponed)

data/app/models/reserve_transition.rb

@@ -12,7 +12,7 @@ end
 #
 #  id          :integer          not null, primary key
 #  to_state    :string
-#  metadata    :text             default({})
+#  metadata    :text             default("{}")
 #  sort_key    :integer
 #  reserve_id  :integer
 #  created_at  :datetime

data/app/models/user_checkout_stat_transition.rb

@@ -12,7 +12,7 @@ end
 #
 #  id                    :integer          not null, primary key
 #  to_state              :string
-#  metadata              :text             default({})
+#  metadata              :text             default("{}")
 #  sort_key              :integer
 #  user_checkout_stat_id :integer
 #  created_at            :datetime

data/app/models/user_group_has_checkout_type.rb

@@ -71,12 +71,12 @@ end
 #  id                              :integer          not null, primary key
 #  user_group_id                   :integer          not null
 #  checkout_type_id                :integer          not null
-#  checkout_limit                  :integer          default(0), not null
-#  checkout_period                 :integer          default(0), not null
-#  checkout_renewal_limit          :integer          default(0), not null
-#  reservation_limit               :integer          default(0), not null
-#  reservation_expired_period      :integer          default(7), not null
-#  set_due_date_before_closing_day :boolean          default(FALSE), not null
+#  checkout_limit                  :integer          default("0"), not null
+#  checkout_period                 :integer          default("0"), not null
+#  checkout_renewal_limit          :integer          default("0"), not null
+#  reservation_limit               :integer          default("0"), not null
+#  reservation_expired_period      :integer          default("7"), not null
+#  set_due_date_before_closing_day :boolean          default("0"), not null
 #  fixed_due_date                  :datetime
 #  note                            :text
 #  position                        :integer

data/app/models/user_reserve_stat_transition.rb

@@ -12,7 +12,7 @@ end
 #
 #  id                   :integer          not null, primary key
 #  to_state             :string
-#  metadata             :text             default({})
+#  metadata             :text             default("{}")
 #  sort_key             :integer
 #  user_reserve_stat_id :integer
 #  created_at           :datetime

data/lib/enju_circulation/version.rb

@@ -1,3 +1,3 @@
 module EnjuCirculation
-  VERSION = "0.3.7"
+  VERSION = "0.3.10".freeze
 end

data/lib/generators/enju_circulation/setup/setup_generator.rb

@@ -32,10 +32,5 @@ ActiveSupport::Inflector.inflections do |inflect|
   inflect.irregular 'reserve', 'reserves'
 end
 EOS
-
-    UserGroup.order(created_at: :desc).first.update!(
-      number_of_day_to_notify_overdue: 7,
-      number_of_day_to_notify_due_date: 3
-    )
   end
 end

data/spec/controllers/checkins_controller_spec.rb

@@ -201,14 +201,14 @@ describe CheckinsController do
           it 'redirects to the created checkin' do
             post :create, params: { checkin: @attrs, basket_id: 9 }
             response.should redirect_to(checkins_url(basket_id: assigns(:checkin).basket_id))
-            assigns(:checkin).item.circulation_status.name.should eq 'Available On Shelf'
+            assigns(:checkin).checkout.item.circulation_status.name.should eq 'Available On Shelf'
           end
 
           it 'should checkin the overdue item' do
             post :create, params: { checkin: { item_identifier: '00014' }, basket_id: 9 }
             response.should redirect_to(checkins_url(basket_id: assigns(:checkin).basket_id))
             assigns(:checkin).checkout.should be_valid
-            assigns(:checkin).item.circulation_status.name.should eq 'Available On Shelf'
+            assigns(:checkin).checkout.item.circulation_status.name.should eq 'Available On Shelf'
           end
         end
       end
@@ -249,14 +249,14 @@ describe CheckinsController do
         it 'should show notification when it is reserved' do
           post :create, params: { checkin: { item_identifier: '00008' }, basket_id: 9 }
           flash[:message].to_s.index(I18n.t('item.this_item_is_reserved')).should be_truthy
-          assigns(:checkin).item.should be_retained
-          assigns(:checkin).item.circulation_status.name.should eq 'Available On Shelf'
+          assigns(:checkin).checkout.item.should be_retained
+          assigns(:checkin).checkout.item.circulation_status.name.should eq 'Available On Shelf'
           response.should redirect_to(checkins_url(basket_id: assigns(:basket).id))
         end
 
         it 'should show notification when an item includes supplements' do
           post :create, params: { checkin: { item_identifier: '00004' }, basket_id: 9 }
-          assigns(:checkin).item.circulation_status.name.should eq 'Available On Shelf'
+          assigns(:checkin).checkout.item.circulation_status.name.should eq 'Available On Shelf'
           flash[:message].to_s.index(I18n.t('item.this_item_include_supplement')).should be_truthy
           response.should redirect_to(checkins_url(basket_id: assigns(:basket).id))
         end
@@ -306,6 +306,128 @@ describe CheckinsController do
     end
   end
 
+  describe 'POST create (json format)' do
+    before(:each) do
+      @attrs = { item_identifier: '00003' }
+      @invalid_attrs = { item_identifier: 'invalid' }
+      request.env["HTTP_ACCEPT"] = 'application/json'
+    end
+
+    describe 'When logged in as Administrator' do
+      login_fixture_admin
+
+      describe 'with valid params' do
+        it 'assigns a newly created checkin as @checkin' do
+          post :create, params: { checkin: @attrs }
+          assigns(:checkin).should be_nil
+        end
+
+        it 'should not create checkin without basket_id' do
+          post :create, params: { checkin: @attrs }
+          json = JSON.parse(response.body)
+          expect(json['error']).to eq('forbidden')
+        end
+
+        describe 'When basket_id is specified' do
+          it 'redirects to the created checkin' do
+            post :create, params: { checkin: @attrs, basket_id: 9 }
+            expect(response).to have_http_status(:created)
+            json = JSON.parse(response.body)
+            expect(json['result']['basket_id']).to eq(9)
+            assigns(:checkin).checkout.item.circulation_status.name.should eq 'Available On Shelf'
+          end
+
+          it 'should checkin the overdue item' do
+            post :create, params: { checkin: { item_identifier: '00014' }, basket_id: 9 }
+            expect(response).to have_http_status(:created)
+            assigns(:checkin).checkout.should be_valid
+            assigns(:checkin).checkout.item.circulation_status.name.should eq 'Available On Shelf'
+          end
+        end
+      end
+
+      describe 'with invalid params' do
+        it 'assigns a newly created but unsaved checkin as @checkin' do
+          post :create, params: { checkin: @invalid_attrs }
+          assigns(:checkin).should be_nil
+        end
+
+        it 'should be forbidden' do
+          post :create, params: { checkin: @invalid_attrs }
+          json = JSON.parse(response.body)
+          expect(json['error']).to eq('forbidden')
+        end
+      end
+
+      it 'should not create checkin without item_id' do
+        post :create, params: { checkin: { item_identifier: nil }, basket_id: 9 }
+        assigns(:checkin).should_not be_valid
+        expect(response).to have_http_status(:unprocessable_entity)
+        json = JSON.parse(response.body)
+        expect(json['messages']['base']).to match_array([I18n.t('checkin.item_not_found')])
+        expect(json['messages']['item_id']).to match_array([I18n.t('errors.messages.blank')])
+      end
+    end
+
+    describe 'When logged in as Librarian' do
+      login_fixture_librarian
+
+      describe 'with valid params' do
+        it 'assigns a newly created checkin as @checkin' do
+          post :create, params: { checkin: @attrs }
+          assigns(:checkin).should be_nil
+        end
+
+        it 'should not create checkin without basket_id' do
+          post :create, params: { checkin: @attrs }
+          json = JSON.parse(response.body)
+          expect(json['error']).to eq('forbidden')
+        end
+
+        it 'should show notification when it is reserved' do
+          post :create, params: { checkin: { item_identifier: '00008' }, basket_id: 9 }
+          flash[:message].to_s.index(I18n.t('item.this_item_is_reserved')).should be_truthy
+          assigns(:checkin).checkout.item.should be_retained
+          assigns(:checkin).checkout.item.circulation_status.name.should eq 'Available On Shelf'
+          expect(response).to have_http_status(:created)
+        end
+
+        it 'should show notification when an item includes supplements' do
+          post :create, params: { checkin: { item_identifier: '00004' }, basket_id: 9 }
+          assigns(:checkin).checkout.item.circulation_status.name.should eq 'Available On Shelf'
+          flash[:message].to_s.index(I18n.t('item.this_item_include_supplement')).should be_truthy
+          expect(response).to have_http_status(:created)
+        end
+      end
+
+      it "should show notice when other library's item is checked in" do
+        sign_in users(:librarian2)
+        post :create, params: { checkin: { item_identifier: '00009' }, basket_id: 9 }
+        assigns(:checkin).should be_valid
+        flash[:message].to_s.index(I18n.t('checkin.other_library_item')).should be_truthy
+        expect(response).to have_http_status(:created)
+      end
+    end
+
+    describe 'When not logged in' do
+      before(:each) do
+        @attrs = { item_identifier: '00003' }
+        @invalid_attrs = { item_identifier: 'invalid' }
+      end
+
+      describe 'with valid params' do
+        it 'assigns a newly created checkin as @checkin' do
+          post :create, params: { checkin: @attrs }
+        end
+
+        it 'should redirect to new session url' do
+          post :create, params: { checkin: @attrs }
+          expect(response).to have_http_status(:ok)
+        end
+      end
+    end
+  end
+
   describe 'PUT update' do
     before(:each) do
       @checkin = checkins(:checkin_00001)
@@ -335,7 +457,7 @@ describe CheckinsController do
 
         it "re-renders the 'edit' template" do
           put :update, params: { id: @checkin.id, checkin: @invalid_attrs }
-          response.should render_template('edit')
+          expect(response).to be_successful
         end
 
         it 'should not update checkin without item_identifier' do
@@ -364,12 +486,12 @@ describe CheckinsController do
       describe 'with invalid params' do
         it 'assigns the checkin as @checkin' do
           put :update, params: { id: @checkin.id, checkin: @invalid_attrs }
-          assigns(:checkin).should_not be_valid
+          expect(assigns(:checkin)).not_to be_valid
         end
 
         it "re-renders the 'edit' template" do
           put :update, params: { id: @checkin.id, checkin: @invalid_attrs }
-          response.should render_template('edit')
+          expect(response).to be_successful
         end
       end
     end

data/spec/controllers/checkouts_controller_spec.rb

@@ -45,8 +45,8 @@ describe CheckoutsController do
         response.should be_successful
       end
 
-      it 'should get index txt' do
-        get :index, format: 'txt'
+      it 'should get index text' do
+        get :index, format: :text
         assigns(:checkouts).count.should eq assigns(:checkouts).total_entries
         response.should be_successful
       end
@@ -117,9 +117,9 @@ describe CheckoutsController do
         response.should redirect_to checkouts_url
       end
 
-      it 'should get my index in txt format' do
-        get :index, params: { user_id: users(:user1).username, format: 'txt' }
-        response.should redirect_to checkouts_url(format: :txt)
+      it 'should get my index in text format' do
+        get :index, params: { user_id: users(:user1).username, format: :text }
+        response.should redirect_to checkouts_url(format: :text)
         assigns(:checkouts).should be_nil
       end
 

data/spec/controllers/reserves_controller_spec.rb

@@ -38,8 +38,8 @@ describe ReservesController do
         assigns(:reserves).should eq(Reserve.order('reserves.id DESC').includes(:manifestation))
       end
 
-      it 'should get index txt without user_id' do
-        get :index, format: 'txt'
+      it 'should get index text without user_id' do
+        get :index, format: :text
         response.should be_successful
         assigns(:reserves).count.should eq assigns(:reserves).total_entries
         assigns(:reserves).should eq(Reserve.order('reserves.id DESC').includes(:manifestation))
@@ -51,8 +51,8 @@ describe ReservesController do
         assigns(:reserves).should eq(users(:user1).reserves.order('reserves.id DESC').includes(:manifestation).page(1))
       end
 
-      it 'should get index txt with user_id' do
-        get :index, params: { user_id: users(:user1).username, format: 'txt' }
+      it 'should get index text with user_id' do
+        get :index, params: { user_id: users(:user1).username, format: :text }
         response.should be_successful
         assigns(:reserves).should eq(users(:user1).reserves.order('reserves.id DESC').includes(:manifestation))
       end
@@ -89,8 +89,8 @@ describe ReservesController do
         response.should render_template('index')
       end
 
-      it 'should get my index txt' do
-        get :index, format: :txt
+      it 'should get my index text' do
+        get :index, format: :text
         response.should be_successful
         response.should render_template('index')
       end
@@ -106,9 +106,9 @@ describe ReservesController do
           response.should redirect_to reserves_url(format: :rss)
         end
 
-        it 'should redirect to my reservation txt' do
-          get :index, params: { user_id: users(:user1).username, format: 'txt' }
-          response.should redirect_to reserves_url(format: :txt)
+        it 'should redirect to my reservation text' do
+          get :index, params: { user_id: users(:user1).username, format: :text }
+          response.should redirect_to reserves_url(format: :text)
         end
       end
 

data/spec/controllers/withdraws_controller_spec.rb

@@ -0,0 +1,26 @@
+require 'rails_helper'
+
+describe WithdrawsController do
+  fixtures :all
+  let(:valid_attributes) do
+    FactoryBot.build(:withdraw).attributes.with_indifferent_access
+  end
+  let(:valid_create_attributes) do
+    { basket_id: Basket.find(valid_attributes[:basket_id]).id,
+      withdraw: { item_identifier: Item.find(valid_attributes[:item_id]).item_identifier }}
+  end
+
+  describe 'POST create' do
+    describe 'When logged in as Administrator' do
+      login_fixture_admin
+
+      it 'should not withdraw a checked-out item' do
+        expect do
+          post :create, params: { basket_id: valid_create_attributes[:basket_id], withdraw: { item_identifier: reserves(:reserve_00014).item.item_identifier } }
+        end.to change(Withdraw, :count).by(0)
+        expect(assigns(:withdraw)).to be_a(Withdraw)
+        expect(response).to be_successful
+      end
+    end
+  end
+end