Merge pull request #468 from chef/chris-rock/integration

Move integration tests to test/integration

Berksfile

@@ -1,5 +1,6 @@
+# encoding: utf-8
 source 'https://supermarket.chef.io'
 
 cookbook 'apt'
-cookbook 'os_prepare', path: './cookbooks/os_prepare'
+cookbook 'os_prepare', path: './test/cookbooks/os_prepare'
 cookbook 'runit', github: 'hw-cookbooks/runit'

Gemfile

@@ -19,10 +19,9 @@ end
 
 group :integration do
   gem 'berkshelf', '~> 4.0'
-  gem 'test-kitchen'
+  gem 'test-kitchen', '~> 1.6'
   gem 'kitchen-vagrant'
-  gem 'kitchen-inspec'
-  gem 'winrm-transport', '~> 1.0'
+  gem 'kitchen-inspec', '0.12.5'
   gem 'kitchen-ec2'
 end
 

test/cookbooks/os_prepare/recipes/mount.rb

@@ -9,7 +9,7 @@ when 'ubuntu', 'rhel', 'centos', 'fedora'
 
   # copy iso file for mount tests
   # NB created using `mkdir empty; mkisofs -o empty.iso empty/`
-  cookbook_file '/root/empty.iso' do
+  cookbook_file '/tmp/empty.iso' do
     owner 'root'
     group 'root'
     mode '0755'
@@ -26,7 +26,7 @@ when 'ubuntu', 'rhel', 'centos', 'fedora'
 
   # mount -o loop /root/empty.iso /mnt/iso-disk
   mount '/mnt/iso-disk' do
-    device '/root/empty.iso'
+    device '/tmp/empty.iso'
     options 'loop'
     action [:mount, :enable]
   end

test/cookbooks/os_prepare/recipes/postgres.rb

@@ -6,6 +6,12 @@
 # hw-cookbooks/postgresql is tested on these platforms
 case node['platform']
 when 'ubuntu', 'centos'
+
+  # also skip it on ubuntu 15.10, because the cookbook is not supported
+  # with `enable_pgdg_apt` yet
+  return if node['platform_version'] == "15.10"
+
+  node.default['postgresql']['enable_pgdg_apt'] = true
   node.default['postgresql']['config']['listen_addresses'] = 'localhost'
   node.default['postgresql']['password']['postgres'] = 'md506be11be01439cb4abd537e454df34ea' # "inspec"
   include_recipe 'postgresql::server'

test/integration/default/file_spec.rb

@@ -125,6 +125,7 @@ if os.unix?
   end
 end
 
+# check file mount on linux
 if os.linux?
   # for server spec compatibility
   # Do not use `.with` or `.only_with`, this syntax is deprecated and will be removed
@@ -133,7 +134,7 @@ if os.linux?
     it { should be_mounted }
     it { should be_mounted.with( :type => 'iso9660' ) }
     it { should be_mounted.with( :type => 'iso9660', :options => { :ro => true } ) }
-    it { should be_mounted.with( :type => 'iso9660', :device => '/root/empty.iso' ) }
+    it { should be_mounted.with( :type => 'iso9660', :device => '/tmp/empty.iso' ) }
     it { should_not be_mounted.with( :type => 'ext4' ) }
     it { should_not be_mounted.with( :type => 'xfs' ) }
   end
@@ -142,7 +143,7 @@ if os.linux?
   # also see mount_spec.rb
   describe file('/mnt/iso-disk') do
     it { should be_mounted.only_with( {
-      :device=>"/root/empty.iso",
+      :device=>"/tmp/empty.iso",
       :type=>"iso9660",
       :options=>{
         :ro=>true}

test/integration/default/mount_spec.rb

@@ -5,7 +5,7 @@ if os.linux?
   describe mount '/mnt/iso-disk' do
     it { should be_mounted }
     its('count') { should eq  1 }
-    its('device') { should eq '/root/empty.iso' }
+    its('device') { should eq '/tmp/empty.iso' }
     its('type') { should eq  'iso9660' }
     its('options') { should eq  ['ro'] }
   end