Merge pull request #1 from sorah/downloading

Add StuffsController#download

app/controllers/stuffs_controller.rb

@@ -1,5 +1,5 @@
 class StuffsController < ApplicationController
-  before_action :set_stuff, only: [:show, :edit, :update, :destroy]
+  before_action :set_stuff, only: [:show, :edit, :update, :destroy, :download]
 
   # GET /stuffs
   # GET /stuffs.json
@@ -39,6 +39,10 @@ def destroy
     end
   end
 
+  def download
+    send_file @stuff.filepath
+  end
+
   private
     # Use callbacks to share common setup or constraints between actions.
     def set_stuff

config/routes.rb

@@ -1,5 +1,9 @@
 Rails.application.routes.draw do
-  resources :stuffs, only: %i(index create show destroy)
+  resources :stuffs, only: %i(index create show destroy) do
+    member do
+      get :download
+    end
+  end
 
   # The priority is based upon order of creation: first created -> highest priority.
   # See how all your routes lay out with "rake routes".

spec/controllers/stuffs_controller_spec.rb

@@ -23,10 +23,14 @@
   # This should return the minimal set of attributes required to create a valid
   # Stuff. As you add validations to Stuff, be sure to
   # adjust the attributes here as well.
+  let(:file) do
+    fixture_file_upload('a.png')
+  end
+
   let(:valid_attributes) do
     {
       title: "MyString",
-      file: fixture_file_upload('a.png'),
+      file: file,
     }
   end
 
@@ -103,4 +107,18 @@
     end
   end
 
+  describe "GET download" do
+    let!(:stuff) do
+      Stuff.create!(valid_attributes)
+    end
+
+    context "when stuff exists" do
+      it "sends the file" do
+        get :download, id: stuff.id
+
+        expect(response.code).to eq '200'
+        expect(response.body).to eq File.binread(file.path)
+      end
+    end
+  end
 end