cask: Call Tempfile#close(true) with the choices file to ensure it deleted

Library/Homebrew/cask/lib/hbc/artifact/pkg.rb

@@ -54,17 +54,25 @@ module Hbc
         ]
         args << "-verboseR" if Hbc.verbose
         args << "-allowUntrusted" if pkg_install_opts :allow_untrusted
-        if pkg_install_opts :choices
+        with_choices_file pkg_install_opts(:choices) do |choices_path|
-          choices_file = choices_xml
+          args << "-applyChoiceChangesXML" << choices_path if choices_path
-          args << "-applyChoiceChangesXML" << choices_file.path
+          @command.run!("/usr/sbin/installer", sudo: true, args: args, print_stdout: true)
         end
-        @command.run!("/usr/sbin/installer", sudo: true, args: args, print_stdout: true)
       end
 
-      def choices_xml
+      def with_choices_file(choices)
-        file = Tempfile.open(["", ".xml"])
+        unless choices
-        file.write Plist::Emit.dump(pkg_install_opts(:choices))
+          yield nil
-        file
+          return
+        end
+
+        begin
+          file = Tempfile.new(["choices", ".xml"])
+          file.write Plist::Emit.dump(choices)
+          yield file.path
+        ensure
+          file.close(true)
+        end
       end
     end
   end

Library/Homebrew/cask/test/cask/artifact/pkg_test.rb

@@ -60,7 +60,9 @@ describe Hbc::Artifact::Pkg do
         </plist>
       EOS
       file.stubs path: Pathname.new("/tmp/choices.xml")
-      Tempfile.expects(:open).returns(file)
+      file.expects(:close).with true
+      Tempfile.expects(:new).returns file
+
       Hbc::FakeSystemCommand.expects_command(["/usr/bin/sudo", "-E", "--", "/usr/sbin/installer", "-pkg", @cask.staged_path.join("MyFancyPkg", "Fancy.pkg"), "-target", "/", "-applyChoiceChangesXML", @cask.staged_path.join("/tmp/choices.xml")])
 
       shutup do