From 795106aaa28293048c9003dc93d88894d2abb1c6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fran=C3=A7ois=20Turbelin?= Date: Tue, 9 Jun 2020 10:11:11 +0200 Subject: [PATCH] Use real orders inside spec to extend coverage --- spec/services/bulk_invoice_service_spec.rb | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/spec/services/bulk_invoice_service_spec.rb b/spec/services/bulk_invoice_service_spec.rb index 1874a0fcb4..52fe085288 100644 --- a/spec/services/bulk_invoice_service_spec.rb +++ b/spec/services/bulk_invoice_service_spec.rb @@ -50,8 +50,11 @@ describe BulkInvoiceService do describe "#orders_from" do it "orders with completed desc" do - expect(service.send(:orders_from, [1, 2]).to_sql) - .to include('ORDER BY completed_at DESC') + order_old = create(:order_with_distributor, :completed, completed_at: 2.minutes.ago) + order_older = create(:order_with_distributor, :completed, completed_at: 3.minutes.ago) + + expect(service.send(:orders_from, [order_older.id, order_old.id]).pluck(:id)) + .to eq([order_old.id, order_older.id]) end end end