vagrant-unbundled 1.8.1.2 → 1.8.4.1

data/test/unit/plugins/guests/suse/cap/configure_networks_test.rb

@@ -0,0 +1,60 @@
+require_relative "../../../../base"
+
+describe "VagrantPlugins::GuestSUSE::Cap::ConfigureNetworks" do
+  let(:caps) do
+    VagrantPlugins::GuestSUSE::Plugin
+      .components
+      .guest_capabilities[:suse]
+  end
+
+  let(:machine) { double("machine") }
+  let(:comm) { VagrantTests::DummyCommunicator::Communicator.new(machine) }
+
+  before do
+    allow(machine).to receive(:communicate).and_return(comm)
+    comm.stub_command("ip -o -0 addr | grep -v LOOPBACK | awk '{print $2}' | sed 's/://'",
+      stdout: "eth1\neth2")
+  end
+
+  after do
+    comm.verify_expectations!
+  end
+
+  describe ".configure_networks" do
+    let(:cap) { caps.get(:configure_networks) }
+
+    let(:network_1) do
+      {
+        interface: 0,
+        type: "dhcp",
+      }
+    end
+
+    let(:network_2) do
+      {
+        interface: 1,
+        type: "static",
+        ip: "33.33.33.10",
+        netmask: "255.255.0.0",
+        gateway: "33.33.0.1",
+      }
+    end
+
+    let(:guest) { double("guest") }
+
+    before do
+      allow(machine).to receive(:guest).and_return(guest)
+      allow(guest).to receive(:capability)
+        .with(:network_scripts_dir)
+        .and_return("/scripts")
+    end
+
+    it "creates and starts the networks" do
+      cap.configure_networks(machine, [network_1, network_2])
+      expect(comm.received_commands[1]).to match(/\/sbin\/ifdown 'eth1'/)
+      expect(comm.received_commands[1]).to match(/\/sbin\/ifup 'eth1'/)
+      expect(comm.received_commands[1]).to match(/\/sbin\/ifdown 'eth2'/)
+      expect(comm.received_commands[1]).to match(/\/sbin\/ifup 'eth2'/)
+    end
+  end
+end

data/test/unit/plugins/guests/suse/cap/halt_test.rb

@@ -0,0 +1,36 @@
+require_relative "../../../../base"
+
+describe "VagrantPlugins::GuestSUSE::Cap::Halt" do
+  let(:caps) do
+    VagrantPlugins::GuestSUSE::Plugin
+      .components
+      .guest_capabilities[:suse]
+  end
+
+  let(:machine) { double("machine") }
+  let(:comm) { VagrantTests::DummyCommunicator::Communicator.new(machine) }
+
+  before do
+    allow(machine).to receive(:communicate).and_return(comm)
+  end
+
+  after do
+    comm.verify_expectations!
+  end
+
+  describe ".halt" do
+    let(:cap) { caps.get(:halt) }
+
+    it "runs the shutdown command" do
+      comm.expect_command("/sbin/shutdown -h now")
+      cap.halt(machine)
+    end
+
+    it "does not raise an IOError" do
+      comm.stub_command("shutdown -h now", raise: IOError)
+      expect {
+        cap.halt(machine)
+      }.to_not raise_error
+    end
+  end
+end

data/test/unit/plugins/guests/suse/cap/network_scripts_dir_test.rb

@@ -0,0 +1,19 @@
+require_relative "../../../../base"
+
+describe "VagrantPlugins::GuestSUSE::Cap::NetworkScriptsDir" do
+  let(:caps) do
+    VagrantPlugins::GuestSUSE::Plugin
+      .components
+      .guest_capabilities[:suse]
+  end
+
+  let(:machine) { double("machine") }
+
+  describe ".network_scripts_dir" do
+    let(:cap) { caps.get(:network_scripts_dir) }
+
+    it "runs /etc/sysconfig/network" do
+      expect(cap.network_scripts_dir(machine)).to eq("/etc/sysconfig/network")
+    end
+  end
+end

data/test/unit/plugins/guests/suse/cap/nfs_client_test.rb

@@ -0,0 +1,31 @@
+require_relative "../../../../base"
+
+describe "VagrantPlugins::GuestSUSE::Cap::NFSClient" do
+  let(:caps) do
+    VagrantPlugins::GuestSUSE::Plugin
+      .components
+      .guest_capabilities[:suse]
+  end
+
+  let(:machine) { double("machine") }
+  let(:comm) { VagrantTests::DummyCommunicator::Communicator.new(machine) }
+
+  before do
+    allow(machine).to receive(:communicate).and_return(comm)
+  end
+
+  after do
+    comm.verify_expectations!
+  end
+
+  describe ".nfs_client_install" do
+    let(:cap) { caps.get(:nfs_client_install) }
+
+    it "installs nfs client utilities" do
+      cap.nfs_client_install(machine)
+      expect(comm.received_commands[0]).to match(/zypper -n install nfs-client/)
+      expect(comm.received_commands[0]).to match(/service rpcbind restart/)
+      expect(comm.received_commands[0]).to match(/service nfs restart/)
+    end
+  end
+end

data/test/unit/plugins/guests/suse/cap/rsync_test.rb

@@ -0,0 +1,38 @@
+require_relative "../../../../base"
+
+describe "VagrantPlugins::GuestSUSE::Cap::RSync" do
+  let(:caps) do
+    VagrantPlugins::GuestSUSE::Plugin
+      .components
+      .guest_capabilities[:suse]
+  end
+
+  let(:machine) { double("machine") }
+  let(:comm) { VagrantTests::DummyCommunicator::Communicator.new(machine) }
+
+  before do
+    allow(machine).to receive(:communicate).and_return(comm)
+  end
+
+  after do
+    comm.verify_expectations!
+  end
+
+  describe ".rsync_install" do
+    let(:cap) { caps.get(:rsync_install) }
+
+    it "installs rsync" do
+      comm.expect_command("zypper -n install rsync")
+      cap.rsync_install(machine)
+    end
+  end
+
+  describe ".rsync_installed" do
+    let(:cap) { caps.get(:rsync_installed) }
+
+    it "checks if rsync is installed" do
+      comm.expect_command("test -f /usr/bin/rsync")
+      cap.rsync_installed(machine)
+    end
+  end
+end

data/test/unit/plugins/guests/tinycore/cap/change_host_name_test.rb

@@ -1,4 +1,4 @@
-require File.expand_path("../../../../../base", __FILE__)
+require_relative "../../../../base"
 
 describe "VagrantPlugins::GuestTinyCore::Cap::ChangeHostName" do
   let(:described_class) do

data/test/unit/plugins/guests/ubuntu/cap/change_host_name_test.rb

@@ -1,34 +1,42 @@
-require File.expand_path("../../../../../base", __FILE__)
-require File.expand_path("../../../support/shared/debian_like_host_name_examples", __FILE__)
+require_relative "../../../../base"
 
 describe "VagrantPlugins::GuestUbuntu::Cap::ChangeHostName" do
-  let(:described_class) do
-    VagrantPlugins::GuestUbuntu::Plugin.components.guest_capabilities[:ubuntu].get(:change_host_name)
+  let(:caps) do
+    VagrantPlugins::GuestUbuntu::Plugin
+      .components
+      .guest_capabilities[:ubuntu]
   end
+
   let(:machine) { double("machine") }
-  let(:communicator) { VagrantTests::DummyCommunicator::Communicator.new(machine) }
-  let(:old_hostname) {'oldhostname.olddomain.tld' }
+  let(:comm) { VagrantTests::DummyCommunicator::Communicator.new(machine) }
 
   before do
-    allow(machine).to receive(:communicate).and_return(communicator)
-    communicator.stub_command('hostname -f', stdout: old_hostname)
+    allow(machine).to receive(:communicate).and_return(comm)
   end
 
   after do
-    communicator.verify_expectations!
+    comm.verify_expectations!
   end
 
   describe ".change_host_name" do
-    it_behaves_like "a debian-like host name change"
+    let(:cap) { caps.get(:change_host_name) }
+
+    let(:name) { 'banana-rama.example.com' }
 
-    it "refreshes the hostname service with upstart" do
-      communicator.expect_command(%q(service hostname start))
-      described_class.change_host_name(machine, 'newhostname.newdomain.tld')
+    it "sets the hostname if not set" do
+      comm.stub_command("hostname -f | grep -w '#{name}'", exit_code: 1)
+      cap.change_host_name(machine, name)
+      expect(comm.received_commands[1]).to match(/hostname -F \/etc\/hostname/)
+      expect(comm.received_commands[1]).to match(/\/etc\/init.d\/hostname.sh start/)
+      expect(comm.received_commands[1]).to match(/\/etc\/init.d\/hostname start/)
+      expect(comm.received_commands[1]).to match(/\/etc\/init.d\/networking force-reload/)
+      expect(comm.received_commands[1]).to match(/\/etc\/init.d\/network-manager force-reload/)
     end
 
-    it "renews dhcp on the system with the new hostname (with hotplug allowed)" do
-      communicator.expect_command(%q(ifdown -a; ifup -a; ifup -a --allow=hotplug))
-      described_class.change_host_name(machine, 'newhostname.newdomain.tld')
+    it "does not set the hostname if unset" do
+      comm.stub_command("hostname -f | grep -w '#{name}'", exit_code: 0)
+      cap.change_host_name(machine, name)
+      expect(comm.received_commands.size).to eq(1)
     end
   end
 end

data/test/unit/plugins/guests/windows/cap/change_host_name_test.rb

@@ -1,4 +1,4 @@
-require File.expand_path("../../../../../base", __FILE__)
+require_relative "../../../../base"
 
 require Vagrant.source_root.join("plugins/guests/windows/cap/change_host_name")
 

data/test/unit/plugins/guests/windows/cap/halt_test.rb

@@ -1,4 +1,4 @@
-require File.expand_path("../../../../../base", __FILE__)
+require_relative "../../../../base"
 
 require Vagrant.source_root.join("plugins/guests/windows/cap/halt")
 

data/test/unit/plugins/guests/windows/cap/mount_shared_folder_test.rb

@@ -1,4 +1,4 @@
-require File.expand_path("../../../../../base", __FILE__)
+require_relative "../../../../base"
 
 require Vagrant.source_root.join("plugins/guests/windows/cap/mount_shared_folder")
 

data/test/unit/plugins/guests/windows/cap/reboot_test.rb

@@ -1,4 +1,4 @@
-require File.expand_path("../../../../../base", __FILE__)
+require_relative "../../../../base"
 
 require Vagrant.source_root.join("plugins/guests/windows/cap/reboot")
 

data/test/unit/plugins/guests/windows/cap/rsync_test.rb

@@ -1,4 +1,4 @@
-require File.expand_path("../../../../../base", __FILE__)
+require_relative "../../../../base"
 
 require Vagrant.source_root.join("plugins/guests/windows/cap/rsync")
 

data/test/unit/plugins/hosts/darwin/cap/rdp_test.rb

@@ -0,0 +1,36 @@
+require_relative "../../../../base"
+
+require_relative "../../../../../../plugins/hosts/darwin/cap/rdp"
+
+describe VagrantPlugins::HostDarwin::Cap::RDP do
+  let(:rdp_info) do
+    {
+      host: "host",
+      port: "port",
+      username: "username",
+    }
+  end
+
+  it "includes the default options" do
+    path = described_class.generate_config_file(rdp_info)
+    result = File.read(path)
+    expect(result).to match("drivestoredirect:s:*")
+    expect(result).to match("full address:s:host:port")
+    expect(result).to match("prompt for credentials:i:1")
+    expect(result).to match("username:s:username")
+  end
+
+  it "includes extra RDP arguments" do
+    rdp_info.merge!(extra_args: ["screen mode id:i:0"])
+    path = described_class.generate_config_file(rdp_info)
+    result = File.read(path)
+    expect(result).to match("screen mode id:i:0")
+  end
+
+  it "opens the RDP file" do
+    env = double(:env)
+    allow(described_class).to receive(:generate_config_file).and_return("/path")
+    expect(Vagrant::Util::Subprocess).to receive(:execute).with("open", "/path")
+    described_class.rdp_client(env, rdp_info)
+  end
+end

data/test/unit/plugins/kernel_v2/config/vm_test.rb

@@ -173,7 +173,7 @@ describe VagrantPlugins::Kernel_V2::VMConfig do
       subject.communicator = "winrm"
       subject.finalize!
       n = subject.networks
-      expect(n.length).to eq(2)
+      expect(n.length).to eq(3)
 
       expect(n[0][0]).to eq(:forwarded_port)
       expect(n[0][1][:guest]).to eq(5985)
@@ -188,6 +188,32 @@ describe VagrantPlugins::Kernel_V2::VMConfig do
       expect(n[1][1][:id]).to eq("winrm-ssl")
     end
 
+    it "forwards ssh even if the communicator is winrm" do
+      subject.communicator = "winrm"
+      subject.finalize!
+      n = subject.networks
+      expect(n.length).to eq(3)
+
+      expect(n[0][0]).to eq(:forwarded_port)
+      expect(n[0][1][:guest]).to eq(5985)
+      expect(n[0][1][:host]).to eq(55985)
+      expect(n[0][1][:host_ip]).to eq("127.0.0.1")
+      expect(n[0][1][:id]).to eq("winrm")
+
+      expect(n[1][0]).to eq(:forwarded_port)
+      expect(n[1][1][:guest]).to eq(5986)
+      expect(n[1][1][:host]).to eq(55986)
+      expect(n[1][1][:host_ip]).to eq("127.0.0.1")
+      expect(n[1][1][:id]).to eq("winrm-ssl")
+
+      expect(n[2][0]).to eq(:forwarded_port)
+      expect(n[2][1][:guest]).to eq(22)
+      expect(n[2][1][:host]).to eq(2222)
+      expect(n[2][1][:host_ip]).to eq("127.0.0.1")
+      expect(n[2][1][:id]).to eq("ssh")
+
+    end
+
     it "allows overriding SSH" do
       subject.network "forwarded_port",
         guest: 22, host: 14100, id: "ssh"

data/test/unit/plugins/providers/docker/action/create_test.rb

@@ -0,0 +1,55 @@
+require_relative "../../../../base"
+require_relative "../../../../../../plugins/providers/docker/action/create"
+
+describe VagrantPlugins::DockerProvider::Action::Create do
+  include_context "unit"
+  include_context "virtualbox"
+
+  let(:sandbox) { isolated_environment }
+
+  let(:iso_env) do
+    # We have to create a Vagrantfile so there is a root path
+    sandbox.vagrantfile("")
+    sandbox.create_vagrant_env
+  end
+
+  let(:machine) do
+    iso_env.machine(iso_env.machine_names[0], :virtualbox).tap do |m|
+      m.provider.stub(driver: driver)
+    end
+  end
+
+  let(:env)    {{ machine: machine, ui: machine.ui, root_path: Pathname.new(".") }}
+  let(:app)    { lambda { |*args| }}
+  let(:driver) { double("driver", create: "abcd1234") }
+
+  subject { described_class.new(app, env) }
+
+  after do
+    sandbox.close
+  end
+
+  describe "#call" do
+    it "calls the next action in the chain" do
+      called = false
+      app = ->(*args) { called = true }
+
+      action = described_class.new(app, env)
+      action.call(env)
+
+      expect(called).to eq(true)
+    end
+  end
+
+  describe "#forwarded_ports" do
+    it "does not clobber ports with different protocols" do
+      subject.instance_variable_set(:@machine, machine)
+      machine.config.vm.network "forwarded_port", guest: 8125, host: 8125, protocol: "tcp"
+      machine.config.vm.network "forwarded_port", guest: 8125, host: 8125, protocol: "udp"
+
+      result = subject.forwarded_ports(false)
+
+      expect(result).to eq(["8125:8125", "8125:8125/udp"])
+    end
+  end
+end

data/test/unit/plugins/providers/docker/command/exec_test.rb

@@ -0,0 +1,44 @@
+require_relative "../../../../base"
+require_relative "../../../../../../plugins/providers/docker/command/exec"
+
+describe VagrantPlugins::DockerProvider::Command::Exec do
+  include_context "unit"
+  include_context "command plugin helpers"
+
+  let(:sandbox) do
+    isolated_environment
+  end
+
+  let(:argv) { [] }
+  let(:env) { sandbox.create_vagrant_env }
+
+  let(:vagrantfile_path) { File.join(env.cwd, "Vagrantfile") }
+
+  subject { described_class.new(argv, env) }
+
+  before(:all) do
+    I18n.load_path << Vagrant.source_root.join("templates/locales/providers_docker.yml")
+    I18n.reload!
+  end
+
+  before do
+    Vagrant.plugin("2").manager.stub(commands: {})
+    allow(subject).to receive(:exec_command)
+  end
+
+  after do
+    sandbox.close
+  end
+
+  describe "#execute" do
+    describe "without a command" do
+      let(:argv) { [] }
+
+      it "raises an error" do
+        expect {
+          subject.execute
+        }.to raise_error(VagrantPlugins::DockerProvider::Errors::ExecCommandRequired)
+      end
+    end
+  end
+end