let class names reflect its purpose (in tests)

spec/models/simple_example.rb

@@ -1,4 +1,4 @@
-class Payment
+class SimpleExample
   include AASM
   aasm do
     state :initialised, :initial => true

spec/models/state_machine_with_failed_event.rb

@@ -1,4 +1,4 @@
-class SimpleStateMachine
+class StateMachineWithFailedEvent
   include AASM
 
   aasm do

spec/unit/event_naming_spec.rb

@@ -1,9 +1,9 @@
 require 'spec_helper'
 
 describe "event naming" do
-  let(:state_machine) { SimpleStateMachine.new }
+  let(:state_machine) { StateMachineWithFailedEvent.new }
 
-  it "allows an event of failed without blowing the stack" do
+  it "allows an event of failed without blowing the stack aka stack level too deep" do
     state_machine.failed
 
     expect { state_machine.failed }.to raise_error(AASM::InvalidTransition)

spec/unit/simple_example_spec.rb

@@ -1,42 +1,42 @@
 require 'spec_helper'
 
 describe 'state machine' do
-  let(:payment) {Payment.new}
+  let(:simple) { SimpleExample.new }
 
   it 'starts with an initial state' do
-    expect(payment.aasm.current_state).to eq(:initialised)
-    expect(payment).to respond_to(:initialised?)
-    expect(payment).to be_initialised
+    expect(simple.aasm.current_state).to eq(:initialised)
+    expect(simple).to respond_to(:initialised?)
+    expect(simple).to be_initialised
   end
 
   it 'allows transitions to other states' do
-    expect(payment).to respond_to(:fill_out)
-    expect(payment).to respond_to(:fill_out!)
-    payment.fill_out!
-    expect(payment).to respond_to(:filled_out?)
-    expect(payment).to be_filled_out
+    expect(simple).to respond_to(:fill_out)
+    expect(simple).to respond_to(:fill_out!)
+    simple.fill_out!
+    expect(simple).to respond_to(:filled_out?)
+    expect(simple).to be_filled_out
 
-    expect(payment).to respond_to(:authorise)
-    expect(payment).to respond_to(:authorise!)
-    payment.authorise
-    expect(payment).to respond_to(:authorised?)
-    expect(payment).to be_authorised
+    expect(simple).to respond_to(:authorise)
+    expect(simple).to respond_to(:authorise!)
+    simple.authorise
+    expect(simple).to respond_to(:authorised?)
+    expect(simple).to be_authorised
   end
 
   it 'denies transitions to other states' do
-    expect {payment.authorise}.to raise_error(AASM::InvalidTransition)
-    expect {payment.authorise!}.to raise_error(AASM::InvalidTransition)
-    payment.fill_out
-    expect {payment.fill_out}.to raise_error(AASM::InvalidTransition)
-    expect {payment.fill_out!}.to raise_error(AASM::InvalidTransition)
-    payment.authorise
-    expect {payment.fill_out}.to raise_error(AASM::InvalidTransition)
-    expect {payment.fill_out!}.to raise_error(AASM::InvalidTransition)
+    expect {simple.authorise}.to raise_error(AASM::InvalidTransition)
+    expect {simple.authorise!}.to raise_error(AASM::InvalidTransition)
+    simple.fill_out
+    expect {simple.fill_out}.to raise_error(AASM::InvalidTransition)
+    expect {simple.fill_out!}.to raise_error(AASM::InvalidTransition)
+    simple.authorise
+    expect {simple.fill_out}.to raise_error(AASM::InvalidTransition)
+    expect {simple.fill_out!}.to raise_error(AASM::InvalidTransition)
   end
 
   it 'defines constants for each state name' do
-    expect(Payment::STATE_INITIALISED).to eq(:initialised)
-    expect(Payment::STATE_FILLED_OUT).to eq(:filled_out)
-    expect(Payment::STATE_AUTHORISED).to eq(:authorised)
+    expect(SimpleExample::STATE_INITIALISED).to eq(:initialised)
+    expect(SimpleExample::STATE_FILLED_OUT).to eq(:filled_out)
+    expect(SimpleExample::STATE_AUTHORISED).to eq(:authorised)
   end
 end