state_machines-activemodel 0.31.0 → 0.31.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ad141a5029b6489fb99da9c9ef4ad7bf2059d9947e993265225bb4ca88a0ff66
-  data.tar.gz: ef7ae48fd6d4d1f712842e2f0057158e5674b5ba59487cc3b0483c89524374e3
+  metadata.gz: 83ec0c17ef0c2f530f146fd28b90e6474a9ff1fde5e37c07f143652d65c60375
+  data.tar.gz: 9d3c8fb5814ed1e8bf03f543aa9c25ef7ff0b8a241cb05dd20d7b64f15a039fb
 SHA512:
-  metadata.gz: 82b54e46bc9713fdda9c22701d4bec699c8065344bb32e10217dc730a0743b0296bb97245d38ffc8d3b18bd65c770058cb1485a723f2620d66051ce93ce4b106
-  data.tar.gz: 3ae6a52dd12a7c7ff56666fc9d691218f043ba2a0915d80262800a873589c0275a538d5b08ab956d046e23a3ba73e9c6d12ceb732ed0c75ee63d89cb77414b09
+  metadata.gz: f47b157cadbf59127defff4c9f9dade32957b98d439be01963718abe482b0adf638ee8e273347e28fbef56598abbc9e91bfe4d1384f803569b42472abf190d25
+  data.tar.gz: 0cd4c365cf6d108b219bd913598b9bfe1425d7fc9cd62a86961b865cea29483480d0421af07db63daa016050135c44da7578c2cf4131e9ab23ff292b5dd75111

data/lib/state_machines/integrations/active_model/version.rb

@@ -3,7 +3,7 @@
 module StateMachines
   module Integrations
     module ActiveModel
-      VERSION = '0.31.0'
+      VERSION = '0.31.1'
     end
   end
 end

data/test/integration_test.rb

@@ -20,7 +20,7 @@ class IntegrationTest < BaseTestCase
   end
 
   def test_should_not_match_if_class_does_not_include_active_model_features
-    refute StateMachines::Integrations::ActiveModel.matches?(new_model)
+    refute StateMachines::Integrations::ActiveModel.matches?(new_plain_model)
   end
 
   def test_should_have_no_defaults

data/test/machine_initialization_compatibility_test.rb

@@ -6,7 +6,6 @@ class MachineInitializationCompatibilityTest < BaseTestCase
   def setup
     @model = new_model do
       include ActiveModel::Validations
-      attr_accessor :state
     end
 
     @machine = StateMachines::Machine.new(@model, initial: :parked)
@@ -31,9 +30,7 @@ class MachineInitializationCompatibilityTest < BaseTestCase
 
   def test_should_handle_attribute_aliases
     @model.class_eval do
-      def self.attribute_aliases
-        { 'status' => 'state' }
-      end
+      alias_attribute :status, :state
     end
 
     record = @model.new(status: 'idling')

data/test/machine_multiple_test.rb

@@ -5,7 +5,7 @@ require 'test_helper'
 class MachineMultipleTest < BaseTestCase
   def setup
     @model = new_model do
-      model_attribute :status
+      attribute :status, :string
     end
 
     @state_machine = StateMachines::Machine.new(@model, initial: :parked, integration: :active_model)

data/test/machine_with_dirty_attribute_and_custom_attributes_during_loopback_test.rb

@@ -5,13 +5,14 @@ require 'test_helper'
 class MachineWithDirtyAttributeAndCustomAttributesDuringLoopbackTest < BaseTestCase
   def setup
     @model = new_model do
-      include ActiveModel::Dirty
-      model_attribute :status
-      define_attribute_methods [:status]
+      attribute :status, :string
 
       def save
-        super.tap do
+        if valid?
           changes_applied
+          true
+        else
+          false
         end
       end
     end

data/test/machine_with_dirty_attribute_and_state_events_test.rb

@@ -5,12 +5,12 @@ require 'test_helper'
 class MachineWithDirtyAttributeAndStateEventsTest < BaseTestCase
   def setup
     @model = new_model do
-      include ActiveModel::Dirty
-      define_attribute_methods [:state]
-
       def save
-        super.tap do
+        if valid?
           changes_applied
+          true
+        else
+          false
         end
       end
     end

data/test/machine_with_dirty_attributes_and_custom_attribute_test.rb

@@ -5,13 +5,14 @@ require 'test_helper'
 class MachineWithDirtyAttributesAndCustomAttributeTest < BaseTestCase
   def setup
     @model = new_model do
-      include ActiveModel::Dirty
-      model_attribute :status
-      define_attribute_methods [:status]
+      attribute :status, :string
 
       def save
-        super.tap do
+        if valid?
           changes_applied
+          true
+        else
+          false
         end
       end
     end
@@ -26,17 +27,17 @@ class MachineWithDirtyAttributesAndCustomAttributeTest < BaseTestCase
   end
 
   def test_should_include_state_in_changed_attributes
-    assert_equal %w(status), @record.changed
+    assert_equal %w[status], @record.changed
   end
 
   def test_should_track_attribute_change
-    assert_equal %w(parked idling), @record.changes['status']
+    assert_equal %w[parked idling], @record.changes['status']
   end
 
   def test_should_not_reset_changes_on_multiple_transitions
     transition = StateMachines::Transition.new(@record, @machine, :ignite, :idling, :idling)
     transition.perform
 
-    assert_equal %w(parked idling), @record.changes['status']
+    assert_equal %w[parked idling], @record.changes['status']
   end
 end

data/test/machine_with_dirty_attributes_during_loopback_test.rb

@@ -5,12 +5,12 @@ require 'test_helper'
 class MachineWithDirtyAttributesDuringLoopbackTest < BaseTestCase
   def setup
     @model = new_model do
-      include ActiveModel::Dirty
-      define_attribute_methods [:state]
-
       def save
-        super.tap do
+        if valid?
           changes_applied
+          true
+        else
+          false
         end
       end
     end

data/test/machine_with_dirty_attributes_test.rb

@@ -5,12 +5,12 @@ require 'test_helper'
 class MachineWithDirtyAttributesTest < BaseTestCase
   def setup
     @model = new_model do
-      include ActiveModel::Dirty
-      define_attribute_methods [:state]
-
       def save
-        super.tap do
+        if valid?
           changes_applied
+          true
+        else
+          false
         end
       end
     end
@@ -25,17 +25,17 @@ class MachineWithDirtyAttributesTest < BaseTestCase
   end
 
   def test_should_include_state_in_changed_attributes
-    assert_equal %w(state), @record.changed
+    assert_equal %w[state], @record.changed
   end
 
   def test_should_track_attribute_change
-    assert_equal %w(parked idling), @record.changes['state']
+    assert_equal %w[parked idling], @record.changes['state']
   end
 
   def test_should_not_reset_changes_on_multiple_transitions
     transition = StateMachines::Transition.new(@record, @machine, :ignite, :idling, :idling)
     transition.perform
 
-    assert_equal %w(parked idling), @record.changes['state']
+    assert_equal %w[parked idling], @record.changes['state']
   end
 end

data/test/machine_with_initialized_aliased_attribute_test.rb

@@ -6,6 +6,7 @@ class MachineWithInitializedAliasedAttributeTest < BaseTestCase
   def test_should_match_original_attribute_value_with_attribute_methods
     model = new_model do
       include ActiveModel::AttributeMethods
+
       alias_attribute :custom_status, :state
     end
 
@@ -19,7 +20,16 @@ class MachineWithInitializedAliasedAttributeTest < BaseTestCase
   end
 
   def test_should_not_match_original_attribute_value_without_attribute_methods
-    model = new_model do
+    model = new_plain_model do
+      include ActiveModel::Model
+
+      attr_accessor :state
+
+      def self.alias_attribute(new_name, old_name)
+        alias_method new_name, old_name
+        alias_method "#{new_name}=", "#{old_name}="
+      end
+
       alias_attribute :custom_status, :state
     end
 
@@ -32,4 +42,3 @@ class MachineWithInitializedAliasedAttributeTest < BaseTestCase
     refute record.state?(:started)
   end
 end
-

data/test/machine_with_non_model_state_attribute_undefined_test.rb

@@ -4,9 +4,8 @@ require 'test_helper'
 
 class MachineWithNonModelStateAttributeUndefinedTest < BaseTestCase
   def setup
-    @model = new_model do
-      def initialize
-      end
+    @model = new_plain_model do
+      def initialize; end
     end
 
     @machine = StateMachines::Machine.new(@model, :status, initial: :parked, integration: :active_model)

data/test/test_helper.rb

@@ -11,45 +11,40 @@ I18n.enforce_available_locales = true
 
 class BaseTestCase < ActiveSupport::TestCase
   protected
-  # Creates a new ActiveModel model (and the associated table)
-  def new_model(&block)
-    # Simple ActiveModel superclass
-    parent = Class.new do
-      def self.model_attribute(name)
-        define_method(name) { instance_variable_defined?(:"@#{name}") ? instance_variable_get(:"@#{name}") : nil }
-        define_method("#{name}=") do |value|
-          send(:"#{name}_will_change!") if self.class <= ActiveModel::Dirty && value != send(name)
-          instance_variable_set("@#{name}", value)
-        end
-      end
 
-      def self.create
-        object = new
-        object.save
-        object
+  # Creates a plain model without ActiveModel features
+  def new_plain_model(&block)
+    model = Class.new do
+      def self.name
+        'Foo'
       end
+    end
 
-      def initialize(attrs = {})
-        attrs.each { |attr, value| send("#{attr}=", value) }
-      end
+    model.class_eval(&block) if block_given?
 
-      def attributes
-        @attributes ||= {}
-      end
+    model
+  end
 
-      def save
-        true
-      end
-    end
+  # Creates a new ActiveModel model (and the associated table)
+  def new_model(&block)
+    model = Class.new do
+      include ActiveModel::Model
+      include ActiveModel::Attributes
+      include ActiveModel::Dirty
+
+      attribute :state, :string
 
-    model = Class.new(parent) do
       def self.name
         'Foo'
       end
 
-      model_attribute :state
+      def self.create
+        new.tap { |instance| instance.save if instance.respond_to?(:save) }
+      end
     end
+
     model.class_eval(&block) if block_given?
+
     model
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: state_machines-activemodel
 version: !ruby/object:Gem::Version
-  version: 0.31.0
+  version: 0.31.1
 platform: ruby
 authors:
 - Abdelkader Boudih
@@ -16,28 +16,28 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.31.0
+        version: 0.100.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.31.0
+        version: 0.100.1
 - !ruby/object:Gem::Dependency
   name: activemodel
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '7.1'
+        version: '7.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '7.1'
+        version: '7.2'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -162,14 +162,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 3.1.0
+      version: 3.2.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.9
+rubygems_version: 3.6.7
 specification_version: 4
 summary: ActiveModel integration for State Machines
 test_files: