stateful_models 0.0.3 → 0.0.5

data/spec/generators/has_states/install_generator_spec.rb

@@ -4,7 +4,7 @@ require 'rails_helper'
 require 'generators/has_states/install/install_generator'
 
 RSpec.describe HasStates::InstallGenerator do
-  let(:destination) { File.expand_path('../tmp', __dir__) }
+  let(:destination) { File.expand_path('../templates', __dir__) }
 
   before do
     FileUtils.rm_rf(destination)

data/spec/has_states/configuration_spec.rb

@@ -74,6 +74,63 @@ RSpec.describe HasStates::Configuration do
         end
       end.to raise_error(ArgumentError, /must be an ActiveRecord model/)
     end
+
+    it 'allows setting a limit on the number of states' do
+      configuration.configure_model User do |model|
+        model.state_type :kyc do |type|
+          type.statuses = %w[pending completed]
+          type.limit = 2
+        end
+      end
+
+      expect(configuration.limit_for(User, 'kyc')).to eq(2)
+    end
+
+    it 'returns nil for limit when no limit is set' do
+      configuration.configure_model User do |model|
+        model.state_type :kyc do |type|
+          type.statuses = %w[pending completed]
+        end
+      end
+
+      expect(configuration.limit_for(User, 'kyc')).to be_nil
+    end
+
+    context 'when a metadata schema is set' do
+      let(:schema) do
+        {
+          type: :object,
+          properties: {
+            name: { type: :string },
+            age: { type: :integer, minimum: 18 }
+          },
+          required: %i[name age]
+        }
+      end
+
+      before do
+        configuration.configure_model User do |model|
+          model.state_type :kyc do |type|
+            type.statuses = %w[pending completed]
+            type.metadata_schema = schema
+          end
+        end
+      end
+
+      it 'allows setting a metadata schema for validation' do
+        expect(configuration.metadata_schema_for(User, 'kyc')).to eq(schema)
+      end
+    end
+
+    it 'returns nil for metadata_schema when no schema is set' do
+      configuration.configure_model User do |model|
+        model.state_type :kyc do |type|
+          type.statuses = %w[pending completed]
+        end
+      end
+
+      expect(configuration.metadata_schema_for(User, 'kyc')).to be_nil
+    end
   end
 
   describe 'callbacks' do

data/spec/has_states/state_limit_spec.rb

@@ -0,0 +1,107 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe 'State Limit Feature', type: :model do
+  before do
+    HasStates.configuration.clear_callbacks!
+    HasStates.configuration.model_configurations.clear
+
+    HasStates.configure do |config|
+      config.configure_model User do |model|
+        model.state_type :double_limit_state do |type|
+          type.statuses = %w[pending completed]
+          type.limit = 2
+        end
+
+        model.state_type :no_limit_state do |type|
+          type.statuses = %w[pending completed]
+          # No limit set for this type
+        end
+
+        model.state_type :single_limit_state do |type|
+          type.statuses = %w[pending completed]
+          type.limit = 1
+        end
+      end
+    end
+  end
+
+  describe 'state limit validation' do
+    let(:user) { create(:user) }
+
+    context 'with no limit set' do
+      it 'allows creating multiple states of the same type' do
+        # Create multiple onboarding states (no limit)
+        expect do
+          3.times { user.add_state('no_limit_state', status: 'pending') }
+        end.not_to raise_error
+
+        expect(user.states.where(state_type: 'no_limit_state').count).to eq(3)
+      end
+    end
+
+    context 'with a limit set' do
+      it 'allows creating states up to the limit' do
+        # Create double_limit_state states up to the limit (2)
+        expect do
+          2.times { user.add_state('double_limit_state', status: 'pending') }
+        end.not_to raise_error
+
+        expect(user.states.where(state_type: 'double_limit_state').count).to eq(2)
+      end
+
+      it 'prevents creating states beyond the limit' do
+        # Create double_limit_state states up to the limit
+        2.times { user.add_state('double_limit_state', status: 'pending') }
+
+        # Try to create one more (exceeding the limit)
+        expect do
+          user.add_state('double_limit_state', status: 'pending')
+        end.to raise_error(ActiveRecord::RecordInvalid, /maximum number of double_limit_state states/)
+      end
+
+      it 'enforces a limit of 1 correctly' do
+        # Create one state
+        user.add_state('single_limit_state', status: 'pending')
+
+        # Try to create another one (exceeding the limit of 1)
+        expect do
+          user.add_state('single_limit_state', status: 'completed')
+        end.to raise_error(ActiveRecord::RecordInvalid, /maximum number of single_limit_state states/)
+      end
+    end
+
+    context 'with different state types' do
+      it 'applies limits independently to each state type' do
+        # Create states up to the limit for double_limit_state
+        2.times { user.add_state('double_limit_state', status: 'pending') }
+
+        # Create states for no_limit_state (no limit)
+        3.times { user.add_state('no_limit_state', status: 'pending') }
+
+        # Create one state for single_limit_state (limit 1)
+        user.add_state('single_limit_state', status: 'pending')
+
+        # Verify counts
+        expect(user.states.where(state_type: 'double_limit_state').count).to eq(2)
+        expect(user.states.where(state_type: 'no_limit_state').count).to eq(3)
+        expect(user.states.where(state_type: 'single_limit_state').count).to eq(1)
+
+        # Try to exceed limits
+        expect do
+          user.add_state('double_limit_state', status: 'completed')
+        end.to raise_error(ActiveRecord::RecordInvalid, /maximum number of double_limit_state states/)
+
+        expect do
+          user.add_state('single_limit_state', status: 'completed')
+        end.to raise_error(ActiveRecord::RecordInvalid, /maximum number of single_limit_state states/)
+
+        # But can still add more no_limit_state states
+        expect do
+          user.add_state('no_limit_state', status: 'completed')
+        end.not_to raise_error
+      end
+    end
+  end
+end

data/spec/has_states/state_metadata_schema_spec.rb

@@ -0,0 +1,75 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe 'Metadata Schema Validation', type: :model do
+  let(:user) { create(:user) }
+  let(:schema) do
+    {
+      type: :object,
+      properties: {
+        name: { type: :string },
+        age: {
+          type: :integer,
+          minimum: 18
+        }
+      },
+      required: %i[name age]
+    }
+  end
+
+  before do
+    HasStates.configuration.clear_callbacks!
+    HasStates.configuration.model_configurations.clear
+
+    HasStates.configure do |config|
+      config.configure_model User do |model|
+        model.state_type :schema_enabled_state do |type|
+          type.statuses = %w[pending completed rejected]
+          type.metadata_schema = schema
+        end
+
+        model.state_type :no_schema_state do |type|
+          type.statuses = %w[active inactive]
+        end
+      end
+    end
+  end
+
+  describe 'validation' do
+    context 'with a valid schema' do
+      it 'validates valid metadata' do
+        # Invalid metadata (too young < 18)
+        expect do
+          user.add_state('schema_enabled_state', status: 'pending', metadata: { name: 'John Doe', age: 17 })
+        end.to raise_error(ActiveRecord::RecordInvalid, /did not have a minimum value of 18/)
+
+        # Invalid metadata (missing required fields)
+        expect do
+          user.add_state('schema_enabled_state', status: 'pending', metadata: { age: 25 })
+        end.to raise_error(ActiveRecord::RecordInvalid, /did not contain a required property of 'name'/)
+
+        # Valid metadata
+        state = user.add_state('schema_enabled_state', status: 'pending', metadata: { name: 'John Doe', age: 25 })
+
+        expect(state).to be_valid
+        expect(state.persisted?).to be true
+      end
+    end
+
+    context 'with no schema defined' do
+      it 'accepts any metadata for state types without a schema' do
+        state = user.add_state('no_schema_state', status: 'active', metadata: {
+                                 anything: 'goes',
+                                 nested: {
+                                   data: 'is fine too'
+                                 },
+                                 numbers: [1, 2, 3]
+                               })
+
+        expect(state).to be_valid
+        expect(state.persisted?).to be true
+      end
+    end
+  end
+end

data/spec/has_states/state_spec.rb

@@ -289,11 +289,12 @@ RSpec.describe HasStates::State, type: :model do
     class KYCState < HasStates::Base
       validates :metadata, presence: true
       validate :required_metadata_fields
-      
+
       private
-      
+
       def required_metadata_fields
         return if metadata&.key?('document_type')
+
         errors.add(:metadata, 'must include document_type')
       end
     end
@@ -328,7 +329,7 @@ RSpec.describe HasStates::State, type: :model do
 
     it 'defaults to HasStates::State when no state_class specified' do
       state = user.add_state('kyc', status: 'pending')
-      
+
       expect(state).to be_a(HasStates::State)
       expect(state).to be_valid
     end

data/spec/has_states/stateable_spec.rb

@@ -9,7 +9,7 @@ RSpec.describe HasStates::Stateable do
     end
   end
 
-  let(:user) { create(:user) }  # Assuming you have a User factory
+  let(:user) { create(:user) } # Assuming you have a User factory
 
   before do
     # HasStates.configuration.clear_callbacks!
@@ -54,7 +54,7 @@ RSpec.describe HasStates::Stateable do
 
     context 'with custom parameters' do
       let(:metadata) { { key: 'value' } }
-      
+
       it 'creates a new state with provided values' do
         state = user.add_state('test_type', status: 'completed', metadata: metadata)
 
@@ -98,7 +98,7 @@ RSpec.describe HasStates::Stateable do
 
   describe '#current_state' do
     let!(:latest_state) { user.add_state('test_type', status: 'completed') }
-    
+
     before do
       travel_to 1.day.ago do
         3.times { user.add_state('test_type', status: 'pending') }
@@ -110,7 +110,6 @@ RSpec.describe HasStates::Stateable do
       expect(user.current_state('test_type')).to eq latest_state
     end
 
-
     it 'returns nil when no state exists for the given type' do
       expect(user.current_state('nonexistent')).to be_nil
     end
@@ -124,7 +123,7 @@ RSpec.describe HasStates::Stateable do
 
     it 'returns all states for the given type ordered by creation time' do
       test_types = user.current_states('test_type')
-      
+
       expect(test_types.size).to eq 3
       expect(test_types).to eq test_types.sort_by(&:created_at).reverse
     end
@@ -144,8 +143,8 @@ RSpec.describe HasStates::Stateable do
 
     context 'find one methods' do
       it 'defines query one methods for configured states' do
-        expect(user).to respond_to("test_type")
-        expect(user).to respond_to("other_test_type")
+        expect(user).to respond_to('test_type')
+        expect(user).to respond_to('other_test_type')
       end
 
       it 'returns the most recent state for the given status' do
@@ -156,8 +155,8 @@ RSpec.describe HasStates::Stateable do
 
     context 'find many methods' do
       it 'defines query all methods for configured states' do
-        expect(user).to respond_to("test_types")
-        expect(user).to respond_to("other_test_types")
+        expect(user).to respond_to('test_types')
+        expect(user).to respond_to('other_test_types')
       end
 
       it 'returns all states for the given status ordered by creation time' do
@@ -181,4 +180,4 @@ RSpec.describe HasStates::Stateable do
       expect { user.destroy }.to change { HasStates::Base.count }.by(-1)
     end
   end
-end
+end

metadata

@@ -1,15 +1,29 @@
 --- !ruby/object:Gem::Specification
 name: stateful_models
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.5
 platform: ruby
 authors:
 - Sebastian Scholl
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-01-14 00:00:00.000000000 Z
-dependencies: []
+date: 2025-04-14 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: json-schema
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: "\n    HasStates provides state management and event system capabilities
   for Ruby objects.\n    It allows tracking states, state transitions, and triggering
   callbacks on state changes.\n  "
@@ -83,11 +97,13 @@ files:
 - spec/dummy/tmp/local_secret.txt
 - spec/factories/has_states.rb
 - spec/generators/has_states/install_generator_spec.rb
-- spec/generators/tmp/config/initializers/has_states.rb
-- spec/generators/tmp/db/migrate/20250114180401_create_has_states_states.rb
-- spec/generators/tmp/db/migrate/20250114180401_create_indexes_on_has_states_states.rb
+- spec/generators/templates/config/initializers/has_states.rb
+- spec/generators/templates/db/migrate/20250414184759_create_has_states_states.rb
+- spec/generators/templates/db/migrate/20250414184759_create_indexes_on_has_states_states.rb
 - spec/has_states/callback_spec.rb
 - spec/has_states/configuration_spec.rb
+- spec/has_states/state_limit_spec.rb
+- spec/has_states/state_metadata_schema_spec.rb
 - spec/has_states/state_spec.rb
 - spec/has_states/stateable_spec.rb
 - spec/has_states_spec.rb
@@ -121,7 +137,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.22
+rubygems_version: 3.5.9
 signing_key:
 specification_version: 4
 summary: Simple and flexible state management for Ruby objects