parallelized_specs 0.4.47 → 0.4.48

data/Rakefile

@@ -12,7 +12,7 @@ begin
     gem.email    = "jake@instructure.com"
     gem.homepage = "http://github.com/jakesorce/#{gem.name}"
     gem.authors  = "Jake Sorce, Bryan Madsen, Shawn Meredith"
-    gem.version = "0.4.47"
+    gem.version = "0.4.48"
   end
   Jeweler::GemcutterTasks.new
 rescue LoadError

data/Readme.md

@@ -1,6 +1,12 @@
 Speedup Test::RSpec by running parallel on multiple CPUs (or cores).<br/>
 ParallelizedSpecs splits tests into even groups(by number of tests or runtime) and runs each group in a single process with its own database.
-Optional rerunning of failed specs allowing intermittent specs to not cause builds to fail with configurable max failures to try to rerun post build.
+
+Setup Requirements
+***IMPORTANT***
+**OutcomeBuilder Formatter and FailuresFormatter must be enabled in the way specified starting from 4.47**
+This allows streamlined pass\fail determination and thorough false positive checking while handing unusual pass\fail conditions in ruby 1.9.3
+See FailuresFormatter read section and set RERUNS to 0 if don't want to allow reruns
+
 
 Setup for Rails
 ===============

data/lib/parallelized_specs/outcome_builder.rb

@@ -26,6 +26,11 @@ module RSpec
       puts "Thread example failure count = #{failure_count}"
       failure_count > 0 ? (File.open(failure_file, 'a+') { |f| f.write(failure_count) }) : (puts "All specs in Thread #{env_test_number} passed")
       puts "Thread #{env_test_number} has completed in #{duration}"
+
+      lock_output do
+        File.open("#{Rails.root}/tmp/parallel_log/total_specs.txt", 'a+') { |f| f.puts("#{example_count}*#{failure_count}*#{pending_count}") }
+      end
+
     end
 
     def dump_failures(*args)

data/lib/parallelized_specs.rb

@@ -85,7 +85,8 @@ class ParallelizedSpecs
 
   def self.run_specs(tests, options)
     formatters = formatters_setup
-
+    @outcome_builder_enabled = formatters.any? { |formatter| formatter.match(/OutcomeBuilder/) }
+    @reruns_enabled = formatters.any? { |formatter| formatter.match(/FailuresFormatter/) }
     num_processes = options[:count] || Parallel.processor_count
     name = 'spec'
 
@@ -109,20 +110,42 @@ class ParallelizedSpecs
     test_results = Parallel.map(groups, :in_processes => num_processes) do |group|
       run_tests(group, groups.index(group), options)
     end
+    failed = test_results.any? { |result| result[:exit_status] != 0 } #ruby 1.8.7 works breaks on 1.9.3
     slowest_spec_determination("#{Rails.root}/tmp/parallel_log/slowest_specs.log")
 
-    #parse and print results
-    results = find_results(test_results.map { |result| result[:stdout] }*"")
-    puts summarize_results(results)
-    puts "INFO: Took #{Time.now - start} seconds"
-
     #determines if any tricky conditions happened that can cause false positives and offers logging into what was the last spec to start or finishing running
-    false_positive_sniffer(num_processes) if formatters.any? { |formatter| formatter.match(/OutcomeBuilder/) }
-    failed = test_results.any? { |result| result[:exit_status] != 0 } #ruby 1.8.7 works breaks on 1.9.3
-    formatters.any? { |formatter| formatter.match(/FailuresFormatter/) } ? rerun_initializer(name, failed) : abort("SEVERE: #{name.capitalize}s Failed") if failed
+    if @outcome_builder_enabled
+      puts "INFO: OutcomeBuilder is enabled now checking for false positives"
+      spec_summary = calculate_total_spec_details
+      puts "INFO: Total specs run #{spec_summary[0]} failed specs #{spec_summary[1]} pending specs #{spec_summary[2]}\n INFO: Took #{Time.now - start} seconds"
+      false_positive_sniffer(num_processes)
+    else
+      puts "INFO: OutcomeBuilder is disabled not checking for false positives"
+      results = find_results(test_results.map { |result| result[:stdout] }*"")
+      puts summarize_results(results)
+      puts "INFO: Took #{Time.now - start} seconds"
+    end
+
+    if @reruns_enabled
+      puts "INFO: RERUNS are enabled starting the rerun process"
+      rerun_initializer(name, failed)
+    else
+      abort("SEVERE: #{name.capitalize}s Failed") if failed
+    end
     puts "INFO: marking build as PASSED"
   end
 
+  def self.calculate_total_spec_details
+    spec_total_details = [0,0,0]
+    File.open("#{Rails.root}/tmp/parallel_log/total_specs.txt").each_line do |count|
+      thread_spec_details = count.split("*")
+      spec_total_details[0] += thread_spec_details[0].to_i
+      spec_total_details[1] += thread_spec_details[1].to_i
+      spec_total_details[2] += thread_spec_details[2].to_i
+    end
+    spec_total_details
+  end
+
   def self.formatters_setup
     formatters = []
     File.open("#{Rails.root}/spec/spec.opts").each_line do |line|
@@ -134,7 +157,7 @@ class ParallelizedSpecs
 
   def self.formatter_directory_management(formatters)
     FileUtils.mkdir_p('parallel_log') if !File.directory?('tmp/parallel_log')
-    if formatters.any? { |formatter| formatter.match(/FailuresFormatter/) }
+    if @reruns_enabled || @outcome_builder_enabled
       begin
         %w['tmp/parallel_log/spec_count','tmp/parallel_log/failed_specs', 'tmp/parallel_log/thread_started'].each do |dir|
           directory_cleanup_and_create(dir)
@@ -156,7 +179,7 @@ class ParallelizedSpecs
   end
 
   def self.rerun_initializer(name, failed)
-    if Dir.glob("#{Rails.root}/tmp/parallel_log/failed_specs/{*,.*}").count > 2 && !File.zero?("#{Rails.root}/tmp/parallel_log/rspec.failures") # works on both 1.8.7\1.9.3
+    if !File.zero?("#{Rails.root}/tmp/parallel_log/rspec.failures") # works on both 1.8.7\1.9.3
       puts "INFO: some specs failed, about to start the rerun process\n INFO: no more than 9 specs may be rerun and shared specs are not allowed\n...\n..\n."
       ParallelizedSpecs.rerun()
     else
@@ -170,7 +193,7 @@ class ParallelizedSpecs
       (puts "INFO: All threads completed")
     elsif Dir.glob("#{Rails.root}/tmp/parallel_log/thread_started/{*,.*}").count != num_processes + 2
       File.open("#{Rails.root}/tmp/parallel_log/error.log", 'a+') { |f| f.write "\n\n\n syntax issues" }
-      File.open("#{Rails.root}/tmp/failure_cause.log", 'a+') { |f| f.write "Syntax errors" }
+      File.open("#{Rails.root}/tmp/failure_cause.log", 'a+') { |f| f.write "syntax errors" }
       abort "SEVERE: one or more threads didn't get started by rspec, this may be caused by a syntax issue in specs, check logs right before specs start running"
     else
       threads = Dir["#{Rails.root}/tmp/parallel_log/spec_count/*"]

data/parallelized_specs.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = "parallelized_specs"
-  s.version = "0.4.47"
+  s.version = "0.4.48"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Jake Sorce, Bryan Madsen, Shawn Meredith"]

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: parallelized_specs
 version: !ruby/object:Gem::Version 
-  hash: 81
+  hash: 111
   prerelease: 
   segments: 
   - 0
   - 4
-  - 47
-  version: 0.4.47
+  - 48
+  version: 0.4.48
 platform: ruby
 authors: 
 - Jake Sorce, Bryan Madsen, Shawn Meredith