tork 18.2.4 → 19.0.0

data/lib/tork/config/coverage/master.rb

@@ -0,0 +1,29 @@
+require 'coverage'
+at_exit do
+  if not $! or ($!.kind_of? SystemExit and $!.success?) and
+     coverage_by_file = Coverage.result rescue nil
+  then
+    report = {}
+    coverage_by_file.each do |file, coverage|
+      # ignore files outside working directory
+      if file.start_with? Dir.pwd
+        nsloc = 0
+        holes = []
+        coverage.each_with_index do |hits, index|
+          # ignore non-source lines of code
+          unless hits.nil?
+            nsloc += 1
+            # +1 because line numbers go 1..N
+            holes << index + 1 if hits.zero?
+          end
+        end
+
+        grade = ((nsloc - holes.length) / nsloc.to_f) * 100
+        report[file] = { grade: grade, nsloc: nsloc, holes: holes }
+      end
+    end
+
+    require 'yaml'
+    YAML.dump report, STDOUT
+  end
+end

data/lib/tork/config/coverage/worker.rb

@@ -0,0 +1 @@
+Coverage.start

data/lib/tork/config/cucumber/driver.rb

@@ -0,0 +1,11 @@
+Tork::Driver::ALL_TEST_FILE_GLOBS.push 'features/**/*.feature'
+
+Tork::Driver::TEST_FILE_GLOBBERS.update(
+  # source files that correspond to test files
+  %r{^(features/(.+/)?)step_definitions/.+\.rb$} => lambda do |matches|
+    matches[1] + '*.feature'
+  end,
+
+  # the actual test files themselves
+  %r{^features/.+\.feature$} => lambda {|matches| matches[0] }
+)

data/lib/tork/config/cucumber/worker.rb

@@ -0,0 +1,14 @@
+if $tork_test_file.end_with? '.feature'
+  original_argv = ARGV.dup
+  begin
+    # pass the feature file to cucumber(1) in ARGV
+    ARGV.push [$tork_test_file, *$tork_line_numbers].join(':')
+    require 'rubygems'
+    require 'cucumber'
+    load Gem.bin_path('cucumber', 'cucumber')
+  ensure
+    # restore ARGV for other at_exit hooks.  otherwise, RSpec's hook will
+    # try to load the non-Ruby feature file from ARGV and fail accordingly.
+    ARGV.replace original_argv
+  end
+end

data/lib/tork/config/default/config.rb

@@ -0,0 +1,5 @@
+ENV['TORK_CONFIGS'] += ':rails' if Dir['script/{rails,console}'].any?
+ENV['TORK_CONFIGS'] += ':test' if File.directory? 'test'
+ENV['TORK_CONFIGS'] += ':spec' if File.directory? 'spec'
+ENV['TORK_CONFIGS'] += ':cucumber' if File.directory? 'features'
+ENV['TORK_CONFIGS'] += ':factory_girl' if Dir['{test,spec}/factories/'].any?

data/lib/tork/config/dotlog/onfork.rb

@@ -0,0 +1,2 @@
+dirname, basename = File.split($tork_log_file)
+$tork_log_file.replace File.join(dirname, '.' + basename)

data/lib/tork/config/factory_girl/onfork.rb

@@ -0,0 +1,2 @@
+require 'factory_girl'
+FactoryGirl.factories.clear

data/lib/tork/config/factory_girl/worker.rb

@@ -0,0 +1 @@
+FactoryGirl.find_definitions

data/lib/tork/config/logdir/onfork.rb

@@ -0,0 +1,4 @@
+require 'fileutils'
+dirname, basename = File.split($tork_log_file)
+FileUtils.mkdir_p log_dir = File.join('log', dirname)
+$tork_log_file.replace File.join(log_dir, basename)

data/lib/tork/config/parallel_tests/worker.rb

@@ -0,0 +1,4 @@
+# parallel_tests expects the sequence "", "2", "3", "4", ...
+if $tork_worker_number > 0
+  ENV['TEST_ENV_NUMBER'] = ($tork_worker_number + 1).to_s
+end

data/lib/tork/config/rails/driver.rb

@@ -0,0 +1,15 @@
+require 'active_support/inflector'
+
+Tork::Driver::REABSORB_FILE_GREPS.push(
+  %r{^config/.+\.(rb|yml)$},
+  %r{^db/schema\.rb$},
+  %r{^Gemfile\.lock$}
+)
+
+Tork::Driver::TEST_FILE_GLOBBERS.update(
+  %r{^(app|lib|test|spec)/.*?([^/]+?)(_factory)?\.rb$} => lambda do |matches|
+    single = matches[2]
+    plural = ActiveSupport::Inflector.pluralize(single)
+    "{test,spec}/**/{#{single},#{plural}_*}_{test,spec}.rb"
+  end
+)

data/lib/tork/config/rails/master.rb

@@ -0,0 +1,12 @@
+begin
+  require 'rails/railtie'
+  class Tork::Railtie < Rails::Railtie
+    config.before_initialize do |app|
+      app.config.cache_classes = false
+    end
+  end
+rescue LoadError => error
+  warn "tork/config/rails/master: could not set configuration using railties;\n"\
+       "you will have to add the following to your test environment manually:\n\t"\
+       'config.cache_classes = false'
+end

data/lib/tork/config/rails/worker.rb

@@ -0,0 +1,21 @@
+# although inherited as a file descriptor during the fork(), the database
+# connection cannot be shared between the master and its workers or amongst
+# the workers themselves because access to the file descriptor isn't mutexed
+if defined? ActiveRecord::Base
+  base = ActiveRecord::Base
+
+  info =
+    if base.respond_to? :connection_info        # rails >= 3.1.0
+      base.connection_info
+    elsif base.respond_to? :connection_pool     # rails >= 2.2.1
+      base.connection_pool.spec.config
+    else
+      warn "#{$0}: config/rails/worker: couldn't read connection information"
+      {}
+    end
+
+  # in-memory databases are an exception; they are simply fork()ed along
+  if info[:database] != ':memory:'
+    base.connection.reconnect!
+  end
+end

data/lib/tork/config/spec/driver.rb

@@ -0,0 +1,17 @@
+Tork::Driver::REABSORB_FILE_GREPS.push 'spec/spec_helper.rb'
+
+Tork::Driver::ALL_TEST_FILE_GLOBS.push 'spec/**/{spec_*,*_spec}.rb'
+
+Tork::Driver::TEST_FILE_GLOBBERS.update(
+  # source files that correspond to test files
+  %r{^lib/.*?([^/]+)\.rb$} => lambda do |matches|
+    target = matches[1]
+    "spec/**/{spec_#{target},#{target}_spec}.rb"
+  end,
+
+  # the actual test files themselves
+  %r{^spec/.+\.rb$} => lambda do |matches|
+    target = matches[0]
+    target if File.basename(target) =~ /^spec_|_spec\./
+  end
+)

data/lib/tork/config/spec/master.rb

@@ -0,0 +1,3 @@
+$LOAD_PATH.unshift 'spec' unless $LOAD_PATH.include? 'spec'
+$LOAD_PATH.unshift 'lib' unless $LOAD_PATH.include? 'lib'
+require 'spec_helper' if File.exist? 'spec/spec_helper.rb'

data/lib/tork/config/spec/worker.rb

@@ -0,0 +1,3 @@
+$tork_line_numbers.each do |line|
+  ARGV.push '--line_number', line.to_s
+end

data/lib/tork/config/test/driver.rb

@@ -0,0 +1,17 @@
+Tork::Driver::REABSORB_FILE_GREPS.push 'test/test_helper.rb'
+
+Tork::Driver::ALL_TEST_FILE_GLOBS.push 'test/**/{test_*,*_test}.rb'
+
+Tork::Driver::TEST_FILE_GLOBBERS.update(
+  # source files that correspond to test files
+  %r{^lib/.*?([^/]+)\.rb$} => lambda do |matches|
+    target = matches[1]
+    "test/**/{test_#{target},#{target}_test}.rb"
+  end,
+
+  # the actual test files themselves
+  %r{^test/.+\.rb$} => lambda do |matches|
+    target = matches[0]
+    target if File.basename(target) =~ /^test_|_test\./
+  end
+)

data/lib/tork/config/test/master.rb

@@ -0,0 +1,3 @@
+$LOAD_PATH.unshift 'test' unless $LOAD_PATH.include? 'test'
+$LOAD_PATH.unshift 'lib' unless $LOAD_PATH.include? 'lib'
+require 'test_helper' if File.exist? 'test/test_helper.rb'

data/lib/tork/config/test/worker.rb

@@ -0,0 +1,23 @@
+unless $tork_line_numbers.empty?
+  test_file_lines = File.readlines($tork_test_file)
+  test_names = $tork_line_numbers.map do |line|
+    catch :found do
+      # search backwards from the desired line number to
+      # the first line in the file for test definitions
+      line.downto(0) do |i|
+        test_name =
+          case test_file_lines[i]
+          when /^\s*def\s+test_(\w+)/ then $1
+          when /^\s*(test|context|should|describe|it)\b.+?(['"])(.*?)\2/
+            # elide string interpolation and invalid method name characters
+            $3.gsub(/\#\{.*?\}/, ' ').strip.gsub(/\W+/, '.*')
+          end \
+        and throw :found, test_name
+      end; nil # prevent unsuccessful search from returning an integer
+    end
+  end.compact.uniq
+
+  unless test_names.empty?
+    ARGV.push '--name', "/(?i:#{test_names.join('|')})/"
+  end
+end

data/lib/tork/driver.rb

@@ -1,59 +1,91 @@
 require 'set'
-require 'tork/client'
 require 'tork/engine'
+require 'tork/server'
 require 'tork/config'
 
 module Tork
-class Driver < Engine
+class Driver < Server
+
+  REABSORB_FILE_GREPS = []
+  ALL_TEST_FILE_GLOBS = []
+  TEST_FILE_GLOBBERS = {}
 
   def initialize
     super
-
-    @herald = Client::Transceiver.new('tork-herald') do |changed_files|
-      changed_files.each do |changed_file|
-        # find and run the tests that correspond to the changed file
-        visited = Set.new
-        visitor = lambda do |source_file|
-          Config.test_file_globbers.each do |regexp, globber|
-            if regexp =~ source_file and globs = globber.call($~)
-              Dir[*globs].each do |test_file|
-                if visited.add? test_file
-                  run_test_file test_file
-                  visitor.call test_file
-                end
-              end
-            end
-          end
-        end
-        visitor.call changed_file
-
-        # reabsorb text execution overhead if overhead files changed
-        if Config.reabsorb_file_greps.any? {|r| r =~ changed_file }
-          @client.send [:over, changed_file]
-          reabsorb_overhead_files
-        end
-      end
-    end
-
-    reabsorb_overhead_files
+    Tork.config :driver
   end
 
-  def quit
-    @herald.quit
+  def loop
+    @herald = popen('tork-herald')
+    @engine = popen('tork-engine')
     super
+  ensure
+    pclose @herald
+    pclose @engine
   end
 
   def run_all_test_files
-    all_test_files = Dir[*Config.all_test_file_globs]
+    all_test_files = Dir[*ALL_TEST_FILE_GLOBS]
     if all_test_files.empty?
-      warn "#{$0}: There are no test files to run."
+      tell @client, 'There are no test files to run.'
     else
       run_test_files all_test_files
     end
   end
 
-  def reabsorb_overhead_files
-    reabsorb_overhead Config.overhead_load_paths, Dir[*Config.overhead_file_globs]
+  # accept and delegate tork-engine(1) commands
+  Engine.public_instance_methods(false).each do |name|
+    unless method_defined? name
+      define_method name do |*args|
+        send @engine, [name, *args]
+      end
+    end
+  end
+
+protected
+
+  def recv client, message
+    case client
+    when @engine
+      send @clients, message # propagate downstream
+
+    when @herald
+      message.each do |changed_file|
+        # reabsorb text execution overhead if overhead files changed
+        overhead_changed = REABSORB_FILE_GREPS.any? do |pattern|
+          if pattern.kind_of? Regexp
+            pattern =~ changed_file
+          else
+            pattern == changed_file
+          end
+        end
+
+        if overhead_changed
+          send @clients, [:reabsorb, changed_file]
+          reabsorb_overhead
+        else
+          run_test_files find_dependent_test_files(changed_file).to_a
+        end
+      end
+
+    else
+      super
+    end
+  end
+
+private
+
+  def find_dependent_test_files source_file, results=Set.new
+    TEST_FILE_GLOBBERS.each do |regexp, globber|
+      if regexp =~ source_file and globs = globber.call($~)
+        Dir[*globs].each do |dependent_file|
+          if results.add? dependent_file
+            find_dependent_test_files dependent_file, results
+          end
+        end
+      end
+    end
+    results
   end
 
 end

data/lib/tork/engine.rb

@@ -1,6 +1,5 @@
 require 'set'
 require 'diff/lcs'
-require 'tork/client'
 require 'tork/server'
 require 'tork/config'
 
@@ -9,49 +8,59 @@ class Engine < Server
 
   def initialize
     super
-    @waiting_test_files = Set.new # dispatched to master but not yet running
-    @running_test_files = Set.new # dispatched to master and started running
+    Tork.config :engine
+
+    @queued_test_files = Set.new
     @passed_test_files = Set.new
     @failed_test_files = Set.new
     @lines_by_file = {}
-    @master = create_master_process
   end
 
-  def quit
-    @master.quit
+  def loop
+    create_master
     super
+  ensure
+    destroy_master
   end
 
-  def reabsorb_overhead load_paths, overhead_files
-    @master.quit
-    @master = create_master_process
-    @master.send [:load, load_paths, overhead_files]
+  def reabsorb_overhead
+    destroy_master
+    create_master
 
     # re-dispatch the previously dispatched files to the new master
-    dispatched_test_files = @running_test_files + @waiting_test_files
-    @waiting_test_files.clear
-    run_test_files dispatched_test_files
+    previous = @queued_test_files.to_a
+    @queued_test_files.clear
+    run_test_files previous
   end
 
-  def run_test_file test_file, line_numbers=nil
-    if File.exist? test_file and @waiting_test_files.add? test_file
-      line_numbers ||= find_changed_line_numbers(test_file)
-      @master.send [:test, test_file, line_numbers]
+  def run_test_file test_file, *line_numbers
+    if File.exist? test_file and @queued_test_files.add? test_file
+      if line_numbers.empty?
+        line_numbers = find_changed_line_numbers(test_file)
+      else
+        line_numbers.map!(&:to_i)
+        line_numbers.clear if line_numbers.any?(&:zero?)
+      end
+      send @master, [:test, test_file, line_numbers]
     end
   end
 
-  def stop_running_test_files
-    if @running_test_files.empty?
-      warn "#{$0}: There are no running test files to stop."
+  def run_test_files test_files_with_optional_line_numbers
+    test_files_with_optional_line_numbers.each {|f| run_test_file(*f) }
+  end
+
+  def stop_running_test_files signal=nil
+    if @queued_test_files.empty?
+      tell @client, 'There are no running test files to stop.'
     else
-      @master.send [:stop]
-      @running_test_files.clear
+      send @master, [:stop, signal].compact
+      @queued_test_files.clear
     end
   end
 
   def rerun_passed_test_files
     if @passed_test_files.empty?
-      warn "#{$0}: There are no passed test files to re-run."
+      tell @client, 'There are no passed test files to re-run.'
     else
       run_test_files @passed_test_files
     end
@@ -59,7 +68,7 @@ class Engine < Server
 
   def rerun_failed_test_files
     if @failed_test_files.empty?
-      warn "#{$0}: There are no failed test files to re-run."
+      tell @client, 'There are no failed test files to re-run.'
     else
       run_test_files @failed_test_files
     end
@@ -67,41 +76,37 @@ class Engine < Server
 
 protected
 
-  def run_test_files files
-    files.each {|f| run_test_file f }
-  end
-
-private
-
-  def create_master_process
-    Client::Transceiver.new('tork-master') do |message|
-      @client.send message # propagate output downstream
+  def recv client, message
+    case client
+    when @master
+      send @clients, message # propagate downstream
 
       event, file, line_numbers = message
       case event.to_sym
       when :test
-        @waiting_test_files.delete file
-        @running_test_files.add file
+        @queued_test_files.delete file
 
       when :pass
-        @running_test_files.delete file
-
-        # only whole test file runs qualify as pass
+        # only whole test file runs should qualify as pass
         if line_numbers.empty?
-          @failed_test_files.delete file
-          @passed_test_files.add file
+          was_fail = @failed_test_files.delete? file
+          now_pass = @passed_test_files.add? file
+          send @clients, [:fail_now_pass, file, message] if was_fail and now_pass
         end
 
       when :fail
-        @running_test_files.delete file
-        @failed_test_files.add file
-        @passed_test_files.delete file
+        was_pass = @passed_test_files.delete? file
+        now_fail = @failed_test_files.add? file
+        send @clients, [:pass_now_fail, file, message] if was_pass and now_fail
       end
 
-      Config.test_event_hooks.each {|hook| hook.call message }
+    else
+      super
     end
   end
 
+private
+
   def find_changed_line_numbers test_file
     # cache test file contents for diffing below
     new_lines = File.readlines(test_file)
@@ -114,5 +119,13 @@ private
       map {|change| change.position + 1 }.uniq
   end
 
+  def create_master
+    @master = popen('tork-master')
+  end
+
+  def destroy_master
+    pclose @master
+  end
+
 end
 end