Update tests to pass travs

Signed-off-by: Alex Pop <[email protected]>

spec/unit/recipes/default_spec.rb

@@ -22,7 +22,7 @@
 describe 'audit::default' do
   context 'When all attributes are default, on an unspecified platform' do
     let(:chef_run) do
-      runner = ChefSpec::ServerRunner.new(platform: 'ubuntu', version: '14.04')
+      runner = ChefSpec::ServerRunner.new(platform: 'ubuntu', version: '18.04')
       runner.converge(described_recipe)
     end
 

test/cookbooks/test_helper/recipes/force_inspec_core.rb

@@ -1,16 +1,20 @@
 chef_gem 'inspec-core' do
-  action :nothing
-end.run_action(:remove)
+  compile_time true
+  action :remove
+end
 
 chef_gem 'inspec-core-bin' do
-  action :nothing
-end.run_action(:remove)
+  compile_time true
+  action :remove
+end
 
 chef_gem 'inspec' do
-  action :nothing
-end.run_action(:remove)
+  compile_time true
+  action :remove
+end
 
 chef_gem 'inspec-core' do
+  compile_time true
   version '4.3.2'
-  action :nothing
-end.run_action(:install)
+  action :install
+end

test/cookbooks/test_helper/recipes/install_inspec.rb

@@ -1,16 +1,20 @@
 chef_gem 'inspec-core' do
-  action :nothing
-end.run_action(:remove)
+  compile_time true
+  action :remove
+end
 
 chef_gem 'inspec-core-bin' do
-  action :nothing
-end.run_action(:remove)
+  compile_time true
+  action :remove
+end
 
 chef_gem 'inspec' do
-  action :nothing
-end.run_action(:remove)
+  compile_time true
+  action :remove
+end
 
 chef_gem 'inspec' do
+  compile_time true
   version '1.19.1'
-  action :nothing
-end.run_action(:install)
+  action :install
+end

test/cookbooks/test_helper/recipes/setup_waiver_fixture.rb

@@ -1,19 +1,12 @@
 
-[
-  "controls",
-  "waivers",
-].each do |subdir|
-  directory "#{node["audit"]["profiles"]["waiver-test-profile"]["path"]}/#{subdir}" do
+%w(controls waivers).each do |subdir|
+  directory "#{node['audit']['profiles']['waiver-test-profile']['path']}/#{subdir}" do
     recursive true
   end
 end
 
-[
-  "inspec.yml",
-  "controls/waiver-check.rb",
-  "waivers/waivers.yaml",
-].each do |file|
-  cookbook_file "#{node["audit"]["profiles"]["waiver-test-profile"]["path"]}/#{file}" do
+%w(inspec.yml controls/waiver-check.rb waivers/waivers.yaml).each do |file|
+  cookbook_file "#{node['audit']['profiles']['waiver-test-profile']['path']}/#{file}" do
     source "waivers-fixture/#{file}"
   end
 end

test/integration/inspec-attributes/default.rb

@@ -9,7 +9,7 @@
     results << r
   end
 end
-attribute_control = results.find { |x| x['code_desc'] == 'File /opt/kitchen/cache/attribute-file-exists.test should exist' }
+attribute_control = results.find { |x| x['code_desc'] == 'File /opt/kitchen/cache/attribute-file-exists.test is expected to exist' }
 attribute_control = {} if attribute_control.nil?
 
 describe 'attribute control' do

test/integration/waivers/default.rb

@@ -5,14 +5,14 @@
 # The test fixture has two controls - the first should pass,
 # the second should be a skip with a waiver justification
 
-control "the unwaivered control" do
+control 'the unwaivered control' do
   describe controls[0]['results'][0]['status'] do
-    it { should cmp "passed" }
+    it { should cmp 'passed' }
   end
 end
 
-control "the waivered control" do
+control 'the waivered control' do
   describe controls[1]['results'][0]['status'] do
-    it { should cmp "skipped" }
+    it { should cmp 'skipped' }
   end
 end

test/kitchen-automate/recipes/bootstrap_localhost.rb

@@ -24,8 +24,8 @@
 end
 
 # Ensure the node is in the expected state
-raise 'Cannot find /tmp/kitchen/client.pem' unless File.exist?('/tmp/kitchen/client.pem')
-raise 'Cannot find /home/ec2-user/.ssh/authorized_keys' unless File.exist?('/home/ec2-user/.ssh/authorized_keys')
+raise 'Cannot find /tmp/kitchen/client.pem' unless ::File.exist?('/tmp/kitchen/client.pem')
+raise 'Cannot find /home/ec2-user/.ssh/authorized_keys' unless ::File.exist?('/home/ec2-user/.ssh/authorized_keys')
 
 # Add a public key to be used for SSH bootstrapping
 execute 'Add SSH public key to be used by self bootstrapping' do
@@ -49,5 +49,5 @@
   EOH
   live_stream true
   action :run
-  not_if { File.exist?('/etc/chef/client.pem') }
+  not_if { ::File.exist?('/etc/chef/client.pem') }
 end

test/kitchen-automate/recipes/configure_services.rb

@@ -33,5 +33,5 @@
     touch /etc/opscode/chef-server-reconfigured.txt
   EOH
   action :run
-  not_if { File.exist?('/etc/opscode/chef-server-reconfigured.txt') }
+  not_if { ::File.exist?('/etc/opscode/chef-server-reconfigured.txt') }
 end

test/kitchen-automate/recipes/converge_localhost.rb

@@ -42,7 +42,7 @@
   EOH
   live_stream true
   action :run
-  only_if { File.exist?('/etc/chef/client.pem') }
+  only_if { ::File.exist?('/etc/chef/client.pem') }
 end
 
 # Create the attributes file to test the 'chef-visibility' collector
@@ -91,5 +91,5 @@
   EOH
   live_stream true
   action :run
-  only_if { File.exist?('/etc/chef/client.pem') }
+  only_if { ::File.exist?('/etc/chef/client.pem') }
 end