statesman 13.0.0 → 13.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b15bf5cd57e2049db1ae4a0e93f41b16528c8a73c5741cf6337ec280f6ed1041
-  data.tar.gz: 28fba6b8a3304b97fb97a56d145243b5da73f79057fc05449dd3c48a6ea76f72
+  metadata.gz: f93bd968475f6cfd090311f9998a2350993df852aa3796fab5d3b52f962d6147
+  data.tar.gz: 3789c10d55df44ee6e921f9d2297d7d0e0767f42521309ac14aed8e49a0986a8
 SHA512:
-  metadata.gz: 5df7c20cbbc737fc8527d959ad8d894cc0e5fece227099287dc1a96d72cbc62000d5d87b854f4a8483b2ee58c726154d9d50ca7568be27fc4126afe5a94818e5
-  data.tar.gz: 29c4efb1b93e7057ba343b8c4894978e3d9660a53043755f8d26e6378fa804085cd83e8d81203ab7f81a2cbcdd89bab5126042a113d9bb0ca376cb5967b65ba3
+  metadata.gz: 13b9e129ec5bb984066a05c408ee99c9965861775269bab232e6aac86ee75b5c5485a14ccc8476b3b96d0bc383cc68a1f67d32cf9d3eea575478e25ed0245d56
+  data.tar.gz: d88507f263405847ea99f8a7cc88ec4e63c3a82addcef3a20e9c0e1c83372df38868278459702c0855e535d7ebca031e18ddaee32031b0beb162932963156325

data/.gitignore

@@ -66,3 +66,4 @@ Gemfile.lock
 
 # JetBrains
 .idea
+spec/support/generator-tmp/

data/CHANGELOG.md

@@ -5,6 +5,21 @@
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## v13.1.0 9th October 2025
+
+### Added
+
+- State constants for defining available states [#515](https://github.com/gocardless/statesman/pull/515)
+- from_state method to transition history for easier access to previous states [#493](https://github.com/gocardless/statesman/pull/493)
+
+### Fixed
+
+- Fixed transition model generator for namespaced models [#555](https://github.com/gocardless/statesman/pull/555)
+
+### Changed
+
+- Updated README and clarified wording [#444](https://github.com/gocardless/statesman/pull/444)
+
 ## v13.0.0 29th August 2025
 
 ### Changes

data/README.md

@@ -33,7 +33,7 @@ protection.
 To get started, just add Statesman to your `Gemfile`, and then run `bundle`:
 
 ```ruby
-gem 'statesman', '~> 12.0.0'
+gem 'statesman', '~> 13.0.0'
 ```
 
 ## Usage
@@ -233,12 +233,14 @@ or 5. To do that
   t.json :metadata, default: {}
   ```
 
-- Remove the `include Statesman::Adapters::ActiveRecordTransition` statement from
-  your transition model. (If you want to customise your transition class's "updated
-  timestamp column", as described above, you should define a
-  `.updated_timestamp_column` method on your class and return the name of the column
-  as a symbol, or `nil` if you don't want to record an updated timestamp on
-  transitions.)
+* Remove the `include Statesman::Adapters::ActiveRecordTransition` statement from
+  your transition model, which would've instructed ActiveRecord to serialize the
+  metadata.
+* The module that you just removed enables customizing the updatated timestamp column
+  as described above. Having removed it, if you want to customise your transition class's
+  "updated timestamp column", you should define a `.updated_timestamp_column` method on
+  your class and return the name of the column as a symbol, or `nil` if you don't want
+  to record an updated timestamp on transitions.
 
 ## Configuration
 
@@ -407,6 +409,24 @@ Machine.successors
 }
 ```
 
+## Class constants
+Adding a state to a state machine will automatically create a constant for the value, for example:
+```ruby
+class OrderStateMachine
+  include Statesman::Machine
+
+  state :pending, initial: true
+  state :checking_out
+  state :cancelled
+
+  # Constants created as a side effect of adding state
+  transition from: PENDING, to: [CHECKING_OUT, CANCELLED]
+end
+
+OrderStateMachine::PENDING #=> "pending"
+OrderStateMachine::CHECKING_OUT # => "checking_out"
+```
+
 ## Instance methods
 
 ### `Machine#current_state`
@@ -541,8 +561,8 @@ model and passing in `transition_class` and `initial_state` as options.
 
 In 4.1.2 and below, these two options had to be defined as methods on the model,
 but 5.0.0 and above allow this style of configuration as well.
-The old method pollutes the model with extra class methods, and is deprecated,
-to be removed in 6.0.0.
+The old way pollutes the model with extra class methods, and is deprecated,
+to be removed in the future.
 
 ```ruby
 class Order < ActiveRecord::Base

data/lib/generators/statesman/active_record_transition_generator.rb

@@ -1,11 +1,13 @@
 # frozen_string_literal: true
 
 require "rails/generators"
+require "rails/generators/active_record"
 require "generators/statesman/generator_helpers"
 
 module Statesman
   class ActiveRecordTransitionGenerator < Rails::Generators::Base
     include Statesman::GeneratorHelpers
+    include ActiveRecord::Generators::Migration
 
     desc "Create an ActiveRecord-based transition model" \
          "with the required attributes"
@@ -16,14 +18,11 @@ module Statesman
     source_root File.expand_path("templates", __dir__)
 
     def create_model_file
-      template("create_migration.rb.erb", migration_file_name)
       template("active_record_transition_model.rb.erb", model_file_name)
     end
 
-    private
-
-    def migration_file_name
-      "db/migrate/#{next_migration_number}_create_#{table_name}.rb"
+    def create_migration_file
+      migration_template("create_migration.rb.erb", File.join(db_migrate_path, "create_#{table_name}.rb"))
     end
   end
 end

data/lib/generators/statesman/generator_helpers.rb

@@ -10,36 +10,44 @@ module Statesman
       "app/models/#{klass.underscore}.rb"
     end
 
-    def migration_class_name
-      klass.gsub("::", "").pluralize
-    end
-
-    def next_migration_number
-      Time.now.utc.strftime("%Y%m%d%H%M%S")
-    end
-
     def parent_name
-      parent.demodulize.underscore
+      parent.underscore.split("/").join("_")
     end
 
     def parent_table_name
-      parent.demodulize.underscore.pluralize
+      parent.underscore.split("/").join("_").tableize
     end
 
     def parent_id
       parent_name + "_id"
     end
 
-    def table_name
+    def association_name
       klass.demodulize.underscore.pluralize
     end
 
+    def table_name
+      klass.underscore.split("/").join("_").tableize
+    end
+
+    def metadata_column_type
+      if ActiveRecord::Base.connection.supports_json?
+        postgres? ? :jsonb : :json
+      else
+        :text
+      end
+    end
+
     def index_name(index_id)
       "index_#{table_name}_#{index_id}"
     end
 
+    def postgres?
+      configuration.adapter.try(:match, /postgres/)
+    end
+
     def mysql?
-      configuration.try(:[], "adapter").try(:match, /mysql/)
+      configuration.adapter.try(:match, /mysql/)
     end
 
     # [] is deprecated and will be removed in 6.2
@@ -52,11 +60,17 @@ module Statesman
     end
 
     def database_supports_partial_indexes?
-      Statesman::Adapters::ActiveRecord.database_supports_partial_indexes?(klass.constantize)
+      Statesman::Adapters::ActiveRecord.database_supports_partial_indexes?(parent.constantize)
     end
 
     def metadata_default_value
-      Utils.rails_5_or_higher? ? "{}" : "{}".inspect
+      Utils.rails_5_or_higher? ? "{}" : "'{}'"
+    end
+
+    def metadata_column_config
+      return if mysql?
+
+      ", default: #{metadata_default_value}"
     end
   end
 end

data/lib/generators/statesman/migration_generator.rb

@@ -1,12 +1,14 @@
 # frozen_string_literal: true
 
 require "rails/generators"
+require "rails/generators/active_record"
 require "generators/statesman/generator_helpers"
 
 # Add statesman attributes to a pre-existing transition class
 module Statesman
   class MigrationGenerator < Rails::Generators::Base
     include Statesman::GeneratorHelpers
+    include ActiveRecord::Generators::Migration
 
     desc "Add the required Statesman attributes to your transition model"
 
@@ -15,14 +17,8 @@ module Statesman
 
     source_root File.expand_path("templates", __dir__)
 
-    def create_model_file
-      template("update_migration.rb.erb", file_name)
-    end
-
-    private
-
-    def file_name
-      "db/migrate/#{next_migration_number}_add_statesman_to_#{table_name}.rb"
+    def create_migration_file
+      migration_template("update_migration.rb.erb", File.join(db_migrate_path, "add_statesman_to_#{table_name}.rb"))
     end
   end
 end

data/lib/generators/statesman/templates/active_record_transition_model.rb.erb

@@ -9,17 +9,17 @@ class <%= klass %> < <%= Statesman::Utils.rails_5_or_higher? ? 'ApplicationRecor
   # self.updated_timestamp_column = nil
 
 <%- unless Statesman::Utils.rails_4_or_higher? -%>
-  attr_accessible :to_state, :metadata, :sort_key
+  attr_accessible :from_state, :to_state, :metadata, :sort_key
 
 <%- end -%>
-  belongs_to :<%= parent_name %><%= class_name_option %>, inverse_of: :<%= table_name %>
+  belongs_to :<%= parent_name %><%= class_name_option %>, inverse_of: :<%= association_name %>
 
   after_destroy :update_most_recent, if: :most_recent?
 
   private
 
   def update_most_recent
-    last_transition = <%= parent_name %>.<%= table_name %>.order(:sort_key).last
+    last_transition = <%= parent_name %>.<%= association_name %>.order(:sort_key).last
     return unless last_transition.present?
     last_transition.update_column(:most_recent, true)
   end

data/lib/generators/statesman/templates/create_migration.rb.erb

@@ -1,8 +1,9 @@
-class Create<%= migration_class_name %> < ActiveRecord::Migration<%= "[#{ActiveRecord::Migration.current_version}]" if Statesman::Utils.rails_5_or_higher? %>
+class <%= migration_class_name %> < ActiveRecord::Migration<%= "[#{ActiveRecord::Migration.current_version}]" if Statesman::Utils.rails_5_or_higher? %>
   def change
     create_table :<%= table_name %> do |t|
+      t.string :from_state, null: false
       t.string :to_state, null: false
-      t.text :metadata<%= ", default: #{metadata_default_value}" unless mysql? %>
+      t.<%= metadata_column_type %> :metadata<%= metadata_column_config %>
       t.integer :sort_key, null: false
       t.integer :<%= parent_id %>, null: false
       t.boolean :most_recent<%= ", null: false" if database_supports_partial_indexes? %>

data/lib/generators/statesman/templates/update_migration.rb.erb

@@ -1,7 +1,8 @@
-class AddStatesmanTo<%= migration_class_name %> < ActiveRecord::Migration<%= "[#{ActiveRecord::Migration.current_version}]" if Statesman::Utils.rails_5_or_higher? %>
+class <%= migration_class_name %> < ActiveRecord::Migration<%= "[#{ActiveRecord::Migration.current_version}]" if Statesman::Utils.rails_5_or_higher? %>
   def change
+    add_column :<%= table_name %>, :from_state, :string, null: false
     add_column :<%= table_name %>, :to_state, :string, null: false
-    add_column :<%= table_name %>, :metadata, :text<%= ", default: #{metadata_default_value}" unless mysql? %>
+    add_column :<%= table_name %>, :metadata, :<%= metadata_column_type %><%= metadata_column_config%>
     add_column :<%= table_name %>, :sort_key, :integer, null: false
     add_column :<%= table_name %>, :<%= parent_id %>, :integer, null: false
     add_column :<%= table_name %>, :most_recent, null: false

data/lib/statesman/adapters/active_record.rb

@@ -81,7 +81,7 @@ module Statesman
 
       def create_transition(from, to, metadata)
         transition = transitions_for_parent.build(
-          default_transition_attributes(to, metadata),
+          default_transition_attributes(from, to, metadata),
         )
 
         transition_class.transaction(requires_new: true) do
@@ -116,13 +116,19 @@ module Statesman
         transition
       end
 
-      def default_transition_attributes(to, metadata)
-        {
+      def default_transition_attributes(from, to, metadata)
+        attributes = {
           to_state: to,
           sort_key: next_sort_key,
           metadata: metadata,
           most_recent: not_most_recent_value(db_cast: false),
         }
+
+        if @transition_class.has_attribute?(:from_state)
+          attributes[:from_state] = from
+        end
+
+        attributes
       end
 
       def add_after_commit_callback(from, to, transition)

data/lib/statesman/adapters/active_record_transition.rb

@@ -19,6 +19,12 @@ module Statesman
         class_attribute :updated_timestamp_column
         self.updated_timestamp_column = DEFAULT_UPDATED_TIMESTAMP_COLUMN
       end
+
+      def from_state
+        if has_attribute?(:from_state)
+          self[:from_state]
+        end
+      end
     end
   end
 end

data/lib/statesman/adapters/memory.rb

@@ -20,7 +20,7 @@ module Statesman
       def create(from, to, metadata = {})
         from = from.to_s
         to = to.to_s
-        transition = transition_class.new(to, next_sort_key, metadata)
+        transition = transition_class.new(from, to, next_sort_key, metadata)
 
         @observer.execute(:before, from, to, transition)
         @history << transition

data/lib/statesman/adapters/memory_transition.rb

@@ -5,13 +5,15 @@ module Statesman
     class MemoryTransition
       attr_accessor :created_at
       attr_accessor :updated_at
+      attr_accessor :from_state
       attr_accessor :to_state
       attr_accessor :sort_key
       attr_accessor :metadata
 
-      def initialize(to, sort_key, metadata = {})
+      def initialize(from, to, sort_key, metadata = {})
         @created_at = Time.now
         @updated_at = Time.now
+        @from_state = from
         @to_state = to
         @sort_key = sort_key
         @metadata = metadata

data/lib/statesman/exceptions.rb

@@ -11,6 +11,8 @@ module Statesman
 
   class MissingTransitionAssociation < StandardError; end
 
+  class StateConstantConflictError < StandardError; end
+
   class TransitionFailedError < StandardError
     def initialize(from, to)
       @from = from

data/lib/statesman/machine.rb

@@ -39,6 +39,8 @@ module Statesman
           validate_initial_state(name)
           @initial_state = name
         end
+        define_state_constant(name)
+
         states << name
       end
 
@@ -163,6 +165,20 @@ module Statesman
 
       private
 
+      def define_state_constant(state_name)
+        constant_name = state_name.upcase.gsub(/[^A-Z0-9]/, "_")
+
+        if const_defined?(constant_name)
+          return if const_get(constant_name) == state_name
+
+          raise StateConstantConflictError, "Name conflict: '#{name}::#{constant_name}' is already " \
+                                            "defined as '#{const_get(constant_name)}' attempting to redefine " \
+                                            "as '#{state_name}'"
+        else
+          const_set(constant_name, state_name)
+        end
+      end
+
       def add_callback(callback_type: nil, callback_class: nil,
                        from: nil, to: nil, &block)
         validate_callback_type_and_class(callback_type, callback_class)

data/lib/statesman/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Statesman
-  VERSION = "13.0.0"
+  VERSION = "13.1.0"
 end

data/spec/generators/statesman/active_record_transition_generator_spec.rb

@@ -11,23 +11,33 @@ describe Statesman::ActiveRecordTransitionGenerator, type: :generator do
     stub_const("Yummy::BaconTransition", Class.new(ActiveRecord::Base))
   end
 
+  around { |e| Timecop.freeze(Time.parse("2025-01-01 00:00:00"), &e) }
+
   it_behaves_like "a generator" do
-    let(:migration_name) { "db/migrate/create_bacon_transitions.rb" }
+    let(:migration_name) { "db/migrate/create_yummy_bacon_transitions.rb" }
   end
 
   describe "creates a migration" do
-    subject(:migration) { file("db/migrate/#{time}_create_bacon_transitions.rb") }
+    subject(:migration) { file("db/migrate/20250101000000_create_yummy_bacon_transitions.rb") }
 
     before do
-      allow(Time).to receive(:now).and_return(mock_time)
       run_generator %w[Yummy::Bacon Yummy::BaconTransition]
     end
 
-    let(:mock_time) { double("Time", utc: double("UTCTime", strftime: time)) }
-    let(:time) { "5678309" }
-
     it "includes a foreign key" do
-      expect(migration).to contain("add_foreign_key :bacon_transitions, :bacons")
+      expect(migration).to contain("add_foreign_key :yummy_bacon_transitions, :yummy_bacons")
+    end
+
+    it "uses the right column type for Postgres", if: postgres? do
+      expect(migration).to contain("t.jsonb :metadata, default: {}")
+    end
+
+    it "uses the right column type for MySQL", if: mysql? do
+      expect(migration).to contain("t.json :metadata")
+    end
+
+    it "uses the right column type for SQLite", if: sqlite? do
+      expect(migration).to contain("t.json :metadata, default: {}")
     end
   end
 
@@ -36,8 +46,7 @@ describe Statesman::ActiveRecordTransitionGenerator, type: :generator do
 
     before { run_generator %w[Yummy::Bacon Yummy::BaconTransition] }
 
-    it { is_expected.to contain(/:bacon_transition/) }
-    it { is_expected.to_not contain(%r{:yummy/bacon}) }
+    it { is_expected.to contain(/:bacon_transitions/) }
     it { is_expected.to contain(/class_name: 'Yummy::Bacon'/) }
   end
 

data/spec/generators/statesman/migration_generator_spec.rb

@@ -2,6 +2,7 @@
 
 require "support/generators_shared_examples"
 require "generators/statesman/migration_generator"
+require "rails/generators/testing/behavior"
 
 describe Statesman::MigrationGenerator, type: :generator do
   before do
@@ -9,40 +10,46 @@ describe Statesman::MigrationGenerator, type: :generator do
     stub_const("Yummy::BaconTransition", Class.new(ActiveRecord::Base))
   end
 
+  around { |e| Timecop.freeze(Time.parse("2025-01-01 00:00:00"), &e) }
+
   it_behaves_like "a generator" do
-    let(:migration_name) { "db/migrate/add_statesman_to_bacon_transitions.rb" }
+    let(:migration_name) { "db/migrate/20250101000000_add_statesman_to_yummy_bacon_transitions.rb" }
   end
 
   describe "the model contains the correct words" do
     subject(:migration) do
       file(
-        "db/migrate/#{migration_number}_add_statesman_to_bacon_transitions.rb",
+        "db/migrate/20250101000000_add_statesman_to_yummy_bacon_transitions.rb",
       )
     end
 
-    let(:migration_number) { "5678309" }
-
-    let(:mock_time) do
-      double("Time", utc: double("UTCTime", strftime: migration_number))
-    end
-
     before do
-      allow(Time).to receive(:now).and_return(mock_time)
       run_generator %w[Yummy::Bacon Yummy::BaconTransition]
     end
 
-    it { is_expected.to contain(/:bacon_transition/) }
-    it { is_expected.to_not contain(%r{:yummy/bacon}) }
+    it { is_expected.to contain(/:yummy_bacon_transition/) }
     it { is_expected.to contain(/null: false/) }
 
     it "names the sorting index appropriately" do
       expect(migration).
-        to contain("name: \"index_bacon_transitions_parent_sort\"")
+        to contain("name: \"index_yummy_bacon_transitions_parent_sort\"")
     end
 
     it "names the most_recent index appropriately" do
       expect(migration).
-        to contain("name: \"index_bacon_transitions_parent_most_recent\"")
+        to contain("name: \"index_yummy_bacon_transitions_parent_most_recent\"")
+    end
+
+    it "uses the right column type for Postgres", if: postgres? do
+      expect(migration).to contain(":metadata, :jsonb")
+    end
+
+    it "uses the right column type for MySQL", if: mysql? do
+      expect(migration).to contain(":metadata, :json")
+    end
+
+    it "uses the right column type for SQLite", if: sqlite? do
+      expect(migration).to contain(":metadata, :json")
     end
   end
 end

data/spec/spec_helper.rb

@@ -15,6 +15,7 @@ require "rspec/rails"
 require "support/exactly_query_databases"
 require "rspec/its"
 require "pry"
+require "timecop"
 
 RSpec.configure do |config|
   config.raise_errors_for_deprecations!

data/spec/statesman/adapters/memory_transition_spec.rb

@@ -4,10 +4,12 @@ require "statesman/adapters/memory_transition"
 
 describe Statesman::Adapters::MemoryTransition do
   describe "#initialize" do
+    let(:from) { :n }
     let(:to) { :y }
     let(:sort_key) { 0 }
-    let(:create) { described_class.new(to, sort_key) }
+    let(:create) { described_class.new(from, to, sort_key) }
 
+    specify { expect(create.from_state).to equal(from) }
     specify { expect(create.to_state).to equal(to) }
     specify { expect(create.created_at).to be_a(Time) }
     specify { expect(create.updated_at).to be_a(Time) }
@@ -15,7 +17,7 @@ describe Statesman::Adapters::MemoryTransition do
 
     context "with metadata passed" do
       let(:metadata) { { some: :hash } }
-      let(:create) { described_class.new(to, sort_key, metadata) }
+      let(:create) { described_class.new(from, to, sort_key, metadata) }
 
       specify { expect(create.metadata).to eq(metadata) }
     end

data/spec/statesman/exceptions_spec.rb

@@ -51,6 +51,16 @@ describe "Exceptions" do
     end
   end
 
+  describe "StateConstantConflictError" do
+    subject(:error) { Statesman::StateConstantConflictError.new }
+
+    its(:message) { is_expected.to eq("Statesman::StateConstantConflictError") }
+
+    its "string matches its message" do
+      expect(error.to_s).to eq(error.message)
+    end
+  end
+
   describe "TransitionFailedError" do
     subject(:error) { Statesman::TransitionFailedError.new("from", "to") }
 

data/spec/statesman/machine_spec.rb

@@ -1,7 +1,15 @@
 # frozen_string_literal: true
 
 describe Statesman::Machine do
-  let(:machine) { Class.new { include Statesman::Machine } }
+  let(:machine) do
+    Class.new do
+      include Statesman::Machine
+
+      def self.name
+        "MyStateMachine"
+      end
+    end
+  end
   let(:my_model) { Class.new { attr_accessor :current_state }.new }
 
   describe ".state" do
@@ -12,10 +20,14 @@ describe Statesman::Machine do
 
     specify { expect(machine.states).to eq(%w[x y]) }
 
+    specify { expect(machine::X).to eq "x" }
+
+    specify { expect(machine::Y).to eq "y" }
+
     context "initial" do
-      before { machine.state(:x, initial: true) }
+      before { machine.state(:z, initial: true) }
 
-      specify { expect(machine.initial_state).to eq("x") }
+      specify { expect(machine.initial_state).to eq("z") }
 
       context "when an initial state is already defined" do
         it "raises an error" do
@@ -24,6 +36,27 @@ describe Statesman::Machine do
         end
       end
     end
+
+    context "when state name constant is already defined" do
+      context "with the same value" do
+        it "does not raise an error" do
+          machine.const_set(:SOME_CONST, "some_const")
+          expect { machine.state(:some_const) }.to_not raise_error
+        end
+      end
+
+      context "with a different value" do
+        it "raises an error about state constant conflict" do
+          machine.const_set(:SOME_CONST, "some_const_different")
+
+          expect { machine.state(:some_const) }.to raise_error(
+            Statesman::StateConstantConflictError, "Name conflict: 'MyStateMachine::SOME_CONST' is " \
+                                                   "already defined as 'some_const_different' " \
+                                                   "attempting to redefine as 'some_const'"
+          )
+        end
+      end
+    end
   end
 
   describe ".remove_state" do

data/spec/support/active_record.rb

@@ -70,6 +70,7 @@ end
 class CreateMyActiveRecordModelTransitionMigration < MIGRATION_CLASS
   def change
     create_table :my_active_record_model_transitions do |t|
+      t.string  :from_state
       t.string  :to_state
       t.integer :my_active_record_model_id
       t.integer :sort_key
@@ -189,6 +190,7 @@ end
 class CreateOtherActiveRecordModelTransitionMigration < MIGRATION_CLASS
   def change
     create_table :other_active_record_model_transitions do |t|
+      t.string  :from_state
       t.string  :to_state
       t.integer :other_active_record_model_id
       t.integer :sort_key
@@ -284,6 +286,7 @@ end
 class CreateNamespacedARModelTransitionMigration < MIGRATION_CLASS
   def change
     create_table :my_namespace_my_active_record_model_transitions do |t|
+      t.string  :from_state
       t.string  :to_state
       t.integer :my_active_record_model_id
       t.integer :sort_key
@@ -411,3 +414,16 @@ class CreateStiActiveRecordModelTransitionMigration < MIGRATION_CLASS
     end
   end
 end
+
+def postgres?
+  ActiveRecord::Base.connection.adapter_name.match?(/postgres/i)
+end
+
+def mysql?
+  ActiveRecord::Base.connection.adapter_name.match?(/mysql/i) ||
+    ActiveRecord::Base.connection.adapter_name.match?(/trilogy/i)
+end
+
+def sqlite?
+  ActiveRecord::Base.connection.adapter_name.match?(/sqlite/i)
+end

data/spec/support/generators_shared_examples.rb

@@ -12,8 +12,8 @@ shared_examples "a generator" do
 
   let(:gen) { generator %w[Yummy::Bacon Yummy::BaconTransition] }
 
-  it "invokes create_model_file method" do
-    expect(gen).to receive(:create_model_file)
+  it "invokes create_migration_file method" do
+    expect(gen).to receive(:create_migration_file)
     gen.invoke_all
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: statesman
 version: !ruby/object:Gem::Version
-  version: 13.0.0
+  version: 13.1.0
 platform: ruby
 authors:
 - GoCardless