From 54d068ee089cb1368430c37acf59acb143a024ea Mon Sep 17 00:00:00 2001 From: David Cook Date: Thu, 20 Jun 2024 16:46:11 +1000 Subject: [PATCH] Add spec for db queries --- .../orders_and_distributors_report_spec.rb | 22 +++++++++++++++++++ 1 file changed, 22 insertions(+) diff --git a/spec/lib/reports/orders_and_distributors_report_spec.rb b/spec/lib/reports/orders_and_distributors_report_spec.rb index 58374912f9..76b86be318 100644 --- a/spec/lib/reports/orders_and_distributors_report_spec.rb +++ b/spec/lib/reports/orders_and_distributors_report_spec.rb @@ -132,6 +132,28 @@ RSpec.describe Reporting::Reports::OrdersAndDistributors::Base do end end end + + it "minimises database queries" do + subject # build context first + + # surely we can do better than that for each row + expect { subject.table_rows }.to query_database [ + "Spree::Role Exists?", + "Spree::Role Exists?", + "SQL", + "Spree::LineItem Load", + "Spree::Order Load", + "Spree::Address Load", + "Spree::Payment Load", + "Spree::PaymentMethod Load", + "Spree::Calculator Load", + "Enterprise Load", + "Spree::Address Load", + "Spree::Shipment Load", + "Spree::ShippingRate Load", + "Spree::ShippingMethod Load", + ] + end end end end