mongo_mapper-unstable 2010.3.8 → 2010.06.23

data/test/functional/test_protected.rb

@@ -13,7 +13,7 @@ class ProtectedTest < Test::Unit::TestCase
       @doc = @doc_class.create(:name => 'Steve Sloan')
     end
 
-    should 'have protected attributes class method' do
+    should "have protected attributes class method" do
       @doc_class.protected_attributes.should == [:admin].to_set
     end
 
@@ -25,13 +25,25 @@ class ProtectedTest < Test::Unit::TestCase
       @doc.protected_attributes.should equal(@doc_class.protected_attributes)
     end
 
+    should "raise error if there are accessible attributes" do
+      doc = Doc('Post')
+      doc.attr_accessible :name
+      lambda { doc.attr_protected :admin }.
+        should raise_error(/Declare either attr_protected or attr_accessible for Post/)
+    end
+
+    should "know if using protected attributes" do
+      @doc_class.protected_attributes?.should be(true)
+      Doc().protected_attributes?.should be(false)
+    end
+
     should "work with :protected shortcut when defining key" do
       Doc() do
         key :user_id, ObjectId, :protected => true
       end.protected_attributes.should == [:user_id].to_set
     end
 
-    should 'assign protected attribute through accessor' do
+    should "assign protected attribute through accessor" do
       @doc.admin = true
       @doc.admin.should be_true
     end
@@ -52,23 +64,27 @@ class ProtectedTest < Test::Unit::TestCase
       doc.name.should == 'John'
     end
 
-    should 'ignore protected attribute on #update_attributes' do
+    should "ignore protected attribute on #update_attributes" do
       @doc.update_attributes(:name => 'Ren Hoek', :admin => true)
       @doc.name.should == 'Ren Hoek'
       @doc.admin.should be_false
     end
 
-    should 'ignore protected attribute on #update_attributes!' do
+    should "ignore protected attribute on #update_attributes!" do
       @doc.update_attributes!(:name => 'Stimpson J. Cat', :admin => true)
       @doc.name.should == 'Stimpson J. Cat'
       @doc.admin.should be_false
     end
 
-    should 'be indifferent to whether the protected keys are strings or symbols' do
+    should "be indifferent to whether the protected keys are strings or symbols" do
       @doc.update_attributes!("name" => 'Stimpson J. Cat', "admin" => true)
       @doc.name.should == 'Stimpson J. Cat'
       @doc.admin.should be_false
     end
+
+    should "accept nil as constructor's argument without raising exception" do
+      lambda { @doc_class.new(nil) }.should_not raise_error
+    end
   end
 
   context "Single collection inherited protected attributes" do
@@ -76,9 +92,7 @@ class ProtectedTest < Test::Unit::TestCase
       class ::GrandParent
         include MongoMapper::Document
 
-        key :_type, String
         key :site_id, ObjectId
-
         attr_protected :site_id
       end
       GrandParent.collection.remove
@@ -129,7 +143,7 @@ class ProtectedTest < Test::Unit::TestCase
       @doc.people << @edoc
     end
 
-    should 'have protected attributes class method' do
+    should "have protected attributes class method" do
       @edoc_class.protected_attributes.should == [:admin].to_set
     end
 
@@ -141,18 +155,18 @@ class ProtectedTest < Test::Unit::TestCase
       @edoc.protected_attributes.should equal(@edoc_class.protected_attributes)
     end
 
-    should 'assign protected attribute through accessor' do
+    should "assign protected attribute through accessor" do
       @edoc.admin = true
       @edoc.admin.should be_true
     end
 
-    should 'ignore protected attribute on #update_attributes' do
+    should "ignore protected attribute on #update_attributes" do
       @edoc.update_attributes(:name => 'Ren Hoek', :admin => true)
       @edoc.name.should == 'Ren Hoek'
       @edoc.admin.should be_false
     end
 
-    should 'ignore protected attribute on #update_attributes!' do
+    should "ignore protected attribute on #update_attributes!" do
       @edoc.update_attributes!(:name => 'Stimpson J. Cat', :admin => true)
       @edoc.name.should == 'Stimpson J. Cat'
       @edoc.admin.should be_false