refactor tests (do not use so many stubs and mocks)

spec/database.rb

@@ -1,5 +1,5 @@
 ActiveRecord::Migration.suppress_messages do
-  %w{gates readers writers transients simples simple_new_dsls no_scopes no_direct_assignments thieves localizer_test_models persisted_states provided_and_persisted_states}.each do |table_name|
+  %w{gates readers writers transients simples simple_new_dsls no_scopes no_direct_assignments thieves localizer_test_models persisted_states provided_and_persisted_states with_enums with_true_enums with_false_enums}.each do |table_name|
     ActiveRecord::Migration.create_table table_name, :force => true do |t|
       t.string "aasm_state"
     end

spec/models/persistence.rb

@@ -14,6 +14,54 @@ class Gate < ActiveRecord::Base
   end
 end
 
+class WithEnum < ActiveRecord::Base
+  include AASM
+
+  # Fake this column for testing purposes
+  attr_accessor :aasm_state
+
+  aasm :enum => :test do
+    state :opened
+    state :closed
+
+    event :view do
+      transitions :to => :read, :from => [:needs_attention]
+    end
+  end
+end
+
+class WithTrueEnum < ActiveRecord::Base
+  include AASM
+
+  # Fake this column for testing purposes
+  attr_accessor :aasm_state
+
+  aasm :enum => true do
+    state :opened
+    state :closed
+
+    event :view do
+      transitions :to => :read, :from => [:needs_attention]
+    end
+  end
+end
+
+class WithFalseEnum < ActiveRecord::Base
+  include AASM
+
+  # Fake this column for testing purposes
+  attr_accessor :aasm_state
+
+  aasm :enum => false do
+    state :opened
+    state :closed
+
+    event :view do
+      transitions :to => :read, :from => [:needs_attention]
+    end
+  end
+end
+
 class Reader < ActiveRecord::Base
   include AASM
 

spec/unit/persistence/active_record_persistence_spec.rb

@@ -64,46 +64,37 @@ describe "instance methods" do
   end
 
   describe "aasm_enum" do
-    subject { lambda{ gate.send(:aasm_enum) } }
-
     context "when AASM enum setting contains an explicit enum method name" do
-      let(:enum) { :test }
+      let(:with_enum) { WithEnum.new }
-
-      before :each do
-        AASM::StateMachine[Gate].config.stub(:enum).and_return(enum)
-      end
 
       it "returns whatever value was set in AASM config" do
-        expect(subject.call).to eq enum
+        expect(with_enum.send(:aasm_enum)).to eq :test
       end
     end
 
     context "when AASM enum setting is simply set to true" do
+      let(:with_true_enum) { WithTrueEnum.new }
       before :each do
-        AASM::StateMachine[Gate].config.stub(:enum).and_return(true)
+        WithTrueEnum.aasm.stub(:attribute_name).and_return(:value)
-        Gate.aasm.stub(:attribute_name).and_return(:value)
+        with_true_enum.stub(:aasm_guess_enum_method).and_return(:values)
-        gate.stub(:aasm_guess_enum_method).and_return(:values)
       end
 
       it "infers enum method name from pluralized column name" do
-        expect(subject.call).to eq :values
+        expect(with_true_enum.send(:aasm_enum)).to eq :values
-        expect(gate).to have_received :aasm_guess_enum_method
+        expect(with_true_enum).to have_received :aasm_guess_enum_method
       end
     end
 
     context "when AASM enum setting is explicitly disabled" do
-      before :each do
+      let(:with_false_enum) { WithFalseEnum.new }
-        AASM::StateMachine[Gate].config.stub(:enum).and_return(false)
-      end
 
       it "returns nil" do
-        expect(subject.call).to be_nil
+        expect(with_false_enum.send(:aasm_enum)).to be_nil
       end
     end
 
     context "when AASM enum setting is not enabled" do
       before :each do
-        AASM::StateMachine[Gate].config.stub(:enum).and_return(nil)
         Gate.aasm.stub(:attribute_name).and_return(:value)
       end
 
@@ -114,7 +105,7 @@ describe "instance methods" do
         end
 
         it "infers enum method name from pluralized column name" do
-          expect(subject.call).to eq :values
+          expect(gate.send(:aasm_enum)).to eq :values
           expect(gate).to have_received :aasm_guess_enum_method
         end
       end
@@ -126,7 +117,7 @@ describe "instance methods" do
         end
 
         it "returns nil, as we're not using enum" do
-          expect(subject.call).to be_nil
+          expect(gate.send(:aasm_enum)).to be_nil
         end
       end
     end