Treat nil just as any other value, allowing to define NilClassAccess,…

… see #4
dnagir · Feb 26, 2014 · b1be9a0 · b1be9a0
1 parent 79c8855
commit b1be9a0
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/lib/allowy/registry.rb b/lib/allowy/registry.rb
@@ -12,8 +12,6 @@ def access_control_for!(subject)
     end
 
     def access_control_for(subject)
-      return unless subject
-
       # Try subject as decorated object
       clazz = class_for "#{subject.class.source_class.name}Access" if subject.class.respond_to?(:source_class)
 

diff --git a/spec/registry_spec.rb b/spec/registry_spec.rb
@@ -25,6 +25,14 @@ module Allowy
         lambda { subject.access_control_for!(nil) }.should raise_error UndefinedAccessControl
       end
 
+      it "should return NilClassAccess when subject is nil and NilClassAccess is defined" do
+        nil_class_access = Class.new do
+          include Allowy::AccessControl
+        end
+        stub_const("NilClassAccess", nil_class_access)
+        subject.access_control_for!(nil).should be_a nil_class_access
+      end
+
       it "should return the same AC instance" do
         first = subject.access_control_for!(Sample)
         secnd = subject.access_control_for!(Sample)