mighty_test 0.3.1 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e0a5ab6b78b6d5b8b4a0073ebc938a9fe2dffb4d83a8ae9c9d6dbd7ce665c538
-  data.tar.gz: ff523cac9bb63929f65a434bb01688eb2c1634ea8ae79394db99eff40236049f
+  metadata.gz: 5c8ad068e4c978f1a38d6ec2d6fe2cf74c8712ff2b7216ff0d9075dfc8e94959
+  data.tar.gz: 685f753bac6c76035572e8d99ae67ec97f10544bfcf34afe3c02c563175e8420
 SHA512:
-  metadata.gz: 0c62977fcd28132def5507fd5ea1023109fa04e61aaffc2f765831caf2f633f9ef6efffc308a28244b50c4b401375e22c9e8a8a70b8acbc7731db4dd7650d99e
-  data.tar.gz: 2058631fd6e439361a4269d7374377df0019a386d0e5186a63064866bed9809bf33cfb2ae3d3f98a817b55ee5e190233960e0d0ad01b295592508aa7747c77db
+  metadata.gz: 94f9ab682bdbc1a61de55ebed5f365615ce8ddf111b4f2689186fb1d8ac763fd6a8cb2075460c88b304a9a9e9db353865c1539d3b62a861f29e3f06f58763891
+  data.tar.gz: fb7dfb42e60dc90f35708065e75a3e3bddc01457e5beb81dfd6d6d5c31a6e42ca831af79e92a8d80feb240e5d7d0ae0001226e79689bd1fbcc543cf6e7306910

data/lib/mighty_test/console.rb

@@ -1,4 +1,5 @@
 require "io/console"
+require "io/wait"
 
 module MightyTest
   class Console
@@ -15,11 +16,14 @@ module MightyTest
       true
     end
 
-    def wait_for_keypress
-      with_raw_input do
-        sleep if stdin.eof?
-        stdin.getc
-      end
+    def with_raw_input(&)
+      return yield unless stdin.respond_to?(:raw) && tty?
+
+      stdin.raw(intr: true, &)
+    end
+
+    def read_keypress_nonblock
+      stdin.getc if stdin.wait_readable(0)
     end
 
     def play_sound(name, wait: false)
@@ -55,11 +59,5 @@ module MightyTest
     def tty?
       $stdout.respond_to?(:tty?) && $stdout.tty?
     end
-
-    def with_raw_input(&)
-      return yield unless stdin.respond_to?(:raw) && tty?
-
-      stdin.raw(intr: true, &)
-    end
   end
 end

data/lib/mighty_test/version.rb

@@ -1,3 +1,3 @@
 module MightyTest
-  VERSION = "0.3.1".freeze
+  VERSION = "0.4.0".freeze
 end

data/lib/mighty_test/watcher/event_queue.rb

@@ -0,0 +1,78 @@
+require "io/console"
+
+module MightyTest
+  class Watcher
+    class EventQueue
+      def initialize(console: Console.new, file_system: FileSystem.new)
+        @console = console
+        @file_system = file_system
+        @file_system_queue = Thread::Queue.new
+      end
+
+      def pop
+        console.with_raw_input do
+          until stopped?
+            if (key = console.read_keypress_nonblock)
+              return [:keypress, key]
+            end
+            if (paths = pop_files_changed)
+              return [:file_system_changed, paths]
+            end
+          end
+        end
+      end
+
+      def start
+        raise "Already started" unless stopped?
+
+        @file_system_listener = file_system.listen do |modified, added, _removed|
+          paths = [*modified, *added].uniq
+          file_system_queue.push(paths) unless paths.empty?
+        end
+        true
+      end
+
+      def restart
+        stop
+        start
+      end
+
+      def stop
+        file_system_listener&.stop
+        @file_system_listener = nil
+      end
+
+      def stopped?
+        !file_system_listener
+      end
+
+      private
+
+      attr_reader :console, :file_system, :file_system_listener, :file_system_queue
+
+      def pop_files_changed
+        paths = try_file_system_pop(timeout: 0.2)
+        return if paths.nil?
+
+        paths += file_system_queue.pop until file_system_queue.empty?
+        paths.uniq
+      end
+
+      if RUBY_VERSION.start_with?("3.1.")
+        # TODO: Remove once we drop support for Ruby 3.1
+        require "timeout"
+        def try_file_system_pop(timeout:)
+          Timeout.timeout(timeout) do
+            file_system_queue.pop
+          end
+        rescue Timeout::Error
+          nil
+        end
+      else
+        def try_file_system_pop(timeout:)
+          file_system_queue.pop(timeout:)
+        end
+      end
+    end
+  end
+end

data/lib/mighty_test/watcher.rb

@@ -1,58 +1,41 @@
+require_relative "watcher/event_queue"
+
 module MightyTest
   class Watcher
-    class ListenerTriggered < StandardError
-      attr_reader :paths
-
-      def initialize(paths)
-        @paths = paths
-        super()
-      end
-    end
-
     WATCHING_FOR_CHANGES = 'Watching for changes to source and test files. Press "h" for help or "q" to quit.'.freeze
 
-    def initialize(console: Console.new, extra_args: [], file_system: FileSystem.new, system_proc: method(:system))
+    def initialize(console: Console.new, extra_args: [], event_queue: nil, file_system: nil, system_proc: nil)
       @console = console
       @extra_args = extra_args
-      @file_system = file_system
-      @system_proc = system_proc
+      @file_system = file_system || FileSystem.new
+      @system_proc = system_proc || method(:system)
+      @event_queue = event_queue || EventQueue.new(console: @console, file_system: @file_system)
     end
 
-    def run(iterations: :indefinitely) # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/MethodLength
-      started = false
-      @foreground_thread = Thread.current
-
-      loop_for(iterations) do
-        start_file_system_listener && puts(WATCHING_FOR_CHANGES) unless started
-        started = true
-
-        case console.wait_for_keypress
-        when "\r", "\n"
-          run_all_tests
-        when "a"
-          run_all_tests(flags: ["--all"])
-        when "d"
-          run_matching_test_files_from_git_diff
-        when "h"
-          show_help
-        when "q"
-          file_system_listener.stop
-          break
+    def run
+      event_queue.start
+      puts WATCHING_FOR_CHANGES
+
+      loop do
+        case event_queue.pop
+        in [:file_system_changed, [_, *] => paths] then run_matching_test_files(paths)
+        in [:keypress, "\r" | "\n"] then run_all_tests
+        in [:keypress, "a"] then run_all_tests(flags: ["--all"])
+        in [:keypress, "d"] then run_matching_test_files_from_git_diff
+        in [:keypress, "h"] then show_help
+        in [:keypress, "q"] then break
+        else
+          nil
         end
-      rescue ListenerTriggered => e
-        run_matching_test_files(e.paths)
-        file_system_listener.start if file_system_listener.paused?
-      rescue Interrupt
-        file_system_listener&.stop
-        raise
       end
     ensure
+      event_queue.stop
       puts "\nExiting."
     end
 
     private
 
-    attr_reader :console, :extra_args, :file_system, :file_system_listener, :system_proc, :foreground_thread
+    attr_reader :console, :extra_args, :file_system, :event_queue, :system_proc
 
     def show_help
       console.clear
@@ -109,25 +92,7 @@ module MightyTest
       $stdout.flush
     rescue Interrupt
       # Pressing ctrl-c kills the fs_event background process, so we have to manually restart it.
-      restart_file_system_listener
-    end
-
-    def start_file_system_listener
-      file_system_listener.stop if file_system_listener && !file_system_listener.stopped?
-
-      @file_system_listener = file_system.listen do |modified, added, _removed|
-        paths = [*modified, *added].uniq
-        next if paths.empty?
-
-        # Pause listener so that subsequent changes are queued up while we are running the tests
-        file_system_listener.pause unless file_system_listener.stopped?
-        foreground_thread.raise ListenerTriggered.new(paths)
-      end
-    end
-    alias restart_file_system_listener start_file_system_listener
-
-    def loop_for(iterations, &)
-      iterations == :indefinitely ? loop(&) : iterations.times(&)
+      event_queue.restart
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mighty_test
 version: !ruby/object:Gem::Version
-  version: 0.3.1
+  version: 0.4.0
 platform: ruby
 authors:
 - Matt Brictson
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-07-09 00:00:00.000000000 Z
+date: 2024-08-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: listen
@@ -101,6 +101,7 @@ files:
 - lib/mighty_test/test_parser.rb
 - lib/mighty_test/version.rb
 - lib/mighty_test/watcher.rb
+- lib/mighty_test/watcher/event_queue.rb
 homepage: https://github.com/mattbrictson/mighty_test
 licenses:
 - MIT
@@ -125,7 +126,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.13
+rubygems_version: 3.5.11
 signing_key:
 specification_version: 4
 summary: A modern Minitest runner for TDD, with watch mode and more