From f8d8089cb6cc5836c6f64ea05bcd4f76e21fd6cb Mon Sep 17 00:00:00 2001 From: Steffen van Bergerem Date: Sat, 9 Jul 2016 12:25:40 +0200 Subject: [PATCH] Add more specs for the admins controller --- spec/controllers/admins_controller_spec.rb | 25 +++++++++++++++++++++- 1 file changed, 24 insertions(+), 1 deletion(-) diff --git a/spec/controllers/admins_controller_spec.rb b/spec/controllers/admins_controller_spec.rb index 3bab4f38a..5e2d5914e 100644 --- a/spec/controllers/admins_controller_spec.rb +++ b/spec/controllers/admins_controller_spec.rb @@ -86,6 +86,12 @@ describe AdminsController, :type => :controller do expect(response).to redirect_to user_search_path expect(flash.notice).to include("invitation sent") end + + it "doesn't invite an existing user" do + get :admin_inviter, identifier: bob.email + expect(response).to redirect_to user_search_path + expect(flash.notice).to include("error sending invite") + end end end @@ -94,10 +100,27 @@ describe AdminsController, :type => :controller do Role.add_admin(@user.person) end - it 'succeeds and renders stats' do + it "succeeds and renders stats" do get :stats expect(response).to be_success expect(response).to render_template(:stats) + expect(response.body).to include( + I18n.translate("admins.stats.display_results", segment: I18n.translate("admins.stats.daily")) + ) + end + + it "succeeds and renders stats for different ranges" do + %w(week 2weeks month).each do |range| + get :stats, range: range + expect(response).to be_success + expect(response).to render_template(:stats) + expect(response.body).not_to include( + I18n.translate("admins.stats.display_results", segment: I18n.translate("admins.stats.daily")) + ) + expect(response.body).to include( + I18n.translate("admins.stats.display_results", segment: I18n.translate("admins.stats.#{range}")) + ) + end end end end