Blacklight::OpenStructWithHashAccess should expose a HashWithIndiffer…

…entAccess
projectblacklight · Dec 3, 2016 · a5437bb · a5437bb
1 parent 061ac75
commit a5437bb
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/lib/blacklight/utils.rb b/lib/blacklight/utils.rb
@@ -16,6 +16,12 @@ def self.protected_attributes_enabled?
   class OpenStructWithHashAccess < OpenStruct
     delegate :keys, :each, :map, :has_key?, :key?, :include?, :empty?, :length, :delete, :delete_if, :keep_if, :clear, :reject!, :select!, :replace, :fetch, :to_json, :as_json, to: :to_h
 
+    def initialize(*args)
+      super
+
+      @table = @table.with_indifferent_access
+    end
+
     ##
     # Expose the internal hash
     # @return [Hash]
@@ -55,6 +61,12 @@ def merge! other_hash
     def deep_dup
       self.class.new @table.deep_dup
     end
+
+    protected
+
+    def convert_key(key)
+      key.is_a?(Symbol) ? key : key.to_sym
+    end
   end
 
   ##

diff --git a/spec/lib/blacklight/utils_spec.rb b/spec/lib/blacklight/utils_spec.rb
@@ -38,14 +38,13 @@
       end
 
       it "exposes the internal hash table" do
-        expect(@h.to_h).to be_a_kind_of(Hash)
+        expect(@h.to_h).to be_a_kind_of(HashWithIndifferentAccess)
         expect(@h.to_h[:a]).to eq 1
       end
 
       it "exposes keys" do
         expect(@h.keys).to include(:a, :b)
       end
-
     end
 
     describe "#key?" do