Merge pull request #320 from skipkayhil/hm-dump-to-file-2

Reapply "Dump JSON directly to file instead of buffering"

Author: ChrisBr

ruby/lib/minitest/queue/build_status_reporter.rb

@@ -107,11 +107,15 @@ def progress
       end
 
       def write_failure_file(file)
-        File.write(file, error_reports.map(&:to_h).to_json)
+        File.open(file, 'w') do |f|
+          JSON.dump(error_reports.map(&:to_h), f)
+        end
       end
 
       def write_flaky_tests_file(file)
-        File.write(file, flaky_reports.to_json)
+        File.open(file, 'w') do |f|
+          JSON.dump(flaky_reports, f)
+        end
       end
 
       private

ruby/lib/minitest/queue/runner.rb

@@ -323,7 +323,9 @@ def display_warnings(build)
         warnings = build.pop_warnings.map do |type, attributes|
           attributes.merge(type: type)
         end.compact
-        File.write(queue_config.warnings_file, warnings.to_json)
+        File.open(queue_config.warnings_file, 'w') do |f|
+          JSON.dump(warnings, f)
+        end
       end
 
       def run_tests_in_fork(queue)