Update tickets controller spec for Rails 5

snap-cloud · Feb 6, 2020 · 231b2a6 · 231b2a6
1 parent d06c3ce
commit 231b2a6
Showing 1 changed file with 24 additions and 14 deletions.
diff --git a/spec/controllers/admin/tickets_controller_spec.rb b/spec/controllers/admin/tickets_controller_spec.rb
@@ -12,7 +12,7 @@
     before { sign_in admin }
 
     describe 'GET #index' do
-      before { get :index, conference_id: conference }
+      before { get :index, params: { conference_id: conference } }
 
       it 'assigns conference and tickets variables via cancancan' do
         expect(assigns(:conference)).to eq conference
@@ -25,7 +25,7 @@
     end
 
     describe 'GET #edit' do
-      before { get :edit, conference_id: conference, id: ticket }
+      before { get :edit, params: { conference_id: conference, id: ticket } }
 
       it 'assigns ticket variable via cancancan' do
         expect(assigns(:ticket)).to eq ticket
@@ -37,7 +37,7 @@
     end
 
     describe 'GET #new' do
-      before { get :new, conference_id: conference }
+      before { get :new, params: { conference_id: conference } }
 
       it 'assigns ticket variable' do
         expect(assigns(:ticket)).to be_instance_of(Ticket)
@@ -51,7 +51,7 @@
     describe 'POST #create' do
       context 'saves successfuly' do
         before(:each, run: true) do
-          post :create, conference_id: conference, ticket: attributes_for(:ticket)
+          post :create, params: { conference_id: conference, ticket: attributes_for(:ticket) }
         end
 
         let!(:ticket_count) { conference.tickets.count }
@@ -68,16 +68,18 @@
 
         it 'creates new ticket' do
           expect do
-            post :create, ticket:        attributes_for(:ticket),
-                          conference_id: conference
+            post :create, params: {
+                ticket: attributes_for(:ticket),
+                conference_id: conference
+            }
           end.to change{ conference.tickets.count }.from(ticket_count).to(ticket_count + 1)
         end
       end
 
       context 'save fails' do
         before do
           allow_any_instance_of(Ticket).to receive(:save).and_return(false)
-          post :create, conference_id: conference, ticket: attributes_for(:ticket)
+          post :create, params: { conference_id: conference, ticket: attributes_for(:ticket) }
         end
 
         let!(:ticket_count) { conference.tickets.count }
@@ -99,8 +101,10 @@
     describe 'PATCH #update' do
       context 'updates successfully' do
         before do
-          patch :update, conference_id: conference, id: ticket,
+          patch :update, params: {
+            conference_id: conference, id: ticket,
             ticket: attributes_for(:ticket, title: new_title)
+          }
         end
 
         it 'redirects to index path' do
@@ -122,8 +126,10 @@
       context 'update fails' do
         before do
           allow_any_instance_of(Ticket).to receive(:save).and_return(false)
-          patch :update, conference_id: conference, id: ticket,
+          patch :update, params: {
+            conference_id: conference, id: ticket,
             ticket: attributes_for(:ticket, title: new_title)
+          }
         end
 
         it 'renders edit template' do
@@ -144,7 +150,7 @@
     describe 'DELETE #destroy' do
       context 'deletes successfully' do
         before(:each, run: true) do
-          delete :destroy, conference_id: conference, id: ticket
+          delete :destroy, params: { conference_id: conference, id: ticket }
         end
 
         let!(:ticket_count) { conference.tickets.count }
@@ -161,7 +167,7 @@
 
         it 'deletes the ticket' do
           expect do
-            delete :destroy, conference_id: conference, id: ticket
+            delete :destroy, params: { conference_id: conference, id: ticket }
           end.to change{ conference.tickets.count }.from(ticket_count).to(ticket_count - 1)
         end
       end
@@ -171,7 +177,7 @@
 
         before do
           allow_any_instance_of(Ticket).to receive(:destroy).and_return(false)
-          delete :destroy, conference_id: conference, id: ticket
+          delete :destroy, params: { conference_id: conference, id: ticket }
         end
 
         it 'redirects to index path' do
@@ -195,8 +201,10 @@
         let!(:purchase_count) { admin.ticket_purchases.count }
 
         before do
-          post :give, conference_id: conference, id: ticket,
+          post :give, params: {
+            conference_id: conference, id: ticket,
             ticket_purchase: { user_id: admin.id }
+          }
         end
 
         it 'redirects to ticket' do
@@ -220,8 +228,10 @@
       context 'giving fails' do
         before do
           allow_any_instance_of(TicketPurchase).to receive(:save).and_return(false)
-          post :give, conference_id: conference, id: ticket,
+          post :give, params: {
+            conference_id: conference, id: ticket,
             ticket_purchase: { user_id: admin.id }
+          }
         end
 
         let(:purchase_count) { admin.ticket_purchases.count }