parallel_tests 2.17.1 → 2.18.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f5a51a34a91c7947c0a43231f159eb47dcf92678
-  data.tar.gz: 7cc48638574403d6d898c3b4bf64396a25268c54
+  metadata.gz: f5b04284134cfd09c0b28c8852041cae9946958b
+  data.tar.gz: 4579bd82392b38f52dac94facf562b42a729331f
 SHA512:
-  metadata.gz: 7b1cc69d8645a0468d7cf9a61326050d77b9f294c8f5db512e8bc3fb16b436cee1ac49cc91be6f456b746fe5ce42f2cad0a011ddb8cb4b77d53993e32765e97b
-  data.tar.gz: 02187019194e0f549ecd8c04902c199f00a473c08e209efd8d333dc0925c6a474adb487adfa5908e2eb0fb2a6423161cebc2c33f6552b637db5ab5f7cbbba6ab
+  metadata.gz: ae0013b187faed8555b62d8e88a46dfcd58acdaeb7f87c058f0eb6e64f9f6f44236d0726f0cae30a3242e37ed2c3f07545b2e39b0dd6e1e3a863a1959c3cbaf7
+  data.tar.gz: 146639b4ab99103164b734247b278752d2b1cdcc2523e843655287ab957a444fa5e4c82696e18b175f8d2ea7efe2ed8f491da9d93cae597bb1845c189814d4d5

data/lib/parallel_tests.rb

@@ -60,14 +60,6 @@ module ParallelTests
       current_process_number == total_processes
     end
 
-    def parent_pid
-      if WINDOWS
-        `wmic process where (processid=#{Process.pid}) get parentprocessid`
-      else
-        `ps -o ppid= -p#{`ps -o ppid= -p#{Process.pid}`}` #the true parent is one layer up.
-      end.to_i
-    end
-
     def with_ruby_binary(command)
       WINDOWS ? "#{RUBY_BINARY} -- #{command}" : command
     end

data/lib/parallel_tests/test/runner.rb

@@ -98,7 +98,8 @@ module ParallelTests
         end
 
         def find_results(test_output)
-          test_output.split("\n").map do |line|
+          test_output.lines.map do |line|
+            line.chomp!
             line.gsub!(/\e\[\d+m/, '') # remove color coding
             next unless line_is_result?(line)
             line

data/lib/parallel_tests/test/runtime_logger.rb

@@ -18,19 +18,28 @@ module ParallelTests
         end
 
         def unique_log
-          lock do
+          with_locked_log do |logfile|
             separator = "\n"
-            groups = File.read(logfile).split(separator).map { |line| line.split(":") }.group_by(&:first)
+            groups = logfile.read.split(separator).map { |line| line.split(":") }.group_by(&:first)
             lines = groups.map do |file, times|
               time = "%.2f" % times.map(&:last).map(&:to_f).inject(:+)
               "#{file}:#{time}"
             end
-            File.write(logfile, lines.join(separator) + separator)
+            logfile.rewind
+            logfile.write(lines.join(separator) + separator)
+            logfile.truncate(logfile.pos)
           end
         end
 
         private
 
+        def with_locked_log
+          File.open(logfile, File::RDWR|File::CREAT) do |logfile|
+            logfile.flock(File::LOCK_EX)
+            yield logfile
+          end
+        end
+
         # ensure folder exists + clean out previous log
         # this will happen in multiple processes, but should be roughly at the same time
         # so there should be no log message lost
@@ -43,8 +52,9 @@ module ParallelTests
 
         def log(test, time)
           return unless message = message(test, time)
-          lock do
-            File.open(logfile, 'a') { |f| f.puts message }
+          with_locked_log do |logfile|
+            logfile.seek(0, IO::SEEK_END)
+            logfile.puts message
           end
         end
 
@@ -54,17 +64,6 @@ module ParallelTests
           "#{filename}:#{delta}"
         end
 
-        def lock
-          File.open(logfile, 'r') do |f|
-            begin
-              f.flock File::LOCK_EX
-              yield
-            ensure
-              f.flock File::LOCK_UN
-            end
-          end
-        end
-
         def logfile
           ParallelTests::Test::Runner.runtime_log
         end

data/lib/parallel_tests/version.rb

@@ -1,3 +1,3 @@
 module ParallelTests
-  VERSION = Version = '2.17.1'
+  VERSION = Version = '2.18.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: parallel_tests
 version: !ruby/object:Gem::Version
-  version: 2.17.1
+  version: 2.18.0
 platform: ruby
 authors:
 - Michael Grosser
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-11-10 00:00:00.000000000 Z
+date: 2017-11-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: parallel