rspec-interactive 0.9.8 → 0.9.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 895c315e10777f7d94c8efdf005129bb532463b31d764af1ddd079692b3e0e62
-  data.tar.gz: 264d523cfc32ffab312ef3d9323611c05f00d851d755d5cf53735680eac0ce4a
+  metadata.gz: 8cced0f02ca892c718d8df21ce4098fb6ef1f8b2f2845e9795811147ee9b4bc0
+  data.tar.gz: 23434b560c9f01d9c4c9342ba180ffb53e35134342e2a87522f913d51eb1d5dd
 SHA512:
-  metadata.gz: 47ca24fe9e49e7ea070c62b3786d72232cb52eabc2faac4777b913099c515a4cb9a0eaeb2515ab204637c344824461fcfdc6e3a460082ca1384bf57d2e04e773
-  data.tar.gz: f7d35eb450b116bb275eac5eaf729d37af936464e18ce9015d3bea953b6204517212299c923eb9e114182b120df3ad3868112be716bd9bd20d6de2c2c8c6326d
+  metadata.gz: a3057fd377b3862bd39962ad7864b7cfed2605ca2a87bb0aae9d5fef24c286cf5804fa86e6edb494152025a79a68e092994f6a176b245612c61de93dc7d085dd
+  data.tar.gz: 79c345903c78993722af61c4294423f733a4f92136f6219e173c494956268ab8bf94f9b8fdbe05e967ffe13ddb62c642047942e46d30c182b3b60771d0505a31

data/Gemfile.lock

@@ -1,8 +1,7 @@
 PATH
   remote: .
   specs:
-    rspec-interactive (0.9.7)
-      listen
+    rspec-interactive (0.9.10)
       pry
       rspec-core
       rspec-teamcity (= 1.0.0)
@@ -14,9 +13,6 @@ GEM
     diff-lcs (1.4.4)
     ffi (1.15.5)
     ffi (1.15.5-java)
-    listen (3.7.1)
-      rb-fsevent (~> 0.10, >= 0.10.3)
-      rb-inotify (~> 0.9, >= 0.9.10)
     method_source (1.0.0)
     pry (0.14.1)
       coderay (~> 1.1)
@@ -25,9 +21,6 @@ GEM
       coderay (~> 1.1)
       method_source (~> 1.0)
       spoon (~> 0.0)
-    rb-fsevent (0.11.1)
-    rb-inotify (0.10.1)
-      ffi (~> 1.0)
     rspec (3.9.0)
       rspec-core (~> 3.9.0)
       rspec-expectations (~> 3.9.0)

data/lib/rspec-interactive/string_output.rb

@@ -18,6 +18,13 @@ module RSpec
       def print(str = "")
         @string += str.to_s
       end
+
+      def flush
+      end
+
+      def closed?
+        @client.closed?
+      end
     end
   end
 end

data/lib/rspec-interactive/threaded_output.rb

@@ -0,0 +1,33 @@
+module RSpec
+  module Interactive
+    class ThreadedOutput
+      attr_reader :string
+
+      def initialize(thread_map:, default:)
+        @thread_map = thread_map
+        @default = default
+        @string = ''
+      end
+
+      def write(name, str = "")
+        (@thread_map[Thread.current] || @default).write(name, str)
+      end
+
+      def puts(str = "")
+        (@thread_map[Thread.current] || @default).puts(str)
+      end
+
+      def print(str = "")
+        (@thread_map[Thread.current] || @default).puts(str)
+      end
+
+      def flush
+        (@thread_map[Thread.current] || @default).flush
+      end
+
+      def closed?
+        (@thread_map[Thread.current] || @default).closed?
+      end
+    end
+  end
+end

data/lib/rspec-interactive/version.rb

@@ -2,6 +2,6 @@
 
 module RSpec
   module Interactive
-    VERSION = "0.9.8"
+    VERSION = "0.9.11"
   end
 end

data/lib/rspec-interactive.rb

@@ -1,8 +1,9 @@
+require 'find'
 require 'json'
-require 'listen'
 require 'pry'
 require 'readline'
 require 'rspec/core'
+require 'set'
 require 'shellwords'
 require 'socket'
 require 'teamcity/spec/runner/formatter/teamcity/formatter'
@@ -19,6 +20,7 @@ require 'rspec-interactive/rubo_cop_command'
 require 'rspec-interactive/runner'
 require 'rspec-interactive/stdio'
 require 'rspec-interactive/string_output'
+require 'rspec-interactive/threaded_output'
 
 module RSpec
   module Interactive
@@ -46,8 +48,7 @@ module RSpec
       @history_file = history_file
       @updated_files = []
       @stty_save = %x`stty -g`.chomp
-      @file_change_mutex = Mutex.new
-      @command_mutex = Mutex.new
+      @rspec_mutex = Mutex.new
       @output_stream = output_stream
       @input_stream = input_stream
       @error_stream = error_stream
@@ -59,27 +60,43 @@ module RSpec
       check_rails
       maybe_trap_interrupt
       configure_pry
+      configure_watched_files
 
       @startup_thread = Thread.start do
+        Thread.current.report_on_exception = false
         if server
           @server_thread = Thread.start do
             server = TCPServer.new port
 
-            while client = server.accept
-              request = client.gets
-              args = Shellwords.split(request)
-              rspec_for_server(client, args)
-              client.close
+            while true
+              break unless client = server.accept
+              begin
+                request = client.gets
+                args = Shellwords.split(request)
+                rspec_for_server(client, args)
+              rescue StandardError => e
+                # It would be nice to log to the client here but it might be
+                # disconnected or disconnect before we successfully write. Any
+                # error here is unexpected so just log to the console.
+                @output_stream.puts
+                @output_stream.puts 'error handling client request'
+                log_exception(@output_stream, e)
+              ensure
+                client.close
+              end
             end
           end
         end
 
-        @config_cache.record_configuration { @configuration.configure_rspec.call }
-        start_file_watcher
+        @startup_output = StringOutput.new
+        output = ThreadedOutput.new(thread_map: { Thread.current => @startup_output }, default: @output_stream)
+
+        Stdio.capture(stdout: output, stderr: output) do
+          @config_cache.record_configuration { @configuration.configure_rspec.call }
+        end
       end
 
       Pry.start
-      @listener.stop if @listener
       @server_thread.exit if @server_thread
       @startup_thread.exit if @startup_thread
       0
@@ -114,18 +131,6 @@ module RSpec
       end
     end
 
-    def self.start_file_watcher
-      return if @configuration.watch_dirs.empty?
-
-      # Only polling seems to work in Docker.
-      @listener = Listen.to(*@configuration.watch_dirs, only: /\.rb$/, force_polling: true) do |modified, added|
-        @file_change_mutex.synchronize do
-          @updated_files.concat(added + modified)
-        end
-      end
-      @listener.start
-    end
-
     def self.configure_pry
       # Set up IO.
       Pry.config.input = Readline
@@ -139,19 +144,16 @@ module RSpec
       Pry.config.history_file = @history_file
     end
 
-    def self.refresh
-      @file_change_mutex.synchronize do
-        @updated_files.uniq.each do |filename|
-          @output_stream.puts "changed: #{filename}"
-          trace = TracePoint.new(:class) do |tp|
-            @configuration.on_class_load.call(tp.self)
-          end
-          trace.enable
-          load filename
-          trace.disable
-          @output_stream.puts
+    def self.refresh(output: @output_stream)
+      get_updated_files.each do |filename|
+        output.puts "changed: #{filename}"
+        trace = TracePoint.new(:class) do |tp|
+          @configuration.on_class_load.call(tp.self)
         end
-        @updated_files.clear
+        trace.enable
+        load filename
+        trace.disable
+        output.puts
       end
       @configuration.refresh.call
     end
@@ -178,7 +180,7 @@ module RSpec
     end
 
     def self.rspec(args)
-      @command_mutex.synchronize do
+      @rspec_mutex.synchronize do
         begin
           @runner = RSpec::Interactive::Runner.new(parse_args(args))
 
@@ -211,25 +213,22 @@ module RSpec
     end
 
     def self.rspec_for_server(client, args)
-      @command_mutex.synchronize do
+      @rspec_mutex.synchronize do
         disable_pry = ENV['DISABLE_PRY']
         output = ClientOutput.new(client)
 
         ENV['TEAMCITY_RAKE_RUNNER_DEBUG_OUTPUT_CAPTURER_ENABLED'] = 'false'
         Rake::TeamCity::RunnerCommon.class_variable_set(:@@original_stdout, output)
 
-        Stdio.capture(
-          stdout: output,
-          stderr: output) do
-
-          await_startup(output: output)
+        return unless await_startup(output: output)
 
+        Stdio.capture(stdout: output, stderr: output) do
           # Prevent the debugger from being used. The server isn't interactive.
           ENV['DISABLE_PRY'] = 'true'
 
-          @runner = RSpec::Interactive::Runner.new(parse_args(args))
+          runner = RSpec::Interactive::Runner.new(parse_args(args))
 
-          refresh
+          refresh(output: output)
 
           # RSpec::Interactive-specific RSpec configuration
           RSpec.configure do |config|
@@ -250,18 +249,17 @@ module RSpec
           RSpec.configuration.formatter = Spec::Runner::Formatter::TeamcityFormatter
 
           # Run.
-          @runner.run
-        rescue Errno::EPIPE => e
+          runner.run
+        rescue Errno::EPIPE, IOError
           # Don't care.
         ensure
           ENV['DISABLE_PRY'] = disable_pry
-          @runner = nil
+          runner = nil
 
           # Reset
           RSpec.clear_examples
           RSpec.reset
 
-          await_startup(output: output)
           @config_cache.replay_configuration
         end
       end
@@ -279,24 +277,69 @@ module RSpec
       return yield if line.nil? # EOF
       return yield if line.empty? # blank line
 
-      begin
-        await_startup
-      rescue Interrupt
+      if await_startup
+        yield
+      else
         @output_stream.puts
-        return true
+        true
       end
-
-      yield
     end
 
     def self.await_startup(output: @output_stream)
-      if @startup_thread
-        if @startup_thread.alive?
-          output.puts 'waiting for configure_rspec...'
-        end
+      return true unless @startup_thread
+
+      if @startup_thread.alive?
+        output.puts 'waiting for configure_rspec...'
+      end
+
+      begin
         @startup_thread.join
         @startup_thread = nil
+        print_startup_output(output: output)
+        true
+      rescue Interrupt
+        false
+      rescue StandardError => e
+        print_startup_output(output: output)
+        output.puts 'configure_rspec failed'
+        log_exception(output, e)
+        false
+      end
+    end
+
+    def self.log_exception(output, e)
+      output.puts "#{e.backtrace[0]}: #{e.message} (#{e.class})"
+      e.backtrace[1..-1].each { |b| output.puts "\t#{b}" }
+    end
+
+    def self.print_startup_output(output: @output_stream)
+      return if @startup_output.nil? || @startup_output.string.empty?
+
+      output.puts(@startup_output.string)
+      @startup_output = nil
+    end
+
+    def self.configure_watched_files
+      @watched_files = get_watched_files
+    end
+
+    def self.get_watched_files
+      return Set.new if @configuration.watch_dirs.empty?
+      entries = Find.find(*@configuration.watch_dirs).flat_map do |file|
+        if FileTest.file?(file)
+          [[file, File.mtime(file).to_i]]
+        else
+          []
+        end
       end
+      entries.to_set
+    end
+
+    def self.get_updated_files
+      new_watched_files = get_watched_files
+      difference = new_watched_files - @watched_files
+      @watched_files = new_watched_files
+      difference.map(&:first)
     end
   end
 end

data/rspec-interactive.gemspec

@@ -27,6 +27,5 @@ Gem::Specification.new do |spec|
 
   spec.add_dependency 'rspec-core'
   spec.add_dependency 'rspec-teamcity', '1.0.0'
-  spec.add_dependency 'listen'
   spec.add_dependency 'pry'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rspec-interactive
 version: !ruby/object:Gem::Version
-  version: 0.9.8
+  version: 0.9.11
 platform: ruby
 authors:
 - Nick Dower
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-02-27 00:00:00.000000000 Z
+date: 2022-02-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec-core
@@ -38,20 +38,6 @@ dependencies:
     - - '='
       - !ruby/object:Gem::Version
         version: 1.0.0
-- !ruby/object:Gem::Dependency
-  name: listen
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: pry
   requirement: !ruby/object:Gem::Requirement
@@ -103,6 +89,7 @@ files:
 - lib/rspec-interactive/runner.rb
 - lib/rspec-interactive/stdio.rb
 - lib/rspec-interactive/string_output.rb
+- lib/rspec-interactive/threaded_output.rb
 - lib/rspec-interactive/version.rb
 - lib/teamcity/spec/runner/formatter/teamcity/formatter.rb
 - rspec-interactive.gemspec