beaker-vagrant 0.7.1 → 1.1.0

data/spec/beaker/hypervisor/vagrant_desktop_spec.rb

@@ -1,19 +1,19 @@
 require 'spec_helper'
 
 describe Beaker::VagrantDesktop do
-  let( :options ) { make_opts.merge({ :hosts_file => 'sample.cfg', 'logger' => double().as_null_object }) }
-  let( :vagrant ) { described_class.new( hosts, options ) }
-  let( :hosts ) { make_hosts }
+  let(:options) { make_opts.merge({ :hosts_file => 'sample.cfg', 'logger' => double.as_null_object }) }
+  let(:vagrant) { described_class.new(hosts, options) }
+  let(:hosts) { make_hosts }
 
-  it "uses the vmware_desktop provider for provisioning" do
+  it 'uses the vmware_desktop provider for provisioning' do
     hosts.each do |host|
       host_prev_name = host['user']
-      expect( vagrant ).to receive( :set_ssh_config ).with( host, 'vagrant' ).once
-      expect( vagrant ).to receive( :copy_ssh_to_root ).with( host, options ).once
-      expect( vagrant ).to receive( :set_ssh_config ).with( host, host_prev_name ).once
+      expect(vagrant).to receive(:set_ssh_config).with(host, 'vagrant').once
+      expect(vagrant).to receive(:copy_ssh_to_root).with(host, options).once
+      expect(vagrant).to receive(:set_ssh_config).with(host, host_prev_name).once
     end
-    expect( vagrant ).to receive( :hack_etc_hosts ).with( hosts, options ).once
-    expect( vagrant ).to receive( :vagrant_cmd ).with( "up --provider vmware_desktop" ).once
+    expect(vagrant).to receive(:hack_etc_hosts).with(hosts, options).once
+    expect(vagrant).to receive(:vagrant_cmd).with('up --provider vmware_desktop').once
     FakeFS do
       vagrant.provision
     end
@@ -27,32 +27,32 @@ describe Beaker::VagrantDesktop do
       end
     end
 
-    it "for a set of hosts" do
-      is_expected.to include( %Q{    v.vm.provider :vmware_desktop do |v|\n      v.vmx['memsize'] = '1024'\n    end})
+    it 'for a set of hosts' do
+      is_expected.to include(%(    v.vm.provider :vmware_desktop do |v|\n      v.vmx['memsize'] = '1024'\n    end))
     end
 
     context 'with whitelist_verified' do
-      let(:hosts) { make_hosts({:whitelist_verified => true}, 1) }
+      let(:hosts) { make_hosts({ whitelist_verified: true }, 1) }
 
-      it { is_expected.to include( %Q{ v.vmx['whitelist_verified'] = 'true'}) }
+      it { is_expected.to include(%( v.vmx['whitelist_verified'] = 'true')) }
     end
 
     context 'with functional_hgfs' do
-      let(:hosts) { make_hosts({:functional_hgfs => true}, 1) }
+      let(:hosts) { make_hosts({ functional_hgfs: true }, 1) }
 
-      it { is_expected.to include( %Q{ v.vmx['functional_hgfs'] = 'true'}) }
+      it { is_expected.to include(%( v.vmx['functional_hgfs'] = 'true')) }
     end
 
     context 'with unmount_default_hgfs' do
-      let(:hosts) { make_hosts({:unmount_default_hgfs => true}, 1) }
+      let(:hosts) { make_hosts({ unmount_default_hgfs: true }, 1) }
 
-      it { is_expected.to include( %Q{ v.vmx['unmount_default_hgfs'] = 'true'}) }
+      it { is_expected.to include(%( v.vmx['unmount_default_hgfs'] = 'true')) }
     end
 
-    context "with gui" do
-      let(:hosts) { make_hosts({:gui => true},1) }
+    context 'with gui' do
+      let(:hosts) { make_hosts({ gui: true }, 1) }
 
-      it { is_expected.to include( %Q{ v.vmx['gui'] = true}) }
+      it { is_expected.to include(%( v.vmx['gui'] = true)) }
     end
   end
 end

data/spec/beaker/hypervisor/vagrant_fusion_spec.rb

@@ -1,32 +1,32 @@
 require 'spec_helper'
 
 describe Beaker::VagrantFusion do
-  let( :options ) { make_opts.merge({ :hosts_file => 'sample.cfg', 'logger' => double().as_null_object }) }
-  let( :vagrant ) { described_class.new( hosts, options ) }
-  let( :hosts ) { make_hosts() }
+  let(:options) { make_opts.merge({ :hosts_file => 'sample.cfg', 'logger' => double.as_null_object }) }
+  let(:vagrant) { described_class.new(hosts, options) }
+  let(:hosts) { make_hosts }
 
-  it "uses the vmware_fusion provider for provisioning" do
+  it 'uses the vmware_fusion provider for provisioning' do
     hosts.each do |host|
       host_prev_name = host['user']
-      expect( vagrant ).to receive( :set_ssh_config ).with( host, 'vagrant' ).once
-      expect( vagrant ).to receive( :copy_ssh_to_root ).with( host, options ).once
-      expect( vagrant ).to receive( :set_ssh_config ).with( host, host_prev_name ).once
+      expect(vagrant).to receive(:set_ssh_config).with(host, 'vagrant').once
+      expect(vagrant).to receive(:copy_ssh_to_root).with(host, options).once
+      expect(vagrant).to receive(:set_ssh_config).with(host, host_prev_name).once
     end
-    expect( vagrant ).to receive( :hack_etc_hosts ).with( hosts, options ).once
-    expect( vagrant ).to receive( :vagrant_cmd ).with( "up --provider vmware_fusion" ).once
+    expect(vagrant).to receive(:hack_etc_hosts).with(hosts, options).once
+    expect(vagrant).to receive(:vagrant_cmd).with('up --provider vmware_fusion').once
     vagrant.provision
   end
 
   context 'Correct vagrant configuration' do
     subject do
       FakeFS do
-        vagrant.make_vfile( hosts, options )
+        vagrant.make_vfile(hosts, options)
         File.read(vagrant.instance_variable_get(:@vagrant_file))
       end
     end
 
     it 'has a provider section' do
-      is_expected.to include( %Q{    v.vm.provider :vmware_fusion do |v|\n      v.vmx['memsize'] = '1024'\n    end})
+      is_expected.to include(%(    v.vm.provider :vmware_fusion do |v|\n      v.vmx['memsize'] = '1024'\n    end))
     end
   end
 end

data/spec/beaker/hypervisor/vagrant_libvirt_spec.rb

@@ -1,27 +1,28 @@
 require 'spec_helper'
 
 describe Beaker::VagrantLibvirt do
-  let( :options ) { make_opts.merge({ :hosts_file => 'sample.cfg',
-                                      'logger' => double().as_null_object,
-                                      'libvirt' => { 'uri' => 'qemu+ssh://root@host/system'},
-                                      'vagrant_cpus' => 2,
-                                    }) }
-  let( :vagrant ) { described_class.new( hosts, options ) }
-  let( :hosts ) do
-    make_hosts().each do |host|
+  let(:options) do
+    make_opts.merge({ :hosts_file => 'sample.cfg',
+                      'logger' => double.as_null_object,
+                      'libvirt' => { 'uri' => 'qemu+ssh://root@host/system' },
+                      'vagrant_cpus' => 2, })
+  end
+  let(:vagrant) { described_class.new(hosts, options) }
+  let(:hosts) do
+    make_hosts.each do |host|
       host.delete('ip')
     end
   end
 
-  it "uses the vagrant_libvirt provider for provisioning" do
+  it 'uses the vagrant_libvirt provider for provisioning' do
     hosts.each do |host|
       host_prev_name = host['user']
-      expect( vagrant ).to receive( :set_ssh_config ).with( host, 'vagrant' ).once
-      expect( vagrant ).to receive( :copy_ssh_to_root ).with( host, options ).once
-      expect( vagrant ).to receive( :set_ssh_config ).with( host, host_prev_name ).once
+      expect(vagrant).to receive(:set_ssh_config).with(host, 'vagrant').once
+      expect(vagrant).to receive(:copy_ssh_to_root).with(host, options).once
+      expect(vagrant).to receive(:set_ssh_config).with(host, host_prev_name).once
     end
-    expect( vagrant ).to receive( :hack_etc_hosts ).with( hosts, options ).once
-    expect( vagrant ).to receive( :vagrant_cmd ).with( "up --provider libvirt" ).once
+    expect(vagrant).to receive(:hack_etc_hosts).with(hosts, options).once
+    expect(vagrant).to receive(:vagrant_cmd).with('up --provider libvirt').once
     FakeFS do
       vagrant.provision
     end
@@ -30,28 +31,28 @@ describe Beaker::VagrantLibvirt do
   context 'Correct vagrant configuration' do
     subject do
       FakeFS do
-        vagrant.make_vfile( hosts, options )
+        vagrant.make_vfile(hosts, options)
         File.read(vagrant.instance_variable_get(:@vagrant_file))
       end
     end
 
     it 'has a provider section' do
-      is_expected.to include( %Q{    v.vm.provider :libvirt do |node|})
+      is_expected.to include(%(    v.vm.provider :libvirt do |node|))
     end
 
-    it "has no private network" do
+    it 'has no private network' do
       is_expected.to include('v.vm.network :private_network')
     end
 
-    it "can specify the memory as an integer" do
+    it 'can specify the memory as an integer' do
       is_expected.to include('node.memory = 1024')
     end
 
-    it "can specify the number of cpus" do
-      is_expected.to include("node.cpus = 2")
+    it 'can specify the number of cpus' do
+      is_expected.to include('node.cpus = 2')
     end
 
-    it "can specify any libvirt option" do
+    it 'can specify any libvirt option' do
       is_expected.to include("node.uri = 'qemu+ssh://root@host/system'")
     end
   end

data/spec/beaker/hypervisor/vagrant_parallels_spec.rb

@@ -1,40 +1,40 @@
 require 'spec_helper'
 
 describe Beaker::VagrantParallels do
-  let( :options ) { make_opts.merge({ :hosts_file => 'sample.cfg', 'logger' => double().as_null_object }) }
-  let( :vagrant ) { Beaker::VagrantParallels.new( hosts, options ) }
-  let( :hosts ) { make_hosts }
+  let(:options) { make_opts.merge({ :hosts_file => 'sample.cfg', 'logger' => double.as_null_object }) }
+  let(:vagrant) { Beaker::VagrantParallels.new(hosts, options) }
+  let(:hosts) { make_hosts }
 
-  it "uses the parallels provider for provisioning" do
+  it 'uses the parallels provider for provisioning' do
     hosts.each do |host|
       host_prev_name = host['user']
-      expect( vagrant ).to receive( :set_ssh_config ).with( host, 'vagrant' ).once
-      expect( vagrant ).to receive( :copy_ssh_to_root ).with( host, options ).once
-      expect( vagrant ).to receive( :set_ssh_config ).with( host, host_prev_name ).once
+      expect(vagrant).to receive(:set_ssh_config).with(host, 'vagrant').once
+      expect(vagrant).to receive(:copy_ssh_to_root).with(host, options).once
+      expect(vagrant).to receive(:set_ssh_config).with(host, host_prev_name).once
     end
-    expect( vagrant ).to receive( :hack_etc_hosts ).with( hosts, options ).once
-    expect( vagrant ).to receive( :vagrant_cmd ).with( "up --provider parallels" ).once
+    expect(vagrant).to receive(:hack_etc_hosts).with(hosts, options).once
+    expect(vagrant).to receive(:vagrant_cmd).with('up --provider parallels').once
     vagrant.provision
   end
 
   context 'Correct vagrant configuration' do
     subject do
       FakeFS do
-        vagrant.make_vfile( hosts, options )
+        vagrant.make_vfile(hosts, options)
         File.read(vagrant.instance_variable_get(:@vagrant_file))
       end
     end
 
-    it "can make a Vagrantfile for a set of hosts" do
-      is_expected.to include( %Q{    v.vm.provider :parallels do |prl|\n      prl.optimize_power_consumption = false\n      prl.memory = '1024'\n    end})
+    it 'can make a Vagrantfile for a set of hosts' do
+      is_expected.to include(%(    v.vm.provider :parallels do |prl|\n      prl.optimize_power_consumption = false\n      prl.memory = '1024'\n    end))
     end
   end
 
   context 'disabled guest tools' do
-    let(:options) { super().merge({ :prl_update_guest_tools => 'disable' }) }
-    subject { vagrant.class.provider_vfile_section( hosts.first, options ) }
+    let(:options) { super().merge({ prl_update_guest_tools: 'disable' }) }
+    subject { vagrant.class.provider_vfile_section(hosts.first, options) }
 
-    it "can disable the auto-update functionality of the Parallels Guest Tools" do
+    it 'can disable the auto-update functionality of the Parallels Guest Tools' do
       is_expected.to match(/prl.update_guest_tools = false/)
     end
   end