Adding respond_to override to the ObjectifiedHash class so it properl…

…y responds to respond_to? calls

lib/gitlab/objectified_hash.rb

@@ -20,5 +20,9 @@ def to_hash
     def method_missing(key)
       @data.key?(key.to_s) ? @data[key.to_s] : nil
     end
+
+    def respond_to?(method_name, include_private = false)
+      @hash.keys.map(&:to_sym).include?(method_name.to_sym) || super
+    end
   end
 end

spec/gitlab/objectified_hash_spec.rb

@@ -2,7 +2,7 @@
 
 describe Gitlab::ObjectifiedHash do
   before do
-    @hash = {a: 1, b: 2}
+    @hash = {a: 1, b: 2, 'string' => 'string', symbol: :symbol}
     @oh = Gitlab::ObjectifiedHash.new @hash
   end
 
@@ -20,4 +20,22 @@
       expect(@oh.respond_to?(:to_h)).to be_truthy
     end
   end
+
+  describe "#respond_to" do
+    it "should return true for methods this object responds to through method_missing as sym" do
+      expect(@oh.respond_to?(:a)).to be_truthy
+    end
+
+    it "should return true for methods this object responds to through method_missing as string" do
+      expect(@oh.respond_to?('string')).to be_truthy
+    end
+
+    it "should not care if you use a string or symbol to reference a method" do
+      expect(@oh.respond_to?(:string)).to be_truthy
+    end
+
+    it "should not care if you use a string or symbol to reference a method" do
+      expect(@oh.respond_to?('symbol')).to be_truthy
+    end
+  end
 end