Fix test issues

test/unit/vagrant/box_test.rb

@@ -245,7 +245,7 @@ describe Vagrant::Box, :skip_windows do
     end
 
     after do
-      metadata_url.unlink if metadata_url.file?
+      metadata_url.unlink
     end
 
     it "loads the url and returns the data" do

test/unit/vagrant/environment_test.rb

@@ -933,7 +933,7 @@ VF
       let(:instance) { described_class.new(local_data_path: local_data_path) }
 
       after do
-        File.unlink(v1_dotfile_tempfile) if File.file?(v1_dotfile_template)
+        File.unlink(v1_dotfile) if File.file?(v1_dotfile)
       end
 
       it "should be fine if dotfile is empty" do

test/unit/vagrant/machine_index_test.rb

@@ -9,7 +9,7 @@ require "vagrant/machine_index"
 describe Vagrant::MachineIndex do
   include_context "unit"
 
-  let(:data_dir) { Dir.mktmpdir }
+  let(:data_dir) { temporary_dir }
   let(:entry_klass) { Vagrant::MachineIndex::Entry }
 
   let(:new_entry) do

test/unit/vagrant/machine_test.rb

@@ -43,7 +43,7 @@ describe Vagrant::Machine do
   let(:instance) { new_instance }
 
   after do
-    FileUtils.rm_rf(data_dir)
+    FileUtils.rm_rf(data_dir) if data_dir
   end
 
   subject { instance }