tiny_ds 0.0.2 → 0.0.3.pre

data/README.rdoc

@@ -1,25 +1,28 @@
-* TinyDS -- tiny datastore library for GAE/JRuby
-  github: http://github.com/takeru/tiny_ds
+= TinyDS -- tiny datastore library for GAE/JRuby
+github: http://github.com/takeru/tiny_ds
 
- - CRUD like a ActiveRecord or DataMapepr
- - set parent, key, id, name, and find by these, YES entity-group-transaction!!
- - BASE transaction : http://blog.notdot.net/2009/9/Distributed-Transactions-on-App-Engine
- - query : very thin wrapper of low level API.
- - Shout demo app. (sinatra)
- - some specs..
+- CRUD like a ActiveRecord or DataMapepr
+- set parent, key, id, name, and find by these, YES entity-group-transaction!!
+- BASE transaction : http://blog.notdot.net/2009/9/Distributed-Transactions-on-App-Engine
+- query : very thin wrapper of low level API.
+- Shout demo app. (sinatra)
+- some specs..
   
-** run shout demo
+== run example
+ % examples/basic
  % dev_appserver.rb .
 
-** how to run specs
+== how to run specs
 install jruby 1.4.0
- % source set_classpath_for_jruby.sh (set classpath, please read spec/spec_helper.rb)
+ % cd tiny_ds
+ % source examples/basic/set_classpath_for_jruby.sh
+   (set classpath, please read spec/spec_helper.rb)
  % jruby -S spec -c -b spec/basic_spec.rb
-   or 
+   or
  % jruby -e "ENV['RSPEC']='true'; ENV['AUTOTEST']='true'; system('jruby -S autotest', *ARGV)"
 
-** demo
+== demo site
 Rubyist Social Graph : http://rubyist-sg.appspot.com/ is running with GAE/JRuby+Rails2.3.5+TinyDS.
 
-** author
-  takeru : sasaki.takeru@gmail.com, twitter:@urekat (almost japanese!), blog:http://d.hatena.ne.jp/urekat
+== author
+takeru: sasaki.takeru@gmail.com, twitter:@urekat (almost japanese!), blog:http://d.hatena.ne.jp/urekat

data/Rakefile

@@ -28,7 +28,7 @@ spec = Gem::Specification.new do |s|
   s.homepage = "http://github.com/takeru/tiny_ds"
   s.require_path = 'lib'
   s.files = %w(LICENSE README.rdoc Rakefile) +
-            Dir.glob("spec/**/*") + Dir.glob("lib/**/*") 
+            Dir.glob("spec/**/*.rb") + Dir.glob("lib/**/*.rb")
   s.add_dependency('appengine-apis')
 end
 

data/lib/tiny_ds/base.rb

@@ -1,31 +1,10 @@
-# TODO
-#   callback before/after
-#   validations
-#   associations
-#x   default value
-#x   property list
-#   property Key
-#   property others
-#   nil=>false
-#   index=>false setUnindexedProperty
-#   version
-#   schema_version
-#x   new_record?
-#   find_by_xxx
-#x   get_by_id(id)         get_by_name(name)         : root entity only!!
-#x   get_by_id(parent, id) get_by_name(parent, name) : ancestor+Kind => key get
-#x   Foo.query.filter(:key=>123)
-#   Foo.filter(:key=>123)
-#   logger
-#   logging low level API calls.
-
 require "time"
 
 module TinyDS
 class Base
   class << self; attr_accessor :_property_definitions; end
   RESERVED_PROPERTY_NAME = [:id, :name, :key, :entity, :parent_key, :parent]
-  VALID_PROPERTY_TYPE = [:string, :integer, :text, :time, :list]
+  VALID_PROPERTY_TYPE = [:string, :integer, :float, :boolean, :text, :time, :list]
   def self.property(pname, ptype, opts={})
     pname = pname.to_sym
     if RESERVED_PROPERTY_NAME.include?(pname)
@@ -42,6 +21,10 @@ class Base
     property_definitions[name.to_sym] or raise "unknown property='#{name}'"
   end
 
+  def self.has_property?(name)
+    property_definitions.has_key?(name.to_sym)
+  end
+
   def self.default_attrs
     attrs = {}
     property_definitions.each do |pname,pdef|
@@ -95,6 +78,22 @@ class Base
     end
   end
 
+  # KeyRange allocateIds(java.lang.String kind, long num)
+  def self.allocate_ids(num)
+    AppEngine::Datastore.allocate_ids(kind, num) # KeyRange
+  end
+  def self.allocate_id
+    allocate_ids(1).start # Key
+  end
+
+  # KeyRange allocateIds(Key parent, java.lang.String kind, long num)
+#  def allocate_ids(kind, num)
+#    todo
+#  end
+#  def allocate_id(kind)
+#    todo
+#  end
+
   # Foo.create({:title=>"hello",...}, :parent=>aaa, :id=>bbb, :name=>ccc, :key=>...)
   def self.create(attrs={}, opts={})
     m = new(attrs, opts)
@@ -104,8 +103,7 @@ class Base
 
   # Foo.new
   def initialize(attrs={}, opts={})
-    @entity = opts.delete(:entity)
-    unless @entity
+    unless opts.has_key?(:entity)
       if opts[:parent] && opts[:parent].kind_of?(Base)
         opts = opts.dup
         opts[:parent] = opts[:parent].entity
@@ -114,6 +112,7 @@ class Base
       self.attributes = self.class.default_attrs.merge(attrs || {})
       @new_record = true
     else
+      @entity = opts.delete(:entity) or raise "opts[:entity] is nil."
       @new_record = false
     end
   end
@@ -257,7 +256,11 @@ class Base
     if ds_v.nil?
       @entity.removeProperty(k)
     else
-      @entity[k] = ds_v
+      if prop_def.index?
+        @entity[k] = ds_v
+      else
+        @entity.setUnindexedProperty(k, ds_v)
+      end
     end
     # todo cache value read/write
   end

data/lib/tiny_ds/base_tx.rb

@@ -72,7 +72,7 @@ class BaseTx
   #   TODO TxIDを指定できるようにする。TxSrc#key.nameにTxIDを指定して重複実行防止
   def create_tx(src, dest, args)
     tx_src = nil
-    TinyDS.tx(tx_retries){
+    TinyDS.tx(:force_begin=>true, :retries=>tx_retries){
       src = src.class.get(src.key)
       src_phase(src, args)
       src.save!
@@ -102,7 +102,7 @@ class BaseTx
   def roll_forward
     tx_src = TxSrc.get(@tx_key)
 
-    TinyDS.tx(tx_retries){
+    TinyDS.tx(:force_begin=>true, :retries=>tx_retries){
       dest_key = LowDS::KeyFactory.stringToKey(tx_src.dest_key)
       dest = dest_key.kind.constantize.get(dest_key)
       done_name = "TxDone_#{@tx_key.to_s}"
@@ -122,7 +122,7 @@ class BaseTx
     }
 
     # TxSrc#statusをdoneに
-    TinyDS.tx(tx_retries){
+    TinyDS.tx(:force_begin=>true, :retries=>tx_retries){
       tx_src = TxSrc.get!(@tx_key)
       if tx_src.status=="pending"
         tx_src.status = "done"
@@ -133,7 +133,7 @@ class BaseTx
     return true
   rescue => e
     puts e.inspect
-    TinyDS.tx(tx_retries){
+    TinyDS.tx(:force_begin=>true, :retries=>tx_retries){
       tx_src = TxSrc.get!(@tx_key)
       tx_src.roll_forward_failed_count += 1
       if roll_forward_retries_limit < tx_src.roll_forward_failed_count

data/lib/tiny_ds/low_ds.rb

@@ -57,11 +57,21 @@ module LowDS
           when String; KeyFactory.stringToKey(key)
           else raise "invalid key type key.class=[#{key.class}] key.inspect=[#{key.inspect}]"
           end
-    ent = AppEngine::Datastore.get(key)
-    if opts[:kind]
-      raise "kind missmatch. #{ent.kind}!=#{opts[:kind]}" if ent.kind!=opts[:kind]
+
+    retries = opts[:retries] || 20
+    while 0<=retries
+      retries -= 1
+      begin
+        ent = AppEngine::Datastore.get(key)
+        if opts[:kind]
+          raise "kind missmatch. #{ent.kind}!=#{opts[:kind]}" if ent.kind!=opts[:kind]
+        end
+        return ent
+      rescue AppEngine::Datastore::Timeout => ex
+        raise ex if retries<=0
+        sleep(0.001)
+      end
     end
-    ent
   end
 
   # update

data/lib/tiny_ds/property_definition.rb

@@ -18,12 +18,22 @@ class PropertyDefinition
   def has_default?
     @opts.has_key?(:default)
   end
+
+  # default is true
+  def index?
+    (!@opts.has_key?(:index)) || @opts[:index]
+  end
+
   def to_ds_value(v)
     case @ptype
     when :string
       v.nil? ? nil : v.to_s
     when :integer
       v.nil? ? nil : v.to_i
+    when :float
+      v.nil? ? nil : v.to_f
+    when :boolean
+      ![nil, false].include?(v)
     when :text
       v.nil? ? nil : com.google.appengine.api.datastore::Text.new(v.to_s)
     when :time
@@ -54,6 +64,10 @@ class PropertyDefinition
       ds_v.nil? ? nil : ds_v.to_s
     when :integer
       ds_v.nil? ? nil : ds_v.to_i
+    when :float
+      ds_v.nil? ? nil : ds_v.to_f
+    when :boolean
+      ds_v
     when :text
       ds_v.nil? ? nil : ds_v.to_s
     when :time

data/lib/tiny_ds/query.rb

@@ -42,8 +42,30 @@ class Query
   def count #todo(tx=nil)
     @q.count
   end
+  def count2
+    _count = 0
+    max_key = nil
+    loop do
+      q = @q.clone # or ruby dup
+      q.filter(:__key__, ">", max_key) if max_key
+      q.sort(:__key__)
+      q.java_query.setKeysOnly
+      entries = q.fetch(:limit=>1000).to_a
+      #p ["entries=", entries.collect{|e| e.key }]
+      c = entries.size
+      break if c==0
+      _count += c
+      max_key = entries.last.key
+      #p ["max_key=", max_key]
+    end
+    _count
+  end
   def one #todo(tx=nil)
-    @model_class.new_from_entity(@q.entity)
+    if @q.entity
+      @model_class.new_from_entity(@q.entity)
+    else
+      nil
+    end
   end
   def all(opts={}) #todo(tx=nil)
     models = []

data/lib/tiny_ds/transaction.rb

@@ -0,0 +1,37 @@
+module TinyDS
+  class Base
+    # @user.tx{                   |u| u.name="john" }
+    # @user.tx(10){               |u| u.name="john" }
+    # @user.tx(    :name=>"john")
+    # @user.tx(10, :name=>"john")
+    # @user.tx(    :name=>"john"){|u| u.name="JOHN" }
+    # @user.tx(10, :name=>"john"){|u| u.name="JOHN" }
+    def tx_update(*args)
+      retries = 3
+      retries = args.shift if args.first.kind_of?(Integer)
+      attrs   = args.shift
+      raise ArgumentError if args.size!=0
+      raise ArgumentError if !block_given? && attrs.nil?
+      obj = nil
+      TinyDS.tx(:retries=>retries) do
+        obj = get_self_and_check_lock_version
+        obj.attributes = attrs if attrs
+        yield(obj) if block_given?
+        obj.save!
+      end
+      obj
+    end
+    def get_self_and_check_lock_version
+      obj = self.class.get(self.key)
+      if self.class.has_property?(:lock_version)
+        if obj.lock_version != self.lock_version
+          raise StaleObjectError.new
+        end
+        obj.lock_version += 1
+      end
+      obj
+    end
+    class StaleObjectError < StandardError
+    end
+  end
+end

data/lib/tiny_ds/version.rb

@@ -1,3 +1,3 @@
 module TinyDS
-  VERSION = '0.0.2'.freeze
+  VERSION = '0.0.3.pre'.freeze
 end

data/lib/tiny_ds.rb

@@ -5,13 +5,35 @@ require File.dirname(__FILE__)+"/tiny_ds/property_definition.rb"
 require File.dirname(__FILE__)+"/tiny_ds/base.rb"
 require File.dirname(__FILE__)+"/tiny_ds/query.rb"
 require File.dirname(__FILE__)+"/tiny_ds/validations.rb"
+require File.dirname(__FILE__)+"/tiny_ds/transaction.rb"
 require File.dirname(__FILE__)+"/tiny_ds/base_tx.rb"
 require File.dirname(__FILE__)+"/tiny_ds/version.rb"
 
 module TinyDS
-  def self.tx(retries=0, &block)
-    AppEngine::Datastore.transaction(retries){
+  # execute block in new transaction.
+  # if current_transaction is exists, no new tx is begin.
+  # if force_begin=true, always begin new tx.
+  def self.tx(opts={}, &block)
+    retries = opts[:retries] || 0
+    cur_tx = nil
+    unless opts[:force_begin]
+      cur_tx = AppEngine::Datastore.current_transaction(nil)
+    end
+    if cur_tx
       yield(block)
-    }
+    else
+      AppEngine::Datastore.transaction(retries){
+        yield(block)
+      }
+    end
+  end
+  def self.readonly(&block)
+    raise "todo"
+  end
+  def batch_get
+    raise "todo"
+  end
+  def batch_put
+    raise "todo"
   end
 end

data/spec/basic_spec.rb

@@ -1,14 +1,12 @@
-#require File.dirname(File.expand_path(__FILE__)) + '/spec_helper.rb'
-
 require File.dirname(__FILE__) + '/spec_helper'
 
 class Comment < TinyDS::Base
-  #property :ekey, String, :key=>true
   property :num,        :integer
   property :title,      :string
   property :body,       :text
   property :flag,       :integer, :default=>5
   property :new_at,     :time,    :default=>proc{ Time.now }
+  property :rate,       :float
   property :updated_at, :time
   property :created_at, :time
   property :view_at,    :time
@@ -16,13 +14,16 @@ end
 
 class Animal < TinyDS::Base
   property :nickname,   :string
-  property :color,      :string
+  property :color,      :string,  :index=>true
+  property :memo,       :string,  :index=>false
+  property :age,        :integer, :index=>nil
 end
 
 class User < TinyDS::Base
   property :nickname,   :string
   property :age,        :integer
   property :favorites,  :list
+  property :height,     :float
 end
 
 describe TinyDS::Base do
@@ -35,9 +36,96 @@ describe TinyDS::Base do
     Comment.kind.should == "Comment"
   end
 
-  #describe "tx" do
-  #  # todo
-  #end
+  describe "tx" do
+    it "should not begin new tx if current_transaction exists." do
+      TinyDS.tx{
+        tx1 = AppEngine::Datastore.current_transaction
+        TinyDS.tx{
+          tx2 = AppEngine::Datastore.current_transaction
+          tx1.getId.to_s.should == tx2.getId.to_s
+        }
+      }
+    end
+    it "should begin new tx if :force_begin=true." do
+      TinyDS.tx{
+        tx1 = AppEngine::Datastore.current_transaction
+        TinyDS.tx(:force_begin=>true){
+          tx2 = AppEngine::Datastore.current_transaction
+          # p [tx1.getId, tx2.getId]
+          tx1.getId.to_s.should_not == tx2.getId.to_s
+        }
+      }
+    end
+    it "should retried if concurrent modify"
+    it "should not overwrite properties modified by other tx"
+    it "should not retry when application exception raised"
+  end
+
+  describe "instance tx_update" do
+    it "should updated with tx(attrs)" do
+      c0 = Comment.create(:body=>"hello")
+      c1 = Comment.get(c0.key)
+      c1.body.should == c0.body
+      c2 = c1.tx_update(:body=>"HELLO")
+      c2.body.should == "HELLO"
+      c1.body.should == c0.body # should not changed! BUT I WANT TO BE CHANGED
+      c3 = Comment.get(c0.key)
+      c3.body.should == "HELLO"
+    end
+    it "should updated with tx(block)" do
+      c0 = Comment.create(:body=>"hello")
+      c1 = Comment.get(c0.key)
+      c1.body.should == c0.body
+      c2 = c1.tx_update{|c|
+        c.body = "HELLO"
+      }
+      c2.body.should == "HELLO"
+      c1.body.should == c0.body # should not changed!
+      c3 = Comment.get(c0.key)
+      c3.body.should == "HELLO"
+    end
+    it "should updated with tx(attrs,block)" do
+      c0 = Comment.create(:body=>"hello", :title=>"world", :num=>10)
+      c1 = Comment.get(c0.key)
+      c1.body.should  == c0.body
+      c1.title.should == c0.title
+      c1.num.should   == c0.num
+      c2 = c1.tx_update(:title=>"WORLD", :num=>20){|c|
+        c.body = "HELLO"
+        c.num  = 30
+      }
+      c2.body.should  == "HELLO"
+      c2.title.should == "WORLD"
+      c2.num.should   == 30
+      c1.body.should  == c0.body  # should not changed!
+      c1.title.should == c0.title # should not changed!
+      c1.num.should   == c0.num   # should not changed!
+      c3 = Comment.get(c0.key)
+      c3.body.should  == "HELLO"
+      c3.title.should == "WORLD"
+      c3.num.should   == 30
+    end
+    it "should retried if concurrent modify" do
+      c0 = Comment.create(:body=>"hello", :num=>0)
+      c1 = Comment.get(c0.key)
+      c1.body.should == c0.body
+      loop_count = 0
+      c2 = c1.tx_update(10){|c|
+        loop_count += 1
+        c.body += " world"
+        if loop_count<=5
+          # modify entity by other tx
+          TinyDS.tx(:force_begin=>true){ com = Comment.get(c0.key); com.num+=1; com.save; }
+        end
+      }
+      loop_count.should == 6
+      c2.num.should  == 5
+      c2.body.should == "hello world" # should not be "hello world world world..."
+      c1.body.should == c0.body # should not be changed!
+    end
+    it "should not overwrite properties modified by other tx"
+    it "should not retry when application exception raised"
+  end
 
   describe :property_definitions do
     it "should convert to Text" do
@@ -45,7 +133,7 @@ describe TinyDS::Base do
       text.class.should == com.google.appengine.api.datastore.Text
     end
     it "should correct properties count" do
-      Comment.property_definitions.size.should == 8
+      Comment.property_definitions.size.should == 9
     end
     it "should initialized with default value" do
       a = Comment.new
@@ -55,9 +143,40 @@ describe TinyDS::Base do
       a = Comment.new
       (Time.now-a.new_at).should <= 2.0
     end
+    it "should not index if :index=>false" do
+      a = Animal.create({:nickname=>"pochi", :color=>"white", :memo=>"POCHI", :age=>5})
+      Animal.query.filter(:nickname, "==", "pochi").one.memo.should == a.memo
+      Animal.query.filter(:color,    "==", "white").one.memo.should == a.memo
+      Animal.query.filter(:memo,     "==", "POCHI").one.should be_nil
+      Animal.query.filter(:age,      "==", 5      ).one.should be_nil
+    end
+    it "should be saved large unindexed list props" do
+      favs = (0...10000).to_a
+      # should be raised. test env skips index count limit=5000???
+      User.create({:favorites=>favs, :nickname=>"john"})
+      u = User.query.filter(:nickname=>"john").one
+      u.favorites.size.should == 10000
+    end
+  end
+
+  describe "allocate_ids" do
+    it "should be allocated keys" do
+      kr = User.allocate_ids(10)
+      kr.should be_kind_of(Java::ComGoogleAppengineApiDatastore::KeyRange)
+      (kr.end.id-kr.start.id).should == 10-1
+      kr.each do |k|
+        k.should be_kind_of(AppEngine::Datastore::Key)
+        k.inspect.should match(/^User\(\d+\)$/)
+      end
+    end
+    it "should be allocate a id" do
+      k0 = User.allocate_id
+      k0.should be_kind_of(AppEngine::Datastore::Key)
+      k0.inspect.should match(/^User\(\d+\)$/)
+    end
   end
 
-  describe "key" do 
+  describe "create with key" do
     it "should be got id/name" do
       k1 = AppEngine::Datastore::Key.from_path("Com", 9999)
       a1 = Comment.create({:title=>"ccccc"}, :key=>k1)
@@ -319,6 +438,18 @@ describe TinyDS::Base do
       Comment.new(              ).num.should be_kind_of(NilClass)
       Comment.new(              ).num.should == nil
     end
+    it "should convert to float" do
+      User.new(:height => "zzzz" ).height.should be_kind_of(Float)
+      User.new(:height => "zzzz" ).height.should be_close(0.0, 0.00001)
+      User.new(:height => 123.5  ).height.should be_kind_of(Float)
+      User.new(:height => 123.5  ).height.should be_close(123.5, 0.00001)
+      User.new(:height => "123.5").height.should be_kind_of(Float)
+      User.new(:height => "123.5").height.should be_close(123.5, 0.00001)
+      User.new(:height => nil    ).height.should be_kind_of(NilClass)
+      User.new(:height => nil    ).height.should == nil
+      User.new(                  ).height.should be_kind_of(NilClass)
+      User.new(                  ).height.should == nil
+    end
     it "should convert to text" do
       Comment.new(:body => "zzzz").body.should be_kind_of(String)
       Comment.new(:body => "zzzz").body.should == "zzzz"
@@ -350,15 +481,20 @@ describe TinyDS::Base do
     before :all do
       Comment.destroy_all
       raise if Comment.count!=0
+      rate = -1.0
       3.times do
-        Comment.create(:num=>10, :title=>"BBB")
+        Comment.create(:num=>10, :title=>"BBB", :rate=>rate)
+        rate += 0.1
       end
       5.times do
-        Comment.create(:num=>10, :title=>"AAA")
+        Comment.create(:num=>10, :title=>"AAA", :rate=>rate)
+        rate += 0.1
       end
       7.times do
-        Comment.create(:num=>50, :title=>"AAA")
+        Comment.create(:num=>50, :title=>"AAA", :rate=>rate)
+        rate += 0.1
       end
+      # -1.0,-0.9,...,0.0,0.1,....,0.3,0.4
     end
     it "should fetched all" do
       Comment.query.count.should == 15
@@ -384,6 +520,15 @@ describe TinyDS::Base do
       Comment.query.filter(:num, ">=", 20).all.all?{|c| c.num==50 }.should be_true
       Comment.query.filter(:num, "<=", 20).all.all?{|c| c.num==10 }.should be_true
     end
+    it "should be fetched by float" do
+      Comment.query.filter(:rate, "<", -0.01).count.should == 10
+      Comment.query.filter(:rate, ">", -0.01).count.should ==  5
+      Comment.query.filter(:rate, "<",  0.01).count.should == 11
+      Comment.query.filter(:rate, ">",  0.01).count.should ==  4
+
+      Comment.query.filter(:rate, ">",  0.01).filter(:rate, "<", 0.21).count.should == 2
+      Comment.query.filter(:rate, ">",  0.01).filter(:rate, "<", 0.09).count.should == 0
+    end
     it "should be sorted" do
       comments = Comment.query.sort(:title).sort(:num).all
       comments[ 0, 5].all?{|c| c.title=="AAA" && c.num==10 }.should be_true
@@ -410,6 +555,7 @@ describe TinyDS::Base do
       Comment.count.should == 2
       Comment.query.filter(:num, "<", 0).count.should == 0
     end
+    it "should return 1000+ count2"
   end
   describe "query(2) parent-children" do
     before :all do
@@ -442,7 +588,7 @@ describe TinyDS::Base do
           c.parent_key.to_s.should == parent.key.to_s
         end
       }
-      Comment.query.                     filter(:num, "==", 10).count.should == 3
+      Comment.query.                 filter(:num, "==", 10).count.should == 3
       Comment.query.ancestor(parent).filter(:num, "==", 10).count.should == 2
       Comment.query.ancestor(parent).filter(:num, "==", 10).each{|c| # [C1,C2]
         c.key.inspect.index(parent.key.inspect).should == 0
@@ -459,6 +605,7 @@ describe TinyDS::Base do
       child2  = Comment.create({:title=>"C2", :num=>10})
     end
     it "should raise error from one" do
+      Comment.query.filter(:num=>999).one.should == nil
       proc{
         Comment.query.one
       }.should      raise_error(AppEngine::Datastore::TooManyResults)
@@ -644,5 +791,57 @@ describe TinyDS::Base do
       User.query.filter(:favorites, ">=", 20).count.should == 6
       User.query.filter(:favorites, ">",  20).count.should == 4
     end
+    it "should be default is []" do
+      u1 = User.new
+      u1.favorites.should == []
+      u1.save
+      u1.favorites.should == []
+      u1 = User.get(u1.key)
+      u1.favorites.should == []
+    end
+    it "should be added" do
+      u1 = User.new
+      u1.favorites.size.should == 0
+      u1.favorites << "DOG"
+      u1.favorites.size.should == 0 #1
+      u1.favorites << "CAT"
+      u1.favorites.size.should == 0 #2
+
+      u1.favorites += ["DOG"]
+      u1.favorites.size.should == 1
+      u1.favorites += ["CAT"]
+      u1.favorites.size.should == 2
+      u1.save
+
+      u1 = User.get(u1.key)
+      u1.favorites.sort.should == ["CAT","DOG"]
+    end
+    it "should be removed" do
+      u1 = User.new
+      u1.favorites += ["CAT","DOG"]
+      u1.save
+
+      u1 = User.get(u1.key)
+      u1.favorites.sort.should == ["CAT","DOG"]
+      u1.favorites -= ["DOG"]
+      u1.favorites.sort.should == ["CAT"]
+      u1.save
+
+      u1 = User.get(u1.key)
+      u1.favorites.sort.should == ["CAT"]
+      u1.favorites -= ["CAT"]
+      u1.favorites.sort.should == []
+      u1.save
+
+      u1 = User.get(u1.key)
+      u1.favorites.sort.should == []
+    end
+  end
+  describe "boolean property" do
+    it "true/false"
+    it "nil"
+    it "123"
+    it "str"
+    it "query"
   end
 end