aboutsummaryrefslogtreecommitdiff
path: root/spec/acceptance/tests/resource/cron/should_remove_cron.rb
diff options
context:
space:
mode:
authorJorie Tappa <jorie@jorietappa.com>2018-07-31 17:01:34 -0500
committerJorie Tappa <jorie@jorietappa.com>2018-08-01 10:30:53 -0500
commit90216d84f8a2da1f77107dc5f0e3d76a3d72aacc (patch)
tree6c8b98d86ed8ea12c20a6e4234b595d99c922ad4 /spec/acceptance/tests/resource/cron/should_remove_cron.rb
parenta2af7dd0b9713f279724d2c7e6f17bfd8ce2d95b (diff)
downloadpuppet-cron_core-90216d84f8a2da1f77107dc5f0e3d76a3d72aacc.tar.gz
puppet-cron_core-90216d84f8a2da1f77107dc5f0e3d76a3d72aacc.tar.bz2
Apply automatic pdk validate fixes
Diffstat (limited to 'spec/acceptance/tests/resource/cron/should_remove_cron.rb')
-rw-r--r--spec/acceptance/tests/resource/cron/should_remove_cron.rb27
1 files changed, 13 insertions, 14 deletions
diff --git a/spec/acceptance/tests/resource/cron/should_remove_cron.rb b/spec/acceptance/tests/resource/cron/should_remove_cron.rb
index 1320272..1759172 100644
--- a/spec/acceptance/tests/resource/cron/should_remove_cron.rb
+++ b/spec/acceptance/tests/resource/cron/should_remove_cron.rb
@@ -1,7 +1,7 @@
-test_name "puppet should remove a crontab entry as expected"
-confine :except, :platform => 'windows'
-confine :except, :platform => /^eos-/ # See PUP-5500
-confine :except, :platform => /^fedora-28/
+test_name 'puppet should remove a crontab entry as expected'
+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,28 +12,27 @@ 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 "apply the resource on the host using puppet resource"
- on(host, puppet_resource("cron", "crontest", "user=tstuser",
- "command=/bin/true", "ensure=absent")) do
- assert_match(/crontest\D+ensure:\s+removed/, stdout, "Didn't remove crobtab entry for tstuser on #{host}")
+ step 'apply the resource on the host using puppet resource'
+ on(host, puppet_resource('cron', 'crontest', 'user=tstuser',
+ 'command=/bin/true', 'ensure=absent')) do
+ assert_match(%r{crontest\D+ensure:\s+removed}, stdout, "Didn't remove crobtab entry for tstuser on #{host}")
end
- step "verify that crontab -l contains what you expected"
+ step 'verify that crontab -l contains what you expected'
run_cron_on(host, :list, 'tstuser') do
assert_no_match(/\/bin\/true/, stderr, "Error: Found entry for tstuser on #{host}")
end
-
end