retest 2.1.0 → 2.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7eb4992f6d7149b443a2833f6481694b551fcf3ca98355acbe014f9209ed68c0
-  data.tar.gz: f1c50bb7340803ae2abc65178c204b983fe0a99471ac502ad0726d94d7219638
+  metadata.gz: d136e02d0fe9258511865619e208809547e4ab021ecd735755b3b76b0893ac6d
+  data.tar.gz: 5e93ba6847b556359e921cb8b4807676937708e8d8f55987b7cf6795082fe649
 SHA512:
-  metadata.gz: 69b59be1bf2c414d66fa9b80eb656e430d32ede6fd88734169096d7b0b7c1f64d48edfcceabc4b48676a5dcf36cf307379926030b75e6fffa95525a504c9e279
-  data.tar.gz: c063a98b171140f21e66567a2c37d1014f8a3aed5c85f935e6a2e99f8d0845db84c046ccd47f7e739cc115e894111afc07d08b3564f02bdce09698c1a6674f03
+  metadata.gz: 1a88d8f871d30a0d1bf74cf04dd19eed678290a74e2ccfa5454f8a8f70d3dafe5f83e76ec6e7177995be57527f54700137ae63995d48957e899e702aaf7607fe
+  data.tar.gz: c7527faa79a247c04de260cab878b40619de1a546980e8d2371c53c26c5056a5d9a8ad0dd489d4ebad63c6e3df2703c3fc6ed52748f40bf54699b2c4fdceb4a7

data/.github/workflows/ci.yml

@@ -20,23 +20,24 @@ jobs:
       matrix:
         os: [ubuntu-latest]
         ruby:
-          - '2.5'
-          - '2.6'
-          - '2.7'
-          - '3.0'
-          - '3.1'
-          - '3.2'
-          - '3.3'
-        include:
-          - os: macos-13
-            ruby: 2.5
-    name: Ruby ${{ matrix.ruby }} test (${{ matrix.os }})
+          - version: '2.5'
+          - version: '2.6'
+          - version: '2.7'
+          - version: '3.0'
+          - version: '3.1'
+          - version: '3.2'
+          - version: '3.3'
+          - version: '3.4'
+            gemfile: Gemfile-3.4
+    name: Ruby ${{ matrix.ruby.version }} test (${{ matrix.os }})
+    env:
+      BUNDLE_GEMFILE: ${{ matrix.ruby.gemfile || 'Gemfile' }}
     steps:
     - uses: actions/checkout@v4
     - name: Set up Ruby
       uses: ruby/setup-ruby@v1
       with:
-        ruby-version: ${{ matrix.ruby }}
+        ruby-version: ${{ matrix.ruby.version }}
         bundler-cache: true
     - run: bundle install
     - run: bundle exec rake

data/Gemfile-3.4

@@ -0,0 +1,8 @@
+source "https://rubygems.org"
+
+# Specify your gem's dependencies in retest.gemspec
+gemspec
+
+gem "rake", "~> 13.0"
+gem "minitest", "~> 5.0"
+gem "byebug", require: false

data/Gemfile-3.4.lock

@@ -0,0 +1,71 @@
+PATH
+  remote: .
+  specs:
+    retest (2.2.1)
+      listen (~> 3.9)
+      observer (~> 0.1)
+      string-similarity (~> 2.1)
+      tty-option (~> 0.1)
+      tty-prompt (~> 0.1)
+
+GEM
+  remote: https://rubygems.org/
+  specs:
+    byebug (11.1.3)
+    ffi (1.17.1)
+    ffi (1.17.1-aarch64-linux-gnu)
+    ffi (1.17.1-aarch64-linux-musl)
+    ffi (1.17.1-arm-linux-gnu)
+    ffi (1.17.1-arm-linux-musl)
+    ffi (1.17.1-arm64-darwin)
+    ffi (1.17.1-x86-linux-gnu)
+    ffi (1.17.1-x86-linux-musl)
+    ffi (1.17.1-x86_64-darwin)
+    ffi (1.17.1-x86_64-linux-gnu)
+    ffi (1.17.1-x86_64-linux-musl)
+    listen (3.9.0)
+      rb-fsevent (~> 0.10, >= 0.10.3)
+      rb-inotify (~> 0.9, >= 0.9.10)
+    minitest (5.25.5)
+    observer (0.1.2)
+    pastel (0.8.0)
+      tty-color (~> 0.5)
+    rake (13.2.1)
+    rb-fsevent (0.11.2)
+    rb-inotify (0.11.1)
+      ffi (~> 1.0)
+    string-similarity (2.1.0)
+    tty-color (0.6.0)
+    tty-cursor (0.7.1)
+    tty-option (0.3.0)
+    tty-prompt (0.23.1)
+      pastel (~> 0.8)
+      tty-reader (~> 0.8)
+    tty-reader (0.9.0)
+      tty-cursor (~> 0.7)
+      tty-screen (~> 0.8)
+      wisper (~> 2.0)
+    tty-screen (0.8.2)
+    wisper (2.0.1)
+
+PLATFORMS
+  aarch64-linux-gnu
+  aarch64-linux-musl
+  arm-linux-gnu
+  arm-linux-musl
+  arm64-darwin
+  ruby
+  x86-linux-gnu
+  x86-linux-musl
+  x86_64-darwin
+  x86_64-linux-gnu
+  x86_64-linux-musl
+
+DEPENDENCIES
+  byebug
+  minitest (~> 5.0)
+  rake (~> 13.0)
+  retest!
+
+BUNDLED WITH
+   2.6.6

data/Gemfile.lock

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

data/bin/list-ruby-versions

@@ -0,0 +1,15 @@
+#!/usr/bin/env ruby
+
+require "rubygems"
+
+`brew upgrade ruby-build`
+
+ruby_list = `rbenv install --list-all`
+
+puts ruby_list
+  .split("\n")
+  .select { |version| version =~ /^\d+\.\d+\.\d+$/ }
+  .select { |version| Gem::Version.new(version) >= Gem::Version.new("2.5") }
+  .group_by { |version| version[0..2] }
+  .transform_values { |version| version.max_by { |number| Gem::Version.new(number) } }
+  .values

data/bin/start

@@ -0,0 +1,3 @@
+#!/usr/bin/env bash
+
+docker run --rm -it -v "$(pwd):/app" -w /app ruby:$1 bash

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.1"
 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,50 @@ 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)
+
+        thread = 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
+
+        at_exit do
+          thread.exit
+          Process.kill("TERM", pid) if pid
+          watch_rd.close
+          watch_wr.close
+        end
+      end
     end
 
     module Watchexec
@@ -42,22 +79,24 @@ 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)
-        at_exit do
-          Process.kill("TERM", pid) if pid
-          watch_rd.close
-          watch_wr.close
-        end
+        # 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)
+
+        thread = Thread.new do
+          files = VersionControl.files(extensions: extensions).zip([]).to_h
 
-        Thread.new do
           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
@@ -82,40 +121,12 @@ module Retest
           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
+        at_exit do
+          thread.exit
+          Process.kill("TERM", pid) if pid
+          watch_rd.close
+          watch_wr.close
+        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,13 @@
 --- !ruby/object:Gem::Specification
 name: retest
 version: !ruby/object:Gem::Version
-  version: 2.1.0
+  version: 2.2.1
 platform: ruby
 authors:
 - Alexandre Barret
-autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-01-27 00:00:00.000000000 Z
+date: 2025-03-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: string-similarity
@@ -80,7 +79,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.1'
-description:
 email:
 - alex@abletech.nz
 executables:
@@ -92,6 +90,8 @@ files:
 - ".github/workflows/ci.yml"
 - ".gitignore"
 - Gemfile
+- Gemfile-3.4
+- Gemfile-3.4.lock
 - Gemfile.lock
 - LICENSE.txt
 - README.md
@@ -100,7 +100,9 @@ files:
 - bin/build/watchexec
 - bin/console
 - bin/debug
+- bin/list-ruby-versions
 - bin/setup
+- bin/start
 - bin/test/bundler-app
 - bin/test/git-ruby
 - bin/test/hanami-app
@@ -138,6 +140,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:
@@ -145,7 +148,6 @@ licenses:
 metadata:
   homepage_uri: https://github.com/AlexB52/retest
   source_code_uri: https://github.com/AlexB52/retest
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -160,8 +162,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3.1
-signing_key:
+rubygems_version: 3.6.2
 specification_version: 4
 summary: A simple command line tool to watch file change and run its matching spec.
 test_files: []