Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Adds shared nested specs #56

Merged
merged 2 commits into from
Aug 11, 2017
Merged

Conversation

NickLaMuro
Copy link
Member

@NickLaMuro NickLaMuro commented Aug 11, 2017

Adds specs around shared/nested.rb for #fetch_path and #store_path for Arrays mixed with Hashes and vice-versa.

Should ideally be merged prior to #54 and #55, and once merged, rebased into those two branches to confirm existing functionality.

@miq-bot
Copy link
Member

miq-bot commented Aug 11, 2017

Checked commits NickLaMuro/more_core_extensions@98bcbc4~...2461778 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 2 offenses detected

spec/core_ext/shared/nested_spec.rb

Adds specs around Array and Hashes where each class is intermixed with
the other.  Further cements the current implementation of #store_path
@Fryguy Fryguy merged commit 4bb2cd2 into ManageIQ:master Aug 11, 2017
@Fryguy Fryguy added the test label Aug 11, 2017
@NickLaMuro
Copy link
Member Author

Wow... well that didn't take much convincing ;)

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants