minitest-substitute 0.1.0 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 82dcc1be480cea4b024afc50aa111065cb8f6762847d40db97e2eae37823d43e
-  data.tar.gz: f9b44bae2d05d0552cbfdc944531ceb71834ee1e017eea62df34b8d5cd952e9e
+  metadata.gz: 288e4fc7e4e20c4b649237471762f8723ce333f2886d4244048033ea281c4ab2
+  data.tar.gz: ee23d391142761ea6be22ecb306944313e612fc697c4849ab4f70c7a6ce8e844
 SHA512:
-  metadata.gz: ea455efb4e176abb0441e9fe57306628f0b5c232fda56ce4b152152127a902ddf0e75cd1a6a40513e610e14dee5d00cb8b187534fbe2ff5cef609be16e1c2803
-  data.tar.gz: 52ffbfd39b06e813c4f7a44fd4dade18ab7c2e1728a0b0993971bb5d51c4607ac1ca29321a7e308de2244732efc33598e7be61e5b4afc38c06a6ae7fdd9a9ba4
+  metadata.gz: 6d2abd17135f9a750c2e7efb89e1fbc4d2b1ce30a7767e60ad368715059de575435e36856128e5e911c495cfc407169ef68baf9176721ee11eb28f55197fb13f
+  data.tar.gz: ba8d78a68880474bd1051887ad5435e0efdda86aa78c643ddad004bdf606e725ee688385da114abdc4dfb110ad5eb42e5c1a7de146de380e49372296ae6ec9de

data/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 Nothing so far
 
+## 0.2.0
+
+#### Additions
+* Refactor to support class variables as well
+* Use declared subject as default target for with helper
+
 ## 0.1.0
 
 #### Initial Implementation

data/README.md

@@ -41,13 +41,6 @@ require 'minitest/substitute'
 
 ## Usage
 
-This lightweight gem implements features on a "as needed" basis, as of now:
-
-* substitution of instance variables
-* substitution of global variables
-
-Please [create an issue describing your use case](https://github.com/svoop/minitest-substitute/issues) in case you need more features such as the substitution of class variables or substitution via accessor methods.
-
 ### Block
 
 To substitute the value of an instance variable for the duration of a block:
@@ -59,11 +52,29 @@ class Config
   end
 end
 
-Config.instance_variable_get('@version')     # => 1
-with '@version', 2, on: Config do
-  Config.instance_variable_get('@version')   # => 2
+config = Config.new
+
+config.instance_variable_get('@version')     # => 1
+with '@version', 2, on: config do
+  config.instance_variable_get('@version')   # => 2
+end
+config.instance_variable_get('@version')     # => 1
+```
+
+:warning: The target `on` is set explicitly in this case. If you omit this argument, `self` will be used as target by default.
+
+Class variables can be substituted as well:
+
+```ruby
+class Config
+  @@counter = 0
+end
+
+Config.class_variable_get('@@counter')     # => 0
+with '@@counter', 42, on: Config do
+  Config.class_variable_get('@@counter')   # => 42
 end
-Config.instance_variable_get('@version')     # => 1
+Config.class_variable_get('@@counter')     # => 0
 ```
 
 Same goes for global variables:
@@ -128,6 +139,8 @@ with '@version', on: Config do
 end
 ```
 
+:warning: The target `on` is set explicitly in this case. If you omit this argument, `:subject` will be used as target by default which refers to the subject defined by the `subject {}` helper.
+
 It's safe to use multiple `with` statements within one `describe` block.
 
 (The spec integration is borrowed from [minitest-around](https://rubygems.org/gems/minitest-around) for elegance and compatibility.)

data/lib/minitest/substitute/spec.rb

@@ -3,23 +3,23 @@
 require 'minitest/spec'
 
 Minitest::Spec::DSL.class_eval do
-  class Substitutor
-    include Minitest::Substitute::With
-  end
 
   # Substitute the variable value for the duration of the current description
   #
   # @param variable [String] instance or global variable name
   # @param substitute [Object] temporary substitution value
-  # @param on [Object, nil] substitute in the context of this object
+  # @param on [Object, Symbol, nil] substitute in the context of this object
+  #   or in the context of the declared subject if +:subject+ is set
   # @yield temporary substitution value (takes precedence over +substitute+ param)
-  def with(variable, substitute=nil, on: self)
+  def with(variable, substitute=nil, on: :subject)
+    substitute = yield if block_given?
+    substitutor = Minitest::Substitute::Substitutor.new(variable, substitute, on: on)
     before do
-      substitute = yield if block_given?
-      Substitutor.send(:commit_substitution, variable, substitute, on: on)
+      substitutor.on = subject if on == :subject && respond_to?(:subject)
+      substitutor.commit
     end
     after do
-      Substitutor.send(:rollback_substitution, variable, on: on)
+      substitutor.rollback
     end
   end
 
@@ -44,4 +44,5 @@ Minitest::Spec::DSL.class_eval do
       end
     end)  #.then &:include
   end
+
 end

data/lib/minitest/substitute/substitutor.rb

@@ -0,0 +1,40 @@
+# frozen_string_literal: true
+
+module Minitest
+  module Substitute
+    class Substitutor
+
+      EVAL_METHODS = [:instance_eval, :instance_eval, :class_eval].freeze
+
+      attr_writer :on
+
+      def initialize(variable, substitute, on:)
+        @variable, @substitute, @on = variable, substitute, on
+      end
+
+      def commit
+        @original = get
+        set @substitute
+      end
+
+      def rollback
+        set @original
+      end
+
+      private
+
+      def eval_method
+        EVAL_METHODS[@variable.count('@')]
+      end
+
+      def get
+        @on.send(eval_method, @variable.to_s)
+      end
+
+      def set(value)
+        @on.send(eval_method, "#{@variable} = value")
+      end
+
+    end
+  end
+end

data/lib/minitest/substitute/version.rb

@@ -2,6 +2,6 @@
 
 module Minitest
   module Substitute
-    VERSION = '0.1.0'
+    VERSION = '0.2.0'
   end
 end

data/lib/minitest/substitute/with.rb

@@ -3,10 +3,6 @@
 module Minitest
   module Substitute
     module With
-      @original = {}
-      class << self
-        attr_accessor :original
-      end
 
       # Substitute the variable value for the duration of the given block
       #
@@ -16,22 +12,13 @@ module Minitest
       # @yield block during which the substitution is made
       # @return [Object] return value of the yielded block
       def with(variable, substitute, on: self)
-        commit_substitution(variable, substitute, on: on)
+        substitutor = Minitest::Substitute::Substitutor.new(variable, substitute, on: on)
+        substitutor.commit
         yield.tap do
-          rollback_substitution(variable, on: on)
+          substitutor.rollback
         end
       end
 
-      private
-
-      def commit_substitution(variable, substitute, on:)
-        Minitest::Substitute::With.original[variable.hash] = on.instance_eval variable.to_s
-        on.instance_eval "#{variable} = substitute"
-      end
-
-      def rollback_substitution(variable, on:)
-        on.instance_eval "#{variable} = Minitest::Substitute::With.original.delete(variable.hash)"
-      end
     end
   end
 end

data/lib/minitest/substitute.rb

@@ -2,6 +2,7 @@
 
 require_relative 'substitute/version'
 
+require_relative 'substitute/substitutor'
 require_relative 'substitute/with'
 require_relative 'substitute/spec'
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: minitest-substitute
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
 - Sven Schwyn
@@ -29,7 +29,7 @@ cert_chain:
   kAyiRqgxF4dJviwtqI7mZIomWL63+kXLgjOjMe1SHxfIPo/0ji6+r1p4KYa7o41v
   fwIwU1MKlFBdsjkd
   -----END CERTIFICATE-----
-date: 2023-08-23 00:00:00.000000000 Z
+date: 2023-08-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: minitest
@@ -157,7 +157,11 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: ''
+description: |
+  Simple Minitest helper to replace values such as an instance variable of an
+  object or an environment variable for the duration of a block or a group of
+  tests. This comes in very handy when you have to derive from default
+  configuration in order to test some aspects of your code.
 email:
 - ruby@bitcetera.com
 executables: []
@@ -172,6 +176,7 @@ files:
 - README.md
 - lib/minitest/substitute.rb
 - lib/minitest/substitute/spec.rb
+- lib/minitest/substitute/substitutor.rb
 - lib/minitest/substitute/version.rb
 - lib/minitest/substitute/with.rb
 homepage: https://github.com/svoop/minitest-substitute