From 46410b5be2710a5d3678f0eb84a88e8d9c6e042d Mon Sep 17 00:00:00 2001 From: Maikel Linke Date: Thu, 29 Jun 2023 16:13:10 +1000 Subject: [PATCH] Test the right invoices controller I was wondering why the spec didn't work and then I noticed that it was testing the wrong controller and passing the wrong parameter. --- spec/controllers/spree/admin/orders/invoices_spec.rb | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/spec/controllers/spree/admin/orders/invoices_spec.rb b/spec/controllers/spree/admin/orders/invoices_spec.rb index c9ec8b7c78..1ec7316ac9 100644 --- a/spec/controllers/spree/admin/orders/invoices_spec.rb +++ b/spec/controllers/spree/admin/orders/invoices_spec.rb @@ -104,7 +104,9 @@ describe Spree::Admin::OrdersController, type: :controller do end end end +end +describe Spree::Admin::InvoicesController, type: :controller do describe "#index" do let(:user) { create(:user) } let(:enterprise_user) { create(:user, enterprises: [create(:enterprise)]) } @@ -113,7 +115,7 @@ describe Spree::Admin::OrdersController, type: :controller do ship_address: create(:address)) } let(:distributor) { order.distributor } - let(:params) { { id: order.number } } + let(:params) { { order_id: order.number } } context "as a normal user" do before { allow(controller).to receive(:spree_current_user) { user } } @@ -141,6 +143,7 @@ describe Spree::Admin::OrdersController, type: :controller do it "should allow me to see the order" do spree_get :index, params expect(response).to be_successful + expect(assigns(:order)).to eq order end end end