raggedstaff/openfoodnetwork
1
0
Fork 0
mirror of https://github.com/openfoodfoundation/openfoodnetwork synced 2026-01-28 21:07:16 +00:00
Code Issues Packages Projects Releases Wiki Activity

In bulk coop report service spec just check array has the same elements but don't check the order.

Browse Source 
The #table_items methos seems to return line items in different order sometimes making this test a bit flaky. The test passed on Semaphore previously and is passing in development. I don't think the order matters so using :match_array instead of :eq.
This commit is contained in:
Cillian O'Ruanaidh
2020-08-28 12:20:47 +01:00
parent a441979649
commit 9efee1b0be
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
engines/order_management/spec/services/order_management/reports/bulk_coop/bulk_coop_report_spec.rb
View File

@@ -36,7 +36,7 @@ describe OrderManagement::Reports::BulkCoop::BulkCoopReport do
o2.line_items << li2
report = OrderManagement::Reports::BulkCoop::BulkCoopReport.new user, {}, true
expect(report.table_items).to eq([li1, li2])
expect(report.table_items).to match_array [li1, li2]
report = OrderManagement::Reports::BulkCoop::BulkCoopReport.new(
user, { q: { completed_at_gt: 2.days.ago } }, true
@@ -60,7 +60,7 @@ describe OrderManagement::Reports::BulkCoop::BulkCoopReport do
o2.line_items << li2
report = OrderManagement::Reports::BulkCoop::BulkCoopReport.new user, {}, true
expect(report.table_items).to eq([li1, li2])
expect(report.table_items).to match_array [li1, li2]
report = OrderManagement::Reports::BulkCoop::BulkCoopReport.new(
user, { q: { distributor_id_in: [d1.id] } }, true