Merge pull request #46 from bdunne/sorting_hashes

Extract Hash sorting from manageiq-gems-pending

lib/more_core_extensions/core_ext/hash.rb

@@ -1,2 +1,3 @@
-require 'more_core_extensions/core_ext/hash/nested'
 require 'more_core_extensions/core_ext/hash/deletes'
+require 'more_core_extensions/core_ext/hash/nested'
+require 'more_core_extensions/core_ext/hash/sorting'

lib/more_core_extensions/core_ext/hash/sorting.rb

@@ -0,0 +1,20 @@
+module MoreCoreExtensions
+  module HashSortBang
+    def sort!(*args, &block)
+      sorted = sort(*args, &block)
+      sorted = self.class[sorted.to_a] unless sorted.instance_of?(self.class)
+      replace(sorted)
+    end
+  end
+
+  module HashSortByBang
+    def sort_by!(*args, &block)
+      sorted = sort_by(*args, &block)
+      sorted = self.class[sorted.to_a] unless sorted.instance_of?(self.class)
+      replace(sorted)
+    end
+  end
+end
+
+Hash.send(:include, MoreCoreExtensions::HashSortBang) unless Hash.method_defined?(:sort!)
+Hash.send(:include, MoreCoreExtensions::HashSortByBang) unless Hash.method_defined?(:sort_by!)

spec/core_ext/hash/sorting.rb

@@ -0,0 +1,21 @@
+describe Hash do
+  it "#sort!" do
+    h = {:x => 1, :b => 2, :y => 3, :a => 4}
+    h_id = h.object_id
+
+    h.sort!
+
+    expect(h.keys).to eq([:a, :b, :x, :y])
+    expect(h.object_id).to eq(h_id)
+  end
+
+  it "#sort_by!" do
+    h = {:x => 1, :b => 2, :y => 3, :a => 4}
+    h_id = h.object_id
+
+    h.sort_by! { |k, _v| k }
+
+    expect(h.keys).to eq([:a, :b, :x, :y])
+    expect(h.object_id).to eq(h_id)
+  end
+end