access_policy 0.0.4 → 0.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c13ac5c6dac57c9f9941984ef3a3cdfb41ea0a2e
-  data.tar.gz: 72b632f8163d91a4655bb8b9fdd4989402c76777
+  metadata.gz: feb81c43a683b9e8237ea9c967e2e23c7ff768a1
+  data.tar.gz: c8ec25b6998f7cc9c6ec47b5de509b3f3226a522
 SHA512:
-  metadata.gz: 08b09203d8dfeda012749a7b6480ecbc594120cf16df57b92734f06eb8fbd5501a9429c5c836670bef0fd6a73e361b28482540c962b9a1aec6bd8986d287f029
-  data.tar.gz: 12c95f04b081718da2b910b68d89942514862d59fdc44ad240c86f30d14273c9e28be567feb6915edf769d6ee9453ec68ecf29105cb7a6e47db17248443f8e34
+  metadata.gz: f2acab607775708c0bf8eb59df2d1282e6afb7718039804fee723b248e72f6e16cd0d6dcce8219f10c77b288ec71fdd3e26941e046cf272857789166f1856926
+  data.tar.gz: 6b7e3d5c100fd6d1a5090a8eaeda60cf1b8daaebbb632fe713b030db9f73ef68f8dad4dbccbb62bdefc35135d212d1953a0488af7012cb88d077baf141a7f66a

data/lib/access_policy/policy_enforcer.rb

@@ -4,7 +4,7 @@ module AccessPolicy
     attr_accessor :current_user_or_role, :object_or_class, :query, :default_error_policy
 
     def initialize(current_user_or_role, object_or_class, query=nil, default_error_policy=->(*) { raise })
-      raise NotDefinedError, 'unable to find policy class for anonymous classes' if class_to_guard(object_or_class).name.nil? || class_to_guard(object_or_class).name.length < 1
+      raise NotDefinedError, 'unable to find policy class for anonymous classes' unless policy_class_can_be_found_for?(object_or_class)
 
       self.current_user_or_role = current_user_or_role
       self.object_or_class = object_or_class
@@ -37,6 +37,11 @@ module AccessPolicy
 
     protected
 
+    def policy_class_can_be_found_for?(object_or_class)
+      subject = class_to_guard(object_or_class)
+      (!subject.name.nil? && subject.name.length > 0) || subject.respond_to?(:policy_class)
+    end
+
     def class_to_guard(obj_or_class=object_or_class)
       obj_or_class.is_a?(Class) ? obj_or_class : obj_or_class.class
     end

data/lib/access_policy/version.rb

@@ -1,3 +1,3 @@
 module AccessPolicy
-  VERSION = "0.0.4"
+  VERSION = "0.0.5"
 end

data/lib/access_policy.rb

@@ -44,7 +44,7 @@ module AccessPolicy
   module ClassMethods
 
     def policy_guarded_method(action_name, &block)
-      unsafe_action_name = :"#{action_name}_unsafe"
+      unsafe_action_name = unsafe_action_name(action_name)
 
       define_method action_name do |*args|
         _authorize "#{action_name}?"
@@ -54,6 +54,10 @@ module AccessPolicy
       define_method unsafe_action_name, block
     end
 
+    def unsafe_action_name(action_name)
+      :"#{action_name}_unsafe"
+    end
+
   end
 
 end

data/spec/unit/lib/access_policy/policy_enforcer_spec.rb

@@ -54,6 +54,16 @@ module AccessPolicy
       it 'raises an error when class to guard has no name' do
         expect{AccessPolicy::PolicyEnforcer.new(current_user, Class.new.new, action, error_policy)}.to raise_error
       end
+
+      it 'raises no error when class to guard has no name but an policy class is defined' do
+        some_class = Class.new do
+          def self.policy_class
+            Class.new
+          end
+        end
+
+        expect { AccessPolicy::PolicyEnforcer.new(current_user, some_class.new, action, error_policy) }.not_to raise_error
+      end
     end
 
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: access_policy
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.5
 platform: ruby
 authors:
 - Dieter Späth