Merge pull request #5117 from james-stocks/js/fix_html2_tests

Address test:isolated failure

Rakefile

@@ -157,6 +157,7 @@ namespace :test do
   end
   task parallel: [:accept_license] # given isolated being green, why is this needed?
 
+  desc "Run test files in multiple threads"
   task :isolated do
     require "fileutils"
 

lib/plugins/inspec-reporter-html2/test/functional/inspec_reporter_html2_test.rb

@@ -6,7 +6,8 @@ describe "inspec-reporter-html2" do
 
   let(:output_file) { f = Tempfile.new(["temp", ".html"]); f.close; f.path }
   def teardown
-    FileUtils.rm(output_file)
+    # Use force because the deletion may occasionally fail in parallel CI
+    FileUtils.rm_f(output_file)
   end
 
   describe "when run on a basic profile" do