From 90216d84f8a2da1f77107dc5f0e3d76a3d72aacc Mon Sep 17 00:00:00 2001
From: Jorie Tappa <jorie@jorietappa.com>
Date: Tue, 31 Jul 2018 17:01:34 -0500
Subject: Apply automatic pdk validate fixes

---
 .../tests/resource/cron/should_remove_cron.rb      | 27 +++++++++++-----------
 1 file changed, 13 insertions(+), 14 deletions(-)

(limited to 'spec/acceptance/tests/resource/cron/should_remove_cron.rb')

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
-- 
cgit v1.2.3