summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJohn Duarte <john.duarte@puppetlabs.com>2014-05-08 11:55:58 -0700
committerJohn Duarte <john.duarte@puppetlabs.com>2014-05-08 11:55:58 -0700
commitd0c2026b64b796482d2e410b336331dc493589d0 (patch)
tree138543526474ee6244136fa47bba6c7a5795056f
parent2a57379e669a6920766548927b08e9ef4ede521e (diff)
downloadpuppet-vcsrepo-d0c2026b64b796482d2e410b336331dc493589d0.tar.gz
puppet-vcsrepo-d0c2026b64b796482d2e410b336331dc493589d0.tar.bz2
Change apply syntax to apply_manifest_on
-rw-r--r--spec/acceptance/git_clone_protocols_spec.rb46
1 files changed, 23 insertions, 23 deletions
diff --git a/spec/acceptance/git_clone_protocols_spec.rb b/spec/acceptance/git_clone_protocols_spec.rb
index f59644e..6c9114d 100644
--- a/spec/acceptance/git_clone_protocols_spec.rb
+++ b/spec/acceptance/git_clone_protocols_spec.rb
@@ -7,8 +7,8 @@ hosts.each do |host|
before(:all) do
# {{{ setup
- on(host,apply_manifest("user{'testuser': ensure => present, managehome => true }"))
- on(host,apply_manifest("user{'vagrant': ensure => present, }"))
+ apply_manifest_on(host, "user{'testuser': ensure => present, managehome => true }")
+ apply_manifest_on(host, "user{'vagrant': ensure => present, }")
# install git
install_package(host, 'git')
install_package(host, 'git-daemon')
@@ -36,9 +36,9 @@ hosts.each do |host|
after(:all) do
# {{{ teardown
- on(host,apply_manifest("user{'testuser': ensure => absent, managehome => true }"))
- on(host,apply_manifest("file{'/root/.ssh/id_rsa': ensure => absent, force => true }"))
- on(host,apply_manifest("file{'/root/.ssh/id_rsa.pub': ensure => absent, force => true }"))
+ apply_manifest_on(host, "user{'testuser': ensure => absent, managehome => true }")
+ apply_manifest_on(host, "file{'/root/.ssh/id_rsa': ensure => absent, force => true }")
+ apply_manifest_on(host, "file{'/root/.ssh/id_rsa.pub': ensure => absent, force => true }")
# }}}
end
@@ -47,7 +47,7 @@ hosts.each do |host|
context 'using local protocol (file URL)' do
before(:all) do
- on(host,apply_manifest("file {'#{tmpdir}/testrepo': ensure => directory, purge => true, recurse => true, recurselimit => 1, force => true; }"))
+ apply_manifest_on(host, "file {'#{tmpdir}/testrepo': ensure => directory, purge => true, recurse => true, recurselimit => 1, force => true; }")
end
it 'should have HEAD pointing to master' do
@@ -60,8 +60,8 @@ hosts.each do |host|
EOS
# Run it twice and test for idempotency
- on(host,apply_manifest(pp, :catch_failures => true))
- on(host,apply_manifest(pp, :catch_changes => true))
+ apply_manifest_on(host, pp, :catch_failures => true)
+ apply_manifest_on(host, pp, :catch_changes => true)
end
describe file("#{tmpdir}/testrepo/.git/HEAD") do
@@ -72,7 +72,7 @@ hosts.each do |host|
context 'using local protocol (file path)' do
before(:all) do
- on(host,apply_manifest("file {'#{tmpdir}/testrepo': ensure => directory, purge => true, recurse => true, recurselimit => 1, force => true; }"))
+ apply_manifest_on(host, "file {'#{tmpdir}/testrepo': ensure => directory, purge => true, recurse => true, recurselimit => 1, force => true; }")
end
it 'should have HEAD pointing to master' do
@@ -85,8 +85,8 @@ hosts.each do |host|
EOS
# Run it twice and test for idempotency
- on(host,apply_manifest(pp, :catch_failures => true))
- on(host,apply_manifest(pp, :catch_changes => true))
+ apply_manifest_on(host, pp, :catch_failures => true)
+ apply_manifest_on(host, pp, :catch_changes => true)
end
describe file("#{tmpdir}/testrepo/.git/HEAD") do
@@ -97,7 +97,7 @@ hosts.each do |host|
context 'using git protocol' do
before(:all) do
- on(host,apply_manifest("file {'#{tmpdir}/testrepo': ensure => directory, purge => true, recurse => true, recurselimit => 1, force => true; }"))
+ apply_manifest_on(host, "file {'#{tmpdir}/testrepo': ensure => directory, purge => true, recurse => true, recurselimit => 1, force => true; }")
host.execute("nohup git daemon --detach --base-path=/#{tmpdir}")
end
@@ -111,8 +111,8 @@ hosts.each do |host|
EOS
# Run it twice and test for idempotency
- apply_manifest(pp, :catch_failures => true)
- apply_manifest(pp, :catch_changes => true)
+ apply_manifest_on(host, pp, :catch_failures => true)
+ apply_manifest_on(host, pp, :catch_changes => true)
end
describe file("#{tmpdir}/testrepo/.git/HEAD") do
it { should contain 'ref: refs/heads/master' }
@@ -125,7 +125,7 @@ hosts.each do |host|
context 'using http protocol' do
before(:all) do
- on(host,apply_manifest("file {'#{tmpdir}/testrepo': ensure => directory, purge => true, recurse => true, recurselimit => 1, force => true; }"))
+ apply_manifest_on(host, "file {'#{tmpdir}/testrepo': ensure => directory, purge => true, recurse => true, recurselimit => 1, force => true; }")
daemon =<<-EOF
require 'webrick'
server = WEBrick::HTTPServer.new(:Port => 8000, :DocumentRoot => "#{tmpdir}")
@@ -146,8 +146,8 @@ hosts.each do |host|
EOS
# Run it twice and test for idempotency
- apply_manifest(pp, :catch_failures => true)
- apply_manifest(pp, :catch_changes => true)
+ apply_manifest_on(host, pp, :catch_failures => true)
+ apply_manifest_on(host, pp, :catch_changes => true)
end
describe file("#{tmpdir}/testrepo/.git/HEAD") do
it { should contain 'ref: refs/heads/master' }
@@ -160,7 +160,7 @@ hosts.each do |host|
context 'using https protocol' do
before(:all) do
- on(host,apply_manifest("file {'#{tmpdir}/testrepo': ensure => directory, purge => true, recurse => true, recurselimit => 1, force => true; }"))
+ apply_manifest_on(host, "file {'#{tmpdir}/testrepo': ensure => directory, purge => true, recurse => true, recurselimit => 1, force => true; }")
daemon =<<-EOF
require 'webrick'
require 'webrick/https'
@@ -190,8 +190,8 @@ hosts.each do |host|
EOS
# Run it twice and test for idempotency
- apply_manifest(pp, :catch_failures => true)
- apply_manifest(pp, :catch_changes => true)
+ apply_manifest_on(host, pp, :catch_failures => true)
+ apply_manifest_on(host, pp, :catch_changes => true)
end
describe file("#{tmpdir}/testrepo/.git/HEAD") do
@@ -205,7 +205,7 @@ hosts.each do |host|
context 'using ssh protocol' do
before(:all) do
- on(host,apply_manifest("file {'#{tmpdir}/testrepo': ensure => directory, purge => true, recurse => true, recurselimit => 1, force => true; }"))
+ apply_manifest_on(host, "file {'#{tmpdir}/testrepo': ensure => directory, purge => true, recurse => true, recurselimit => 1, force => true; }")
end
it 'should have HEAD pointing to master' do
pp = <<-EOS
@@ -217,8 +217,8 @@ hosts.each do |host|
EOS
# Run it twice and test for idempotency
- apply_manifest(pp, :catch_failures => true)
- apply_manifest(pp, :catch_changes => true)
+ apply_manifest_on(host, pp, :catch_failures => true)
+ apply_manifest_on(host, pp, :catch_changes => true)
end
describe file("#{tmpdir}/testrepo/.git/HEAD") do