Merge pull request #3711 from consul/fix_managers_suggestions

Allow managers to read investment suggestions

app/models/abilities/manager.rb

@@ -0,0 +1,11 @@
+module Abilities
+  class Manager
+    include CanCan::Ability
+
+    def initialize(user)
+      merge Abilities::Common.new(user)
+
+      can :suggest, Budget::Investment
+    end
+  end
+end

app/models/ability.rb

@@ -14,6 +14,8 @@ class Ability
         merge Abilities::Administrator.new(user)
       elsif user.moderator?
         merge Abilities::Moderator.new(user)
+      elsif user.manager?
+        merge Abilities::Manager.new(user)
       else
         merge Abilities::Common.new(user)
       end

spec/features/management/budget_investments_spec.rb

@@ -94,6 +94,29 @@ describe "Budget Investments" do
 
       expect(page).to have_content "User is not verified"
     end
+
+    scenario "Shows suggestions to unverified managers", :js do
+      expect(manager.user.level_two_or_three_verified?).to be false
+
+      create(:budget_investment, budget: budget, title: "More parks")
+      create(:budget_investment, budget: budget, title: "No more parks")
+      create(:budget_investment, budget: budget, title: "Plant trees")
+      login_managed_user(create(:user, :level_two))
+
+      click_link "Create budget investment"
+      within "#budget_#{budget.id}" do
+        click_link "Create budget investment"
+      end
+
+      fill_in "Title", with: "Park"
+      fill_in_ckeditor "Description", with: "Wish I had one"
+
+      within(".js-suggest") do
+        expect(page).to have_content "More parks"
+        expect(page).to have_content "No more parks"
+        expect(page).not_to have_content "Plant trees"
+      end
+    end
   end
 
   context "Searching" do