hydra-access-controls 6.3.2 → 6.3.3

data/lib/hydra/datastream/rights_metadata.rb

@@ -107,7 +107,9 @@ module Hydra
           end
         else
           remove_all_permissions(selector)
-          unless new_access_level == "none" 
+          if new_access_level == "none" 
+            self.content = self.to_xml
+          else
             access_type_symbol = "#{new_access_level}_access".to_sym
             current_values = term_values(access_type_symbol, type)
             self.update_values([access_type_symbol, type] => current_values + [actor] )

data/spec/unit/hydra_rights_metadata_persistence_spec.rb

@@ -0,0 +1,72 @@
+require 'spec_helper'
+
+describe Hydra::Datastream::RightsMetadata do
+  before :all do
+    class RightsTest < ActiveFedora::Base
+      include Hydra::ModelMixins::RightsMetadata
+      has_metadata name: 'rightsMetadata', type: Hydra::Datastream::RightsMetadata
+    end
+  end
+
+  after :all do
+    Object.send(:remove_const,:RightsTest)
+  end
+
+  describe "rightsMetadata" do
+    let!(:thing) {RightsTest.new}
+
+    [:discover,:read,:edit].each do |mode|
+      describe "##{mode}_users" do
+        let(:get_method) {"#{mode}_users".to_sym}
+        let(:set_method) {"#{mode}_users=".to_sym}
+
+        before :each do
+          thing.send(set_method, ['locutus@borg.collective.mil'])
+          thing.save
+        end
+
+        it "should persist initial setting" do
+          thing.reload.send(get_method).should == ['locutus@borg.collective.mil']
+        end
+
+        it "should persist changes" do
+          thing.send(set_method, ['locutus@borg.collective.mil','sevenofnine@borg.collective.mil'])
+          thing.save
+          thing.reload.send(get_method).should =~ ['locutus@borg.collective.mil','sevenofnine@borg.collective.mil']
+        end
+
+        it "should persist emptiness" do
+          thing.send(set_method, [])
+          thing.save
+          thing.reload.send(get_method).should == []
+        end
+      end
+
+      describe "##{mode}_groups" do
+        let(:get_method) {"#{mode}_groups".to_sym}
+        let(:set_method) {"#{mode}_groups=".to_sym}
+
+        before :each do
+          thing.send(set_method, ['borg'])
+          thing.save
+        end
+
+        it "should persist initial setting" do
+          thing.reload.send(get_method).should == ['borg']
+        end
+
+        it "should persist changes" do
+          thing.send(set_method, ['borg','federation'])
+          thing.save
+          thing.reload.send(get_method).should =~ ['borg','federation']
+        end
+
+        it "should persist emptiness" do
+          thing.send(set_method, [])
+          thing.save
+          thing.reload.send(get_method).should == []
+        end
+      end
+    end
+  end
+end

metadata

@@ -1,7 +1,8 @@
 --- !ruby/object:Gem::Specification
 name: hydra-access-controls
 version: !ruby/object:Gem::Version
-  version: 6.3.2
+  version: 6.3.3
+  prerelease: 
 platform: ruby
 authors:
 - Chris Beer
@@ -10,25 +11,28 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-07-22 00:00:00.000000000 Z
+date: 2013-08-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: active-fedora
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -36,6 +40,7 @@ dependencies:
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -43,34 +48,39 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: cancan
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: deprecation
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: blacklight
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -78,6 +88,7 @@ dependencies:
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -85,29 +96,33 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 description: Access controls for project hydra
@@ -152,6 +167,7 @@ files:
 - spec/unit/ability_spec.rb
 - spec/unit/access_controls_enforcement_spec.rb
 - spec/unit/admin_policy_spec.rb
+- spec/unit/hydra_rights_metadata_persistence_spec.rb
 - spec/unit/hydra_rights_metadata_spec.rb
 - spec/unit/inheritable_rights_metadata_spec.rb
 - spec/unit/policy_aware_ability_spec.rb
@@ -162,26 +178,27 @@ files:
 homepage: http://projecthydra.org
 licenses:
 - APACHE2
-metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
+  none: false
   requirements:
-  - - '>='
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: 1.9.3
 required_rubygems_version: !ruby/object:Gem::Requirement
+  none: false
   requirements:
-  - - '>='
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.3
+rubygems_version: 1.8.23
 signing_key: 
-specification_version: 4
+specification_version: 3
 summary: Access controls for project hydra
 test_files:
 - spec/factories.rb
@@ -196,6 +213,7 @@ test_files:
 - spec/unit/ability_spec.rb
 - spec/unit/access_controls_enforcement_spec.rb
 - spec/unit/admin_policy_spec.rb
+- spec/unit/hydra_rights_metadata_persistence_spec.rb
 - spec/unit/hydra_rights_metadata_spec.rb
 - spec/unit/inheritable_rights_metadata_spec.rb
 - spec/unit/policy_aware_ability_spec.rb

checksums.yaml

@@ -1,7 +0,0 @@
----
-SHA1:
-  metadata.gz: d02a342a99ab54d6be798a0f0f2d2ae479f518a1
-  data.tar.gz: bb95f71e73450a6878d6fad434b5640cf58b633a
-SHA512:
-  metadata.gz: be56500f51d49197e1add0a38082474fab5d0ef2a15afec5117b8a765cd6c5a1a59b6e6cfdedc3ff31bff017cbf2e584c0eac0b9d4a258d0f00cadd48b7535e6
-  data.tar.gz: 6ed343fe7cadccc9ffd0be60a508179159538ba6a8434e38ac5ad49333dc0b2488f2bb8e79c034ed8e4b00b54c9b6a18f56121ae1ea735bf7392d039943cb7f1