Update specs and fix FM-1361
[puppet_vcsrepo.git] / spec / acceptance / beaker / git / user_checkout / user_checkout_http.rb
index 6faad95..03ea27b 100644 (file)
@@ -5,6 +5,7 @@ repo_name = 'testrepo_user_checkout'
 user = 'myuser'
 
 hosts.each do |host|
+  ruby = (host.is_pe? && '/opt/puppet/bin/ruby') || 'ruby'
   tmpdir = host.tmpdir('vcsrepo')
   step 'setup - create repo' do
     install_package(host, 'git')
@@ -21,17 +22,17 @@ hosts.each do |host|
     server.start
     EOF
     create_remote_file(host, '/tmp/http_daemon.rb', http_daemon)
-    on(host, "ruby /tmp/http_daemon.rb")
+    on(host, "#{ruby} /tmp/http_daemon.rb")
   end
 
   step 'setup - create user' do
-    apply_manifest_on(host, "user { '#{user}': ensure => present, }")
+    apply_manifest_on(host, "user { '#{user}': ensure => present, }", :catch_failures => true)
   end
 
   teardown do
     on(host, "rm -fr #{tmpdir}")
-    on(host, 'ps ax | grep "ruby /tmp/http_daemon.rb" | grep -v grep | awk \'{print "kill -9 " $1}\' | sh')
-    apply_manifest_on(host, "user { '#{user}': ensure => absent, }")
+    on(host, "ps ax | grep '#{ruby} /tmp/http_daemon.rb' | grep -v grep | awk '{print \"kill -9 \" $1}' | sh ; sleep 1")
+    apply_manifest_on(host, "user { '#{user}': ensure => absent, }", :catch_failures => true)
   end
 
   step 'checkout a user with puppet' do
@@ -44,8 +45,8 @@ hosts.each do |host|
     }
     EOS
 
-    apply_manifest_on(host, pp)
-    apply_manifest_on(host, pp)
+    apply_manifest_on(host, pp, :catch_failures => true)
+    apply_manifest_on(host, pp, :catch_changes  => true)
   end
 
   step "verify git checkout is owned by user #{user}" do