(MODULES-5364) All version negative/prop_file fix

[Iristyle]

• 0284928 made a naive fix to the acceptance test negative/prop_file
  under the assumption that the test output changed based on just
  recent module changes in 4c73468.

• In reality, the 4c73468 bugfix changed how the output appears in
  both Puppet 4 and Puppet 5, as Puppet 5 changed formatting around
  basic Ruby types.

  That bugfix stopped emitting a [Puppet::Type::Acl::Ace] and instead
  emits a [Hash] representing Aces.

  Those formatting changes apply to pupept apply and puppet agent
  for the ACL module, but not for puppet resource.

• The test updates in this commit vary how output is checked based
  on which version of Puppet the test is running on.

  For Puppet 4, the manifest application emits a message like:

   Notice: /Stage[main]/Main/Acl[c:/temp/prop_file]/permissions: permissions changed [
   ] to [
    { affects => 'self_only', identity => 'IHTEWFY92S03JZM\bob', rights => ['full'
   ] },
    { affects => 'self_only', identity => 'BUILTIN\Administrators', rights => ['full'
   ] }
   ]

For Puppet 5, the manifest application emits a mesaage like:

   Notice: /Stage[main]/Main/Acl[c:/temp/prop_file]/permissions: permissions changed [] to [{"identity"=>"AAR8J38RQU6AJ62\\bob", "rights"=>["full"], "affects"=>:self_only}, {"identity"=>"BUILTIN\\Administrators", "rights"=>["full"], "affects"=>:self_only}]

• This fix is temporary to resolve CI failures, until more time can
  be dedicated to some refactorings around the type structure inside
  the module.

[ghost]

This commit, combined with the test documentation in PA-1374, looks good to me. Merging.