refactor moderator role and add specs

#5324

app/controllers/application_controller.rb

@@ -91,7 +91,7 @@ class ApplicationController < ActionController::Base
   end
 
   def redirect_unless_moderator
-    return if current_user.moderator? || current_user.admin?
+    return if current_user.moderator?
     redirect_to stream_url, notice: "you need to be an admin or moderator to do that"
   end
 

app/models/role.rb

@@ -19,7 +19,7 @@ class Role < ActiveRecord::Base
   end
 
   def self.moderator?(person)
-    exists?(person_id: person.id, name: "moderator")
+    moderators.exists?(person_id: person.id)
   end
 
   def self.add_moderator(person)

spec/models/role_spec.rb

@@ -6,7 +6,7 @@ describe Role do
   describe "validations" do
     it { should validate_presence_of(:person) }
     it { should validate_uniqueness_of(:name).scoped_to(:person_id) }
-    it { should validate_inclusion_of(:name).in_array(%w(admin spotlight)) }
+    it { should validate_inclusion_of(:name).in_array(%w(admin spotlight moderator)) }
   end
 
   describe "associations" do
@@ -15,6 +15,7 @@ describe Role do
 
   describe "scopes" do
     let!(:admin_role) { person.roles.create(name: "admin") }
+    let!(:moderator_role) { person.roles.create(name: "moderator") }
     let!(:spotlight_role) { person.roles.create(name: "spotlight") }
 
     describe ".admins" do
@@ -22,6 +23,20 @@ describe Role do
         expect(Role.admins).to match_array([admin_role])
       end
     end
+
+    describe ".moderators" do
+      it "should include admins" do
+        expect(Role.moderators).to include(admin_role)
+      end
+
+      it "should include moderators" do
+        expect(Role.moderators).to include(moderator_role)
+      end
+
+      it "should not include normal users" do
+        expect(Role.moderators).to_not include(person)
+      end
+    end
   end
 
   describe ".is_admin?" do