fix #13 (#194)

* fix #13

* ereased double request_return function

app/controllers/items_controller.rb

@@ -106,19 +106,6 @@ def request_return
     redirect_to item_url(@item)
   end
 
-  def request_return
-    @item = Item.find(params[:id])
-    @item.set_status_pending_return
-    @item.save
-    @user = current_user
-    unless ReturnRequestNotification.find_by(item: @item)
-      @notification = ReturnRequestNotification.new(user: User.find(@item.owner),
-                                                    date: Time.zone.now, item: @item, borrower: @user)
-      @notification.save
-    end
-    redirect_to item_url(@item)
-  end
-
   def accept_return
     @item = Item.find(params[:id])
     @notification = ReturnRequestNotification.find_by(item: @item)

spec/features/request/accept_lend_request_spec.rb

@@ -0,0 +1,19 @@
+require "rails_helper"
+
+describe "Requests handling", type: :feature do
+
+  it "owner can start lending by accepting the request" do
+    owner = create(:max)
+    borrower = create(:peter)
+    item = create(:item, owner: owner.id)
+    sign_in borrower
+    visit item_path(item)
+    click_button('Lend')
+    expect(item.reload.lend_status).to eq('pending_lend_request')
+    sign_in owner
+    visit notifications_path
+    click_button('Check')
+    click_button('Accept')
+    expect(item.reload.lend_status).to eq('lent')
+  end
+end