added specs and validations for Role model

- added shoulda-matchers gem for one-line specs
- added FactoryGirl syntax methods

Author: MothOnMars

Gemfile

@@ -254,6 +254,7 @@ group :test do
   gem "factory_girl_rails", "4.5.0"
   gem "timecop",            "0.7.3"
   gem "webmock",            "1.20.4", require: false
+  gem "shoulda-matchers",   "2.8.0", require: false
 end
 
 group :development, :test do

Gemfile.lock

@@ -619,6 +619,8 @@ GEM
       rubyzip (~> 1.0)
       websocket (~> 1.0)
     shellany (0.0.1)
+    shoulda-matchers (2.8.0)
+      activesupport (>= 3.0.0)
     sidekiq (3.3.2)
       celluloid (>= 0.16.0)
       connection_pool (>= 2.1.1)
@@ -803,6 +805,7 @@ DEPENDENCIES
   ruby-oembed (= 0.8.12)
   sass-rails (= 5.0.1)
   selenium-webdriver (= 2.45.0)
+  shoulda-matchers (= 2.8.0)
   sidekiq (= 3.3.2)
   sidetiq (= 0.6.3)
   simple_captcha2 (= 0.3.4)

app/models/role.rb

@@ -1,11 +1,16 @@
 #NOTE add the person object you want to attach role to...
+
 class Role < ActiveRecord::Base
   belongs_to :person
 
+  validates :person, presence: true
+  validates :name, uniqueness: {scope: :person_id}
+  validates :name, inclusion: {in: %w(admin spotlight)}
+
   scope :admins, -> { where(name: 'admin') }
 
   def self.is_admin?(person)
-    find_by_person_id_and_name(person.id, 'admin')
+    self.exists?(person_id: person.id, name: 'admin')
   end
 
   def self.add_admin(person)

spec/models/role_spec.rb

@@ -1,5 +1,54 @@
-require 'spec_helper'
+require "spec_helper"
 
-describe Role, :type => :model do
-  skip "add some examples to (or delete) #{__FILE__}"
+describe Role do
+  let(:person) { create(:person) }
+
+  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)) }
+  end
+
+  describe "associations" do
+    it { should belong_to(:person) }
+  end
+
+  describe "scopes" do
+    let!(:admin_role) { person.roles.create(name: "admin") }
+    let!(:spotlight_role) { person.roles.create(name: "spotlight") }
+
+    describe ".admins" do
+      it "includes admin roles" do
+        expect(Role.admins).to match_array([admin_role])
+      end
+    end
+  end
+
+  describe ".is_admin?" do
+    it "defaults to false" do
+      expect(Role.is_admin?(person)).to be false
+    end
+
+    context "when the person is an admin" do
+      before { person.roles.create(name: "admin") }
+
+      it "is true" do
+        expect(Role.is_admin?(person)).to be true
+      end
+    end
+  end
+
+  describe ".add_admin" do
+    it "creates the admin role" do
+      Role.add_admin(person)
+      expect(person.roles.where(name: "admin")).to exist
+    end
+  end
+
+  describe ".add_spotlight" do
+    it "creates the spotlight role" do
+      Role.add_spotlight(person)
+      expect(person.roles.where(name: "spotlight")).to exist
+    end
+  end
 end

spec/spec_helper.rb

@@ -10,6 +10,7 @@
 require 'webmock/rspec'
 require 'factory_girl'
 require 'sidekiq/testing'
+require 'shoulda/matchers'
 
 include HelperMethods
 
@@ -100,4 +101,6 @@ def photo_fixture_name
   config.after(:each) do
     ActionMailer::Base.deliveries.clear
   end
+
+  config.include FactoryGirl::Syntax::Methods
 end