vagrant-libvirt 0.8.2 → 0.9.0

data/spec/unit/action/create_domain_spec.rb

@@ -21,15 +21,9 @@ describe VagrantPlugins::ProviderLibvirt::Action::CreateDomain do
   let(:domain_volume) { instance_double(::Fog::Libvirt::Compute::Volume) }
 
   let(:domain_xml) { File.read(File.join(File.dirname(__FILE__), File.basename(__FILE__, '.rb'), domain_xml_file)) }
-  let(:storage_pool_xml) { File.read(File.join(File.dirname(__FILE__), File.basename(__FILE__, '.rb'), storage_pool_xml_file)) }
-  let(:libvirt_storage_pool) { double('storage_pool') }
 
   describe '#call' do
     before do
-      allow_any_instance_of(VagrantPlugins::ProviderLibvirt::Driver)
-        .to receive(:connection).and_return(connection)
-      allow(connection).to receive(:client).and_return(libvirt_client)
-
       allow(connection).to receive(:servers).and_return(servers)
       allow(connection).to receive(:volumes).and_return(volumes)
       allow(volumes).to receive(:all).and_return([domain_volume])
@@ -43,15 +37,17 @@ describe VagrantPlugins::ProviderLibvirt::Action::CreateDomain do
 
       env[:domain_name] = "vagrant-test_default"
 
-      env[:box_volumes] = []
-      env[:box_volumes].push({
+      env[:domain_volumes] = []
+      env[:domain_volumes].push({
+        :device=>'vda',
+        :bus=>'virtio',
+        :cache=>'default',
+        :absolute_path=>'/var/lib/libvirt/images/vagrant-test_default.img',
         :path=>"/test/box.img",
-        :name=>"test_vagrant_box_image_1.1.1_0.img", 
+        :name=>'test_vagrant_box_image_1.1.1_0.img',
         :virtual_size=> ByteNumber.new(5),
+        :pool=>'default',
       })
-      # should be ignored for system session and used for user session
-      allow(Process).to receive(:uid).and_return(9999)
-      allow(Process).to receive(:gid).and_return(9999)
     end
 
     context 'connection => qemu:///system' do
@@ -61,123 +57,99 @@ describe VagrantPlugins::ProviderLibvirt::Action::CreateDomain do
         allow(machine.provider_config).to receive(:qemu_use_session).and_return(false)
       end
 
-      context 'default pool' do
-        it 'should execute correctly' do
-          expect(servers).to receive(:create).with(xml: domain_xml).and_return(machine)
-          expect(volumes).to_not receive(:create) # additional disks only
-
-          expect(subject.call(env)).to be_nil
-        end
-
-        context 'with no box' do
-          let(:storage_pool_xml_file) { 'default_system_storage_pool.xml' }
-          let(:vagrantfile) do
-            <<-EOF
-            Vagrant.configure('2') do |config|
-              config.vm.define :test
-            end
-            EOF
-          end
+      it 'should execute correctly' do
+        expect(servers).to receive(:create).with(xml: domain_xml).and_return(machine)
+        expect(volumes).to_not receive(:create) # additional disks only
 
-          it 'should query for the storage pool path' do
-            expect(libvirt_client).to receive(:lookup_storage_pool_by_name).and_return(libvirt_storage_pool)
-            expect(libvirt_storage_pool).to receive(:xml_desc).and_return(storage_pool_xml)
-            expect(servers).to receive(:create).and_return(machine)
+        expect(subject.call(env)).to be_nil
+      end
 
-            expect(subject.call(env)).to be_nil
-          end
+      context 'additional disks' do
+        let(:disks) do
+          [
+            :device        => 'vdb',
+            :cache         => 'default',
+            :bus           => 'virtio',
+            :type          => 'qcow2',
+            :absolute_path => '/var/lib/libvirt/images/vagrant-test_default-vdb.qcow2',
+            :virtual_size  => ByteNumber.new(20*1024*1024*1024),
+            :pool          => 'default',
+          ]
         end
 
-        context 'additional disks' do
-          let(:vagrantfile_providerconfig) do
-            <<-EOF
-            libvirt.storage :file, :size => '20G'
-            EOF
-          end
-
-          context 'volume create failed' do
-            it 'should raise an exception' do
-              expect(volumes).to receive(:create).and_raise(Libvirt::Error)
-
-              expect{ subject.call(env) }.to raise_error(VagrantPlugins::ProviderLibvirt::Errors::FogCreateDomainVolumeError)
-            end
-          end
+        before do
+          env[:disks] = disks
+        end
 
-          context 'volume create succeeded' do
-            let(:domain_xml_file) { 'additional_disks_domain.xml' }
-
-            it 'should complete' do
-              expect(volumes).to receive(:create).with(
-                hash_including(
-                  :path        => "/var/lib/libvirt/images/vagrant-test_default-vdb.qcow2",
-                  :owner       => 0,
-                  :group       => 0,
-                  :pool_name   => "default",
-                )
-              )
-              expect(servers).to receive(:create).with(xml: domain_xml).and_return(machine)
+        context 'volume create failed' do
+          it 'should raise an exception' do
+            expect(volumes).to receive(:create).and_raise(Libvirt::Error)
 
-              expect(subject.call(env)).to be_nil
-            end
+            expect{ subject.call(env) }.to raise_error(VagrantPlugins::ProviderLibvirt::Errors::FogCreateDomainVolumeError)
           end
         end
 
-        context 'with custom disk device setting' do
-          let(:domain_xml_file) { 'custom_disk_settings.xml' }
-          let(:vagrantfile_providerconfig) {
-            <<-EOF
-              libvirt.disk_device = 'sda'
-            EOF
-          }
+        context 'volume create succeeded' do
+          let(:domain_xml_file) { 'additional_disks_domain.xml' }
 
-          it 'should set the domain device' do
-            expect(ui).to receive(:info).with(/ -- Image\(sda\):.*/)
+          it 'should complete' do
+            expect(volumes).to receive(:create).with(
+              hash_including(
+                :path        => "/var/lib/libvirt/images/vagrant-test_default-vdb.qcow2",
+                :owner       => 0,
+                :group       => 0,
+                :pool_name   => "default",
+              )
+            )
             expect(servers).to receive(:create).with(xml: domain_xml).and_return(machine)
 
             expect(subject.call(env)).to be_nil
           end
         end
+      end
 
-        context 'with two domain disks' do
-          let(:domain_xml_file) { 'two_disk_settings.xml' }
-          let(:domain_volume_2) { double('domain_volume 2') }
-
-          before do
-            expect(volumes).to receive(:all).and_return([domain_volume])
-            expect(volumes).to receive(:all).and_return([domain_volume_2])
-            expect(domain_volume_2).to receive(:pool_name).and_return('default')
-            expect(domain_volume_2).to receive(:path).and_return('/var/lib/libvirt/images/vagrant-test_default_1.img')
-
-            env[:box_volumes].push({
-              :path=>"/test/box_1.img",
-              :name=>"test_vagrant_box_image_1.1.1_1.img",
-              :virtual_size=> ByteNumber.new(5),
-            })
-          end
+      context 'with custom disk device setting' do
+        let(:domain_xml_file) { 'custom_disk_settings.xml' }
 
-          it 'should correctly assign device entries' do
-            expect(ui).to receive(:info).with(/ -- Image\(vda\):.*/)
-            expect(ui).to receive(:info).with(/ -- Image\(vdb\):.*/)
-            expect(servers).to receive(:create).with(xml: domain_xml).and_return(machine)
+        before do
+          env[:domain_volumes][0][:device] = 'sda'
+        end
 
-            expect(subject.call(env)).to be_nil
-          end
+        it 'should set the domain device' do
+          expect(ui).to receive(:info).with(/ -- Image\(sda\):.*/)
+          expect(servers).to receive(:create).with(xml: domain_xml).and_return(machine)
+
+          expect(subject.call(env)).to be_nil
         end
       end
 
-      context 'no default pool' do
-        let(:vagrantfile) do
-          <<-EOF
-          Vagrant.configure('2') do |config|
-            config.vm.define :test
-          end
-          EOF
+      context 'with two domain disks' do
+        let(:domain_xml_file) { 'two_disk_settings.xml' }
+        let(:domain_volume_2) { double('domain_volume 2') }
+
+        before do
+          expect(volumes).to receive(:all).with(name: 'vagrant-test_default.img').and_return([domain_volume])
+          expect(volumes).to receive(:all).with(name: 'vagrant-test_default_1.img').and_return([domain_volume_2])
+          expect(domain_volume_2).to receive(:pool_name).and_return('default')
+
+          env[:domain_volumes].push({
+            :device=>'vdb',
+            :bus=>'virtio',
+            :cache=>'default',
+            :absolute_path=>'/var/lib/libvirt/images/vagrant-test_default_1.img',
+            :path=>"/test/box_1.img",
+            :name=>"test_vagrant_box_image_1.1.1_1.img",
+            :virtual_size=> ByteNumber.new(5),
+            :pool=>'default',
+          })
         end
 
-        it 'should raise an exception' do
-          expect(libvirt_client).to receive(:lookup_storage_pool_by_name).and_return(nil)
+        it 'should list multiple device entries' do
+          expect(ui).to receive(:info).with(/ -- Image\(vda\):.*/)
+          expect(ui).to receive(:info).with(/ -- Image\(vdb\):.*/)
+          expect(servers).to receive(:create).with(xml: domain_xml).and_return(machine)
 
-          expect{ subject.call(env) }.to raise_error(VagrantPlugins::ProviderLibvirt::Errors::NoStoragePool)
+          expect(subject.call(env)).to be_nil
         end
       end
     end
@@ -187,57 +159,51 @@ describe VagrantPlugins::ProviderLibvirt::Action::CreateDomain do
         allow(machine.provider_config).to receive(:qemu_use_session).and_return(true)
       end
 
-      context 'default pool' do
-        it 'should execute correctly' do
-          expect(servers).to receive(:create).and_return(machine)
+      it 'should execute correctly' do
+        expect(servers).to receive(:create).and_return(machine)
 
-          expect(subject.call(env)).to be_nil
+        expect(subject.call(env)).to be_nil
+      end
+
+      context 'additional disks' do
+        let(:vagrantfile_providerconfig) do
+          <<-EOF
+          libvirt.qemu_use_session = true
+          EOF
         end
 
-        context 'with no box' do
-          let(:storage_pool_xml_file) { 'default_user_storage_pool.xml' }
-          let(:vagrantfile) do
-            <<-EOF
-            Vagrant.configure('2') do |config|
-              config.vm.define :test
-              config.vm.provider :libvirt do |libvirt|
-                #{vagrantfile_providerconfig}
-              end
-            end
-            EOF
-          end
+        let(:disks) do
+          [
+            :device        => 'vdb',
+            :cache         => 'default',
+            :bus           => 'virtio',
+            :type          => 'qcow2',
+            :absolute_path => '/var/lib/libvirt/images/vagrant-test_default-vdb.qcow2',
+            :virtual_size  => ByteNumber.new(20*1024*1024*1024),
+            :pool          => 'default',
+          ]
+        end
 
-          it 'should query for the storage pool path' do
-            expect(libvirt_client).to receive(:lookup_storage_pool_by_name).and_return(libvirt_storage_pool)
-            expect(libvirt_storage_pool).to receive(:xml_desc).and_return(storage_pool_xml)
-            expect(servers).to receive(:create).and_return(machine)
+        before do
+          expect(Process).to receive(:uid).and_return(9999).at_least(:once)
+          expect(Process).to receive(:gid).and_return(9999).at_least(:once)
 
-            expect(subject.call(env)).to be_nil
-          end
+          env[:disks] = disks
         end
 
-        context 'additional disks' do
-          let(:vagrantfile_providerconfig) do
-            <<-EOF
-            libvirt.qemu_use_session = true
-            libvirt.storage :file, :size => '20G'
-            EOF
-          end
-
-          context 'volume create succeeded' do
-            it 'should complete' do
-              expect(volumes).to receive(:create).with(
-                hash_including(
-                  :path        => "/var/lib/libvirt/images/vagrant-test_default-vdb.qcow2",
-                  :owner       => 9999,
-                  :group       => 9999,
-                  :pool_name   => "default",
-                )
+        context 'volume create succeeded' do
+          it 'should complete' do
+            expect(volumes).to receive(:create).with(
+              hash_including(
+                :path        => "/var/lib/libvirt/images/vagrant-test_default-vdb.qcow2",
+                :owner       => 9999,
+                :group       => 9999,
+                :pool_name   => "default",
               )
-              expect(servers).to receive(:create).and_return(machine)
+            )
+            expect(servers).to receive(:create).and_return(machine)
 
-              expect(subject.call(env)).to be_nil
-            end
+            expect(subject.call(env)).to be_nil
           end
         end
       end

data/spec/unit/action/create_domain_volume_spec.rb

@@ -26,9 +26,6 @@ describe VagrantPlugins::ProviderLibvirt::Action::CreateDomainVolume do
 
   describe '#call' do
     before do
-      allow_any_instance_of(VagrantPlugins::ProviderLibvirt::Driver)
-        .to receive(:connection).and_return(connection)
-      allow(connection).to receive(:client).and_return(libvirt_client)
       allow(connection).to receive(:volumes).and_return(volumes)
       allow(volumes).to receive(:all).and_return(all)
       allow(all).to receive(:first).and_return(box_volume)

data/spec/unit/action/destroy_domain_spec.rb

@@ -19,10 +19,14 @@ describe VagrantPlugins::ProviderLibvirt::Action::DestroyDomain do
 
   let(:domain_xml) { File.read(File.join(File.dirname(__FILE__), File.basename(__FILE__, '.rb'), domain_xml_file)) }
 
+  let(:destroy_method) { double('destroy_method') }
+
   before do
     allow(machine.provider).to receive('driver').and_return(driver)
     allow(driver).to receive(:connection).and_return(connection)
     allow(logger).to receive(:info)
+    allow(domain).to receive(:method).with(:destroy).and_return(destroy_method)
+    allow(destroy_method).to receive(:parameters).and_return([[:opt, :options, :flags]])
   end
 
   describe '#call' do
@@ -49,7 +53,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::DestroyDomain do
 
       context 'when box only has one root disk' do
         it 'calls fog to destroy volumes' do
-          expect(domain).to receive(:destroy).with(destroy_volumes: true)
+          expect(domain).to receive(:destroy).with(destroy_volumes: true, flags: 0)
           expect(subject.call(env)).to be_nil
         end
 
@@ -69,7 +73,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::DestroyDomain do
           end
 
           it 'destroys disks individually' do
-            expect(domain).to receive(:destroy).with(destroy_volumes: false)
+            expect(domain).to receive(:destroy).with(destroy_volumes: false, flags: 0)
             expect(extra_disk).to receive(:destroy) # extra disk remove
             expect(root_disk).to receive(:destroy)  # root disk remove
             expect(subject.call(env)).to be_nil
@@ -81,7 +85,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::DestroyDomain do
         let(:domain_xml_file) { 'box_multiple_disks.xml' }
 
         it 'calls fog to destroy volumes' do
-          expect(domain).to receive(:destroy).with(destroy_volumes: true)
+          expect(domain).to receive(:destroy).with(destroy_volumes: true, flags: 0)
           expect(subject.call(env)).to be_nil
         end
 
@@ -111,7 +115,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::DestroyDomain do
               expect(disk).to receive(:name).and_return(name).at_least(:once)
               expect(disk).to receive(:destroy)
             end
-            expect(domain).to receive(:destroy).with(destroy_volumes: false)
+            expect(domain).to receive(:destroy).with(destroy_volumes: false, flags: 0)
             expect(subject.call(env)).to be_nil
           end
 
@@ -133,7 +137,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::DestroyDomain do
                 next if disk == domain_disks.last.first
                 expect(disk).to receive(:destroy)
               end
-              expect(domain).to receive(:destroy).with(destroy_volumes: false)
+              expect(domain).to receive(:destroy).with(destroy_volumes: false, flags: 0)
               expect(subject.call(env)).to be_nil
             end
 
@@ -141,7 +145,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::DestroyDomain do
               let(:domain_xml_file) { 'box_multiple_disks_and_additional_and_custom_disks_no_aliases.xml' }
 
               it 'only destroys expected disks' do
-                expect(ui).to receive(:warn).with(/Destroying machine that was originally created without device aliases.*/)
+                expect(ui).to receive(:warn).with(/Machine that was originally created without device aliases.*/)
                 expect(ui).to receive(:warn).with(/Unexpected number of volumes detected/)
                 expect(ui).to receive(:warn).with(/box metadata not available to get volume list during destroy, assuming inferred list/)
                 domain_disks.each do |disk, name|
@@ -150,7 +154,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::DestroyDomain do
                   next if domain_disks.last.first == disk
                   expect(disk).to receive(:destroy)
                 end
-                expect(domain).to receive(:destroy).with(destroy_volumes: false)
+                expect(domain).to receive(:destroy).with(destroy_volumes: false, flags: 0)
                 expect(subject.call(env)).to be_nil
               end
 
@@ -168,7 +172,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::DestroyDomain do
                 end
 
                 it 'only destroys expected disks' do
-                  expect(ui).to receive(:warn).with(/Destroying machine that was originally created without device aliases.*/)
+                  expect(ui).to receive(:warn).with(/Machine that was originally created without device aliases.*/)
                   expect(ui).to receive(:warn).with(/Unexpected number of volumes detected/)
                   domain_disks.each do |disk, name|
                     expect(disk).to receive(:name).and_return(name).at_least(:once)
@@ -176,7 +180,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::DestroyDomain do
                     next if domain_disks.last.first == disk
                     expect(disk).to receive(:destroy)
                   end
-                  expect(domain).to receive(:destroy).with(destroy_volumes: false)
+                  expect(domain).to receive(:destroy).with(destroy_volumes: false, flags: 0)
                   expect(subject.call(env)).to be_nil
                 end
               end
@@ -185,6 +189,35 @@ describe VagrantPlugins::ProviderLibvirt::Action::DestroyDomain do
         end
       end
 
+      context 'when has nvram' do
+        let(:vagrantfile) do
+          <<-EOF
+          Vagrant.configure('2') do |config|
+            config.vm.define :test
+            config.vm.provider :libvirt do |libvirt|
+              libvirt.nvram = "test"
+            end
+          end
+          EOF
+        end
+
+        it 'sets destroy flags to keep nvram' do
+          expect(domain).to receive(:destroy).with(destroy_volumes: true, flags: VagrantPlugins::ProviderLibvirt::Util::DomainFlags::VIR_DOMAIN_UNDEFINE_KEEP_NVRAM)
+          expect(subject.call(env)).to be_nil
+        end
+
+        context 'when fog does not support destroy with flags' do
+          before do
+            expect(destroy_method).to receive(:parameters).and_return([[:opt, :options]])
+          end
+
+          it 'skips setting additional destroy flags' do
+            expect(domain).to receive(:destroy).with(destroy_volumes: true)
+            expect(subject.call(env)).to be_nil
+          end
+        end
+      end
+
       context 'when has CDROMs attached' do
         let(:vagrantfile_providerconfig) do
           <<-EOF
@@ -197,7 +230,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::DestroyDomain do
           expect(domain).to receive(:volumes).and_return([root_disk, nil])
           expect(libvirt_domain).to receive(:xml_desc).and_return(domain_xml)
 
-          expect(domain).to_not receive(:destroy).with(destroy_volumes: true)
+          expect(domain).to_not receive(:destroy).with(destroy_volumes: true, flags: 0)
           expect(root_disk).to receive(:destroy)  # root disk remove
           expect(subject.call(env)).to be_nil
         end

data/spec/unit/action/handle_box_image_spec.rb

@@ -15,7 +15,6 @@ describe VagrantPlugins::ProviderLibvirt::Action::HandleBoxImage do
   include_context 'unit'
   include_context 'libvirt'
 
-  let(:libvirt_client) { double('libvirt_client') }
   let(:volumes) { double('volumes') }
   let(:all) { double('all') }
   let(:box_volume) { double('box_volume') }
@@ -54,17 +53,13 @@ describe VagrantPlugins::ProviderLibvirt::Action::HandleBoxImage do
   byte_number_20G = ByteNumber.new(21474836480)
 
 
-
-
   describe '#call' do
     before do
-      allow_any_instance_of(VagrantPlugins::ProviderLibvirt::Driver)
-        .to receive(:connection).and_return(connection)
-      allow(connection).to receive(:client).and_return(libvirt_client)
       allow(connection).to receive(:volumes).and_return(volumes)
       allow(volumes).to receive(:all).and_return(all)
       allow(env[:ui]).to receive(:clear_line)
 
+      env[:machine].provider_config.disk_device = 'vda'
     end
 
     context 'when one disk in metadata.json' do
@@ -92,7 +87,8 @@ describe VagrantPlugins::ProviderLibvirt::Action::HandleBoxImage do
               :path=>"/test/box.img",
               :name=>"test_vagrant_box_image_1.1.1_box.img",
               :virtual_size=>byte_number_5G,
-              :format=>"qcow2"
+              :format=>"qcow2",
+              :device=>'vda',
             }
           ]
         )
@@ -121,7 +117,8 @@ describe VagrantPlugins::ProviderLibvirt::Action::HandleBoxImage do
                 :path=>"/test/box.img",
                 :name=>"test_vagrant_box_image_0_#{box_mtime.to_i}_box.img",
                 :virtual_size=>byte_number_5G,
-                :format=>"qcow2"
+                :format=>"qcow2",
+                :device=>'vda',
               }
             ]
           )
@@ -130,7 +127,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::HandleBoxImage do
 
       context 'When config.machine_virtual_size is set and smaller than box_virtual_size' do
         before do
-          allow(env[:machine]).to receive_message_chain("provider_config.machine_virtual_size").and_return(1)
+          env[:machine].provider_config.machine_virtual_size = 1
         end
         it 'should warning must be raise' do
           expect(ui).to receive(:warn).with("Ignoring requested virtual disk size of '1' as it is below\nthe minimum box image size of '5'.")
@@ -141,7 +138,8 @@ describe VagrantPlugins::ProviderLibvirt::Action::HandleBoxImage do
                 :path=>"/test/box.img",
                 :name=>"test_vagrant_box_image_1.1.1_box.img",
                 :virtual_size=>byte_number_5G,
-                :format=>"qcow2"
+                :format=>"qcow2",
+                :device=>'vda',
               }
             ]
           )
@@ -150,7 +148,7 @@ describe VagrantPlugins::ProviderLibvirt::Action::HandleBoxImage do
 
       context 'When config.machine_virtual_size is set and higher than box_virtual_size' do
         before do
-          allow(env[:machine]).to receive_message_chain("provider_config.machine_virtual_size").and_return(20)
+          env[:machine].provider_config.machine_virtual_size = 20
         end
         it 'should be use' do
           expect(ui).to receive(:info).with("Created volume larger than box defaults, will require manual resizing of\nfilesystems to utilize.")
@@ -161,7 +159,8 @@ describe VagrantPlugins::ProviderLibvirt::Action::HandleBoxImage do
                 :path=>"/test/box.img",
                 :name=>"test_vagrant_box_image_1.1.1_box.img",
                 :virtual_size=>byte_number_20G,
-                :format=>"qcow2"
+                :format=>"qcow2",
+                :device=>'vda',
               }
             ]
           )
@@ -252,7 +251,8 @@ describe VagrantPlugins::ProviderLibvirt::Action::HandleBoxImage do
               :path=>"/test/box.img",
               :name=>"test_vagrant_box_image_1.1.1_send_box_name.img",
               :virtual_size=>byte_number_5G,
-              :format=>"qcow2"
+              :format=>"qcow2",
+              :device=>'vda',
             },
             {
               :path=>"/test/disk.qcow2",

data/spec/unit/action/package_domain_spec.rb

@@ -12,8 +12,6 @@ describe VagrantPlugins::ProviderLibvirt::Action::PackageDomain do
   include_context 'libvirt'
   include_context 'temporary_dir'
 
-  let(:libvirt_client) { double('libvirt_client') }
-  let(:libvirt_domain) { double('libvirt_domain') }
   let(:servers) { double('servers') }
   let(:volumes) { double('volumes') }
   let(:metadata_file) { double('file') }
@@ -21,9 +19,6 @@ describe VagrantPlugins::ProviderLibvirt::Action::PackageDomain do
 
   describe '#call' do
     before do
-      allow_any_instance_of(VagrantPlugins::ProviderLibvirt::Driver)
-        .to receive(:connection).and_return(connection)
-      allow(connection).to receive(:client).and_return(libvirt_client)
       allow(libvirt_client).to receive(:lookup_domain_by_uuid).and_return(libvirt_domain)
 
       allow(connection).to receive(:servers).and_return(servers)

data/spec/unit/action/resolve_disk_settings_spec/default_domain.xml

@@ -0,0 +1,43 @@
+<domain type='kvm' xmlns:qemu='http://libvirt.org/schemas/domain/qemu/1.0'>
+  <name>vagrant-test_default</name>
+  <title></title>
+  <description>Source: /rootpath/Vagrantfile</description>
+  <uuid></uuid>
+  <memory>524288</memory>
+  <vcpu>1</vcpu>
+  <cpu mode='host-model'>
+    <model fallback='allow'></model>
+  </cpu>
+  <os>
+    <type>hvm</type>
+    <kernel></kernel>
+    <initrd></initrd>
+    <cmdline></cmdline>
+  </os>
+  <features>
+    <acpi/>
+    <apic/>
+    <pae/>
+  </features>
+  <clock offset='utc'>
+  </clock>
+  <devices>
+    <disk type='file' device='disk'>
+      <alias name='ua-box-volume-0'/>
+      <driver name='qemu' type='qcow2' cache='default'/>
+      <source file='/var/lib/libvirt/images/vagrant-test_default.img'/>
+      <target dev='vda' bus='virtio'/>
+    </disk>
+    <serial type='pty'>
+      <target port='0'/>
+    </serial>
+    <console type='pty'>
+      <target port='0'/>
+    </console>
+    <input type='mouse' bus='ps2'/>
+    <graphics type='vnc' port='-1' autoport='yes' listen='127.0.0.1' keymap='en-us'/>
+    <video>
+      <model type='cirrus' vram='16384' heads='1'/>
+    </video>
+  </devices>
+</domain>