aboutsummaryrefslogtreecommitdiff
path: root/spec/acceptance/tests/resource/cron/should_update_existing.rb
diff options
context:
space:
mode:
Diffstat (limited to 'spec/acceptance/tests/resource/cron/should_update_existing.rb')
-rw-r--r--spec/acceptance/tests/resource/cron/should_update_existing.rb30
1 files changed, 15 insertions, 15 deletions
diff --git a/spec/acceptance/tests/resource/cron/should_update_existing.rb b/spec/acceptance/tests/resource/cron/should_update_existing.rb
index a4bf177..eff634b 100644
--- a/spec/acceptance/tests/resource/cron/should_update_existing.rb
+++ b/spec/acceptance/tests/resource/cron/should_update_existing.rb
@@ -1,7 +1,7 @@
-test_name "puppet should update existing crontab entry"
-confine :except, :platform => 'windows'
-confine :except, :platform => /^eos-/ # See PUP-5500
-confine :except, :platform => /^fedora-28/
+test_name 'puppet should update existing crontab entry'
+confine :except, platform: 'windows'
+confine :except, platform: %r{^eos-} # See PUP-5500
+confine :except, platform: %r{^fedora-28}
tag 'audit:medium',
'audit:refactor', # Use block style `test_name`
'audit:acceptance' # Could be done at the integration (or unit) layer though
@@ -12,31 +12,31 @@ require 'puppet/acceptance/common_utils'
extend Puppet::Acceptance::CronUtils
teardown do
- step "Cron: cleanup"
+ step 'Cron: cleanup'
agents.each do |agent|
clean agent
end
end
agents.each do |host|
- step "ensure the user exist via puppet"
+ step 'ensure the user exist via puppet'
setup host
- step "create the existing job by hand..."
- run_cron_on(host,:add,'tstuser',"* * * * * /bin/true")
+ step 'create the existing job by hand...'
+ run_cron_on(host, :add, 'tstuser', '* * * * * /bin/true')
- step "verify that crontab -l contains what you expected"
- run_cron_on(host,:list,'tstuser') do
+ step 'verify that crontab -l contains what you expected'
+ run_cron_on(host, :list, 'tstuser') do
assert_match(/\* \* \* \* \* \/bin\/true/, stdout, "Didn't find correct crobtab entry for tstuser on #{host}")
end
- step "apply the resource change on the host"
- on(host, puppet_resource("cron", "crontest", "user=tstuser", "command=/bin/true", "ensure=present", "hour='0-6'")) do
- assert_match(/hour\s+=>\s+\['0-6'\]/, stdout, "Modifying cron entry failed for tstuser on #{host}")
+ step 'apply the resource change on the host'
+ on(host, puppet_resource('cron', 'crontest', 'user=tstuser', 'command=/bin/true', 'ensure=present', "hour='0-6'")) do
+ assert_match(%r{hour\s+=>\s+\['0-6'\]}, stdout, "Modifying cron entry failed for tstuser on #{host}")
end
- step "verify that crontab -l contains what you expected"
- run_cron_on(host,:list,'tstuser') do
+ step 'verify that crontab -l contains what you expected'
+ run_cron_on(host, :list, 'tstuser') do
assert_match(/\* 0-6 \* \* \* \/bin\/true/, stdout, "Didn't find correctly modified time entry in crobtab entry for tstuser on #{host}")
end
end