openstax_utilities 4.2.0 → 4.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7336e5f9c9952756f9a981da7b79d5dece466022
-  data.tar.gz: e6a8621a11547eae03d63fc0d94e05053cc86664
+  metadata.gz: 4772e799075a8e6e6fb5d6b670be36448369e87e
+  data.tar.gz: e7fd112367bbfec2cc3047963c71289a980deb78
 SHA512:
-  metadata.gz: 7b17cb61f65fe32b021ce90b736ff163fedc34ffcb3173f9ba0af024d165a4b53ea586547df15cda7b3d787bcf2afc3cf5a2c39a69ce1bc546ddce1056c6a6cf
-  data.tar.gz: a5a03f359924bc9de55fd70e3b433fd3ef84dbed7c94d1c7ff40e802b1c5cc65d452a6bf09d77198872d94f82f3e3da6ca65a773f1becd7d6b003a122605c16c
+  metadata.gz: 364297c1c70a6218b52ca4eac44e8917bbb97cea1981a55c54fa174488cf831b34fc2a418e5dae66c01e9ccacf9939d7e01042aefbbc80b6cb1116e3588569d2
+  data.tar.gz: af1ec77a057b82d86586e644fae8546bab74fcccc92f00810a35c692880ebd3e73c876e3f62d50f481b14cf049c207c166a2a503ab89418f23a8e6224f84d723

data/lib/openstax/utilities/access_policy.rb

@@ -38,7 +38,7 @@ module OpenStax
         end
 
         resource_class = resource.is_a?(Class) ? resource : resource.class
-        policy_class = instance.resource_policy_map[resource_class.to_s]
+        policy_class = instance.resource_policy_map[resource_class.to_s].try(:constantize)
 
         # If there is no policy registered, we by default deny access
         return false if policy_class.nil?
@@ -47,7 +47,7 @@ module OpenStax
       end
 
       def self.register(resource_class, policy_class)
-        self.instance.resource_policy_map[resource_class.to_s] = policy_class
+        self.instance.resource_policy_map[resource_class.to_s] = policy_class.to_s
       end
 
     end

data/lib/openstax/utilities/version.rb

@@ -1,5 +1,5 @@
 module OpenStax
   module Utilities
-    VERSION = "4.2.0"
+    VERSION = "4.2.1"
   end
 end

data/spec/dummy/app/access_policies/dummier_access_policy.rb

@@ -0,0 +1,10 @@
+class DummierAccessPolicy
+  cattr_accessor :last_action, :last_requestor, :last_resource
+
+  def self.action_allowed?(action, requestor, resource)
+    @@last_action = action
+    @@last_requestor = requestor
+    @@last_resource = resource
+    true
+  end
+end

data/spec/dummy/config/environments/test.rb

@@ -13,7 +13,7 @@ Rails.application.configure do
   config.eager_load = false
 
   # Configure static asset server for tests with Cache-Control for performance.
-  config.serve_static_assets  = true
+  config.serve_static_files  = true
   config.static_cache_control = 'public, max-age=3600'
 
   # Show full error reports and disable caching.

data/spec/lib/openstax/utilities/access_policy_spec.rb

@@ -24,30 +24,31 @@ module OpenStax
 
       it 'delegates checks to policy classes based on resource class' do
         dummy_object = double('Dummy')
-        dummy_policy = double('Dummy Policy', :action_allowed? => true)
 
         AccessPolicy.register(User, DummyAccessPolicy)
-        AccessPolicy.register(dummy_object.class, dummy_policy)
+        AccessPolicy.register(dummy_object.class, DummierAccessPolicy)
         
         DummyAccessPolicy.last_action = nil
         DummyAccessPolicy.last_requestor = nil
         DummyAccessPolicy.last_resource = nil
 
-        expect(AccessPolicy.action_allowed?(:read, user, dummy_object)).to(
-          eq(true))
-      
-        expect{AccessPolicy.require_action_allowed!(:read, user, dummy_object)
-          }.not_to raise_error
+        DummierAccessPolicy.last_action = nil
+        DummierAccessPolicy.last_requestor = nil
+        DummierAccessPolicy.last_resource = nil
 
-        expect(DummyAccessPolicy.last_action).to be_nil
-        expect(DummyAccessPolicy.last_requestor).to be_nil
-        expect(DummyAccessPolicy.last_resource).to be_nil
+        expect(AccessPolicy.action_allowed?(:read, user, dummy_object)).to eq true
+        expect{
+          AccessPolicy.require_action_allowed!(:read, user, dummy_object)
+        }.not_to raise_error
 
-        expect(AccessPolicy.action_allowed?(:create, user, User.new)).to(
-          eq(true))
+        expect(DummierAccessPolicy.last_action).to eq(:read)
+        expect(DummierAccessPolicy.last_requestor).to eq(user)
+        expect(DummierAccessPolicy.last_resource).to eq(dummy_object)
 
-        expect{AccessPolicy.require_action_allowed!(:create, user, User.new)
-          }.not_to raise_error
+        expect(AccessPolicy.action_allowed?(:create, user, User.new)).to eq true
+        expect{
+          AccessPolicy.require_action_allowed!(:create, user, User.new)
+        }.not_to raise_error
 
         expect(DummyAccessPolicy.last_action).to eq(:create)
         expect(DummyAccessPolicy.last_requestor).to eq(user)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: openstax_utilities
 version: !ruby/object:Gem::Version
-  version: 4.2.0
+  version: 4.2.1
 platform: ruby
 authors:
 - JP Slavinsky
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-12-16 00:00:00.000000000 Z
+date: 2015-05-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -160,6 +160,7 @@ files:
 - lib/openstax_utilities.rb
 - spec/dummy/README.md
 - spec/dummy/Rakefile
+- spec/dummy/app/access_policies/dummier_access_policy.rb
 - spec/dummy/app/access_policies/dummy_access_policy.rb
 - spec/dummy/app/assets/javascripts/application.js
 - spec/dummy/app/assets/stylesheets/application.css
@@ -230,6 +231,7 @@ signing_key:
 specification_version: 4
 summary: Utilities for OpenStax web sites
 test_files:
+- spec/dummy/app/access_policies/dummier_access_policy.rb
 - spec/dummy/app/access_policies/dummy_access_policy.rb
 - spec/dummy/app/assets/javascripts/application.js
 - spec/dummy/app/assets/stylesheets/application.css