datts_right 0.0.19 → 0.0.20

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    datts_right (0.0.18)
+    datts_right (0.0.19)
       datts_right
       rails (>= 3.0.0)
 

data/VERSION

@@ -1 +1 @@
-0.0.19
+0.0.20

data/datts_right.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{datts_right}
-  s.version = "0.0.19"
+  s.version = "0.0.20"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Ramon Tayag"]
-  s.date = %q{2011-04-19}
+  s.date = %q{2011-04-26}
   s.description = %q{Creates a separate table that saves all your dynamic attributes.}
   s.email = %q{ramon@tayag.net}
   s.extra_rdoc_files = [
@@ -33,17 +33,25 @@ Gem::Specification.new do |s|
     "lib/datts_right/category.rb",
     "lib/datts_right/dynamic_attribute.rb",
     "lib/datts_right/dynamic_attribute_definition.rb",
+    "lib/datts_right/errors.rb",
     "lib/datts_right/exceptions.rb",
     "lib/datts_right/instance_methods.rb",
     "lib/datts_right/page.rb",
     "lib/datts_right/query_methods.rb",
     "spec/datt_spec.rb",
+    "spec/datts_right/add_definition_spec.rb",
+    "spec/datts_right/add_dynamic_attribute_spec.rb",
     "spec/datts_right/add_dynamic_attributes_spec.rb",
     "spec/datts_right/attributes_spec.rb",
     "spec/datts_right/create_definitions_spec.rb",
+    "spec/datts_right/definition_spec.rb",
     "spec/datts_right/dynamic_attribute_definition_spec.rb",
+    "spec/datts_right/find_by_dynamic_attribute_spec.rb",
     "spec/datts_right/inheritance_spec.rb",
+    "spec/datts_right/read_dynamic_attribute_spec.rb",
+    "spec/datts_right/remove_definition_spec.rb",
     "spec/datts_right/remove_dynamic_attribute_spec.rb",
+    "spec/datts_right/where_dynamic_attribute_spec.rb",
     "spec/datts_right_spec.rb",
     "spec/spec_helper.rb"
   ]
@@ -54,12 +62,19 @@ Gem::Specification.new do |s|
   s.summary = %q{Allows saving of dynamic attributes in your ActiveRecord model}
   s.test_files = [
     "spec/datt_spec.rb",
+    "spec/datts_right/add_definition_spec.rb",
+    "spec/datts_right/add_dynamic_attribute_spec.rb",
     "spec/datts_right/add_dynamic_attributes_spec.rb",
     "spec/datts_right/attributes_spec.rb",
     "spec/datts_right/create_definitions_spec.rb",
+    "spec/datts_right/definition_spec.rb",
     "spec/datts_right/dynamic_attribute_definition_spec.rb",
+    "spec/datts_right/find_by_dynamic_attribute_spec.rb",
     "spec/datts_right/inheritance_spec.rb",
+    "spec/datts_right/read_dynamic_attribute_spec.rb",
+    "spec/datts_right/remove_definition_spec.rb",
     "spec/datts_right/remove_dynamic_attribute_spec.rb",
+    "spec/datts_right/where_dynamic_attribute_spec.rb",
     "spec/datts_right_spec.rb",
     "spec/spec_helper.rb"
   ]
@@ -275,6 +290,12 @@ Gem::Specification.new do |s|
       s.add_development_dependency(%q<bundler>, ["~> 1.0.0"])
       s.add_development_dependency(%q<jeweler>, ["~> 1.5.2"])
       s.add_development_dependency(%q<rcov>, [">= 0"])
+      s.add_development_dependency(%q<autotest>, [">= 0"])
+      s.add_development_dependency(%q<sqlite3>, [">= 0"])
+      s.add_development_dependency(%q<rspec>, ["~> 2.3.0"])
+      s.add_development_dependency(%q<bundler>, ["~> 1.0.0"])
+      s.add_development_dependency(%q<jeweler>, ["~> 1.5.2"])
+      s.add_development_dependency(%q<rcov>, [">= 0"])
     else
       s.add_dependency(%q<datts_right>, [">= 0"])
       s.add_dependency(%q<rails>, [">= 3.0.0"])
@@ -483,6 +504,12 @@ Gem::Specification.new do |s|
       s.add_dependency(%q<bundler>, ["~> 1.0.0"])
       s.add_dependency(%q<jeweler>, ["~> 1.5.2"])
       s.add_dependency(%q<rcov>, [">= 0"])
+      s.add_dependency(%q<autotest>, [">= 0"])
+      s.add_dependency(%q<sqlite3>, [">= 0"])
+      s.add_dependency(%q<rspec>, ["~> 2.3.0"])
+      s.add_dependency(%q<bundler>, ["~> 1.0.0"])
+      s.add_dependency(%q<jeweler>, ["~> 1.5.2"])
+      s.add_dependency(%q<rcov>, [">= 0"])
     end
   else
     s.add_dependency(%q<datts_right>, [">= 0"])
@@ -692,6 +719,12 @@ Gem::Specification.new do |s|
     s.add_dependency(%q<bundler>, ["~> 1.0.0"])
     s.add_dependency(%q<jeweler>, ["~> 1.5.2"])
     s.add_dependency(%q<rcov>, [">= 0"])
+    s.add_dependency(%q<autotest>, [">= 0"])
+    s.add_dependency(%q<sqlite3>, [">= 0"])
+    s.add_dependency(%q<rspec>, ["~> 2.3.0"])
+    s.add_dependency(%q<bundler>, ["~> 1.0.0"])
+    s.add_dependency(%q<jeweler>, ["~> 1.5.2"])
+    s.add_dependency(%q<rcov>, [">= 0"])
   end
 end
 

data/lib/datts_right.rb

@@ -2,5 +2,6 @@ require 'datts_right/base'
 require 'datts_right/instance_methods'
 require 'datts_right/dynamic_attribute'
 require 'datts_right/dynamic_attribute_definition'
+require 'datts_right/errors'
 
 ActiveRecord::Base.extend DattsRight::Base

data/lib/datts_right/errors.rb

@@ -0,0 +1,13 @@
+module DattsRight
+  class DattsRightError < StandardError
+  end
+
+  class NoDefinitionError < DattsRightError
+  end
+
+  class AlreadyDefined < DattsRightError
+  end
+
+  class NotDefinedError < DattsRightError
+  end
+end

data/lib/datts_right/instance_methods.rb

@@ -119,6 +119,32 @@ module DattsRight
       end
     end
 
+    def add_definition(key, value)
+      key = key.to_sym
+      if dynamic_attributes_options[:definition]
+        if definition[key]
+          raise AlreadyDefined, "#{key} is already defined"
+        else
+          definition.merge!({key => value})
+        end
+      else
+        raise NoDefinitionError
+      end
+    end
+
+    def remove_definition(key)
+      key = key.to_sym
+      if dynamic_attributes_options[:definition]
+        if definition && definition[key]
+          definition.delete(key)
+        else
+          raise NotDefinedError, "#{key} is not defined"
+        end
+      else
+        raise NoDefinitionError
+      end
+    end
+
     private
 
     # Called after validation on update so that dynamic attributes behave

data/spec/datts_right/add_definition_spec.rb

@@ -0,0 +1,27 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+describe DattsRight, ".add_definition(key, value)" do
+  before do
+    reset_database
+  end
+  
+  it "should add a definition to the existing definition" do
+    c = Category.create
+    c.definition = {:robot => {:object_type => "string"}}
+    c.save
+    c.add_definition(:name, :object_type => "string", :description => "This is great")
+    c.save
+    c.definition[:name].should == {:object_type => "string", :description => "This is great"}
+  end
+
+  it "should raise DattsRight::AttributeAlreadyExists if the attribute is already there" do
+    c = Category.create
+    c.definition = {:robot => {:object_type => "string"}}
+    c.save
+    lambda {c.add_definition(:robot, "whatever")}.should raise_error(DattsRight::AlreadyDefined, "robot is already defined")
+  end
+
+  it "should raise NoMethodError if it doesn't have definition => true" do
+    lambda {Page.create.add_definition(:fake, :object_type => "string")}.should raise_error(DattsRight::NoDefinitionError)
+  end
+end

data/spec/datts_right/add_dynamic_attribute_spec.rb

@@ -0,0 +1,26 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+describe DattsRight, ".remove_dynamic_attribute(attr_key)" do
+  before do
+    reset_database
+    @page = Page.create
+  end
+
+  it "should be aliased by remove_datt" do
+    Page.instance_method(:remove_dynamic_attribute).should == Page.instance_method(:remove_datt)
+  end
+
+  it "should remove the attribute completely" do
+    @page.add_dynamic_attribute(:rocks, "string")
+    @page.remove_dynamic_attribute(:rocks)
+    lambda { @page.rocks }.should raise_error(NoMethodError)
+  end
+
+  it "should not explode if the attribute being removed isn't there" do
+    @page.add_dynamic_attribute(:rocks, "string")
+    @page.remove_dynamic_attribute(:rocks)
+    lambda {
+      @page.remove_dynamic_attribute(:rocks)
+    }.should_not raise_error(NoMethodError)
+  end
+end

data/spec/datts_right/definition_spec.rb

@@ -0,0 +1,7 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+describe DattsRight, "on the definition" do
+  it "should not be created if it is not defined" do
+    Page.create.dynamic_attribute_definition.should be_nil
+  end
+end

data/spec/datts_right/find_by_dynamic_attribute_spec.rb

@@ -0,0 +1,91 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+describe DattsRight, "on dynamic find_by_dynamic_attribute methods" do
+  before do
+    reset_database
+    @page = Page.create
+  end
+
+  it "should be aliased by find_by_datt" do
+    @page.add_dynamic_attribute(:price, "integer")
+    @page.write_dynamic_attribute :price, 400
+    @page.save
+    Page.find_by_datt_price(400).should == Page.find_by_dynamic_attribute_price(400)
+  end
+
+  it "should be able to return single records" do
+    @page.add_dynamic_attribute(:price, "integer")
+    @page.write_dynamic_attribute :price, 400
+    @page.save
+
+    @page_2 = Page.create
+    @page_2.add_dynamic_attribute(:price, "integer")
+    @page_2.write_dynamic_attribute :price, 500
+    @page_2.save
+
+    @results = Page.find_by_dynamic_attribute_price(400)
+    @results.should == @page
+    @results.should_not == @page_2
+  end
+
+  it "should be able to return all matching records" do
+    @page.add_dynamic_attribute(:price, "integer")
+    @page.write_dynamic_attribute :price, 400
+    @page.save
+
+    @page_2 = Page.create
+    @page_2.add_dynamic_attribute(:price, "integer")
+    @page_2.write_dynamic_attribute :price, 500
+    @page_2.save
+
+    @page_3 = Page.create
+    @page_3.add_dynamic_attribute(:price, "integer")
+    @page_3.write_dynamic_attribute :price, 400
+    @page_3.save
+
+    @results = Page.find_all_by_dynamic_attribute_price(400)
+    @results.should include(@page)
+    @results.should_not include(@page_2)
+    @results.should include(@page_3)
+  end
+
+  it "should still allow raising of NoMethodError if the original attribute does not exist" do
+    lambda { Page.find_by_faker("hi") }.should raise_error(NoMethodError)
+  end
+
+  it "should be able to return the last record" do
+    @page.add_dynamic_attribute(:price, "integer")
+    @page.write_dynamic_attribute :price, 400
+    @page.save
+
+    @page_2 = Page.create
+    @page_2.add_dynamic_attribute(:price, "integer")
+    @page_2.write_dynamic_attribute :price, 400
+    @page_2.save
+
+    Page.find_last_by_dynamic_attribute_price(400) == @page_2
+  end
+
+  it "should allow normal attributes to work" do
+    @page.name = "Roland"
+    @page.save
+
+    Page.find_by_name("Roland").should == @page
+  end
+
+  it "should allow chaining" do
+    @page.add_dynamic_attribute(:price, "integer")
+    @page.name = "fixed"
+    @page.write_dynamic_attribute :price, 400
+    @page.save
+
+    @page_2 = Page.create
+    @page_2.add_dynamic_attribute(:price, "integer")
+    @page_2.write_dynamic_attribute :price, 500
+    @page_2.save
+
+    @pages = Page.where(:name => "fixed").find_by_dynamic_attribute_price(400)
+    @pages.should == @page
+    @pages.should_not == @page_2
+  end
+end

data/spec/datts_right/inheritance_spec.rb

@@ -2,11 +2,13 @@ require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
 
 describe DattsRight, "when inheriting the definition of an association" do
   before do
+    reset_database
     @category = Category.create
     @category.definition = {:teaser => {:object_type => "text"}, :price => {:object_type => "integer"}}
     @category.save
     @page = Page.create :category => @category
   end
+
   it "should create the dynamic attributes" do
     @page.teaser = "hi"
     @page.teaser.should == "hi"

data/spec/datts_right/read_dynamic_attribute_spec.rb

@@ -0,0 +1,7 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+describe DattsRight, ".read_dynamic_attribute" do
+  it "should be aliased by read_datt" do
+    Page.instance_method(:read_dynamic_attribute).should == Page.instance_method(:read_datt)
+  end
+end

data/spec/datts_right/remove_definition_spec.rb

@@ -0,0 +1,23 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+describe DattsRight, ".remove_definition(key, value)" do
+  before do
+    reset_database
+  end
+  
+  it "should remove a definition from the existing definition" do
+    c = Category.create
+    c.definition = {:robot => {:object_type => "string"}}
+    c.save
+    c.remove_definition(:robot)
+    c.definition[:robot].should be_nil
+  end
+
+  it "should raise NotDefinedError if it doesn't exist" do
+    lambda {Category.create.remove_definition(:robot)}.should raise_error(DattsRight::NotDefinedError, "robot is not defined")
+  end
+
+  it "should raise NoDefinitionError if it doesn't have definition => true" do
+    lambda {Page.create.remove_definition(:fake)}.should raise_error(DattsRight::NoDefinitionError)
+  end
+end

data/spec/datts_right/where_dynamic_attribute_spec.rb

@@ -0,0 +1,53 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+describe DattsRight, ".where_dynamic_attribute" do
+  before do
+    reset_database
+    @page = Page.create
+  end
+
+  it "should be aliased by where_datt" do
+    @page.add_dynamic_attribute(:price, "float")
+    @page.write_dynamic_attribute :price, 200.0
+    Page.where_dynamic_attribute(:price => 200.0).should == Page.where_datt(:price => 200.0)
+  end
+
+  it "should automatically join in the dynamic_attributes table" do
+    @page.add_dynamic_attribute(:price, "float")
+    @page.write_dynamic_attribute :price, 200.0
+    @page.add_dynamic_attribute(:farce, "boolean")
+    @page.write_dynamic_attribute :farce, true
+    @page.save
+    
+    @page_2 = Page.create
+    @page_2.add_dynamic_attribute(:price, "integer")
+    @page_2.add_dynamic_attribute(:farce, "boolean")
+    @page_2.write_dynamic_attribute :farce, true
+    @page_2.write_dynamic_attribute :price, 1
+    @page_2.save
+
+    @result = Page.where_dynamic_attribute(:price => 200.0, :farce => true)
+    @result.should include(@page)
+    @result.should_not include(@page_2)
+  end
+
+  it "should allow chaining with where scopes" do
+    @page.add_dynamic_attribute(:price, "integer")
+    @page.name = "aardvark"
+    @page.write_dynamic_attribute :price, 200
+    @page.save
+    
+    @page_2 = Page.create
+    @page_2.add_dynamic_attribute(:price, "integer")
+    @page_2.write_dynamic_attribute :price, 200
+    @page_2.save
+
+    @results = Page.where_dynamic_attribute(:price => 200).where(:name => "aardvark")
+    @results.should include(@page)
+    @results.should_not include(@page_2)
+  end
+
+  it "should return an empty array if we're looking for a field that doesn't exist" do
+    Page.where_dynamic_attribute(:bogus_field => "none").should be_empty
+  end
+end

data/spec/datts_right_spec.rb

@@ -1,7 +1,4 @@
 require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
-# Raised when unknown attributes are supplied via mass assignment.
-#class UnknownAttributeError < NoMethodError
-#end
 
 describe DattsRight do
   before do
@@ -9,56 +6,6 @@ describe DattsRight do
     @page = Page.create
   end
 
-  describe ".add_dynamic_attribute(attr_key, object_type)" do
-    it "should be aliased by add_datt" do
-      Page.instance_method(:add_dynamic_attribute).should == Page.instance_method(:add_datt)
-    end
-
-    it "should add a dynamic attribute" do
-      @page.add_dynamic_attribute(:rocks, "string")
-      @page.dynamic_attribute_details(:rocks).value.should be_nil
-      @page.add_dynamic_attribute(:rock, "string", "123")
-      @page.read_datt(:rock).should == "123"
-      @page.dynamic_attribute_details(:rock).value.should == "123"
-    end
-
-    it "should ignore when trying to add same attribute" do
-      @page.add_dynamic_attribute(:rocks, "string")
-      @page.add_dynamic_attribute(:rocks, "integer")
-      @page.dynamic_attribute_details(:rocks).object_type.should == "string"
-    end
-
-    it "should return false if the method that is being added already exists" do
-      @page.add_dynamic_attribute(:name, "text").should be_false
-    end
-
-    it "should not make any changes to the original attribute" do
-      @page.update_attribute(:name, "juno")
-      @page.add_dynamic_attribute(:name, "text")
-      @page.name.should == "juno"
-    end
-  end
-
-  describe ".remove_dynamic_attribute(attr_key)" do
-    it "should be aliased by remove_datt" do
-      Page.instance_method(:remove_dynamic_attribute).should == Page.instance_method(:remove_datt)
-    end
-
-    it "should remove the attribute completely" do
-      @page.add_dynamic_attribute(:rocks, "string")
-      @page.remove_dynamic_attribute(:rocks)
-      lambda { @page.rocks }.should raise_error(NoMethodError)
-    end
-
-    it "should not explode if the attribute being removed isn't there" do
-      @page.add_dynamic_attribute(:rocks, "string")
-      @page.remove_dynamic_attribute(:rocks)
-      lambda {
-        @page.remove_dynamic_attribute(:rocks)
-      }.should_not raise_error(NoMethodError)
-    end
-  end
-
   it "should not allow arbitrary creation of attributes" do
     lambda {
       @page.some_field = "woot"
@@ -145,97 +92,6 @@ describe DattsRight do
     @page.dynamic_attribute?(:farce).should be_true
   end
 
-  describe ".read_dynamic_attribute" do
-    it "should be aliased by read_datt" do
-      Page.instance_method(:read_dynamic_attribute).should == Page.instance_method(:read_datt)
-    end
-  end
-
-  describe "on dynamic find_by_dynamic_attribute methods" do
-    it "should be aliased by find_by_datt" do
-      @page.add_dynamic_attribute(:price, "integer")
-      @page.write_dynamic_attribute :price, 400
-      @page.save
-      Page.find_by_datt_price(400).should == Page.find_by_dynamic_attribute_price(400)
-    end
-
-    it "should be able to return single records" do
-      @page.add_dynamic_attribute(:price, "integer")
-      @page.write_dynamic_attribute :price, 400
-      @page.save
-
-      @page_2 = Page.create
-      @page_2.add_dynamic_attribute(:price, "integer")
-      @page_2.write_dynamic_attribute :price, 500
-      @page_2.save
-
-      @results = Page.find_by_dynamic_attribute_price(400)
-      @results.should == @page
-      @results.should_not == @page_2
-    end
-
-    it "should be able to return all matching records" do
-      @page.add_dynamic_attribute(:price, "integer")
-      @page.write_dynamic_attribute :price, 400
-      @page.save
-
-      @page_2 = Page.create
-      @page_2.add_dynamic_attribute(:price, "integer")
-      @page_2.write_dynamic_attribute :price, 500
-      @page_2.save
-
-      @page_3 = Page.create
-      @page_3.add_dynamic_attribute(:price, "integer")
-      @page_3.write_dynamic_attribute :price, 400
-      @page_3.save
-
-      @results = Page.find_all_by_dynamic_attribute_price(400)
-      @results.should include(@page)
-      @results.should_not include(@page_2)
-      @results.should include(@page_3)
-    end
-
-    it "should still allow raising of NoMethodError if the original attribute does not exist" do
-      lambda { Page.find_by_faker("hi") }.should raise_error(NoMethodError)
-    end
-
-    it "should be able to return the last record" do
-      @page.add_dynamic_attribute(:price, "integer")
-      @page.write_dynamic_attribute :price, 400
-      @page.save
-
-      @page_2 = Page.create
-      @page_2.add_dynamic_attribute(:price, "integer")
-      @page_2.write_dynamic_attribute :price, 400
-      @page_2.save
-
-      Page.find_last_by_dynamic_attribute_price(400) == @page_2
-    end
-
-    it "should allow normal attributes to work" do
-      @page.name = "Roland"
-      @page.save
-
-      Page.find_by_name("Roland").should == @page
-    end
-
-    it "should allow chaining" do
-      @page.add_dynamic_attribute(:price, "integer")
-      @page.name = "fixed"
-      @page.write_dynamic_attribute :price, 400
-      @page.save
-
-      @page_2 = Page.create
-      @page_2.add_dynamic_attribute(:price, "integer")
-      @page_2.write_dynamic_attribute :price, 500
-      @page_2.save
-
-      @pages = Page.where(:name => "fixed").find_by_dynamic_attribute_price(400)
-      @pages.should == @page
-      @pages.should_not == @page_2
-    end
-  end
-
   it "should allow multiple attributes" do
     @page.add_dynamic_attribute(:price, "integer")
     @page.write_dynamic_attribute :price, 400
@@ -345,57 +201,6 @@ describe DattsRight do
     end
   end
 
-  describe "where_dynamic_attribute" do
-    it "should be aliased by where_datt" do
-      @page.add_dynamic_attribute(:price, "float")
-      @page.write_dynamic_attribute :price, 200.0
-      Page.where_dynamic_attribute(:price => 200.0).should == Page.where_datt(:price => 200.0)
-    end
-
-    it "should automatically join in the dynamic_attributes table" do
-      @page.add_dynamic_attribute(:price, "float")
-      @page.write_dynamic_attribute :price, 200.0
-      @page.add_dynamic_attribute(:farce, "boolean")
-      @page.write_dynamic_attribute :farce, true
-      @page.save
-      
-      @page_2 = Page.create
-      @page_2.add_dynamic_attribute(:price, "integer")
-      @page_2.add_dynamic_attribute(:farce, "boolean")
-      @page_2.write_dynamic_attribute :farce, true
-      @page_2.write_dynamic_attribute :price, 1
-      @page_2.save
-
-      @result = Page.where_dynamic_attribute(:price => 200.0, :farce => true)
-      @result.should include(@page)
-      @result.should_not include(@page_2)
-
-      #@result = Page.where_dynamic_attribute("price < :price", :price => 199.0)
-      #@result.should_not include(@page)
-      #@result.should include(@page_2)
-    end
-
-    it "should allow chaining with where scopes" do
-      @page.add_dynamic_attribute(:price, "integer")
-      @page.name = "aardvark"
-      @page.write_dynamic_attribute :price, 200
-      @page.save
-      
-      @page_2 = Page.create
-      @page_2.add_dynamic_attribute(:price, "integer")
-      @page_2.write_dynamic_attribute :price, 200
-      @page_2.save
-
-      @results = Page.where_dynamic_attribute(:price => 200).where(:name => "aardvark")
-      @results.should include(@page)
-      @results.should_not include(@page_2)
-    end
-
-    it "should return an empty array if we're looking for a field that doesn't exist" do
-      Page.where_dynamic_attribute(:bogus_field => "none").should be_empty
-    end
-  end
-
   describe ".dynamic_attribute_details(key)" do
     it "should be aliased by datt_details" do
       Page.instance_method(:dynamic_attribute_details).should == Page.instance_method(:datt_details)
@@ -438,25 +243,4 @@ describe DattsRight do
     @page.save
     @page.body.should == "dude"
   end
-
-  describe ".attributes=" do
-    it "should accept a mix of normal and dynamic attributes" do
-      @page.add_dynamic_attribute(:price, "integer", 200)
-      @page.attributes = {:name => "fark", :price => 300}
-      @page.name.should == "fark"
-      @page.read_datt(:price).should == 300
-    end
-
-    it "should raise UnknownAttributeError if the attribute doesn't exist" do
-      lambda {
-        @page.attributes = {:name => "fark", :body => "lark"}
-      }.should raise_error(ActiveRecord::UnknownAttributeError, "unknown attribute: body")
-    end
-  end
-
-  describe "on the definition" do
-    it "should not be created if it is not defined" do
-      Page.create.dynamic_attribute_definition.should be_nil
-    end
-  end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: datts_right
 version: !ruby/object:Gem::Version 
-  hash: 57
+  hash: 55
   prerelease: 
   segments: 
   - 0
   - 0
-  - 19
-  version: 0.0.19
+  - 20
+  version: 0.0.20
 platform: ruby
 authors: 
 - Ramon Tayag
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-04-19 00:00:00 +08:00
+date: 2011-04-26 00:00:00 +08:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -3124,6 +3124,96 @@ dependencies:
   name: rcov
   version_requirements: *id207
   prerelease: false
+- !ruby/object:Gem::Dependency 
+  type: :development
+  requirement: &id208 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  name: autotest
+  version_requirements: *id208
+  prerelease: false
+- !ruby/object:Gem::Dependency 
+  type: :development
+  requirement: &id209 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  name: sqlite3
+  version_requirements: *id209
+  prerelease: false
+- !ruby/object:Gem::Dependency 
+  type: :development
+  requirement: &id210 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ~>
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 2
+        - 3
+        - 0
+        version: 2.3.0
+  name: rspec
+  version_requirements: *id210
+  prerelease: false
+- !ruby/object:Gem::Dependency 
+  type: :development
+  requirement: &id211 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ~>
+      - !ruby/object:Gem::Version 
+        hash: 23
+        segments: 
+        - 1
+        - 0
+        - 0
+        version: 1.0.0
+  name: bundler
+  version_requirements: *id211
+  prerelease: false
+- !ruby/object:Gem::Dependency 
+  type: :development
+  requirement: &id212 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ~>
+      - !ruby/object:Gem::Version 
+        hash: 7
+        segments: 
+        - 1
+        - 5
+        - 2
+        version: 1.5.2
+  name: jeweler
+  version_requirements: *id212
+  prerelease: false
+- !ruby/object:Gem::Dependency 
+  type: :development
+  requirement: &id213 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  name: rcov
+  version_requirements: *id213
+  prerelease: false
 description: Creates a separate table that saves all your dynamic attributes.
 email: ramon@tayag.net
 executables: []
@@ -3150,17 +3240,25 @@ files:
 - lib/datts_right/category.rb
 - lib/datts_right/dynamic_attribute.rb
 - lib/datts_right/dynamic_attribute_definition.rb
+- lib/datts_right/errors.rb
 - lib/datts_right/exceptions.rb
 - lib/datts_right/instance_methods.rb
 - lib/datts_right/page.rb
 - lib/datts_right/query_methods.rb
 - spec/datt_spec.rb
+- spec/datts_right/add_definition_spec.rb
+- spec/datts_right/add_dynamic_attribute_spec.rb
 - spec/datts_right/add_dynamic_attributes_spec.rb
 - spec/datts_right/attributes_spec.rb
 - spec/datts_right/create_definitions_spec.rb
+- spec/datts_right/definition_spec.rb
 - spec/datts_right/dynamic_attribute_definition_spec.rb
+- spec/datts_right/find_by_dynamic_attribute_spec.rb
 - spec/datts_right/inheritance_spec.rb
+- spec/datts_right/read_dynamic_attribute_spec.rb
+- spec/datts_right/remove_definition_spec.rb
 - spec/datts_right/remove_dynamic_attribute_spec.rb
+- spec/datts_right/where_dynamic_attribute_spec.rb
 - spec/datts_right_spec.rb
 - spec/spec_helper.rb
 has_rdoc: true
@@ -3199,11 +3297,18 @@ specification_version: 3
 summary: Allows saving of dynamic attributes in your ActiveRecord model
 test_files: 
 - spec/datt_spec.rb
+- spec/datts_right/add_definition_spec.rb
+- spec/datts_right/add_dynamic_attribute_spec.rb
 - spec/datts_right/add_dynamic_attributes_spec.rb
 - spec/datts_right/attributes_spec.rb
 - spec/datts_right/create_definitions_spec.rb
+- spec/datts_right/definition_spec.rb
 - spec/datts_right/dynamic_attribute_definition_spec.rb
+- spec/datts_right/find_by_dynamic_attribute_spec.rb
 - spec/datts_right/inheritance_spec.rb
+- spec/datts_right/read_dynamic_attribute_spec.rb
+- spec/datts_right/remove_definition_spec.rb
 - spec/datts_right/remove_dynamic_attribute_spec.rb
+- spec/datts_right/where_dynamic_attribute_spec.rb
 - spec/datts_right_spec.rb
 - spec/spec_helper.rb