Merge pull request #26 from PRX/feat/stringify_hash_access

Automatically stringify index access on ResourceMap and define ScopeList#==

lib/prx_auth/resource_map.rb

@@ -37,6 +37,14 @@ def contains?(resource, namespace=nil, scope=nil)
       end
     end
 
+    def [](key)
+      super(key.to_s)
+    end
+
+    def []=(key, value)
+      super(key.to_s, value)
+    end
+
     def condense
       condensed_wildcard = @wildcard.condense
       condensed_map = Hash[map do |resource, list|

lib/prx_auth/scope_list.rb

@@ -129,6 +129,10 @@ def &(other_list)
       self - (self - other_list)
     end
 
+    def ==(other)
+      condense.sort_by(&:to_s) == other.condense.sort_by(&:to_s)
+    end
+
     private
 
     def symbolize(value)

lib/prx_auth/version.rb

@@ -1,3 +1,3 @@
 module PrxAuth
-  VERSION = "1.2.1"
+  VERSION = "1.3.0"
 end

test/prx_auth/resource_map_test.rb

@@ -169,4 +169,17 @@ def new_map(val)
       assert map.as_json.has_key?('*')
     end
   end
+
+  describe '#[]' do
+    it 'automatically stringifies' do
+      refute_nil map[123]
+    end
+  end
+
+  describe '#[]=' do
+    it 'automatically stringifies' do
+      map[789] = PrxAuth::ScopeList.new("")
+      refute_nil map["789"]
+    end
+  end
 end

test/prx_auth/scope_list_test.rb

@@ -105,4 +105,15 @@ def new_list(val)
       assert !sl.contains?(:one)
     end
   end
+
+  describe '==' do
+
+    it 'is equal when they are functionally equal' do
+      assert_equal PrxAuth::ScopeList.new("foo ns:foo bar ns2:baz"), PrxAuth::ScopeList.new("ns2:baz bar foo")
+    end
+
+    it 'is not equal when they are not functionally equal' do
+      refute_equal PrxAuth::ScopeList.new("foo bar"), PrxAuth::ScopeList.new("foo:bar bar:foo")
+    end
+  end
 end