Skip HashWithIndifferentAccess specs if it isn't present

spec/fabrication/support_spec.rb

@@ -1,49 +1,44 @@
 require 'spec_helper'
 
 describe Fabrication::Support do
-
-  describe ".class_for" do
-
-    context "with a class that exists" do
-
-      it "returns the class for a class" do
+  describe '.class_for' do
+    context 'with a class that exists' do
+      it 'returns the class for a class' do
         expect(Fabrication::Support.class_for(Object)).to eq(Object)
       end
 
-      it "returns the class for a class name string" do
+      it 'returns the class for a class name string' do
         expect(Fabrication::Support.class_for('object')).to eq(Object)
       end
 
-      it "returns the class for a class name symbol" do
+      it 'returns the class for a class name symbol' do
         expect(Fabrication::Support.class_for(:object)).to eq(Object)
       end
-
     end
 
     context "with a class that doesn't exist" do
-
-      it "returns nil for a class name string" do
-        expect { Fabrication::Support.class_for('your_mom') }.
-          to raise_error(Fabrication::UnfabricatableError)
+      it 'returns nil for a class name string' do
+        expect { Fabrication::Support.class_for('your_mom') }
+          .to raise_error(Fabrication::UnfabricatableError)
       end
 
-      it "returns nil for a class name symbol" do
-        expect { Fabrication::Support.class_for(:your_mom) }.
-          to raise_error(Fabrication::UnfabricatableError)
+      it 'returns nil for a class name symbol' do
+        expect { Fabrication::Support.class_for(:your_mom) }
+          .to raise_error(Fabrication::UnfabricatableError)
       end
 
-      context "and custom const_missing is defined" do
+      context 'and custom const_missing is defined' do
         before do
           module Family
-            def self.const_missing(name)
-              raise NameError, "original message"
+            def self.const_missing(_name)
+              raise NameError, 'original message'
             end
           end
         end
 
-        it "raises an exception with the message from the original exception" do
-          expect { Fabrication::Support.class_for("Family::Mom") }.
-            to raise_error(Fabrication::UnfabricatableError, /original message/)
+        it 'raises an exception with the message from the original exception' do
+          expect { Fabrication::Support.class_for('Family::Mom') }
+            .to raise_error(Fabrication::UnfabricatableError, /original message/)
         end
       end
     end
@@ -52,6 +47,10 @@ def self.const_missing(name)
   describe '.hash_class', depends_on: :active_support do
     subject { Fabrication::Support.hash_class }
 
+    before do
+      pending unless defined?(HashWithIndifferentAccess)
+    end
+
     context 'with HashWithIndifferentAccess defined' do
       it { should == HashWithIndifferentAccess }
     end
@@ -62,12 +61,13 @@ def self.const_missing(name)
         Fabrication::Support.instance_variable_set('@hash_class', nil)
         Object.send(:remove_const, :HashWithIndifferentAccess)
       end
+
       after do
         Fabrication::Support.instance_variable_set('@hash_class', nil)
         HashWithIndifferentAccess = TempHashWithIndifferentAccess
       end
+
       it { should == Hash }
     end
   end
-
 end