test_name "should not delete data when existing content is malformed"

tag 'audit:low',
    '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.

agents.each do |agent|
  file = agent.tmpfile('host-not-delete-data')

  teardown do
    on(agent, "rm -f #{file}", :acceptable_exit_codes => (0..255))
  end

  step "(setup) populate test file with host information"
  on(agent, "printf '127.0.0.2 existing alias\n' > #{file}")

  step "(setup) populate test file with a malformed line"
  on(agent, "printf '==\n' >> #{file}")

  step "tell puppet to add another host entry"
  on(agent, puppet_resource('host', 'test', "target=#{file}",
    'ensure=present', 'ip=127.0.0.3', 'host_aliases=foo'))

  step "verify that the initial host entry was not deleted"
  on(agent, "cat #{file}") do |res|
    fail_test "existing host data was deleted" unless
      res.stdout.include? 'existing'
  end

end