fix: update tests to avoid warning syntax (#47)

Author: seuros

.github/workflows/ruby.yml

@@ -12,9 +12,8 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        ruby-version: ['3.2', '3.3', '3.4']
+        ruby-version: ['3.2', '3.4']
         gemfiles:
-          - gemfiles/active_model_7.1.gemfile
           - gemfiles/active_model_7.2.gemfile
           - gemfiles/active_model_8.0.gemfile
           - gemfiles/active_model_edge.gemfile

.rubocop.yml

@@ -0,0 +1,34 @@
+AllCops:
+  NewCops: enable
+  TargetRubyVersion: 3.0
+  SuggestExtensions: false
+
+# Allow nested method definitions in tests - they're used for test setup
+Lint/NestedMethodDefinition:
+  Exclude:
+    - 'test/**/*_test.rb'
+
+# Test setup methods can be longer
+Metrics/MethodLength:
+  Exclude:
+    - 'test/**/*_test.rb'
+    - 'test/test_helper.rb'
+  Max: 10
+
+# Use bracket style for percent literals
+Style/PercentLiteralDelimiters:
+  PreferredDelimiters:
+    '%w': '[]'
+    '%W': '[]'
+    '%i': '[]'
+    '%I': '[]'
+
+# The save method in tests returns a boolean, it's not a predicate method
+Naming/PredicateMethod:
+  Exclude:
+    - 'test/**/*_test.rb'
+
+# In tests, we sometimes need empty initialize methods for stubbing
+Style/RedundantInitialize:
+  Exclude:
+    - 'test/**/*_test.rb'

state_machines-activemodel.gemspec

@@ -16,7 +16,7 @@ Gem::Specification.new do |spec|
   spec.test_files    = Dir.glob('test/**/{*_test,test_*}.rb')
   spec.require_paths = ['lib']
   spec.required_ruby_version     = '>= 3.2.0'
-  spec.add_dependency 'state_machines', '>= 0.100.0'
+  spec.add_dependency 'state_machines', '>= 0.100.1'
   spec.add_dependency 'activemodel', '>= 7.2'
 
   spec.add_development_dependency 'bundler', '>= 1.6'

test/integration_test.rb

@@ -20,7 +20,7 @@ def test_should_match_if_class_includes_validations_feature
   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

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 @@ def test_should_accept_empty_initialization
 
   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')

test/machine_multiple_test.rb

@@ -5,7 +5,7 @@
 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)

test/machine_with_dirty_attribute_and_custom_attributes_during_loopback_test.rb

@@ -5,13 +5,14 @@
 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

test/machine_with_dirty_attribute_and_state_events_test.rb

@@ -5,12 +5,12 @@
 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

test/machine_with_dirty_attributes_and_custom_attribute_test.rb

@@ -5,13 +5,14 @@
 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 @@ def save
   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

test/machine_with_dirty_attributes_during_loopback_test.rb

@@ -5,12 +5,12 @@
 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