minitest-tagz 1.0.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8743f494fc917d96a638d22e0d98cfaa5e94a3ae
-  data.tar.gz: 1cdd782cfd5edc53da8e3a25c1817e6d05a6023f
+  metadata.gz: 79b82c59731bb66b312c83e52de074ba6e9c09ab
+  data.tar.gz: 08d75b5c1fdebde4f370e700356dfd215e92936c
 SHA512:
-  metadata.gz: 712ba39c5104fe7e02d80eb45026fc414a149853d3e86687a87a5670e301116b92e5e128aa1c24e993a458b9e383cfc8fb64085840ba06aaee176f78bde7ea66
-  data.tar.gz: 95a77a32b2cf90933512784b451e26471cf32b6e15071aec741bf5c23ed1cf1d8ac34f65194fb098056a89d8825f42963f82b3b7cc9758938e60112e52eae267
+  metadata.gz: 480bc3751e1bdf1e4c89fe32f3831373bf329ed2aecaa90df6568999836ad45f5edffbc7f6e2392fe44dba78239d8cb22954cb761db01d4c100b156d60aa2824
+  data.tar.gz: 441ec6724c581f2c5c3652b130a6a81fd972dde66163bacdf655dceae2a4e37226d8c2589e967142e723a440fcabe80f3638a00dcf929b59459af6c562983d08

data/README.md

@@ -89,6 +89,20 @@ class MySpec < Minitest::Spec
 end
 ```
 
+## Debugging
+
+You can save a reference to the tagger and watch the internal state machine:
+
+```rb
+tagger = tag :focus
+it 'should work' do
+  require 'rubygems'; require 'pry'; binding.pry
+end
+
+pry(main)> tagger
+#=> #<Minitest::Tagz::Tagger:0x007fa296102008 @owner=#<Class:0x007fa2957317b8>, @patchers=[Minitest::Tagz::MinitestPatcher], @pending_tags=[:focus], @state="applying_tags">
+```
+
 ## Contributing
 
 1. Fork it ( https://github.com/backupify/minitest-tagz/fork )

data/lib/minitest/tagz/version.rb

@@ -1,5 +1,5 @@
 module Minitest
   module Tagz
-    VERSION = "1.0.0"
+    VERSION = "1.0.1"
   end
 end

data/lib/minitest/tagz.rb

@@ -9,25 +9,26 @@ module Minitest
           "#{owner} >> #{test_name}"
         end
 
-        def patch
-          ::Minitest::Test.singleton_class.class_eval do
-            attr_reader :tag_map
-
-            old_runnable_methods = instance_method(:runnable_methods)
-            define_method :runnable_methods do
-              if Tagz.chosen_tags && Tagz.chosen_tags.any?
-                all_runnables = old_runnable_methods.bind(self).call
-                all_runnables.select do |r|
-                  next false unless MinitestRunnerStrategy.tag_map[MinitestRunnerStrategy.serialize(self, r)]
-                  (Tagz.chosen_tags - MinitestRunnerStrategy.tag_map[MinitestRunnerStrategy.serialize(self, r)]).empty?
-                end
-              else
-                old_runnable_methods.bind(self).call
+        module Patch
+          def runnable_methods
+            all_runnables = super
+            if Tagz.chosen_tags && Tagz.chosen_tags.any?
+              all_runnables.select do |r|
+                serialized = MinitestRunnerStrategy.serialize(self, r)
+                tags_on_runnable = MinitestRunnerStrategy.tag_map[serialized] 
+                next false unless tags_on_runnable
+                (Tagz.chosen_tags - tags_on_runnable).empty?
               end
+            else
+              all_runnables
             end
           end
         end
 
+        def patch
+          ::Minitest::Test.singleton_class.prepend(Patch)
+        end
+
         def tag_map
           @tag_map ||= {}
         end
@@ -133,18 +134,21 @@ module Minitest
         end
 
         def patch_minitest_spec(state_machine)
-          @old_describe = old_describe = Minitest::Spec.method(:describe)
-          Minitest::Spec.class_eval do
-            define_singleton_method(:describe) do |*args, &block|
+          @old_describe = old_describe = Kernel.instance_method(:describe)
+          Kernel.module_eval do
+            define_method(:describe) do |*args, &block|
               state_machine.handle_initial_test_definition do
-                old_describe.unbind.bind(self).call(*args, &block)
+                old_describe.bind(self).call(*args, &block)
               end
             end
           end
         end
 
         def unpatch_minitest_spec
-          Minitest::Spec.define_singleton_method(:describe, @old_describe)
+          old_describe = @old_describe
+          Kernel.module_eval do
+            define_method(:describe, old_describe)
+          end
         end
       end
     end
@@ -156,7 +160,7 @@ module Minitest
       # want to run tests with tags in this set
       # @param [Enumerable<Symbol>] tags - a list of tags you want to test
       def choose_tags(*tags)
-        @chosen_tags = tags
+        @chosen_tags = tags.map(&:to_sym)
       end
 
       def declare_tag_assignment(owner, pending_tags)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: minitest-tagz
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
 platform: ruby
 authors:
 - Josh Bodah