retest 2.1.0 → 2.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7eb4992f6d7149b443a2833f6481694b551fcf3ca98355acbe014f9209ed68c0
-  data.tar.gz: f1c50bb7340803ae2abc65178c204b983fe0a99471ac502ad0726d94d7219638
+  metadata.gz: af4a7f8046c06b50e0d59639373c1cd9fac41a99f3a68a7529b98095b2ef10ae
+  data.tar.gz: 93e64c8ac28fe1ae32bc79e5719a8a775c5524544a0c7bd96ede1a3668dd8c6c
 SHA512:
-  metadata.gz: 69b59be1bf2c414d66fa9b80eb656e430d32ede6fd88734169096d7b0b7c1f64d48edfcceabc4b48676a5dcf36cf307379926030b75e6fffa95525a504c9e279
-  data.tar.gz: c063a98b171140f21e66567a2c37d1014f8a3aed5c85f935e6a2e99f8d0845db84c046ccd47f7e739cc115e894111afc07d08b3564f02bdce09698c1a6674f03
+  metadata.gz: b7142bef6579a44f0534253113f4646823a83fbe50b9417cdb0009ba9d99d01fe800fbf93500657343a626cd60efac2848852b1527f355144d901f128d8ea45d
+  data.tar.gz: 48c33eb530ca3752430afd48b3968fb8359b6fb88007e304b7b47b14da2d7283a4401f46d034429623efd804165c26e0928734b541f3f223f2f9f8ba12db7e96

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    retest (2.1.0)
+    retest (2.2.0)
       listen (~> 3.9)
       observer (~> 0.1)
       string-similarity (~> 2.1)

data/exe/retest

@@ -4,12 +4,6 @@ require 'retest'
 
 $stdout.sync = true
 listen_rd, listen_wr = IO.pipe
-Signal.trap(:INT) do
-  puts "Goodbye"
-  listen_rd.close
-  listen_wr.close
-  exit
-end
 
 options = Retest::Options.new(ARGV)
 
@@ -50,6 +44,16 @@ end
 
 puts "Command: '#{command}'"
 
+# === TRAP INTERRUPTION ===
+Signal.trap(:INT) do
+  if !runner.interrupt_run
+    puts "Goodbye"
+    listen_rd.close
+    listen_wr.close
+    exit
+  end
+end
+
 # === DIFF ACTION ===
 if options.params[:diff]
   program.diff(options.params[:diff])
@@ -98,7 +102,7 @@ def run_command(input:, program:)
     Process.kill("INT", 0)
   when 'r', 'reset'
     program.reset_selection
-    puts "command reset to '#{program.runner.command.to_s}'"
+    puts "command reset to '#{program.runner.command}'"
   when 'f', 'force'
     require 'tty-prompt'
     prompt = TTY::Prompt.new

data/lib/retest/runner.rb

@@ -1,3 +1,4 @@
+require 'forwardable'
 require_relative "runner/cached_test_file"
 
 module Retest
@@ -17,6 +18,14 @@ module Retest
       end
     end
 
+    def interrupt_run
+      return false unless @pid
+
+      Process.kill('INT', @pid)
+    rescue Errno::ESRCH
+      false
+    end
+
     def run_last_command
       unless last_command
         return log('Error - Not enough information to run a command. Please trigger a run first.')
@@ -89,7 +98,10 @@ module Retest
 
     def system_run(command)
       log("\n")
-      result = system(command) ? :tests_pass : :tests_fail
+      @pid = spawn(command)
+      Process.wait
+      @pid = nil
+      result = $?.exitstatus&.zero? ? :tests_pass : :tests_fail
       changed
       notify_observers(result)
     end

data/lib/retest/version.rb

@@ -1,3 +1,3 @@
 module Retest
-  VERSION = "2.1.0"
+  VERSION = "2.2.0"
 end

data/lib/retest/version_control/git.rb

@@ -21,7 +21,7 @@ module Retest
       end
 
       def diff_files(branch)
-        `git diff #{branch} --name-only --diff-filter=ACMRT -z`.split("\x0")
+        `git diff #{branch}...HEAD --name-only --diff-filter=ACMRT -z`.split("\x0")
       end
 
       def untracked_files

data/lib/retest/watcher.rb

@@ -1,3 +1,5 @@
+require 'pathname'
+
 module Retest
   module Watcher
     def self.for(watcher)
@@ -24,15 +26,49 @@ module Retest
         true
       end
 
-      def self.watch(dir:, extensions:, polling: false)
-        Listen.to(dir, only: extensions_regex(extensions), relative: true, polling: polling) do |modified, added, removed|
-          yield modified, added, removed
-        end.start
-      end
-
       def self.extensions_regex(extensions)
         Regexp.new("\\.(?:#{extensions.join("|")})$")
       end
+
+      def self.watch(dir:, extensions:, polling: false)
+        executable = File.expand_path("../../scripts/listen", __FILE__)
+        command = "#{executable} --exts #{extensions.join(',')} -w #{dir} --polling #{polling}"
+
+        watch_rd, watch_wr = IO.pipe
+        # Process needs its own process group otherwise the process gets killed on INT signal
+        # We need the process to still run when trying to stop the current test run
+        # Maybe there is another way to prevent killing these but for now a new process groups works
+        # Process group created with: pgroup: true
+        pid = Process.spawn(command, out: watch_wr, pgroup: true)
+
+        at_exit do
+          Process.kill("TERM", pid) if pid
+          watch_rd.close
+          watch_wr.close
+        end
+
+        Thread.new do
+          loop do
+            ready = IO.select([watch_rd])
+            readable_connections = ready[0]
+            readable_connections.each do |conn|
+              data = conn.readpartial(4096)
+              change = /^(?<action>create|remove|modify):(?<path>.*)/.match(data.strip)
+
+              next unless change
+
+              modified, added, removed = result = [[], [], []]
+              case change[:action]
+              when 'modify' then modified << change[:path]
+              when 'create' then added << change[:path]
+              when 'remove' then removed << change[:path]
+              end
+
+              yield result
+            end
+          end
+        end
+      end
     end
 
     module Watchexec
@@ -42,10 +78,14 @@ module Retest
 
       def self.watch(dir:, extensions:, polling: false)
         command = "watchexec --exts #{extensions.join(',')} -w #{dir} --emit-events-to stdio --no-meta --only-emit-events"
-        files = VersionControl.files(extensions: extensions).zip([]).to_h
 
         watch_rd, watch_wr = IO.pipe
-        pid = Process.spawn(command, out: watch_wr)
+        # Process needs its own process group otherwise the process gets killed on INT signal
+        # We need the process to still run when trying to stop the current test run
+        # Maybe there is another way to prevent killing these but for now a new process groups works
+        # Process group created with: pgroup: true
+        pid = Process.spawn(command, out: watch_wr, pgroup: true)
+
         at_exit do
           Process.kill("TERM", pid) if pid
           watch_rd.close
@@ -53,11 +93,15 @@ module Retest
         end
 
         Thread.new do
+          files = VersionControl.files(extensions: extensions).zip([]).to_h
+
           loop do
             ready = IO.select([watch_rd])
             readable_connections = ready[0]
             readable_connections.each do |conn|
               data = conn.readpartial(4096)
+              # Watchexec is not great at figuring out whether a file has been deleted and comes as an update.
+              # This is why we're not looking at the action like we do with Listen.
               change = /^(?:create|remove|rename|modify):(?<path>.*)/.match(data.strip)
 
               next unless change
@@ -81,41 +125,6 @@ module Retest
             end
           end
         end
-
-        # require 'open3'
-        # Thread.new do
-        #   files = VersionControl.files(extensions: extensions).zip([]).to_h
-
-        #   Open3.popen3(command) do |stdin, stdout, stderr, wait_thr|
-        #     loop do
-        #       ready = IO.select([stdout])
-        #       readable_connections = ready[0]
-        #       readable_connections.each do |conn|
-        #         data = conn.readpartial(4096)
-        #         change = /^(?:create|remove|rename|modify):(?<path>.*)/.match(data.strip)
-
-        #         next unless change
-
-        #         path = Pathname(change[:path]).relative_path_from(Dir.pwd).to_s
-        #         file_exist = File.exist?(path)
-        #         file_cached = files.key?(path)
-
-        #         modified, added, removed = result = [[], [], []]
-        #         if file_exist && file_cached
-        #           modified << path
-        #         elsif file_exist && !file_cached
-        #           added << path
-        #           files[path] = nil
-        #         elsif !file_exist && file_cached
-        #           removed << path
-        #           files.delete(path)
-        #         end
-
-        #         yield result
-        #       end
-        #     end
-        #   end
-        # end
       end
     end
   end

data/lib/retest.rb

@@ -1,5 +1,3 @@
-require 'listen'
-
 require 'string/similarity'
 require 'observer'
 
@@ -17,8 +15,6 @@ require "retest/prompt"
 require "retest/sounds"
 require "retest/watcher"
 
-Listen.adapter_warn_behavior = :log
-
 module Retest
   class Error < StandardError; end
   class FileNotFound < StandardError; end

data/lib/scripts/listen

@@ -0,0 +1,57 @@
+#!/usr/bin/env ruby
+
+$stdout.sync = true
+
+require 'listen'
+require 'optparse'
+
+options = {}
+OptionParser.new do |opts|
+  opts.banner = "Usage: scripts/listen.rb [options]"
+
+  opts.on("--exts rb,js,ts", Array, "Extensions to watch for") do |list|
+    options[:extensions] = list
+  end
+
+  opts.on("--polling BOOLEAN", "Force Listen to use polling") do |value|
+    options[:polling] = value
+  end
+
+  opts.on("-w", "--watch .", "Directory to listen to") do |value|
+    options[:dir] = value
+  end
+
+  opts.on("-h", "--help", "Prints help") do
+    puts opts
+    exit
+  end
+end.parse!
+
+unless options.key?(:extensions)
+  raise ArgumentError, 'must provide the files extensions to watch for'
+end
+
+unless options.key?(:polling)
+  raise ArgumentError, 'must provide the polling option'
+end
+
+unless options.key?(:dir)
+  raise ArgumentError, 'must provide the directory path to watch'
+end
+
+def extensions_regex(extensions)
+  Regexp.new("\\.(?:#{extensions.join("|")})$")
+end
+
+Listen.adapter_warn_behavior = :log
+
+Listen.to(options[:dir], only: extensions_regex(options[:extensions]), relative: true, polling: options[:polling]) do |modified, added, removed|
+  if modified.any?
+    $stdout.puts "modify:#{modified.first}"
+  elsif added.any?
+    $stdout.puts "create:#{added.first}"
+  elsif removed.any?
+    $stdout.puts "remove:#{removed.first}"
+  end
+end.start
+sleep

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: retest
 version: !ruby/object:Gem::Version
-  version: 2.1.0
+  version: 2.2.0
 platform: ruby
 authors:
 - Alexandre Barret
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-01-27 00:00:00.000000000 Z
+date: 2025-03-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: string-similarity
@@ -138,6 +138,7 @@ files:
 - lib/retest/version_control/git.rb
 - lib/retest/version_control/no_version_control.rb
 - lib/retest/watcher.rb
+- lib/scripts/listen
 - retest.gemspec
 homepage: https://github.com/AlexB52/retest
 licenses:
@@ -160,7 +161,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3.1
+rubygems_version: 3.1.6
 signing_key:
 specification_version: 4
 summary: A simple command line tool to watch file change and run its matching spec.