]> gitweb.fluxo.info Git - puppet-stdlib.git/commitdiff
(FM-1587) Fix test issues on solaris 10
authorHunter Haugen <hunter@puppetlabs.com>
Thu, 19 Jun 2014 22:38:23 +0000 (15:38 -0700)
committerHunter Haugen <hunter@puppetlabs.com>
Thu, 19 Jun 2014 22:38:23 +0000 (15:38 -0700)
- ensure_packages fails because Error: Sun packages must specify a package source
- ensure_resource fails for the same reason
- get_module_path fails because the modulepath is different
- has_interface_with fails because the interface is lo0 not lo

spec/acceptance/ensure_packages_spec.rb
spec/acceptance/ensure_resource_spec.rb
spec/acceptance/get_module_path_spec.rb
spec/acceptance/has_interface_with_spec.rb

index aa7b14c9f7e59bb9e1d24f88be2232353e4487ee..f3d2256753790556b7fb0b1cb70a76d600fbd60e 100755 (executable)
@@ -6,8 +6,8 @@ describe 'ensure_packages function', :unless => UNSUPPORTED_PLATFORMS.include?(f
     it 'ensure_packages a package' do
       apply_manifest('package { "zsh": ensure => absent, }')
       pp = <<-EOS
-      $a = "zsh"
-      ensure_packages($a)
+      $a = "rake"
+      ensure_packages($a,{'provider' => 'gem'})
       EOS
 
       apply_manifest(pp, :expect_changes => true) do |r|
index c4d8887df290894e10a46ca284d11799bdcdc703..725f5df590a16151ab1672acbfd9fd65faeeb3d7 100755 (executable)
@@ -6,8 +6,8 @@ describe 'ensure_resource function', :unless => UNSUPPORTED_PLATFORMS.include?(f
     it 'ensure_resource a package' do
       apply_manifest('package { "zsh": ensure => absent, }')
       pp = <<-EOS
-      $a = "zsh"
-      ensure_resource('package', $a)
+      $a = "rake"
+      ensure_resource('package', $a, {'provider' => 'gem'})
       EOS
 
       apply_manifest(pp, :expect_changes => true) do |r|
index 34d91fa3dab5ba5279fcdec3edd43ef37b97bbb2..6ac690c16b5f463a7f4f6fbeba62603c0d0b97c5 100755 (executable)
@@ -3,22 +3,6 @@ require 'spec_helper_acceptance'
 
 describe 'get_module_path function', :unless => UNSUPPORTED_PLATFORMS.include?(fact('operatingsystem')) do
   describe 'success' do
-    it 'get_module_paths stdlib' do
-      pp = <<-EOS
-      $a = $::is_pe ? {
-        'true'  => '/opt/puppet/share/puppet/modules/stdlib',
-        'false' => '/etc/puppet/modules/stdlib',
-      }
-      $o = get_module_path('stdlib')
-      if $o == $a {
-        notify { 'output correct': }
-      }
-      EOS
-
-      apply_manifest(pp, :catch_failures => true) do |r|
-        expect(r.stdout).to match(/Notice: output correct/)
-      end
-    end
     it 'get_module_paths dne' do
       pp = <<-EOS
       $a = $::is_pe ? {
@@ -28,6 +12,8 @@ describe 'get_module_path function', :unless => UNSUPPORTED_PLATFORMS.include?(f
       $o = get_module_path('dne')
       if $o == $a {
         notify { 'output correct': }
+      } else {
+        notify { "failed; module path is '$o'": }
       }
       EOS
 
index 41ae19fd1c5c37d8e548281227b57b685768da49..7b38c95eb883aaae5e3be5c44e633957a59e26ab 100755 (executable)
@@ -27,7 +27,11 @@ describe 'has_interface_with function', :unless => UNSUPPORTED_PLATFORMS.include
     end
     it 'has_interface_with existing interface' do
       pp = <<-EOS
-      $a = 'lo'
+      if $osfamily == 'Solaris' {
+        $a = 'lo0'
+      } else {
+        $a = 'lo'
+      }
       $o = has_interface_with($a)
       notice(inline_template('has_interface_with is <%= @o.inspect %>'))
       EOS