1 files changed, 42 insertions, 0 deletions
diff --git a/spec/acceptance/tests/resource/cron/should_update_existing.rb b/spec/acceptance/tests/resource/cron/should_update_existing.rb
new file mode 100644
index 0000000..a4bf177
--- /dev/null
+++ b/spec/acceptance/tests/resource/cron/should_update_existing.rb
@@ -0,0 +1,42 @@
+test_name "puppet should update existing crontab entry"
+confine :except, :platform => 'windows'
+confine :except, :platform => /^eos-/ # See PUP-5500
+confine :except, :platform => /^fedora-28/
+tag 'audit:medium',
+    'audit:refactor',  # Use block style `test_name`
+    'audit:acceptance' # Could be done at the integration (or unit) layer though
+                       # actual changing of resources could irreparably damage a
+                       # host running this, or require special permissions.
+
+require 'puppet/acceptance/common_utils'
+extend Puppet::Acceptance::CronUtils
+
+teardown do
+  step "Cron: cleanup"
+  agents.each do |agent|
+    clean agent
+  end
+end
+
+agents.each do |host|
+  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 "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}")
+  end
+
+  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