From d89945bbb2979e9b0eb49949fd31565cc92610dc Mon Sep 17 00:00:00 2001 From: Rohan Mitchell Date: Thu, 1 Nov 2012 10:20:06 +1100 Subject: [PATCH] Add smoke test for reports, fix use of old Distributor model --- .../admin/reports_controller_decorator.rb | 2 +- spec/requests/admin/enterprises_spec.rb | 2 +- spec/requests/admin/reports_spec.rb | 27 +++++++++++++++++++ 3 files changed, 29 insertions(+), 2 deletions(-) create mode 100644 spec/requests/admin/reports_spec.rb diff --git a/app/controllers/spree/admin/reports_controller_decorator.rb b/app/controllers/spree/admin/reports_controller_decorator.rb index b4d6585ec2..88424d31c7 100644 --- a/app/controllers/spree/admin/reports_controller_decorator.rb +++ b/app/controllers/spree/admin/reports_controller_decorator.rb @@ -53,7 +53,7 @@ Spree::Admin::ReportsController.class_eval do @search = Spree::Order.complete.search(params[:q]) orders = @search.result - @distributors = Distributor.all + @distributors = Enterprise.is_distributor @report = OpenFoodWeb::GroupBuyReport.new orders unless params[:csv] diff --git a/spec/requests/admin/enterprises_spec.rb b/spec/requests/admin/enterprises_spec.rb index 19edb6d772..dae560df69 100644 --- a/spec/requests/admin/enterprises_spec.rb +++ b/spec/requests/admin/enterprises_spec.rb @@ -1,7 +1,7 @@ require "spec_helper" feature %q{ - As an administration + As an administrator I want manage enterprises } do include AuthenticationWorkflow diff --git a/spec/requests/admin/reports_spec.rb b/spec/requests/admin/reports_spec.rb new file mode 100644 index 0000000000..e3e2d8fcd2 --- /dev/null +++ b/spec/requests/admin/reports_spec.rb @@ -0,0 +1,27 @@ +require "spec_helper" + +feature %q{ + As an administrator + I want numbers, all the numbers! +} do + include AuthenticationWorkflow + include WebHelper + + + scenario "orders and distributors report" do + login_to_admin_section + click_link 'Reports' + click_link 'Orders And Distributors' + + page.should have_content 'Order date' + end + + scenario "group buys report" do + login_to_admin_section + click_link 'Reports' + click_link 'Group Buys' + + page.should have_content 'Supplier' + end + +end