Fix remote hash ordering for unit tests
authorColleen Murphy <colleen@puppetlabs.com>
Wed, 25 Mar 2015 19:04:16 +0000 (12:04 -0700)
committerColleen Murphy <colleen@puppetlabs.com>
Wed, 25 Mar 2015 19:04:16 +0000 (12:04 -0700)
commitd27759141ec5592fd5d76861efa77a55d07c90a8
treef2ab9d3e3439a676911a0ede12b7b5d515bb611d
parent9a955b8746f373a1a3e47ebf67d4186b98e0094f
Fix remote hash ordering for unit tests

Without this commit, the unit tests for the git provider changing
multiple remotes mocks the remotes in a particular order. While in
practice it doesn't matter which remote the update_remotes method
updates first, the unit tests must be able to mock them in the correct
order. For ruby 1.8.7, a Hash will not necessarily produce key value
pairs in the same order on each run, which causes intermittent failures
in the unit tests. This change sorts the :source property values before
trying to update them, and updates the unit tests to expect the values
in alphabetical order.
lib/puppet/provider/vcsrepo/git.rb
spec/unit/puppet/provider/vcsrepo/git_spec.rb