api_resource 0.4.3 → 0.5.0

data/spec/lib/associations_spec.rb

@@ -8,6 +8,35 @@ describe "Associations" do
     TestResource.reload_class_attributes
   end
 
+  let(:ap) do
+    Associations::SingleObjectProxy.new(
+      "TestResource", 
+      HasManyObject.new, {
+        :service_uri => '/single_object_association'
+      }
+    )
+  end
+
+  context "Comparison" do
+
+    context "&group_by" do
+
+      it "should allow grouping by resources with the same id" do
+
+        tr1 = TestResource.find(1)
+        tr2 = TestResource.find(1)
+
+
+        tr1.has_one_object
+        tr2.has_one_object
+
+        [tr1, tr2].group_by(&:has_one_object).keys.length.should be 1
+      end
+
+    end
+
+  end
+
   context "creating and testing for associations of various types" do
 
     it "should be able to give a list of all associations" do
@@ -200,27 +229,29 @@ describe "Associations" do
     end
 
     it "should be able to extract a service uri from the contents hash" do
-      ap = Associations::SingleObjectProxy.new("TestResource",{:service_uri => "/path"})
+      ap = Associations::SingleObjectProxy.new(
+        "TestResource", 
+        HasManyObject.new, {
+          :service_uri => '/path'
+        }
+      )
       ap.remote_path.should eql("/path")
     end
 
-    it "should be able to recognize the attributes of an object and not make them scopes" do
+    it "should be able to recognize the attributes of an object 
+      and not make them scopes" do
+      
       TestResource.define_attributes :test
-      ap = Associations::SingleObjectProxy.new("TestResource",{:service_uri => "/path", :test => "testval"})
+      ap = Associations::SingleObjectProxy.new(
+        "TestResource", 
+        HasManyObject.new, {
+          :service_uri => '/path',
+          :test => "testval"
+        }
+      )
       ap.scope?("test").should be_false
-      ap.remote_path.should eql("/path")
-    end
-
-    it "should make all attributes except the service uri into scopes given the scopes_only option" do
-      ap = Associations::SingleObjectProxy.new("TestResource",{:service_uri => "/path", :test_scope => {"testval" => true}, :scopes_only => true})
-      ap.scope?("test_scope").should be_true
-      ap.remote_path.should eql("/path")
-    end
+      ap.test.should eql("testval")
 
-    it "should pass the attributes that are not scopes and make them attributes of the object" do
-      TestResource.define_attributes :test
-      ap = Associations::SingleObjectProxy.new("TestResource",{:service_uri => "/path", :test => "testval"})
-      ap.internal_object.attributes.keys.should include("test")
     end
   end
 
@@ -239,19 +270,29 @@ describe "Associations" do
       context "Loading array contents" do
 
         it "should be able to load a blank array" do
-          ap = Associations::MultiObjectProxy.new("TestResource",[])
+          ap = Associations::MultiObjectProxy.new(
+            "TestResource",
+            BelongsToObject.new
+          )
           ap.remote_path.should be_nil
-          ap.scopes.keys.should eql([])
         end
 
         it "should be able to recognize a settings hash if it has a service_uri" do
-          ap = Associations::MultiObjectProxy.new("TestResource",[{:service_uri => "/route"}])
+          ap = Associations::MultiObjectProxy.new(
+            "TestResource",
+            BelongsToObject.new,
+            [{:service_uri => "/route"}]
+          )
           ap.remote_path.should eql("/route")
         end
 
         it "should be able to recognize a settings hash if it has a 'service_uri' with another preset name" do
           Associations::MultiObjectProxy.remote_path_element = :the_element
-          ap = Associations::MultiObjectProxy.new("TestResource",[{:the_element => "/route"}])
+          ap = Associations::MultiObjectProxy.new(
+            "TestResource",
+            BelongsToObject.new,
+            [{:the_element => "/route"}]
+          )
           ap.remote_path.should eql("/route")
         end
 
@@ -260,38 +301,71 @@ describe "Associations" do
       context "Loading hash contents" do
         it "should not be able to load a hash without a 'service_uri'" do
           lambda {
-            Associations::MultiObjectProxy.new("TestResource", {:hi => 3})
+            Associations::MultiObjectProxy.new(
+              "TestResource", 
+              BelongsToObject,
+              {:hi => 3}
+            )
           }.should raise_error
         end
 
         it "should be able to recognize settings from a hash" do
-          ap = Associations::MultiObjectProxy.new("TestResource", {:service_uri => "/route"})
+          ap = Associations::MultiObjectProxy.new(
+            "TestResource", 
+            BelongsToObject,
+            {:service_uri => "/route"}
+          )
+          ap.remote_path.should eql("/route")
+        end
+
+        it "should be able to recognize settings from a hash as a string" do
+          ap = Associations::MultiObjectProxy.new(
+            "TestResource", 
+            BelongsToObject,
+            {"service_uri" => "/route"}
+          )
           ap.remote_path.should eql("/route")
         end
 
         it "should recognize settings with differing 'service_uri' names" do
           Associations::MultiObjectProxy.remote_path_element = :the_element
-          ap = Associations::MultiObjectProxy.new("TestResource",{:the_element => "/route"})
+          ap = Associations::MultiObjectProxy.new(
+            "TestResource", 
+            BelongsToObject,
+            {:the_element => "/route"}
+          )
           ap.remote_path.should eql("/route")
         end
       end
 
       context "Defining scopes" do
 
-        it "should define scopes based on the other keys in a settings hash" do
-          ap = Associations::MultiObjectProxy.new("TestResource", [{:service_uri => "/route", :scope1 => {"scope1" => true}, :scope2 => {"scope2" => true}}])
-          [:scope1, :scope2].each{|s| ap.scopes.include?(s).should be_true }
-        end
-
-        it "should identify known scopes based on the scopes defined on the object it is a proxy to" do
+        it "should identify known scopes based on the scopes defined on 
+          the object it is a proxy to" do
+          
           TestResource.scope :class_scope, "class_scope" => "true"
-          ap = Associations::MultiObjectProxy.new("TestResource", [{:service_uri => "/route", :scope1 => {"scope1" => true}, :scope2 => {"scope2" => true}}])
-          [:scope1, :scope2, :class_scope].each{|s| ap.scopes.include?(s).should be_true}
+          
+          ap = Associations::MultiObjectProxy.new(
+            "TestResource",
+            BelongsToObject.new,
+            [{:service_uri => "/route"}]
+          )
+          ap.scope?(:class_scope).should be_true
         end
 
         it "scopes in the response should shadow class defined scopes" do
-          TestResource.scope :scope1, "scope1" => "true"
-          ap = Associations::MultiObjectProxy.new("TestResource", [{:service_uri => "/route", :scope1 => {"scope1" => true}, :scope2 => {"scope2" => true}}])
+          TestResource.scope(:scope1, "scope1" => "true")
+          ap = Associations::MultiObjectProxy.new(
+            "TestResource",
+            BelongsToObject.new,
+            [
+              {
+                :service_uri => "/route", 
+                :scope1 => {"scope1" => true}, 
+                :scope2 => {"scope2" => true}
+              }
+            ]
+          )
           ap.scopes[:scope1].should eql({"scope1" => true})
         end
       end
@@ -313,42 +387,32 @@ describe "Associations" do
       end
 
       it "should be able to query scopes on the current model" do
-        ScopeResource.no_arg.to_query.should eql                "no_arg=true"
-        # ScopeResource.one_arg(5).to_query.should eql            "one_arg[id]=5"
-        # ScopeResource.one_array_arg([3, 5]).to_query.should eql "one_array_arg[ids][]=3&one_array_arg[ids][]=5"
-        # ScopeResource.two_args(1, 20).to_query.should eql       "two_args[page]=1&two_args[per_page]=20"
-        # ScopeResource.opt_arg.to_query.should eql               "opt_args=true"
-        # ScopeResource.opt_args(3).to_query.should eql           "opt_args[arg1]=3"
-        # ScopeResource.var_args(1, 2).to_query.should eql        "var_args[ids][]=1&var_args[ids][]=2"
-        # ScopeResource.mix_args("a", {:opt1 => 1}, {:opt2 => 2}).to_query.should eql "mix_args[arg1]=a&mix_args[vararg][][opt1]=1&mix_arg[vararg][][opt2]=2"
-      end
-
-      it "should be able to change scopes" do
-        ap = Associations::MultiObjectProxy.new("TestResource", [{:service_uri => "/route", :scope1 => {"scope1" => true}, :scope2 => {"scope2" => true}}])
-        ap.scope1.should be_a(Associations::RelationScope)
-        ap.scope1.current_scope.scope1_scope?.should be_true
-      end
-
-      it "should be able to chain scope calls together" do
-        ap = Associations::MultiObjectProxy.new("TestResource", [{:service_uri => "/route", :scope1 => {"scope1" => true}, :scope2 => {"scope2" => true}}])
-        ap.scope1.scope2.current_scope.scope1_and_scope2_scope?.should be_true
-        ap.scope1.scope2.to_query.should eql("scope1=true&scope2=true")
-      end
-
-      it "should support scopes that contain underscores" do
-        ap = Associations::MultiObjectProxy.new("TestResource", [{:service_uri => "/route", :scope_1 => {"scope_1" => true}, :scope_2 => {"scope_2" => true}}])
-        ap.scope_1.scope_2.current_scope.scope_1_and_scope_2_scope?.should be_true
-      end
-
-      it "should be able to return the current query string" do
-        ap = Associations::MultiObjectProxy.new("TestResource", [{:service_uri => "/route", :scope_1 => {"scope_1" => true}, :scope_2 => {"scope_2" => true}}])
-        ap.scope_1.scope_2.to_query.should eql("scope_1=true&scope_2=true")
-      end
-
-      it "should be able to substitute values into the scope query strings by passing a hash to the methods" do
-        ap = Associations::MultiObjectProxy.new("TestResource", [{:service_uri => "/route", :scope_1 => {"scope_1" => true, :test_sub => false}, :scope_2 => {"scope_2" => true}}])
-        obj = ap.scope_1(:test_sub => true).scope_2
-        obj.to_query.should eql("scope_1=true&scope_2=true&test_sub=true")
+        ScopeResource.no_arg.to_query.should eql(
+          "no_arg=true"
+        )
+        ScopeResource.one_arg(5).to_query.should eql(
+          "one_arg[id]=5"
+        )
+        ScopeResource.one_array_arg([3, 5]).to_query.should eql(
+          "one_array_arg[ids][]=3&one_array_arg[ids][]=5"
+        )
+        ScopeResource.two_args(1, 20).to_query.should eql(
+          "two_args[page]=1&two_args[per_page]=20"
+        )
+        $DEB = true
+        ScopeResource.opt_args.to_query.should eql(
+          "opt_args=true"
+        )
+        ScopeResource.opt_args(3).to_query.should eql(
+          "opt_args[arg1]=3"
+        )
+        ScopeResource.var_args(1, 2).to_query.should eql(
+          "var_args[ids][]=1&var_args[ids][]=2"
+        )
+        args = ["a", {:opt1 => 1}, {:opt2 => 2}]
+        ScopeResource.mix_args(*args).to_query.should eql(
+          "mix_args[id]=a&mix_args[vararg][][opt1]=1&mix_args[vararg][][opt2]=2"
+        )
       end
     end
 
@@ -364,89 +428,96 @@ describe "Associations" do
       end
 
       it "should be able to force load an object" do
-        ap = Associations::SingleObjectProxy.new("TestResource",{:service_uri => '/single_object_association', :active => {:active => true}, :scopes_only => true})
-        ap.loaded.should be_blank
+        ap.should_not be_loaded
         name = ap.name
         name.should_not be_blank
-        ap.loaded.should_not be_blank
+        ap.should be_loaded
         # Make sure it isn't reloaded
         ap.name.should eql(name)
       end
 
-      it "should be able to load a scope" do
-        ap = Associations::SingleObjectProxy.new("TestResource",{:service_uri => '/single_object_association', :active => {:active => true}, :scopes_only => true})
-        ap.internal_object.active.should be_false
-        ap.times_loaded.should eql(1)
-        ap.active.should be_a Associations::RelationScope
-        ap.active.name.should_not be_blank
-        ap.times_loaded.should eql(2)
-        ap.active.internal_object.active.should be_true
-        # another check that the resource wasn't reloaded
-        ap.times_loaded.should eql(2)       
-      end
-
-      it "should be able to load a chain of scopes" do
-        ap = Associations::SingleObjectProxy.new("TestResource",{:service_uri => '/single_object_association', :active => {:active => true}, :with_birthday => {:birthday => true}, :scopes_only => true})
-        first = ap.active.with_birthday.id
-        ap.with_birthday.active.id.should eql(first)
-        ap.times_loaded.should eql(1)
-        ap.active.with_birthday.birthday.should_not be_blank
-      end
 
       it "should proxy unknown methods to the object loading if it hasn't already" do
-        ap = Associations::SingleObjectProxy.new("TestResource",{:service_uri => '/single_object_association', :active => {:active => false}, :with_birthday => {:birthday => true}, :scopes_only => true})
-        ap.times_loaded.should eql(0)
+        ap = Associations::SingleObjectProxy.new(
+          "TestResource", 
+          HasManyObject.new, {
+            :service_uri => '/single_object_association'
+          }
+        )
+        ap.should_not be_loaded
         ap.id.should_not be_blank
-        ap.times_loaded.should eql(1)
+        ap.should be_loaded
       end
 
       it "should load scopes with caching" do
-        ap = Associations::SingleObjectProxy.new("TestResource",{:service_uri => '/single_object_association', :active => {:active => true}, :scopes_only => true})
-        ap.times_loaded.should eql(0)
+        ap = Associations::SingleObjectProxy.new(
+          "TestResource", 
+          HasManyObject.new, {
+            :service_uri => '/single_object_association'
+          }
+        )
+        ap.should_not be_loaded
         ap.active.expires_in(30).internal_object
+
+        # should only be called once
+        TestResource.connection.expects(:get).never
         ap.active.expires_in(30).internal_object
-        ap.times_loaded.should eql(1)
+        ap.should be_loaded
       end
 
       it "should check that ttl matches the expiration parameter" do
-        ap = Associations::SingleObjectProxy.new("TestResource",{:service_uri => '/single_object_association', :active => {:active => true}, :scopes_only => true})
+       ap = Associations::SingleObjectProxy.new(
+          "TestResource", 
+          HasManyObject.new, {
+            :service_uri => '/single_object_association'
+          }
+        )
         ap.active.expires_in(10).ttl.should eql(10)
       end
 
       it "should cache scopes when caching enabled" do
-        ap = Associations::SingleObjectProxy.new("TestResource",{:service_uri => '/single_object_association', :active => {:active => false}, :with_birthday => {:birthday => true}, :scopes_only => true})
-        ApiResource.expects(:with_ttl).with(10)
-        ap.active(:active => true, :expires_in => 10).internal_object
-      end
-
-      it "should only load each distinct set of scopes once" do
-        ap = Associations::SingleObjectProxy.new("TestResource",{:service_uri => '/single_object_association', :active => {:active => false}, :with_birthday => {:birthday => true}, :scopes_only => true})
-        ap.times_loaded.should eql(0)
-        ap.active.with_birthday.internal_object
-        ap.active.with_birthday.internal_object
-        ap.with_birthday.active.internal_object
-        ap.times_loaded.should eql(1)
+        ap = Associations::SingleObjectProxy.new(
+          "TestResource", 
+          HasManyObject.new, {
+            :service_uri => '/single_object_association'
+          }
+        )
+        ap.active(:expires_in => 10).internal_object
       end
 
       it "should be able to clear it's loading cache" do
-        ap = Associations::SingleObjectProxy.new("TestResource",{:service_uri => '/single_object_association', :active => {:active => true}, :with_birthday => {:birthday => true}, :scopes_only => true})
-        ap.active.internal_object
-        ap.times_loaded.should eql(1)
-        ap.reload
-        ap.active.internal_object
-        ap.times_loaded.should eql(1)
+        ap = Associations::SingleObjectProxy.new(
+          "TestResource", 
+          HasManyObject.new, {
+            :service_uri => '/single_object_association'
+          }
+        )
+        active = ap.active
+
+        active.internal_object
+        active.should be_loaded
+        active.reload
+        active.should_not be_loaded
+        active.internal_object
+        active.should be_loaded
       end
 
     end
 
     it "should be able to reload a single-object association" do
-      ap = Associations::SingleObjectProxy.new("TestResource",{:service_uri => '/single_object_association', :active => {:active => true}, :scopes_only => true})
+      ap = Associations::SingleObjectProxy.new(
+        "TestResource", 
+        HasManyObject.new, {
+          :service_uri => '/single_object_association'
+        }
+      )
 
       old_name = ap.name
 
       str = "krdflkjsd"
 
       ap.name = str
+
       ap.name.should eql str
       ap.reload
 
@@ -454,7 +525,13 @@ describe "Associations" do
     end
 
     it "should be able to reload a multi-object association" do
-      ap = Associations::MultiObjectProxy.new("TestResource",{:service_uri => '/multi_object_association', :active => {:active => false}, :inactive => {:active => false}, :with_birthday => {:birthday => true}})
+      
+      ap = Associations::MultiObjectProxy.new(
+        "TestResource", 
+        BelongsToObject.new, {
+          :service_uri => '/multi_object_association'
+        }
+      )
 
       old_name = ap.first.name
 
@@ -471,7 +548,10 @@ describe "Associations" do
     it "should propagate the scopes from the associated class" do
 
       ap = Associations::MultiObjectProxy.new(
-        "TestResource", {:service_uri => "/multi_object_association"}
+        "TestResource", 
+        BelongsToObject.new, {
+          :service_uri => '/multi_object_association'
+        }
       )
       ap.scopes.should eql(TestResource.scopes)
       true
@@ -481,63 +561,68 @@ describe "Associations" do
     context "Multi Object" do
 
       it "should be able to load 'all'" do
-        ap = Associations::MultiObjectProxy.new("TestResource",{:service_uri => '/multi_object_association', :active => {:active => false}, :inactive => {:active => false}, :with_birthday => {:birthday => true}})
+        ap = Associations::MultiObjectProxy.new(
+          "TestResource", 
+          BelongsToObject.new, {
+            :service_uri => '/multi_object_association'
+          }
+        )
         results = ap.all
         results.size.should eql(5)
-        results.first.active.should be_false
+        results.first.is_active?.should be_false
       end
 
       it "should be able to load a scope" do
-        ap = Associations::MultiObjectProxy.new("TestResource",{:service_uri => '/multi_object_association', :active => {:active => false}, :inactive => {:active => false}, :with_birthday => {:birthday => true}})
-        results = ap.active.internal_object
+        ap = Associations::MultiObjectProxy.new(
+          "TestResource", 
+          BelongsToObject.new, {
+            :service_uri => '/multi_object_association'
+          }
+        )
+        results = ap.active
         results.size.should eql(5)
-        results.first.active.should be_true
+        record = results.first
+        record.is_active.should be_true
       end
 
       it "should be able to load a chain of scopes" do
-        ap = Associations::MultiObjectProxy.new("TestResource",{:service_uri => '/multi_object_association', :active => {:active => true}, :inactive => {:active => false}, :with_birthday => {:birthday => true}})
-        results = ap.active.with_birthday.internal_object
-        results.first.active.should be_true
-        results.first.birthday.should_not be_blank
+        ap = Associations::MultiObjectProxy.new(
+          "TestResource", 
+          BelongsToObject.new, {
+            :service_uri => '/multi_object_association'
+          }
+        )
+        results = ap.active.birthday(Date.today)
+        results.first.is_active.should be_true
+        results.first.bday.should_not be_blank
       end
 
-      it "should be able to load a chain of scopes with substitution" do
-        ap = Associations::MultiObjectProxy.new("TestResource",{:service_uri => '/multi_object_association', :active => {:active => true}, :inactive => {:active => false}, :with_birthday => {:birthday => true}})
-        results = ap.inactive(:active => true).with_birthday.internal_object
-        results.first.active.should be_true
-        results.first.birthday.should_not be_blank
-      end
 
-      it "should proxy unknown methods to the object array loading if it hasn't already" do
-        ap = Associations::MultiObjectProxy.new("TestResource",{:service_uri => '/multi_object_association', :active => {:active => true}, :inactive => {:active => false}, :with_birthday => {:birthday => true}})
-        ap.first.should be_a TestResource
-        ap.active.first.should be_a TestResource
-        ap.times_loaded.should eql(2)
-      end
+      it "should be able to clear it's loading cache" do
 
-      it "should only load each distinct set of scopes once" do
-        ap = Associations::MultiObjectProxy.new("TestResource",{:service_uri => '/multi_object_association', :active => {:active => true}, :inactive => {:active => false}, :with_birthday => {:birthday => true}})
-        ap.first
-        ap.active.first
-        ap.times_loaded.should eql(2)
-        ap.active.first
-        ap.times_loaded.should eql(2)
-        ap.active.with_birthday.first
-        ap.with_birthday.active.first
-        ap.times_loaded.should eql(3)
-      end
+        ap = Associations::MultiObjectProxy.new(
+          "TestResource", 
+          BelongsToObject.new, {
+            :service_uri => '/multi_object_association'
+          }
+        )
+        active = ap.active
 
-      it "should be able to clear it's loading cache" do
-        ap = Associations::MultiObjectProxy.new("TestResource",{:service_uri => '/multi_object_association', :active => {:active => true}, :inactive => {:active => false}, :with_birthday => {:birthday => true}})
-        ap.active.first
-        ap.times_loaded.should eql(1)
-        ap.reload
-        ap.active.first
-        ap.times_loaded.should eql(1)
+        active.internal_object
+        active.should be_loaded
+        active.reload
+        active.should_not be_loaded
+        active.internal_object
+        active.should be_loaded
       end
 
       it "should be enumerable" do
-        ap = Associations::MultiObjectProxy.new("TestResource",{:service_uri => '/multi_object_association', :active => {:active => true}, :inactive => {:active => false}, :with_birthday => {:birthday => true}})
+        ap = Associations::MultiObjectProxy.new(
+          "TestResource", 
+          BelongsToObject.new, {
+            :service_uri => '/multi_object_association'
+          }
+        )
         ap.each do |tr|
           tr.name.should_not be_blank
         end
@@ -562,9 +647,11 @@ describe "Associations" do
       end
 
       it "should be able to chain scopes" do
-        scp = TestResource.active.paginate
-        scp.should be_a Associations::ResourceScope
-        scp.to_query.should eql("active=true&paginate=true")
+        scp = TestResource.active.paginate(20, 1)
+        scp.should be_a Associations::Scope
+        scp.to_query.should eql(
+          "active=true&paginate[current_page]=1&paginate[per_page]=20"
+        )
       end
 
       it "should load when calling all" do
@@ -718,7 +805,9 @@ describe "Associations" do
           it "should attempt to load a single remote object for a has_one relationship" do
             tar = TestAR.new
             tar.stubs(:id).returns(1)
-            TestResource.connection.expects(:get).with("/test_resources.json?test_ar_id=1").once.returns([{"name" => "testing"}])
+            TestResource.connection.expects(:get)
+              .with("/test_resources.json?test_ar_id=1")
+              .returns([{"name" => "testing"}])
             # load the test resource
             tar.test_resource.name.should eql "testing"
           end

data/spec/lib/attributes_spec.rb

@@ -3,11 +3,39 @@ require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
 include ApiResource
 
 describe "Attributes" do
+
+  before(:all) do
+    TestResource.reload_class_attributes
+  end
   
   after(:all) do
     TestResource.reload_class_attributes
   end
-  
+
+  context "setters" do
+
+    it "should allow setting of protected attributes individually" do
+      test_resource = TestResource.new
+      test_resource.protected_attr = 100
+      test_resource.protected_attr.should eql(100)
+    end
+
+    it "should not allow mass assignment of protected attributes" do
+      test_resource = TestResource.new
+      lambda{
+        test_resource.attributes = {:protected_attr => 100}  
+      }.should raise_error
+    end
+
+    it "should not allow mass assignment in the constructor" do
+      lambda{
+        TestResource.new({:protected_attr => 100})
+      }.should raise_error
+    end
+
+  end
+
+
   context "Defining, getting, and setting attributes" do
     it "should be able to define known attributes" do
       TestResource.define_attributes :attr1, :attr2
@@ -76,12 +104,12 @@ describe "Attributes" do
       end
     
       it "should create protected attributes for unknown attributes trying to be loaded" do
-        tst = TestResource.new({:attr1 => "attr1", :attr3 => "attr3"})
+        
+        TestResource.connection.stubs(:get => {:attr1 => "attr1", :attr3 => "attr3"})
+        tst = TestResource.find(1)
+
         tst.attr3?.should be_true
         tst.attr3.should eql("attr3")
-        lambda {
-          tst.attr3 = "test"
-        }.should raise_error
       end
     end
 
@@ -116,16 +144,6 @@ describe "Attributes" do
     
   end
   
-  context "Protected attributes" do
-    it "should allow protected attributes that cannot be changed" do
-      TestResource.define_protected_attributes :pattr3
-      lambda {
-        tst = TestResource.new
-        tst.pattr3 = "test"
-      }.should raise_error
-    end
-  end
-  
   context "Dirty tracking" do
     context "Changes to attributes" do
       it "should implement dirty tracking for attributes" do