kitchen-nodes-lobatoa 0.10.0

data/spec/unit/nodes_spec.rb

@@ -0,0 +1,285 @@
+require 'erb'
+require 'fakefs/safe'
+require 'kitchen'
+require 'kitchen/driver/dummy'
+require 'kitchen/provisioner/nodes'
+require 'kitchen/transport/dummy'
+require 'kitchen/transport/winrm'
+require 'kitchen/transport/ssh'
+require 'winrm'
+
+# rubocop:disable Metrics/BlockLength
+describe Kitchen::Provisioner::Nodes do
+  let(:config) do
+    {
+      test_base_path: '/b',
+      kitchen_root: '/r',
+      run_list: ['recipe[cookbook::default]'],
+      attributes: { att_key: 'att_val' },
+      client_rb: { environment: 'my_env' },
+      reset_node_files: false
+    }
+  end
+  let(:instance) do
+    double(
+      'instance',
+      name: 'test_suite',
+      suite: suite,
+      platform: platform,
+      transport: transport,
+      driver: Kitchen::Driver::Dummy.new
+    )
+  end
+  let(:transport) { Kitchen::Transport::Ssh.new }
+  let(:platform) { double('platform', os_type: nil, name: 'ubuntu') }
+  let(:suite) { double('suite', name: 'suite') }
+  let(:state) { { hostname: '192.168.1.10' } }
+  let(:node) { JSON.parse(File.read(subject.node_file), symbolize_names: true) }
+
+  before do
+    FakeFS.activate!
+    FileUtils.mkdir_p(config[:test_base_path])
+    allow_any_instance_of(Kitchen::StateFile)
+      .to receive(:read).and_return(state)
+    allow(transport).to receive(:connection)
+      .and_return(Kitchen::Transport::Base::Connection.new)
+    allow_any_instance_of(Kitchen::Transport::Base::Connection)
+      .to receive(:node_execute).with('hostname -f')
+      .and_return("fakehostname\n\n")
+  end
+  after do
+    FakeFS.deactivate!
+    FakeFS::FileSystem.clear
+  end
+
+  subject { Kitchen::Provisioner::Nodes.new(config).finalize_config!(instance) }
+
+  describe '#create_node' do
+    context 'node file does not exist' do
+      before do
+        allow(Dir).to receive(:exist?).and_return(false)
+      end
+
+      it 'creates node' do
+        subject.create_node
+
+        expect(File).to exist(subject.node_file)
+      end
+    end
+    context 'node file exists' do
+      before do
+        allow(Dir).to receive(:exist?).and_return(true)
+        expect(File).to receive(:exist?).with(subject.node_file).and_return(true)
+      end
+
+      context 'config[:reset_node_files] = false' do
+        it 'does not update the node file' do
+          expect(File).not_to receive(:open)
+
+          subject.create_node
+        end
+      end
+
+      context 'config[:reset_node_files] = true' do
+        before do
+          config[:reset_node_files] = true
+        end
+
+        it 'updates the node file' do
+          expect(File).to receive(:open)
+
+          subject.create_node
+        end
+      end
+    end
+  end
+
+  it 'sets the id' do
+    subject.create_node
+
+    expect(node[:id]).to eq instance.name
+  end
+
+  it 'sets the environment' do
+    subject.create_node
+
+    expect(node[:chef_environment]).to eq config[:client_rb][:environment]
+  end
+
+  it 'sets the runlist' do
+    subject.create_node
+
+    expect(node[:run_list]).to eq config[:run_list]
+  end
+
+  it 'expands the runlist' do
+    subject.create_node
+
+    expect(node[:automatic][:recipes]).to eq ['cookbook::default']
+  end
+
+  it 'sets the named_runlist' do
+    subject.create_node
+
+    expect(node[:named_run_list]).to eq config[:named_run_list]
+  end
+
+  it 'sets the normal attributes' do
+    subject.create_node
+
+    expect(node[:normal]).to eq config[:attributes]
+  end
+
+  it 'sets the ip address' do
+    subject.create_node
+
+    expect(node[:automatic][:ipaddress]).to eq state[:hostname]
+  end
+
+  it 'sets the fqdn' do
+    subject.create_node
+
+    expect(node[:automatic][:fqdn]).to eq 'fakehostname'
+  end
+
+  context 'cannot obtain fqdn' do
+    before do
+      allow_any_instance_of(Kitchen::Transport::Base::Connection)
+        .to receive(:node_execute).with('hostname -f')
+        .and_raise(Kitchen::Transport::TransportFailed.new(''))
+    end
+
+    it 'sets the fqdn to nil' do
+      subject.create_node
+      expect(node[:automatic][:fqdn]).to be_nil
+    end
+  end
+
+  context 'no environment explicitly set' do
+    before { config.delete(:client_rb) }
+
+    it 'sets the environment' do
+      subject.create_node
+
+      expect(node[:chef_environment]).to eq '_default'
+    end
+  end
+
+  context 'instance is localhost' do
+    let(:state) { { hostname: '127.0.0.1' } }
+    let(:machine_ips) { ['192.168.1.1', '192.168.1.2', '192.168.1.3'] }
+
+    before do
+      allow_any_instance_of(Net::Ping::External).to receive(:ping)
+        .and_return(true)
+    end
+
+    context 'cannot find an ip' do
+      let(:ifconfig_response) do
+        FakeFS.deactivate!
+        template = File.read('spec/unit/stubs/ifconfig.txt')
+        FakeFS.activate!
+        template.gsub!('', machine_ips[0])
+        template.gsub!('', machine_ips[1])
+        template.gsub!('', machine_ips[2])
+      end
+      let(:transport) { Kitchen::Transport::Ssh.new }
+
+      before do
+        allow_any_instance_of(Kitchen::Transport::Base::Connection)
+          .to receive(:node_execute).and_return(ifconfig_response)
+      end
+
+      it 'fails' do
+        expect { subject.create_node }.to raise_error('Unable to retrieve IPs')
+      end
+    end
+
+    context 'platform is windows' do
+      let(:transport) { Kitchen::Transport::Winrm.new }
+
+      before do
+        data = WinRM::Output.new
+        data << { stdout: "\r\n" }
+        machine_ips.map { |ip| data << { stdout: "IPv4 Address .: #{ip}\r\n" } }
+        allow_any_instance_of(Kitchen::Transport::Base::Connection).to(
+          receive(:node_execute).and_return(data)
+        )
+        allow(platform).to receive(:name).and_return('windows')
+      end
+
+      it 'sets the ip address to the first reachable IP' do
+        subject.create_node
+
+        expect(node[:automatic][:ipaddress]).to eq machine_ips.first
+      end
+
+      context 'only the last ip is reachable' do
+        before do
+          allow_any_instance_of(Net::Ping::TCP).to receive(:ping)
+            .and_return(false)
+          allow_any_instance_of(Net::Ping::External).to receive(:ping)
+            .and_return(false)
+          allow_any_instance_of(Net::Ping::External).to receive(:ping)
+            .with(machine_ips.last).and_return(true)
+        end
+
+        it 'sets the ip address to the last IP' do
+          subject.create_node
+
+          expect(node[:automatic][:ipaddress]).to eq machine_ips.last
+        end
+      end
+    end
+
+    context 'platform is *nix' do
+      let(:ifconfig_response) do
+        FakeFS.deactivate!
+        template = File.read('spec/unit/stubs/ifconfig.txt')
+        FakeFS.activate!
+        template.gsub!('1.1.1.1', machine_ips[0])
+        template.gsub!('2.2.2.2', machine_ips[1])
+        template.gsub!('3.3.3.3', machine_ips[2])
+      end
+      let(:transport) { Kitchen::Transport::Ssh.new }
+
+      before do
+        allow_any_instance_of(Kitchen::Transport::Base::Connection)
+          .to receive(:node_execute).and_return(ifconfig_response)
+      end
+
+      it 'sets the ip address to the RUNNING IP that is not localhost' do
+        subject.create_node
+
+        expect(node[:automatic][:ipaddress]).to eq machine_ips[1]
+      end
+
+      context 'ifconfig not supported' do
+        let(:ip_response) do
+          FakeFS.deactivate!
+          template = File.read('spec/unit/stubs/ip.txt')
+          FakeFS.activate!
+          template.gsub!('1.1.1.1', machine_ips[0])
+          template.gsub!('2.2.2.2', machine_ips[1])
+          template.gsub!('3.3.3.3', machine_ips[2])
+        end
+
+        before do
+          allow_any_instance_of(Kitchen::Transport::Base::Connection)
+            .to receive(:node_execute).with('/sbin/ifconfig -a')
+            .and_raise(Kitchen::Transport::TransportFailed.new(''))
+
+          allow_any_instance_of(Kitchen::Transport::Base::Connection)
+            .to receive(:node_execute).with('/sbin/ip -4 addr show')
+            .and_return(ip_response)
+        end
+
+        it 'sets the ip address to the connected IP that is not localhost' do
+          subject.create_node
+
+          expect(node[:automatic][:ipaddress]).to eq machine_ips[0]
+        end
+      end
+    end
+  end
+end

data/spec/unit/stubs/ifconfig.txt

@@ -0,0 +1,36 @@
+bash: warning: setlocale: LC_ALL: cannot change locale (en_US)
+docker0   Link encap:Ethernet  HWaddr 56:84:7a:fe:97:99
+          inet addr:1.1.1.1  Bcast:0.0.0.0  Mask:255.255.0.0
+          UP BROADCAST MULTICAST  MTU:1500  Metric:1
+          RX packets:0 errors:0 dropped:0 overruns:0 frame:0
+          TX packets:0 errors:0 dropped:0 overruns:0 carrier:0
+          collisions:0 txqueuelen:0
+          RX bytes:0 (0.0 B)  TX bytes:0 (0.0 B)
+
+eth0      Link encap:Ethernet  HWaddr 08:00:27:88:0c:a6
+          inet addr:2.2.2.2  Bcast:10.0.2.255  Mask:255.255.255.0
+          inet6 addr: fe80::a00:27ff:fe88:ca6/64 Scope:Link
+          UP BROADCAST RUNNING MULTICAST  MTU:1500  Metric:1
+          RX packets:10262 errors:0 dropped:0 overruns:0 frame:0
+          TX packets:7470 errors:0 dropped:0 overruns:0 carrier:0
+          collisions:0 txqueuelen:1000
+          RX bytes:1497781 (1.4 MB)  TX bytes:1701791 (1.7 MB)
+
+lo        Link encap:Local Loopback
+          inet addr:127.0.0.1  Mask:255.0.0.0
+          inet6 addr: ::1/128 Scope:Host
+          UP LOOPBACK RUNNING  MTU:65536  Metric:1
+          RX packets:0 errors:0 dropped:0 overruns:0 frame:0
+          TX packets:0 errors:0 dropped:0 overruns:0 carrier:0
+          collisions:0 txqueuelen:0
+          RX bytes:0 (0.0 B)  TX bytes:0 (0.0 B)
+
+# The following represents the format of ifconfig from CentOS 7.1
+enp0s8: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1500
+        inet 3.3.3.3  netmask 255.255.255.0  broadcast 3.3.3.0
+        inet6 fe80::a00:27ff:fe5e:e9b0  prefixlen 64  scopeid 0x20<link>
+        ether 08:00:27:5e:e9:b0  txqueuelen 1000  (Ethernet)
+        RX packets 7961  bytes 823710 (804.4 KiB)
+        RX errors 0  dropped 0  overruns 0  frame 0
+        TX packets 263  bytes 50868 (49.6 KiB)
+        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

data/spec/unit/stubs/ip.txt

@@ -0,0 +1,15 @@
+bash: warning: setlocale: LC_ALL: cannot change locale (en_US)
+1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default
+    inet 127.0.0.1/8 scope host lo
+       valid_lft forever preferred_lft forever
+3: wlan0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc mq state UP group default qlen 1000
+    inet 1.1.1.1/24 brd 192.168.1.255 scope global wlan0
+       valid_lft forever preferred_lft forever
+5: docker0: <NO-CARRIER,BROADCAST,MULTICAST,UP> mtu 1500 qdisc noqueue state DOWN group default
+    inet 2.2.2.2/16 scope global docker0
+       valid_lft forever preferred_lft forever
+# The following represents the format of ipaddr from CentOS 7.1 (which is the same in this case, but
+# adding for completeness)
+6: enp0s8: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc pfifo_fast state UP qlen 1000
+    inet 3.3.3.3/24 brd 3.3.3.255 scope global dynamic enp0s8
+       valid_lft 966sec preferred_lft 966sec

data/test/fixtures/roles/test_json_role.json

@@ -0,0 +1,10 @@
+{
+  "name": "test_json_role",
+  "chef_type": "role",
+  "json_class": "Chef::Role",
+  "default_attributes": {
+  },
+  "run_list": [
+    "recipe[hurry-up-and-test::set_non_nat_vbox_ip]"
+  ]
+}

data/test/fixtures/roles/test_ruby_role.rb

@@ -0,0 +1,2 @@
+name('test_ruby_role')
+run_list 'recipe[hurry-up-and-test::set_non_nat_vbox_ip]'

data/test/integration/cookbooks/node-tests/libraries/helper.rb

@@ -0,0 +1,18 @@
+require 'timeout'
+
+def search_for_nodes(query, timeout = 120)
+  nodes = []
+  Timeout.timeout(timeout) do
+    nodes = search(:node, query)
+    until nodes.count > 0 && nodes[0].key?('ipaddress')
+      sleep 5
+      nodes = search(:node, query)
+    end
+  end
+
+  if nodes.count.zero? || !nodes[0].key?('ipaddress')
+    raise "Unable to find any nodes meeting the search criteria '#{query}'!"
+  end
+
+  nodes
+end

data/test/integration/cookbooks/node-tests/metadata.rb

@@ -0,0 +1,4 @@
+name 'node-tests'
+version '0.1.0'
+maintainer ''
+maintainer_email ''

data/test/integration/cookbooks/node-tests/recipes/node2.rb

@@ -0,0 +1,10 @@
+first_node = search_for_nodes(
+  "run_list:*node1* AND platform:#{node['platform']}"
+)
+
+ruby_block 'Save first attributes' do
+  block do
+    parent = File.join(ENV['TEMP'] || '/tmp', 'kitchen')
+    IO.write(File.join(parent, 'other_node.json'), first_node[0].to_json)
+  end
+end

data/test/integration/node2/serverspec/Gemfile

@@ -0,0 +1,3 @@
+source 'https://rubygems.org'
+
+gem 'winrm' if RUBY_PLATFORM =~ /mingw/

data/test/integration/node2/serverspec/default_spec.rb

@@ -0,0 +1,69 @@
+require 'serverspec'
+require 'json'
+
+if RUBY_PLATFORM =~ /mingw/
+  set :backend, :cmd
+  set :os, family: 'windows'
+else
+  set :backend, :exec
+end
+
+describe 'other node' do
+  let(:node) do
+    JSON.parse(
+      IO.read(File.join(ENV['TEMP'] || '/tmp', 'kitchen/other_node.json'))
+    )
+  end
+  let(:ip) { node['automatic']['ipaddress'] }
+  let(:fqdn) { node['automatic']['fqdn'] }
+  let(:connection) do
+    if RUBY_PLATFORM =~ /mingw/
+      require 'winrm'
+      ::WinRM::Connection.new(
+        endpoint: "http://#{ip}:5985/wsman",
+        user: 'vagrant',
+        password: 'vagrant'
+      ).shell(:cmd)
+    else
+      Net::SSH.start(
+        ip,
+        'vagrant',
+        password: 'vagrant',
+        paranoid: false
+      )
+    end
+  end
+
+  it 'has an non localhost ip' do
+    expect(ip).not_to eq('127.0.0.1')
+  end
+
+  it 'has a valid ip' do
+    expect(ip).to match(/\d{1,3}\.\d{1,3}\.\d{1,3}\.\d{1,3}/)
+  end
+
+  # Skip this test on the 2008 box bc its not sysprepped....
+  unless ENV['computername'] =~ /VAGRANT\-2008R2/i
+    describe command('hostname') do
+      its(:stdout) { should_not match(/#{Regexp.quote(fqdn)}/) }
+    end
+  end
+
+  if RUBY_PLATFORM =~ /mingw/
+    it 'has a computername matching node fqdn' do
+      expect(connection.run('hostname').stdout.chomp).to eq(fqdn)
+    end
+  else
+    it 'has a computername matching node fqdn' do
+      connection.open_channel do |channel|
+        channel.request_pty
+        channel.exec('hostname') do |_ch, _success|
+          channel.on_data do |_ch, data|
+            expect(data.chomp).to eq(fqdn)
+          end
+        end
+      end
+      connection.loop
+    end
+  end
+end