Make sure we clean up the result path on failure

Author: eightbitraptor

lib/benchmark_suite.rb

@@ -55,6 +55,7 @@ def run(ruby:, ruby_description:)
         bench_data[entry.name] = process_benchmark_result(result_json_path, result[:command], delete_file: !caller_json_path)
       else
         bench_failures[entry.name] = result[:status].exitstatus
+        FileUtils.rm_f(result_json_path) unless caller_json_path
       end
     end
 

test/benchmark_suite_test.rb

@@ -8,6 +8,7 @@
 
 describe BenchmarkSuite do
   before do
+    ENV.delete('RESULT_JSON_PATH')
     @original_dir = Dir.pwd
     @temp_dir = Dir.mktmpdir
     Dir.chdir(@temp_dir)