hydra-access-controls 0.0.4 → 0.0.5

data/lib/hydra-access-controls/version.rb

@@ -1,7 +1,7 @@
 module Hydra
   module Access
     module Controls
-      VERSION = "0.0.4"
+      VERSION = "0.0.5"
     end
   end
 end

data/lib/hydra/role_mapper_behavior.rb

@@ -15,11 +15,11 @@ module Hydra::RoleMapperBehavior
       if user_or_uid.kind_of?(String)
         user = ::User.find_by_user_key(user_or_uid)
         user_id = user_or_uid
-      elsif user_or_uid.kind_of?(::User) && !user_or_uid.uid.nil?  
+      elsif user_or_uid.kind_of?(::User) && user_or_uid.user_key  
         user = user_or_uid
         user_id = user.user_key
       end
-      array = byname[user_id]||[]
+      array = byname[user_id].dup || []
       array = array << 'registered' unless (user.nil? || user.new_record?) 
       array
     end

data/spec/support/user.rb

@@ -6,6 +6,7 @@ class User
 
   def initialize(params={})
     self.email = params[:email] if params[:email]
+    self.uid = params[:uid] if params[:uid]
   end
   
   def new_record?

data/spec/unit/role_mapper_spec.rb

@@ -16,6 +16,13 @@ describe RoleMapper do
     RoleMapper.roles('archivist2@example.com').should == ['archivist']
   end
 
+  it "should not change it's response when it's called repeatedly" do
+    u = User.new(:uid=>'leland_himself@example.com')
+    u.stub(:new_record?).and_return(false)
+    RoleMapper.roles(u).sort.should == ['archivist', 'donor', 'patron', "registered"]
+    RoleMapper.roles(u).sort.should == ['archivist', 'donor', 'patron', "registered"]
+  end
+
   it "should return an empty array if there are no roles" do
     RoleMapper.roles('zeus@olympus.mt').empty?.should == true
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hydra-access-controls
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.5
   prerelease: 
 platform: ruby
 authors:
@@ -225,4 +225,3 @@ test_files:
 - spec/unit/policy_aware_access_controls_enforcement_spec.rb
 - spec/unit/rights_metadata_spec.rb
 - spec/unit/role_mapper_spec.rb
-has_rdoc: