Set controller up to distinguish valid params

app/controllers/shelf_selection_reports_controller.rb

@@ -8,8 +8,13 @@ def new
 
   def create
     @shelf_selection_report = ShelfSelectionReport.new(params[:shelf_selection_report])
-    flash[:notice] = 'Shelf Selection Report params submitted!'
-    redirect_to root_url
+    if @shelf_selection_report.valid?
+      flash[:notice] = 'Shelf Selection Report params submitted!'
+      # save to shelf_sel_search
+      redirect_to root_url
+    else
+      render action: 'new'
+    end
   end
 
   def home_locations

app/models/shelf_selection_report.rb

@@ -7,6 +7,7 @@ class ShelfSelectionReport
                 :mhlds, :has_dups, :multvol, :multcop, :no_boundw,
                 :range_type, :call_alpha, :subj_name,
                 :save_opt, :search_name, :call_lo, :call_hi
+  validates :email, presence: true
 
   def self.generic_options
     [["Doesn't matter", 0], ['INCLUDE only', 1], ['EXCLUDE', 2]]

spec/controllers/shelf_selection_reports_controller_spec.rb

@@ -27,10 +27,15 @@
   end
 
   describe 'post#create' do
-    it 'redirects to root_url' do
+    it 'redirects to root_url when params are valid' do
       stub_current_user(FactoryGirl.create(:authorized_user))
-      post :create, shelf_selection_report: { email: '' }
+      post :create, shelf_selection_report: { email: 'testuser@test.org' }
       expect(response).to redirect_to root_url
     end
+    it 'renders the new action when params are invalid' do
+      stub_current_user(FactoryGirl.create(:authorized_user))
+      post :create, shelf_selection_report: { email: '' }
+      expect(response).to render_template('new')
+    end
   end
 end