chef / cheffish

Resources and tools for testing and interacting with Chef and Chef Server.
Apache License 2.0
38 stars 28 forks source link

MergedConfig: Support both string & symbol for [] #114

Closed elthariel closed 8 years ago

elthariel commented 8 years ago

When you have a MergedConfig with two config defining the 'same' key, but one as a string and one as a symbol, you end up having this very confusing behavior:

config['bootstrap_options']
=> {some hash}
config[:bootstrap_options]
=> {another hash}

This PR address this behavior. I hope but doubt this is an appropriate fix, so let me know what you think about it