From adcc8b02f16f7051b009d63fae155790b1180998 Mon Sep 17 00:00:00 2001
From: Dan Kubb <dan.kubb@gmail.com>
Date: Mon, 12 Nov 2012 00:00:05 -0800
Subject: [PATCH] Add spec for Yardstick::Rule#hash

* Refactor existing Yardstick::Rule specs to match modern style
---
 spec/semipublic/yardstick/rule_spec.rb | 31 ++++++++++++++++----------
 1 file changed, 19 insertions(+), 12 deletions(-)

diff --git a/spec/semipublic/yardstick/rule_spec.rb b/spec/semipublic/yardstick/rule_spec.rb
index 1616944..23d069f 100644
--- a/spec/semipublic/yardstick/rule_spec.rb
+++ b/spec/semipublic/yardstick/rule_spec.rb
@@ -1,27 +1,34 @@
 require 'spec_helper'
 
 describe Yardstick::Rule do
+  let(:object)      { described_class.new(description) { true } }
+  let(:description) { 'test rule'                               }
+
   describe '#eql?' do
+    subject { object.eql?(other) }
+
     describe 'when rules are equal' do
-      before do
-        @rule  = Yardstick::Rule.new('test rule') { true }
-        @other = Yardstick::Rule.new('test rule') { true }
-      end
+      let(:other) { described_class.new(description) { true } }
 
-      it 'should return true' do
-        @rule.eql?(@other).should be_true
+      it 'is true' do
+        should be(true)
       end
     end
 
     describe 'when rules are not equal' do
-      before do
-        @rule  = Yardstick::Rule.new('test rule')  { true }
-        @other = Yardstick::Rule.new('other rule') { true }
-      end
+      let(:other) { described_class.new('other rule') { true } }
 
-      it 'should return false' do
-        @rule.eql?(@other).should be_false
+      it 'is false' do
+        should be(false)
       end
     end
   end
+
+  describe '#hash' do
+    subject { object.hash }
+
+    it 'is the expected hash' do
+      should == description.hash
+    end
+  end
 end