parallelized_specs 0.4.48 → 0.4.49

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.48"
+    gem.version = "0.4.49"
   end
   Jeweler::GemcutterTasks.new
 rescue LoadError

data/lib/parallelized_specs.rb

@@ -87,6 +87,7 @@ class ParallelizedSpecs
     formatters = formatters_setup
     @outcome_builder_enabled = formatters.any? { |formatter| formatter.match(/OutcomeBuilder/) }
     @reruns_enabled = formatters.any? { |formatter| formatter.match(/FailuresFormatter/) }
+    @slow_specs_enabled = formatters.any? { |formatter| formatter.match(/SlowestSpecLogger/) }
     num_processes = options[:count] || Parallel.processor_count
     name = 'spec'
 
@@ -111,32 +112,33 @@ class ParallelizedSpecs
       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")
+    slowest_spec_determination("#{Rails.root}/tmp/parallel_log/slowest_specs.log") if @slow_specs_enabled
 
     #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
+    results = find_results(test_results.map { |result| result[:stdout] }*"")
+    @outcome_builder_enabled ? spec_summary = calculate_total_spec_details : spec_summary = summarize_results(results)
+
     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)
+      if @reruns_enabled
+        puts "INFO: RERUNS are enabled"
+        rerun_initializer(name, failed, spec_summary)
+      else
+        abort("SEVERE: #{name.capitalize}s Failed") if failed || spec_summary[2].to_i != 0
+      end
     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 "WARNING: OutcomeBuilder is disabled not checking for false positives its likely things like thread failures and rspec non 0 exit codes will cause false positives"
+      puts spec_summary
       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]
+    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
@@ -178,13 +180,13 @@ class ParallelizedSpecs
     end
   end
 
-  def self.rerun_initializer(name, failed)
-    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."
+  def self.rerun_initializer(name, failed, spec_summary)
+    if spec_summary[2].to_i != 0 && !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...\n..\n."
       ParallelizedSpecs.rerun()
     else
       #works on both 1.8.7\1.9.3
-      abort "SEVERE: #{name.capitalize}s Failed" if Dir.glob("#{Rails.root}/tmp/parallel_log/failed_specs/{*,.*}").count > 2 || failed
+      abort "SEVERE: #{name.capitalize}s Failed" if spec_summary[2].to_i != 0 || failed
     end
   end
 

data/parallelized_specs.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = "parallelized_specs"
-  s.version = "0.4.48"
+  s.version = "0.4.49"
 
   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: 111
+  hash: 109
   prerelease: 
   segments: 
   - 0
   - 4
-  - 48
-  version: 0.4.48
+  - 49
+  version: 0.4.49
 platform: ruby
 authors: 
 - Jake Sorce, Bryan Madsen, Shawn Meredith