raggedstaff/openfoodnetwork
1
0
Fork 0
mirror of https://github.com/openfoodfoundation/openfoodnetwork synced 2026-03-03 02:21:33 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fixing broken enterprise controller spec

Browse Source
This commit is contained in:
Rob Harrington
2015-04-10 13:16:21 +10:00
parent 1e6fd94663
commit 72403fd021
2 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
app/controllers/admin/enterprises_controller.rb
View File

@@ -68,7 +68,6 @@ module Admin
def for_order_cycle
respond_to do |format|
format.html
format.json do
render json: ActiveModel::ArraySerializer.new( @collection,
each_serializer: Api::Admin::ForOrderCycle::EnterpriseSerializer, spree_current_user: spree_current_user

11
spec/controllers/admin/enterprises_controller_spec.rb
View File

@@ -419,32 +419,33 @@ module Admin
OrderCycle.stub new: "new OrderCycle"
OpenFoodNetwork::Permissions.stub(:new) { permission_mock }
allow(permission_mock).to receive :order_cycle_enterprises_for
allow(permission_mock).to receive(:order_cycle_enterprises_for)
allow(ActiveModel::ArraySerializer).to receive(:new) { "" }
end
context "when no order_cycle or coordinator is provided in params" do
before { spree_get :for_order_cycle }
before { spree_get :for_order_cycle, format: :json }
it "returns an empty scope" do
expect(permission_mock).to have_received(:order_cycle_enterprises_for).with(nil)
end
end
context "when an order_cycle_id is provided in params" do
before { spree_get :for_order_cycle, order_cycle_id: 1 }
before { spree_get :for_order_cycle, format: :json, order_cycle_id: 1 }
it "calls order_cycle_enterprises_for() with the existing OrderCycle" do
expect(permission_mock).to have_received(:order_cycle_enterprises_for).with("existing OrderCycle")
end
end
context "when a coordinator is provided in params" do
before { spree_get :for_order_cycle, coordinator_id: 1 }
before { spree_get :for_order_cycle, format: :json, coordinator_id: 1 }
it "calls order_cycle_enterprises_for() with a new OrderCycle" do
expect(permission_mock).to have_received(:order_cycle_enterprises_for).with("new OrderCycle")
end
end
context "when both an order cycle and a coordinator are provided in params" do
before { spree_get :for_order_cycle, order_cycle_id: 1, coordinator_id: 1 }
before { spree_get :for_order_cycle, format: :json, order_cycle_id: 1, coordinator_id: 1 }
it "calls order_cycle_enterprises_for() with the existing OrderCycle" do
expect(permission_mock).to have_received(:order_cycle_enterprises_for).with("existing OrderCycle")
end