smash_the_state 1.4.2 → 1.4.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6f0527df75d9fad254bc32ca2abece0d9a220ee86cb729dbbe87522c29a4150e
-  data.tar.gz: 48ba7b4e6405244a8dbd65d4b3aa6ff9474d15d4283e34d8078e60e23b8be50b
+  metadata.gz: 591b5f213e895130ff25cf2bc2638275d6f89a054abbe01c0fb55ff5c6f3cc84
+  data.tar.gz: 249752c9f3aa7a7efea1adf1dcc4af85bfe135235bf0d07b6961fb3a5eca2a80
 SHA512:
-  metadata.gz: 32815be847a37111eae15e282aab36384914fb99b17e3dad1c27b4bc8269b91fe448ba5daacc59a1a9c3a200fd6ebf55a9a70895bf32d76b8313fd543a93351d
-  data.tar.gz: abf40fd2f6fa63d786d7433fc4fea4437058987bd2e46d44f6fad515b95d6b77da6252b46b80decd6598639b19e5897f57a8a8bf79cc403dd75b32f8f1d81a5f
+  metadata.gz: c74f691d3f1d7d9e6fe008e67d97b7da9cf24cef2d444a57744c1a4ee41f559358d39d6e1e74f5222f93c5d41c1eb9a53f3a61f9635a0cd522798fc77e0d2395
+  data.tar.gz: fbf4c5c27f114281a9db33f2d76d64a50f8f42594f1e2a5a84dfce2414893e1728dae5702e0ea8a65ccd4a3c82790ace05cba760e4634f08ab157e8b115c2e91

data/lib/smash_the_state/operation.rb

@@ -50,6 +50,11 @@ module SmashTheState
 
       def override_step(step_name, options = {}, &block)
         sequence.override_step(step_name, options, &block)
+
+        # also override the dry run step
+        return if dry_run_sequence.steps_for_name(step_name).length.zero?
+
+        dry_run_sequence.override_step(step_name, options, &block)
       end
 
       def error(*steps, &block)
@@ -131,6 +136,9 @@ module SmashTheState
 
       # also copy the state class over
       child_class.instance_variable_set(:@state_class, state_class && state_class.dup)
+
+      # also copy the dry run sequence
+      child_class.dry_run_sequence.steps.concat(dry_run_sequence.steps.map(&:dup))
     end
   end
 end

data/lib/smash_the_state/operation/dry_run.rb

@@ -61,6 +61,7 @@ module SmashTheState
                 sequence.side_effect_free
               end
 
+        seq.run_options[:dry] = true
         run_sequence(seq, params)
       end
       alias dry_call dry_run

data/lib/smash_the_state/operation/sequence.rb

@@ -32,7 +32,6 @@ module SmashTheState
       # return a copy without the steps that produce side-effects
       def side_effect_free
         dup.tap do |seq|
-          seq.run_options[:dry] = true
           seq.instance_eval do
             @steps = seq.steps.select(&:side_effect_free?)
           end

data/lib/smash_the_state/version.rb

@@ -1,3 +1,3 @@
 module SmashTheState
-  VERSION = "1.4.2".freeze
+  VERSION = "1.4.3".freeze
 end

data/spec/unit/operation_spec.rb

@@ -120,9 +120,9 @@ describe SmashTheState::Operation do
         step_name,
         step_options,
         &implementation
-      ).and_return(token_result)
+      )
 
-      expect(klass.override_step(step_name, step_options, &implementation)).to eq(token_result)
+      klass.override_step(step_name, step_options, &implementation)
     end
   end
 
@@ -147,6 +147,15 @@ describe SmashTheState::Operation do
             state.countup += "two"
             state
           end
+
+          dry_run_sequence do
+            step :one
+            step :two
+            step :three do |state|
+              state.countup += "blar"
+              state
+            end
+          end
         end
       end
     end
@@ -189,6 +198,10 @@ describe SmashTheState::Operation do
       too_large = child.call(countup: "")
       expect(too_large.errors[:countup]).to eq(["can't be blank"])
     end
+
+    it "populates the dry run sequence and honors overridden steps" do
+      expect(child.dry_run(countup: "zero").countup).to eq("zerooneoneandahalfblar")
+    end
   end
 
   describe "self#dynamic_schema" do
@@ -223,8 +236,8 @@ describe SmashTheState::Operation do
   describe "self#dry_run_sequence" do
     context "with a custom dry run sequence block" do
       before do
-        klass.step :step_one do |state|
-          state.name = state.name + " one"
+        klass.step :step_one do |state, _original_state, run_options|
+          state.name = state.name + " one " + run_options.to_json
           state
         end
 
@@ -253,16 +266,17 @@ describe SmashTheState::Operation do
       end
 
       it "provides a custom sequence for the dry run that " \
-         "contains only side-effect free steps" do
+         "contains only side-effect free steps, and specifies " \
+         "whether the run is dry the run options" do
         result = klass.call(name: "Sam")
-        expect(result.name).to eq("Sam one two three")
+        expect(result.name).to eq("Sam one {\"dry\":false} two three")
 
         expect(
           klass.dry_run_sequence.steps.all?(&:side_effect_free?)
         ).to eq(true)
 
         dry_result = klass.dry_run(name: "Sam")
-        expect(dry_result.name).to eq("Sam one custom")
+        expect(dry_result.name).to eq("Sam one {\"dry\":true} custom")
       end
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: smash_the_state
 version: !ruby/object:Gem::Version
-  version: 1.4.2
+  version: 1.4.3
 platform: ruby
 authors:
 - Dan Connor
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-02-02 00:00:00.000000000 Z
+date: 2021-04-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: active_model_attributes
@@ -83,8 +83,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project:
-rubygems_version: 2.7.6.2
+rubygems_version: 3.1.4
 signing_key:
 specification_version: 4
 summary: A useful utility for transforming state that provides step sequencing, middleware,