stepper_motor 0.1.12 → 0.1.16

data/test/stepper_motor/journey/if_condition_test.rb

@@ -0,0 +1,355 @@
+# frozen_string_literal: true
+
+require "test_helper"
+require "minitest/mock"
+
+class IfConditionTest < ActiveSupport::TestCase
+  include ActiveJob::TestHelper
+  include SideEffects::TestHelper
+  include StepperMotor::TestHelper
+
+  test "supports skip_if: with symbol condition that returns false (performs step)" do
+    journey_class = create_journey_subclass do
+      step :one, skip_if: :should_skip do
+        SideEffects.touch!("step executed")
+      end
+
+      def should_skip
+        false
+      end
+    end
+
+    journey = journey_class.create!
+    assert_produced_side_effects("step executed") do
+      journey.perform_next_step!
+    end
+    assert journey.finished?
+  end
+
+  test "supports skip_if: with symbol condition that returns true (skips step)" do
+    journey_class = create_journey_subclass do
+      step :one, skip_if: :should_skip do
+        SideEffects.touch!("step executed")
+      end
+
+      step :two do
+        SideEffects.touch!("second step executed")
+      end
+
+      def should_skip
+        true
+      end
+    end
+
+    journey = journey_class.create!
+    speedrun_journey(journey)
+    assert SideEffects.produced?("second step executed")
+    refute SideEffects.produced?("step executed")
+  end
+
+  test "supports skip_if: with block condition that returns false (performs step)" do
+    journey_class = create_journey_subclass do
+      step :one, skip_if: -> { hero.nil? } do
+        SideEffects.touch!("step executed")
+      end
+    end
+
+    journey = journey_class.create!(hero: create_journey_subclass.create!)
+    assert_produced_side_effects("step executed") do
+      journey.perform_next_step!
+    end
+    assert journey.finished?
+  end
+
+  test "supports skip_if: with block condition that returns true (skips step)" do
+    journey_class = create_journey_subclass do
+      step :one, skip_if: -> { hero.present? } do
+        SideEffects.touch!("step executed")
+      end
+
+      step :two do
+        SideEffects.touch!("second step executed")
+      end
+    end
+
+    journey = journey_class.create!(hero: create_journey_subclass.create!)
+    speedrun_journey(journey)
+    assert SideEffects.produced?("second step executed")
+    refute SideEffects.produced?("step executed")
+  end
+
+  test "supports skip_if: with block condition that accesses journey instance variables" do
+    journey_class = create_journey_subclass do
+      step :one, skip_if: -> { @condition_met } do
+        SideEffects.touch!("step executed")
+      end
+
+      step :two do
+        SideEffects.touch!("second step executed")
+      end
+
+      def initialize(*args)
+        super
+        @condition_met = true
+      end
+    end
+
+    journey = journey_class.create!
+    speedrun_journey(journey)
+    assert SideEffects.produced?("second step executed")
+    refute SideEffects.produced?("step executed")
+  end
+
+  test "supports skip_if: with block condition that can be changed during journey execution" do
+    journey_class = create_journey_subclass do
+      step :one, skip_if: -> { @condition_met } do
+        SideEffects.touch!("first step executed")
+      end
+
+      step :two do
+        SideEffects.touch!("second step executed")
+        @condition_met = false
+      end
+
+      step :three, skip_if: -> { @condition_met } do
+        SideEffects.touch!("third step executed")
+      end
+
+      def initialize(*args)
+        super
+        @condition_met = true
+      end
+    end
+
+    journey = journey_class.create!
+    speedrun_journey(journey)
+    assert SideEffects.produced?("second step executed")
+    assert SideEffects.produced?("third step executed")
+    refute SideEffects.produced?("first step executed")
+  end
+
+  test "skips step when skip_if: condition is true and continues to next step" do
+    journey_class = create_journey_subclass do
+      step :one, skip_if: :true_condition do
+        SideEffects.touch!("first step executed")
+      end
+
+      step :two do
+        SideEffects.touch!("second step executed")
+      end
+
+      step :three do
+        SideEffects.touch!("third step executed")
+      end
+
+      def true_condition
+        true
+      end
+    end
+
+    journey = journey_class.create!
+    speedrun_journey(journey)
+    assert SideEffects.produced?("second step executed")
+    assert SideEffects.produced?("third step executed")
+    refute SideEffects.produced?("first step executed")
+  end
+
+  test "skips step when skip_if: condition is true and finishes journey if no more steps" do
+    journey_class = create_journey_subclass do
+      step :one, skip_if: :true_condition do
+        SideEffects.touch!("step executed")
+      end
+
+      def true_condition
+        true
+      end
+    end
+
+    journey = journey_class.create!
+    speedrun_journey(journey)
+    refute SideEffects.produced?("step executed")
+  end
+
+  test "supports skip_if: with literal false (performs step)" do
+    journey_class = create_journey_subclass do
+      step :one, skip_if: false do
+        SideEffects.touch!("step executed")
+      end
+    end
+
+    journey = journey_class.create!
+    assert_produced_side_effects("step executed") do
+      journey.perform_next_step!
+    end
+    assert journey.finished?
+  end
+
+  test "supports skip_if: with literal true (skips step)" do
+    journey_class = create_journey_subclass do
+      step :one, skip_if: true do
+        SideEffects.touch!("step executed")
+      end
+
+      step :two do
+        SideEffects.touch!("second step executed")
+      end
+    end
+
+    journey = journey_class.create!
+    speedrun_journey(journey)
+    assert SideEffects.produced?("second step executed")
+    refute SideEffects.produced?("step executed")
+  end
+
+  test "supports skip_if: with literal true and finishes journey if no more steps" do
+    journey_class = create_journey_subclass do
+      step :one, skip_if: true do
+        SideEffects.touch!("step executed")
+      end
+    end
+
+    journey = journey_class.create!
+    speedrun_journey(journey)
+    refute SideEffects.produced?("step executed")
+  end
+
+  test "defaults to false when skip_if: is not specified" do
+    journey_class = create_journey_subclass do
+      step :one do
+        SideEffects.touch!("step executed")
+      end
+    end
+
+    journey = journey_class.create!
+    assert_produced_side_effects("step executed") do
+      journey.perform_next_step!
+    end
+    assert journey.finished?
+  end
+
+  test "treats nil as false in skip_if condition (performs step)" do
+    journey_class = create_journey_subclass do
+      step :one, skip_if: nil do
+        SideEffects.touch!("step executed")
+      end
+
+      step :two do
+        SideEffects.touch!("second step executed")
+      end
+    end
+
+    journey = journey_class.create!
+    speedrun_journey(journey)
+    assert SideEffects.produced?("step executed")
+    assert SideEffects.produced?("second step executed")
+  end
+
+  test "raises ArgumentError when skip_if: condition is neither symbol nor callable" do
+    assert_raises(ArgumentError) do
+      create_journey_subclass do
+        step :one, skip_if: "not a symbol or callable" do
+          # noop
+        end
+      end
+    end
+  end
+
+  test "passes skip_if: parameter to step definition" do
+    step_def = StepperMotor::Step.new(name: "a_step", seq: 1, on_exception: :reattempt!)
+    assert_skip_if_parameter = ->(**options) {
+      assert options.key?(:skip_if)
+      assert_equal :test_condition, options[:skip_if]
+      # Return the original definition
+      step_def
+    }
+
+    StepperMotor::Step.stub :new, assert_skip_if_parameter do
+      create_journey_subclass do
+        step :test_step, skip_if: :test_condition do
+          # noop
+        end
+      end
+    end
+  end
+
+  # Backward compatibility tests for if: parameter
+  test "supports if: with symbol condition that returns true (performs step, backward compatibility)" do
+    journey_class = create_journey_subclass do
+      step :one, if: :should_run do
+        SideEffects.touch!("step executed")
+      end
+
+      def should_run
+        true
+      end
+    end
+
+    journey = journey_class.create!
+    assert_produced_side_effects("step executed") do
+      journey.perform_next_step!
+    end
+    assert journey.finished?
+  end
+
+  test "supports if: with symbol condition that returns false (skips step, backward compatibility)" do
+    journey_class = create_journey_subclass do
+      step :one, if: :should_run do
+        SideEffects.touch!("step executed")
+      end
+
+      step :two do
+        SideEffects.touch!("second step executed")
+      end
+
+      def should_run
+        false
+      end
+    end
+
+    journey = journey_class.create!
+    speedrun_journey(journey)
+    assert SideEffects.produced?("second step executed")
+    refute SideEffects.produced?("step executed")
+  end
+
+  test "supports if: with literal true (performs step, backward compatibility)" do
+    journey_class = create_journey_subclass do
+      step :one, if: true do
+        SideEffects.touch!("step executed")
+      end
+    end
+
+    journey = journey_class.create!
+    assert_produced_side_effects("step executed") do
+      journey.perform_next_step!
+    end
+    assert journey.finished?
+  end
+
+  test "supports if: with literal false (skips step, backward compatibility)" do
+    journey_class = create_journey_subclass do
+      step :one, if: false do
+        SideEffects.touch!("step executed")
+      end
+
+      step :two do
+        SideEffects.touch!("second step executed")
+      end
+    end
+
+    journey = journey_class.create!
+    speedrun_journey(journey)
+    assert SideEffects.produced?("second step executed")
+    refute SideEffects.produced?("step executed")
+  end
+
+  test "raises error when both skip_if: and if: are specified" do
+    assert_raises(StepperMotor::StepConfigurationError) do
+      create_journey_subclass do
+        step :one, skip_if: :condition1, if: :condition2 do
+          # noop
+        end
+      end
+    end
+  end
+end

data/test/stepper_motor/journey/step_definition_test.rb

@@ -6,6 +6,7 @@ require "minitest/mock"
 class StepDefinitionTest < ActiveSupport::TestCase
   include ActiveJob::TestHelper
   include SideEffects::TestHelper
+  include StepperMotor::TestHelper
 
   test "requires either a block or a name" do
     assert_raises(StepperMotor::StepConfigurationError) do

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: stepper_motor
 version: !ruby/object:Gem::Version
-  version: 0.1.12
+  version: 0.1.16
 platform: ruby
 authors:
 - Julik Tarkhanov
-autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-06-08 00:00:00.000000000 Z
+date: 2025-06-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -94,6 +93,34 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: mysql2
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: pg
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -206,6 +233,7 @@ files:
 - lib/generators/stepper_motor_migration_002.rb.erb
 - lib/generators/stepper_motor_migration_003.rb.erb
 - lib/generators/stepper_motor_migration_004.rb.erb
+- lib/generators/stepper_motor_migration_005.rb.erb
 - lib/stepper_motor.rb
 - lib/stepper_motor/base_job.rb
 - lib/stepper_motor/cyclic_scheduler.rb
@@ -246,6 +274,9 @@ files:
 - test/dummy/config/application.rb
 - test/dummy/config/boot.rb
 - test/dummy/config/cable.yml
+- test/dummy/config/database.mysql2.yml
+- test/dummy/config/database.postgres.yml
+- test/dummy/config/database.sqlite3.yml
 - test/dummy/config/database.yml
 - test/dummy/config/environment.rb
 - test/dummy/config/environments/development.rb
@@ -263,6 +294,7 @@ files:
 - test/dummy/db/migrate/20250525132524_stepper_motor_migration_002.rb
 - test/dummy/db/migrate/20250525132525_stepper_motor_migration_003.rb
 - test/dummy/db/migrate/20250528141038_stepper_motor_migration_004.rb
+- test/dummy/db/migrate/20250609221201_stepper_motor_migration_005.rb
 - test/dummy/db/schema.rb
 - test/dummy/public/400.html
 - test/dummy/public/404.html
@@ -280,6 +312,7 @@ files:
 - test/stepper_motor/journey/exception_handling_test.rb
 - test/stepper_motor/journey/flow_control_test.rb
 - test/stepper_motor/journey/idempotency_test.rb
+- test/stepper_motor/journey/if_condition_test.rb
 - test/stepper_motor/journey/step_definition_test.rb
 - test/stepper_motor/journey/uniqueness_test.rb
 - test/stepper_motor/journey_test.rb
@@ -297,7 +330,6 @@ metadata:
   homepage_uri: https://steppermotor.dev
   source_code_uri: https://github.com/stepper-motor/stepper_motor
   changelog_uri: https://github.com/stepper-motor/stepper_motor/blob/main/CHANGELOG.md
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -312,8 +344,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.10
-signing_key:
+rubygems_version: 3.6.6
 specification_version: 4
 summary: Effortless step workflows that embed nicely inside Rails
 test_files: []