-
Notifications
You must be signed in to change notification settings - Fork 302
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
(MODULES-9479) Fix nested array merge behavior #593
(MODULES-9479) Fix nested array merge behavior #593
Conversation
Fix regression in behavior introduced in the 5.2.0 release with the fix for MODULES-8287. Restore the behavior where arrays nested inside hashes will be merged.
spec/acceptance/force_spec.rb
Outdated
it 'applies the manifest twice with no stderr' do | ||
idempotent_apply(pp) | ||
expect(file("#{@basedir}/file")).to be_file | ||
expect(file("#{@basedir}/file").content).to match '{"one":[1,2]}' || '{"one":[1,2]}\n' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could it be expect(file("#{@basedir}/file").content).to match '{"one":[1,2]}\R?'
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what about .to contain '{"one":[1,2]}'
we dont specifically care if the new line is there or not,
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
apologies for closing the pr, it was to force kick appveyor |
[martin.ewings@martin puppetlabs-concat]$ bundle exec rake litmus:acceptance:parallel
|
Fix regression in behavior introduced in the 5.2.0 release
with the fix for MODULES-8287. Restore the behavior where arrays
nested inside hashes will be merged.