Corrective change should use property values directly, bypassing any munging

It should be possible to apply the two following manifests in sequence without Puppet apply failing with an error, and the report should indicate that it was NOT a corrective change, since the manifest has changed: group { 'bar': ensure => present }   user { 'foo': ensure => present, membership => inclusive, groups => ['bar'] } group { 'bar': ensure => present }   group { 'baz': ensure => present }   user { 'foo': ensure => present, membership => inclusive, groups => ['bar', 'baz'] } User[foo]: !ruby/object:Puppet::Resource::Status title: foo file: "/home/branan/proj/pl/puppet/test.pp" line: 9 resource: User[foo] resource_type: User containment_path: - Stage[main] - Main - User[foo] evaluation_time: 0.033121403 tags: - user - foo - class time: '2016-08-19T12:41:22.752345544-07:00' failed: false changed: true out_of_sync: true skipped: false change_count: 1 out_of_sync_count: 1 events: - !ruby/object:Puppet::Transaction::Event audited: false property: groups previous_value: - baz desired_value: bar,baz historical_value: message: groups changed 'baz' to ['bar', 'baz'] name: :groups_changed status: success time: 2016-08-19 12:41:22.752746299 -07:00 redacted: corrective_change: false corrective_change: false