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
                       # 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(%r{\* \* \* \* \* /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(%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
    assert_match(%r{\* 0-6 \* \* \* /bin/true}, stdout, "Didn't find correctly modified time entry in crobtab entry for tstuser on #{host}")
  end
end