paulcarey-relaxdb 0.3.2 → 0.3.3

data/README.textile

@@ -1,5 +1,7 @@
 h3. What's New?
 
+* 2009-05-27
+** Added minimal support for data migrations. Although CouchDB's nature removes the necessity for migrations, certain knowledge that all objects possess a particular property can simplify client logic. This desire for simplification is the rationale behind this change.
 * 2009-04-19 
 ** Defaults to taf2-curb, falling back to Net/HTTP if it taf2-curb can't be loaded. Thanks to "Fred Cheung":http://www.spacevatican.org/2009/4/13/fun-with-ruby-http-clients.
 ** For those interested in using RelaxDB with an ETag based cache, "look here":http://github.com/fcheung/relaxdb/commit/1d9acfd5f6b3c23da0d275252b6a6e064865440e
@@ -171,6 +173,20 @@ h3. Creating views by hand
 </code>
 </pre>
 
+h3. Migrations
+
+<pre>
+<code>
+  $ cat 001_double.rb
+  RelaxDB::Migration.run Primitives do |p| 
+    p.num *= 2 
+    p
+  end
+  
+  $ ruby -e 'RelaxDB::Migration.run_all Dir["./*.rb"]'
+</code>
+</pre>
+
 h3. Visualise
 
 "Fuschia":http://github.com/paulcarey/fuschia/tree/master offers a web front end for visualising inter-document relationships.

data/Rakefile

@@ -4,7 +4,7 @@ require 'spec/rake/spectask'
 
 PLUGIN = "relaxdb"
 NAME = "relaxdb"
-GEM_VERSION = "0.3.2"
+GEM_VERSION = "0.3.3"
 AUTHOR = "Paul Carey"
 EMAIL = "paul.p.carey@gmail.com"
 HOMEPAGE = "http://github.com/paulcarey/relaxdb/"

data/lib/relaxdb/migration.rb

@@ -0,0 +1,40 @@
+module RelaxDB
+  
+  class Migration
+                
+    def self.run klass, limit = 1000
+      query = lambda do |page_params|
+        RelaxDB.paginate_view "#{klass}_all", :startkey => nil, :endkey => {}, :attributes => [:_id],
+          :page_params => page_params, :limit => limit
+      end
+      
+      objs = query.call({})
+      until objs.empty?
+        migrated = objs.map { |o| yield o }.flatten.reject { |o| o.nil? }
+        RelaxDB.bulk_save! *migrated
+        objs = objs.next_params ? query.call(objs.next_params) : []
+      end
+    end
+    
+    #
+    # Runs all outstanding migrations in a given directory
+    #
+    # ==== Example
+    #   RelaxDB::Migration.run_all Dir["couchdb/migrations/**/*.rb"]
+    #
+    def self.run_all file_names, action = lambda { |fn| require fn }
+      v = RelaxDB::MigrationVersion.version
+      file_names.select { |fn| fv(fn) > v }.each do |fn|
+        RelaxDB.logger.info "Applying #{fn}"
+        action.call fn
+        RelaxDB::MigrationVersion.update fv(fn)
+      end
+    end
+    
+    def self.fv file_name
+      File.basename(file_name).split("_")[0].to_i
+    end
+    
+  end
+  
+end

data/lib/relaxdb/migration_version.rb

@@ -0,0 +1,21 @@
+class RelaxDB::MigrationVersion < RelaxDB::Document
+  
+  DOC_ID = "relaxdb_migration_version"
+  
+  property :version, :default => 0
+  
+  def self.version
+    retrieve.version
+  end
+      
+  def self.update v
+    mv = retrieve
+    mv.version = v
+    mv.save!
+  end
+    
+  def self.retrieve
+    (v = RelaxDB.load(DOC_ID)) ? v : new(:_id => DOC_ID).save!
+  end
+  
+end

data/lib/relaxdb/relaxdb.rb

@@ -73,7 +73,7 @@ module RelaxDB
       end
       
       pre_save_success = objs.inject(true) { |s, o| s &= o.pre_save }
-      raise ValidationFailure, objs unless pre_save_success
+      raise ValidationFailure, objs.inspect unless pre_save_success
       
       docs = {}
       objs.each { |o| docs[o._id] = o }

data/lib/relaxdb/views.rb

@@ -8,7 +8,7 @@ module RelaxDB
       function(doc) {        
         var class_match = #{kls_check kls}
         if (class_match) {
-          emit(null, doc);
+          emit(doc._id, doc);
         }
       }
       QUERY

data/lib/relaxdb.rb

@@ -30,6 +30,7 @@ require 'relaxdb/document'
 require 'relaxdb/extlib'
 require 'relaxdb/has_many_proxy'
 require 'relaxdb/has_one_proxy'
+require 'relaxdb/migration'
 require 'relaxdb/paginate_params'
 require 'relaxdb/paginator'
 require 'relaxdb/query'
@@ -43,5 +44,7 @@ require 'relaxdb/view_uploader'
 require 'relaxdb/views'
 require 'more/grapher.rb'
 
+require 'relaxdb/migration_version'
+
 module RelaxDB
 end

data/spec/doc_inheritable_spec.rb

@@ -0,0 +1,100 @@
+require File.dirname(__FILE__) + '/spec_helper.rb'
+require File.dirname(__FILE__) + '/spec_models.rb'
+
+describe "Inheritance" do
+  
+  before(:each) do
+    setup_test_db
+  end
+  
+  describe "properties" do
+
+    it "should by inherited from a parent document" do
+      d = SubDescendant.new(:x => 1).save!
+      RelaxDB.reload(d).x.should == 1
+    end
+    
+    it "should store its own properties" do
+      r = RichDescendant.new(:x => 1, :foo => :bar).save!
+      RelaxDB.reload(r).x.should == 1
+      RelaxDB.reload(r).foo.should == "bar"
+    end
+    
+    it "validators should behave as normal" do
+      d = SubDescendant.new(:y => false)
+      d.save.should be_false
+      d.errors[:y].should == "Uh oh"
+    end
+    
+  end      
+    
+  describe "_all views" do
+    
+    it "should be rewritten" do
+      a = Ancestor.new(:x => 0).save!
+      d = Descendant.new(:x => 1).save!
+
+      Ancestor.all.should == [a, d]
+      Descendant.all.should == [d]
+    end
+
+    it "should function with inheritance trees" do
+      Inh::X.new.save!
+      
+      Inh::Y.new.save!
+      Inh::Y1.new.save!
+      
+      Inh::Z.new.save!
+      Inh::Z1.new.save!
+      Inh::Z2.new.save!
+      
+      Inh::X.all.size.should == 6
+      Inh::Y.all.size.should == 2
+      Inh::Y1.all.size.should == 1
+      Inh::Z.all.size.should == 3
+      Inh::Z1.all.size.should == 1
+      Inh::Z2.all.size.should == 1
+    end
+    
+  end
+  
+  describe "_by views" do
+    
+    it "should be rewritten for ancestors and generated for descendants" do
+      a = Ancestor.new(:x => 0).save!
+      d = Descendant.new(:x => 1).save!
+      
+      Ancestor.by_x.should == [a, d]
+      Descendant.by_x.should == [d]
+    end
+    
+  end
+  
+  describe "derived properties" do
+    
+    it "should be stored" do
+      u = User.new(:_id => "foo", :name => "u").save!
+      s = SubDescendant.new(:user => u).save!
+      r = RichDescendant.new(:user => u, :ukulele => u).save!
+      
+      RelaxDB.reload(s).user_name.should == "u"
+      RelaxDB.reload(r).user_name.should == "u"
+      RelaxDB.reload(r).ukulele_name.should == "u"
+    end
+    
+  end
+
+  describe "references" do
+    
+    it "should function as normal" do
+      u = User.new(:name => "u").save!
+      s = SubDescendant.new(:user => u).save!
+      r = RichDescendant.new(:user => u).save!
+      
+      RelaxDB.reload(s).user.name.should == "u"
+      RelaxDB.reload(r).user.name.should == "u"
+    end
+    
+  end
+    
+end

data/spec/migration_spec.rb

@@ -0,0 +1,97 @@
+require File.join(File.dirname(__FILE__), "spec_helper")
+require File.join(File.dirname(__FILE__), "spec_models")
+
+describe RelaxDB::Migration do
+          
+  before(:each) do
+    @mig = RelaxDB::Migration
+    setup_test_db
+  end
+      
+  it "should yield each obj to the block and save the result" do
+    Primitives.new(:num => 5).save!
+    r = @mig.run Primitives do |p|
+      pn = Primitives.new(:num => p.num * 2)
+      [p, pn]
+    end
+    Primitives.by_num.map { |p| p.num}.should == [5, 10]
+  end
+    
+  it "should raise an exception if a save results in a conflict" do
+    op = Primitives.new.save!
+    lambda do 
+      @mig.run Primitives do |p|
+        op.save!
+        p
+      end
+    end.should raise_error(RelaxDB::UpdateConflict)
+  end
+  
+  it "should not save docs for blocks that return nil" do
+    Primitives.new.save!
+    @mig.run Primitives do |p| 
+      nil
+    end
+  end
+      
+  describe "multiple docs" do
+  
+    before(:each) do
+      ps = (1..5).map { |i| Primitives.new :num => i }
+      RelaxDB.bulk_save *ps
+      RelaxDB.db.reset_req_count
+    end
+    
+    # Note: two extra requests per paginate_view request are required
+    it "should operate on a doc set of the given size aka limit" do
+      @mig.run(Primitives, 1) { |p| p.num *= p.num; p }
+      RelaxDB.db.req_count.should == 10 + 5 * 2 
+      Primitives.by_num.map { |p| p.num }.should == [1, 4, 9, 16, 25]      
+    end
+
+    it "should operate on a doc set of default size" do
+      @mig.run(Primitives) { |p| p.num *= p.num; p }
+      RelaxDB.db.req_count.should == 2 + 1 * 2
+      Primitives.by_num.map { |p| p.num }.should == [1, 4, 9, 16, 25]      
+    end
+  
+  end
+
+  describe "#fv" do
+    it "should return valid numbers" do
+      @mig.fv("foo/bar/001_foo.rb").should == 1 
+    end
+  end
+  
+  describe ".run_all" do
+    
+    it "should save the version after each successful migration" do
+      @mig.run_all "a/b/1_", lambda {}
+      RelaxDB::MigrationVersion.version.should == 1
+    end
+
+    it "should not run those migrations whose version is less than the current version" do
+      v = RelaxDB::MigrationVersion.retrieve
+      v.version = 2
+      v.save!
+      @mig.run_all "3_", lambda {}
+      RelaxDB::MigrationVersion.version.should == 3
+    end
+    
+    it "should run those migrations whose version is greater than the current version" do
+      v = RelaxDB::MigrationVersion.retrieve
+      v.version = 2
+      v.save!
+      @mig.run_all "1_foo", lambda {}
+      RelaxDB::MigrationVersion.version.should == 2
+    end
+    
+    it "should raise an exception on failure" do
+      lambda do
+        @mig.run_all "1_foo", lambda { raise "Expected" }
+      end.should raise_error(RuntimeError, "Expected")
+    end
+    
+  end
+  
+end

data/spec/migration_version_spec.rb

@@ -0,0 +1,28 @@
+require File.join(File.dirname(__FILE__), "spec_helper")
+require File.join(File.dirname(__FILE__), "spec_models")
+
+describe RelaxDB::MigrationVersion do
+  
+  before(:each) do
+    setup_test_db
+  end
+  
+  it "should not exist in a clean db" do
+    RelaxDB.load(RelaxDB::MigrationVersion::DOC_ID).should be_nil
+  end
+  
+  it "should return zero when it doesnt exist" do
+    RelaxDB::MigrationVersion.version.should == 0
+  end
+  
+  it "should autosave on retrieval when when it doesnt exist" do
+    RelaxDB::MigrationVersion.version
+    RelaxDB.load(RelaxDB::MigrationVersion::DOC_ID).should be
+  end
+  
+  it "should return the saved version" do
+    RelaxDB::MigrationVersion.update 10
+    RelaxDB::MigrationVersion.version.should == 10
+  end
+  
+end

data/spec/paginate_params_spec.rb

@@ -0,0 +1,15 @@
+require File.dirname(__FILE__) + '/spec_helper.rb'
+require File.dirname(__FILE__) + '/spec_models.rb'
+
+describe RelaxDB::PaginateParams do
+    
+  it "should be invalid if hasn't been initialized with both a startkey and endkey" do
+    RelaxDB::PaginateParams.new({}).should be_invalid
+  end
+
+  it "should be valid if initialized with both a startkey and endkey" do
+    pp = RelaxDB::PaginateParams.new :startkey => nil, :endkey => nil
+    pp.should_not be_invalid
+  end
+  
+end

data/spec/paginate_spec.rb

@@ -0,0 +1,351 @@
+require File.dirname(__FILE__) + '/spec_helper.rb'
+require File.dirname(__FILE__) + '/spec_models.rb'
+
+describe "RelaxDB Pagination" do
+    
+  before(:each) do
+    setup_test_db    
+    
+    letters = [ 
+      ["a", 1], ["a", 2], ["a", 3],
+      ["b", 1], ["b", 2], ["b", 3], ["b", 4], ["b", 5], 
+      ["c", 1], ["c", 2] 
+    ].map { |o| Letter.new :letter => o[0], :number => o[1], :_id => "#{o[0]}#{o[1]}" }
+
+    RelaxDB.bulk_save *letters        
+  end
+
+  # helper function
+  def s(letters)
+    letters.map { |l| "#{l.letter}#{l.number}"}.join(", ")
+  end
+  
+  def n(letters)
+    letters.map { |l| "#{l.number}"}.join(", ")
+  end
+    
+  describe "functional tests" do
+
+    it "should navigate through a series" do
+      query = lambda do |page_params|
+         Letter.paginate_by_letter_and_number :page_params => page_params,
+           :startkey => ["a"], :endkey => ["a",{}], :limit => 2
+      end
+    
+      letters = query.call({})
+      s(letters).should == "a1, a2"
+      letters.prev_params.should be_false
+    
+      letters = query.call(letters.next_params)
+      s(letters).should == "a3"
+      letters.next_params.should be_false
+    
+      letters = query.call(letters.prev_params)
+      s(letters).should == "a1, a2"
+      letters.prev_params.should be_false
+      letters.next_params.should be    
+    end
+
+    it "should navigate through b series with descending false" do
+      query = lambda do |page_params|
+         Letter.paginate_by_letter_and_number :page_params => page_params,
+           :startkey => ["b"], :endkey => ["b",{}], :limit => 2
+      end
+    
+      letters = query.call({})
+      s(letters).should == "b1, b2"
+      letters.prev_params.should be_false
+      
+      letters = query.call(letters.next_params)
+      s(letters).should == "b3, b4"
+      letters.next_params.should be
+
+      letters = query.call(letters.prev_params)
+      s(letters).should == "b1, b2"
+      letters.prev_params.should be_false
+    
+      letters = query.call(letters.next_params)
+      s(letters).should == "b3, b4"
+      letters.prev_params.should be
+    
+      letters = query.call(letters.next_params)
+      s(letters).should == "b5"
+      letters.next_params.should be_false
+
+      letters = query.call(letters.prev_params)
+      s(letters).should == "b3, b4"
+      letters.next_params.should be
+    
+      letters = query.call(letters.prev_params)
+      s(letters).should == "b1, b2"
+      letters.prev_params.should be_false
+      letters.next_params.should be
+    end
+
+    it "should navigate through b series with descending true" do      
+      query = lambda do |page_params|
+         Letter.paginate_by_letter_and_number :page_params => page_params,
+           :startkey => ["b", {}], :endkey => ["b"], :descending => true, :limit => 2
+      end      
+    
+      letters = query.call({})
+      s(letters).should == "b5, b4"
+      letters.prev_params.should be_false
+    
+      letters = query.call(letters.next_params)
+      s(letters).should == "b3, b2"
+      letters.prev_params.should be
+    
+      letters = query.call(letters.next_params)
+      s(letters).should == "b1"
+      letters.next_params.should be_false
+
+      letters = query.call(letters.prev_params)
+      s(letters).should == "b3, b2"
+      letters.next_params.should be
+    
+      letters = query.call(letters.prev_params)
+      s(letters).should == "b5, b4"
+      letters.prev_params.should be_false
+      letters.next_params.should be
+    end
+  
+    it "should not display pagination options for c series" do
+      letters = Letter.paginate_by_letter_and_number :page_params => {},
+        :startkey => ["c"], :endkey => ["c", {}], :limit => 2      
+    
+      letters.next_params.should be_false
+      letters.prev_params.should be_false
+    end
+    
+  end
+  
+  describe "next_query" do
+
+    it "should emit a url encoded and json encoded string with query name page_params" do
+      letters = Letter.paginate_by_letter_and_number :page_params => {:startkey => ["b", 2]},
+         :startkey => ["b"], :endkey => ["b",{}],:limit => 2
+      
+      # unescape and parse required as param order is implementation dependent
+      hash = JSON.parse(CGI.unescape(letters.next_query.split("=")[1]))
+      
+      hash["descending"].should be_false
+      hash["startkey"].should == ["b", 4]
+    end
+    
+    it "should be treated as next_param by the paginator" do
+      page_params = {}
+      query = lambda do
+        Letter.paginate_by_letter_and_number :page_params => page_params,
+          :startkey => ["b"], :endkey => ["b",{}], :limit => 2        
+      end
+      
+      letters = query.call      
+      page_params = CGI::unescape(letters.next_query.split("=")[1])
+      letters = query.call
+      s(letters).should == "b3, b4"
+    end
+    
+  end
+  
+  describe "prev_query" do
+    
+    it "should be treated as prev_query by the paginator" do
+      page_params = {}
+      query = lambda do
+        Letter.paginate_by_letter_and_number :page_params => page_params,
+          :startkey => ["b", {}], :endkey => ["b"], :descending => true, :limit => 2
+      end
+      
+      letters = query.call      
+      page_params = CGI::unescape(letters.next_query.split("=")[1])
+      letters = query.call
+      s(letters).should == "b3, b2"
+    end
+    
+    it "should emit a url encoded and json encoded string with query name page_params" do      
+      letters = Letter.paginate_by_letter_and_number :page_params => {:startkey => ["b", 2]},
+         :startkey => ["b"], :endkey => ["b",{}],:limit => 2
+      
+      hash = JSON.parse(CGI.unescape(letters.prev_query.split("=")[1]))
+      
+      hash["descending"].should be_true
+      hash["startkey"].should == ["b", 3]
+    end
+    
+  end
+    
+  describe "multiple keys per document, simple (non array) keys" do
+    
+    it "should work when descending is false" do
+      query = lambda do |page_params|
+        Letter.paginate_by_number :page_params => page_params,
+          :startkey => 1, :endkey => {}, :limit => 4
+      end
+      
+      numbers = query.call({})
+      n(numbers).should == "1, 1, 1, 2"
+      numbers.prev_params.should be_false
+      
+      numbers = query.call(numbers.next_params)
+      n(numbers).should == "2, 2, 3, 3"
+      numbers.next_params.should be
+      
+      numbers = query.call(numbers.prev_params)
+      n(numbers).should == "1, 1, 1, 2"
+      numbers.prev_params.should be_false
+      
+      numbers = query.call(numbers.next_params)
+      n(numbers) == "2, 2, 3, 3"
+      numbers.prev_params.should be
+      
+      numbers = query.call(numbers.next_params)
+      n(numbers) == "4, 5"
+      numbers.next_params.should be_false
+      
+      numbers = query.call(numbers.prev_params)
+      n(numbers) == "2, 2, 3, 3"
+      numbers.next_params.should be
+      
+      numbers = query.call(numbers.prev_params)
+      n(numbers) == "1, 1, 1, 2"
+      numbers.next_params.should be
+      numbers.prev_params.should be_false
+    end
+
+    it "should work when descending is true" do
+      query = lambda do |page_params|
+        Letter.paginate_by_number :page_params => page_params,
+          :startkey => 5, :endkey => nil, :descending => true, :limit => 4        
+      end
+      
+      numbers = query.call({})
+      n(numbers).should == "5, 4, 3, 3"
+      numbers.prev_params.should be_false
+      
+      numbers = query.call(numbers.next_params)
+      n(numbers).should == "2, 2, 2, 1"
+      numbers.next_params.should be
+
+      numbers = query.call(numbers.prev_params)
+      n(numbers).should == "5, 4, 3, 3"
+      numbers.prev_params.should be_false
+
+      numbers = query.call(numbers.next_params)
+      n(numbers).should == "2, 2, 2, 1"
+      numbers.prev_params.should be
+
+      numbers = query.call(numbers.next_params)
+      n(numbers).should == "1, 1"
+      numbers.next_params.should be_false
+
+      numbers = query.call(numbers.prev_params)
+      n(numbers).should == "2, 2, 2, 1"
+      numbers.next_params.should be
+
+      numbers = query.call(numbers.prev_params)
+      n(numbers).should == "5, 4, 3, 3"
+      numbers.prev_params.should be_false
+      numbers.next_params.should be
+    end
+    
+    it "should not get stuck when the number of keys exceeds the limit" do
+      query = lambda do |page_params|
+        Letter.paginate_by_number :page_params => page_params,
+          :startkey => 1, :endkey => {}, :limit => 2
+      end
+      
+      numbers = query.call({})
+      n(numbers).should == "1, 1"
+      numbers = query.call(numbers.next_params)
+      n(numbers).should == "1, 2"
+    end
+    
+  end
+  
+  describe ".paginate_by" do
+    
+    it "should throw an error unless both startkey and endkey are specified" do
+      lambda do
+        Letter.paginate_by_number :page_params => page_params, :startkey => 1, :limit => 2
+      end.should raise_error
+    end
+    
+    it "should return an empty array when no documents exist" do
+      Letter.all.destroy!
+      letters = Letter.paginate_by_number :page_params => {}, :startkey => 1, :endkey => 3
+      letters.should be_empty
+    end
+
+    it "should return an array that responds negatively to next_query and prev_query when no documents exist" do
+      Letter.all.destroy!
+      letters = Letter.paginate_by_number :page_params => {}, :startkey => 1, :endkey => 3
+      letters.prev_query.should be_false
+      letters.next_query.should be_false
+    end
+    
+  end
+  
+  describe ".paginate_view functional tests" do
+    
+    def navigate_b_series(query)
+      letters = query.call({})
+      s(letters).should == "b1, b2"
+      letters.prev_params.should be_false
+      
+      letters = query.call(letters.next_params)
+      s(letters).should == "b3, b4"
+      letters.prev_params.should be
+      
+      letters = query.call(letters.next_params)
+      s(letters).should == "b5"
+      letters.next_params.should be_false
+      
+      letters = query.call(letters.prev_params)
+      s(letters).should == "b3, b4"
+      letters.next_params.should be
+      
+      letters = query.call(letters.prev_params)
+      s(letters).should == "b1, b2"
+      letters.next_params.should be
+      letters.prev_params.should be_false      
+    end
+    
+    before(:each) do
+      map = <<-FUNC
+        function (doc) {
+          if (doc.relaxdb_class === "Letter") {
+            emit([doc.letter, doc.number], doc);
+          }
+        }
+      FUNC
+      
+      reduce = <<-FUNC
+        function (keys, values, combine) {
+          return values.length;
+        }
+      FUNC
+      
+      view_name = "Letter_by_letter_and_number"
+      RelaxDB::DesignDocument.get(RelaxDB.dd).add_map_view(view_name, map).add_reduce_view(view_name, reduce).save      
+    end
+    
+    it "should pass using symbols as view_keys" do
+      query = lambda do |page_params|
+        RelaxDB.paginate_view "Letter_by_letter_and_number", :page_params => page_params,
+          :startkey => ["b"], :endkey => ["b", {}], :limit => 2, :attributes => [:letter, :number]
+      end
+      navigate_b_series query
+    end
+
+    it "should pass using symbols and values as view_keys" do
+      query = lambda do |page_params|
+        RelaxDB.paginate_view "Letter_by_letter_and_number", :page_params => page_params,
+          :startkey => ["b"], :endkey => ["b", {}], :limit => 2, :attributes => ["b", :number]
+      end
+      navigate_b_series query
+    end
+    
+  end
+    
+end

data/spec/relaxdb_spec.rb

@@ -39,11 +39,7 @@ describe RelaxDB do
       ta.should == t1
       tb.should == t2
     end
-    
-    it "should succeed when passed no args" do
-      RelaxDB.bulk_save
-    end
-    
+        
     it "should return false on failure" do
       c = Class.new(RelaxDB::Document) do
         property :foo, :validator => lambda { false }
@@ -128,6 +124,16 @@ describe RelaxDB do
   
   describe ".bulk_save!" do
     
+    it "should succeed when passed no args" do
+      RelaxDB.bulk_save!
+    end
+    
+    it "should raise when passed a nil value" do
+      lambda do
+        RelaxDB.bulk_save! *[nil]
+      end.should raise_error
+    end
+    
     it "should raise an exception if a obj fails validation" do
       c = Class.new(RelaxDB::Document) do
         property :foo, :validator => lambda { false }
@@ -207,6 +213,16 @@ describe RelaxDB do
       ar2.should == a2
     end
     
+    it "should load multiple documents in order" do
+      ns = (0...100).map { rand(1_000_000_000).to_s }
+      objs = ns.map { |n| Primitives.new :_id => n }
+      RelaxDB.bulk_save! *objs
+      objs = RelaxDB.load! ns
+      (0...100).each do |i|
+        ns[i].should == objs[i]._id
+      end
+    end
+    
     it "should throw an exception if given a single id for a non-existant doc" do
       lambda do
         RelaxDB.load! "nothere"

data/spec/server_spec.rb

@@ -0,0 +1,32 @@
+require File.dirname(__FILE__) + '/spec_helper.rb'
+require File.dirname(__FILE__) + '/spec_models.rb'
+
+describe RelaxDB do
+
+  before(:all) do
+    RelaxDB.configure :host => "localhost", :port => 5984, :design_doc => "spec_doc"
+    @server = RelaxDB::Server.new("localhost", 5984)
+  end
+
+  before(:each) do
+    RelaxDB.delete_db "relaxdb_spec" rescue "ok"
+    RelaxDB.use_db "relaxdb_spec"    
+  end
+        
+  describe "GET" do
+    
+    it "should raise a HTTP_404 for a non existant doc" do
+      lambda do
+        @server.get "/relaxdb_spec/foo"
+      end.should raise_error(RelaxDB::HTTP_404)
+    end
+
+    it "should raise a RuntimeError for non specific errors" do
+      lambda do
+        @server.get "/relaxdb_spec/_design/spec_doc/_view?fail=true"
+      end.should raise_error(RuntimeError)
+    end
+    
+  end
+  
+end

data/spec/spec_helper.rb

@@ -9,8 +9,12 @@ end
 $:.unshift(File.dirname(__FILE__) + '/../lib')
 require 'relaxdb'
 
+class RdbFormatter; def call(sv, time, progname, msg); puts msg; end; end
+
 def setup_test_db
   # RelaxDB.configure :host => "localhost", :port => 5984, :design_doc => "spec_doc", :logger => Logger.new(STDOUT)
+  # RelaxDB.logger.formatter = RdbFormatter.new
+  
   RelaxDB.configure :host => "localhost", :port => 5984, :design_doc => "spec_doc"
   
   RelaxDB.delete_db "relaxdb_spec" rescue "ok"

data/spec/view_by_spec.rb

@@ -0,0 +1,76 @@
+require File.dirname(__FILE__) + '/spec_helper.rb'
+require File.dirname(__FILE__) + '/spec_models.rb'
+
+describe "view_by" do
+    
+  before(:all) do
+    RelaxDB.configure :host => "localhost", :port => 5984, :design_doc => "spec_doc"    
+  end
+    
+  describe "view_by" do
+    
+    before(:each) do
+      RelaxDB.delete_db "relaxdb_spec" rescue "ok"
+      RelaxDB.use_db "relaxdb_spec"
+      RelaxDB.enable_view_creation
+
+      class ViewByFoo < RelaxDB::Document
+        property :foo
+        view_by :foo, :descending => true
+      end
+
+    end
+    
+    it "should create corresponding views" do
+      dd = RelaxDB::DesignDocument.get "spec_doc"
+      dd.data["views"]["ViewByFoo_by_foo"].should be
+    end
+      
+    it "should create a by_ att list method" do
+      ViewByFoo.new(:foo => :bar).save!
+      res = ViewByFoo.by_foo
+      res.first.foo.should == "bar"
+    end
+        
+    it "should create a paginate_by_ att list method" do
+      ViewByFoo.new(:foo => :bar).save!      
+      res = ViewByFoo.paginate_by_foo :page_params => {}, :startkey => {}, :endkey => nil
+      res.first.foo.should == "bar"
+    end
+        
+    it "should apply query defaults to by_" do
+      ViewByFoo.new(:foo => "a").save!
+      ViewByFoo.new(:foo => "b").save!
+      
+      ViewByFoo.by_foo.map{ |o| o.foo }.should == ["b", "a"]
+    end
+    
+    it "should allow a single arg to be passed to by_" do
+      vbf = ViewByFoo.new(:foo => "a").save!
+      ViewByFoo.by_foo("a").should == vbf
+    end
+    
+    it "should apply query defaults to paginate_by_" do
+      ViewByFoo.new(:foo => "a").save!
+      ViewByFoo.new(:foo => "b").save!
+      
+      res = ViewByFoo.paginate_by_foo :page_params => {}, :startkey => {}, :endkey => nil
+      res.map{ |o| o.foo }.should == ["b", "a"]
+    end
+    
+    it "should allow query defaults to be overridden for paginate_by_" do
+      ViewByFoo.new(:foo => :bar).save!      
+      res = ViewByFoo.paginate_by_foo :page_params => {}, :startkey => nil, :endkey => {}, :descending => false
+      res.first.foo.should == "bar"      
+    end
+    
+    it "should allow query defaults to be overridden for by_" do
+      ViewByFoo.new(:foo => "a").save!
+      ViewByFoo.new(:foo => "b").save!
+      
+      ViewByFoo.by_foo(:descending => false).map{ |o| o.foo }.should == ["a", "b"]
+    end
+        
+  end
+    
+end

data/spec/view_spec.rb

@@ -0,0 +1,23 @@
+require File.dirname(__FILE__) + '/spec_helper.rb'
+require File.dirname(__FILE__) + '/spec_models.rb'
+
+describe RelaxDB::View do
+    
+  describe "exists" do
+    
+    before(:each) do
+      create_test_db
+    end
+    
+    it "should return nil if a view doesnt exist" do
+      RelaxDB::ViewCreator.all([Atom]).should_not be_exists
+    end
+    
+    it "should return the view if it exits" do
+      RelaxDB::ViewCreator.all([Atom]).save
+      RelaxDB::ViewCreator.all([Atom]).should be_exists
+    end
+
+  end
+
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: paulcarey-relaxdb
 version: !ruby/object:Gem::Version 
-  version: 0.3.2
+  version: 0.3.3
 platform: ruby
 authors: 
 - Paul Carey
@@ -9,7 +9,7 @@ autorequire: relaxdb
 bindir: bin
 cert_chain: []
 
-date: 2009-05-22 00:00:00 -07:00
+date: 2009-05-27 00:00:00 -07:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -65,6 +65,8 @@ files:
 - lib/relaxdb/has_many_proxy.rb
 - lib/relaxdb/has_one_proxy.rb
 - lib/relaxdb/net_http_server.rb
+- lib/relaxdb/migration.rb
+- lib/relaxdb/migration_version.rb
 - lib/relaxdb/paginate_params.rb
 - lib/relaxdb/paginator.rb
 - lib/relaxdb/query.rb
@@ -83,18 +85,26 @@ files:
 - lib/more/atomic_bulk_save_support.rb
 - spec/belongs_to_spec.rb
 - spec/callbacks_spec.rb
-- spec/design_doc_spec.rb
 - spec/derived_properties_spec.rb
+- spec/design_doc_spec.rb
+- spec/doc_inheritable_spec.rb
 - spec/document_spec.rb
 - spec/has_many_spec.rb
 - spec/has_one_spec.rb
+- spec/migration_spec.rb
+- spec/migration_version_spec.rb
+- spec/paginate_params_spec.rb
+- spec/paginate_spec.rb
 - spec/query_spec.rb
 - spec/references_many_spec.rb
 - spec/relaxdb_spec.rb
+- spec/server_spec.rb
 - spec/spec.opts
 - spec/spec_helper.rb
 - spec/spec_models.rb
+- spec/view_by_spec.rb
 - spec/view_object_spec.rb
+- spec/view_spec.rb
 has_rdoc: false
 homepage: http://github.com/paulcarey/relaxdb/
 post_install_message: