From b6ce9ca3cab21f0fc73f9a9e58ac4c168911e52d Mon Sep 17 00:00:00 2001 From: Pau Perez Date: Tue, 2 Mar 2021 17:25:12 +0100 Subject: [PATCH] Test bulk coop report includes canceled orders Without them numbers in the report don't match with /admin/customers and /account where their order total is considered towards the customer balance. --- .../reports/bulk_coop/bulk_coop_report_spec.rb | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/engines/order_management/spec/services/order_management/reports/bulk_coop/bulk_coop_report_spec.rb b/engines/order_management/spec/services/order_management/reports/bulk_coop/bulk_coop_report_spec.rb index a6cb1666b3..9adc4b047a 100644 --- a/engines/order_management/spec/services/order_management/reports/bulk_coop/bulk_coop_report_spec.rb +++ b/engines/order_management/spec/services/order_management/reports/bulk_coop/bulk_coop_report_spec.rb @@ -18,15 +18,16 @@ describe OrderManagement::Reports::BulkCoop::BulkCoopReport do context "as a site admin" do it "fetches completed orders" do - o2 = create(:order) + o2 = create(:order, state: 'cart') o2.line_items << build(:line_item) expect(subject.table_items).to eq([li1]) end - it "does not show cancelled orders" do - o2 = create(:order, state: "canceled", completed_at: 1.day.ago) - o2.line_items << build(:line_item_with_shipment) - expect(subject.table_items).to eq([li1]) + it "shows cancelled orders" do + o2 = create(:order, state: 'canceled', completed_at: 1.day.ago, order_cycle: oc1, distributor: d1) + line_item = build(:line_item_with_shipment) + o2.line_items << line_item + expect(subject.table_items).to include(line_item) end end