diff --git a/lib/couchrest.rb b/lib/couchrest.rb
index 5c05b02..cff840f 100644
--- a/lib/couchrest.rb
+++ b/lib/couchrest.rb
@@ -28,6 +28,9 @@ require 'couchrest/monkeypatches'
module CouchRest
autoload :Server, 'couchrest/core/server'
autoload :Database, 'couchrest/core/database'
+ autoload :Document, 'couchrest/core/document'
+ autoload :Design, 'couchrest/core/design'
+ autoload :View, 'couchrest/core/view'
autoload :Model, 'couchrest/core/model'
autoload :Pager, 'couchrest/helper/pager'
autoload :FileManager, 'couchrest/helper/file_manager'
diff --git a/lib/couchrest/core/database.rb b/lib/couchrest/core/database.rb
index fc0d0e2..6f19128 100644
--- a/lib/couchrest/core/database.rb
+++ b/lib/couchrest/core/database.rb
@@ -70,7 +70,14 @@ module CouchRest
# GET a document from CouchDB, by id. Returns a Ruby Hash.
def get id
slug = CGI.escape(id)
- CouchRest.get "#{@root}/#{slug}"
+ hash = CouchRest.get("#{@root}/#{slug}")
+ doc = if /^_design/ =~ hash["_id"]
+ Design.new(hash)
+ else
+ Document.new(hash)
+ end
+ doc.database = self
+ doc
end
# GET an attachment directly from CouchDB
@@ -103,7 +110,7 @@ module CouchRest
if doc['_attachments']
doc['_attachments'] = encode_attachments(doc['_attachments'])
end
- if doc['_id']
+ result = if doc['_id']
slug = CGI.escape(doc['_id'])
CouchRest.put "#{@root}/#{slug}", doc
else
@@ -114,6 +121,12 @@ module CouchRest
CouchRest.post @root, doc
end
end
+ if result['ok']
+ doc['_id'] = result['id']
+ doc['_rev'] = result['rev']
+ doc.database = self if doc.respond_to?(:database=)
+ end
+ result
end
# POST an array of documents to CouchDB. If any of the documents are
@@ -131,6 +144,8 @@ module CouchRest
# DELETE the document from CouchDB that has the given _id and
# _rev.
def delete doc
+ raise ArgumentError, "_id and _rev required for deleting" unless doc['_id'] && doc['_rev']
+
slug = CGI.escape(doc['_id'])
CouchRest.delete "#{@root}/#{slug}?rev=#{doc['_rev']}"
end
diff --git a/lib/couchrest/core/design.rb b/lib/couchrest/core/design.rb
new file mode 100644
index 0000000..f9b8e22
--- /dev/null
+++ b/lib/couchrest/core/design.rb
@@ -0,0 +1,89 @@
+module CouchRest
+ class Design < Document
+ def view_by *keys
+ opts = keys.pop if keys.last.is_a?(Hash)
+ opts ||= {}
+ self['views'] ||= {}
+ method_name = "by_#{keys.join('_and_')}"
+
+ if opts[:map]
+ view = {}
+ view['map'] = opts.delete(:map)
+ if opts[:reduce]
+ view['reduce'] = opts.delete(:reduce)
+ opts[:reduce] = false
+ end
+ self['views'][method_name] = view
+ else
+ doc_keys = keys.collect{|k|"doc['#{k}']"} # this is where :require => 'doc.x == true' would show up
+ key_emit = doc_keys.length == 1 ? "#{doc_keys.first}" : "[#{doc_keys.join(', ')}]"
+ guards = opts.delete(:guards) || []
+ guards.concat doc_keys
+ map_function = <<-JAVASCRIPT
+function(doc) {
+ if (#{guards.join(' && ')}) {
+ emit(#{key_emit}, null);
+ }
+}
+JAVASCRIPT
+ self['views'][method_name] = {
+ 'map' => map_function
+ }
+ end
+ self['views'][method_name]['couchrest-defaults'] = opts unless opts.empty?
+ method_name
+ end
+
+ # Dispatches to any named view.
+ def view view_name, query={}, &block
+ view_name = view_name.to_s
+ view_slug = "#{name}/#{view_name}"
+ defaults = (self['views'][view_name] && self['views'][view_name]["couchrest-defaults"]) || {}
+ fetch_view(view_slug, defaults.merge(query), &block)
+ end
+
+ def name
+ id.sub('_design/','') if id
+ end
+
+ def name= newname
+ self['_id'] = "_design/#{newname}"
+ end
+
+ def save
+ raise ArgumentError, "_design docs require a name" unless name && name.length > 0
+ super
+ end
+
+ private
+
+ # returns stored defaults if the there is a view named this in the design doc
+ def has_view?(view)
+ view = view.to_s
+ self['views'][view] &&
+ (self['views'][view]["couchrest-defaults"]||{})
+ end
+
+ # def fetch_view_with_docs name, opts, raw=false, &block
+ # if raw
+ # fetch_view name, opts, &block
+ # else
+ # begin
+ # view = fetch_view name, opts.merge({:include_docs => true}), &block
+ # view['rows'].collect{|r|new(r['doc'])} if view['rows']
+ # rescue
+ # # fallback for old versions of couchdb that don't
+ # # have include_docs support
+ # view = fetch_view name, opts, &block
+ # view['rows'].collect{|r|new(database.get(r['id']))} if view['rows']
+ # end
+ # end
+ # end
+
+ def fetch_view view_name, opts, &block
+ database.view(view_name, opts, &block)
+ end
+
+ end
+
+end
\ No newline at end of file
diff --git a/lib/couchrest/core/document.rb b/lib/couchrest/core/document.rb
new file mode 100644
index 0000000..1df61ea
--- /dev/null
+++ b/lib/couchrest/core/document.rb
@@ -0,0 +1,60 @@
+module CouchRest
+ class Response < Hash
+ def initialize keys = {}
+ keys.each do |k,v|
+ self[k.to_s] = v
+ end
+ end
+ def []= key, value
+ super(key.to_s, value)
+ end
+ def [] key
+ super(key.to_s)
+ end
+ end
+
+ class Document < Response
+
+ attr_accessor :database
+
+ # alias for self['_id']
+ def id
+ self['_id']
+ end
+
+ # alias for self['_rev']
+ def rev
+ self['_rev']
+ end
+
+ # returns true if the document has never been saved
+ def new_document?
+ !rev
+ end
+
+ # Saves the document to the db using create or update. Also runs the :save
+ # callbacks. Sets the _id and _rev fields based on
+ # CouchDB's response.
+ def save
+ raise ArgumentError, "doc.database required for saving" unless database
+ result = database.save self
+ result['ok']
+ end
+
+ # Deletes the document from the database. Runs the :delete callbacks.
+ # Removes the _id and _rev fields, preparing the
+ # document to be saved to a new _id.
+ def destroy
+ raise ArgumentError, "doc.database required to destroy" unless database
+ result = database.delete self
+ if result['ok']
+ self['_rev'] = nil
+ self['_id'] = nil
+ end
+ result['ok']
+ end
+
+ end
+
+
+end
\ No newline at end of file
diff --git a/lib/couchrest/core/model.rb b/lib/couchrest/core/model.rb
index 463eed5..4ac2da3 100644
--- a/lib/couchrest/core/model.rb
+++ b/lib/couchrest/core/model.rb
@@ -1,7 +1,7 @@
require 'rubygems'
require 'extlib'
require 'digest/md5'
-
+require File.dirname(__FILE__) + '/document'
# = CouchRest::Model - ORM, the CouchDB way
module CouchRest
# = CouchRest::Model - ORM, the CouchDB way
@@ -68,15 +68,12 @@ module CouchRest
#
# Article.by_tags :key => "ruby", :reduce => true
#
- class Model < Hash
+ class Model < Document
# instantiates the hash by converting all the keys to strings.
def initialize keys = {}
- super()
+ super(keys)
apply_defaults
- keys.each do |k,v|
- self[k.to_s] = v
- end
cast_keys
unless self['_id'] && self['_rev']
self['couchrest-type'] = self.class.to_s
@@ -90,7 +87,7 @@ module CouchRest
class_inheritable_accessor :casts
class_inheritable_accessor :default_obj
class_inheritable_accessor :class_database
- class_inheritable_accessor :generated_design_doc
+ class_inheritable_accessor :design_doc
class_inheritable_accessor :design_doc_slug_cache
class_inheritable_accessor :design_doc_fresh
@@ -114,14 +111,15 @@ module CouchRest
# Load all documents that have the "couchrest-type" field equal to the
# name of the current class. Take thes the standard set of
# CouchRest::Database#view options.
- def all opts = {}
- self.generated_design_doc ||= default_design_doc
+ def all opts = {}, &block
+ self.design_doc ||= Design.new(default_design_doc)
unless design_doc_fresh
refresh_design_doc
end
- view_name = "#{design_doc_slug}/all"
- raw = opts.delete(:raw)
- fetch_view_with_docs(view_name, opts, raw)
+ # view_name = "#{design_doc_slug}/all"
+ # raw = opts.delete(:raw)
+ # fetch_view_with_docs(view_name, opts, raw)
+ view :all, opts, &block
end
# Cast a field as another class. The class must be happy to have the
@@ -266,46 +264,61 @@ module CouchRest
# To understand the capabilities of this view system more compeletly,
# it is recommended that you read the RSpec file at
# spec/core/model_spec.rb.
+
def view_by *keys
+ self.design_doc ||= Design.new(default_design_doc)
opts = keys.pop if keys.last.is_a?(Hash)
opts ||= {}
- type = self.to_s
-
- method_name = "by_#{keys.join('_and_')}"
- self.generated_design_doc ||= default_design_doc
ducktype = opts.delete(:ducktype)
- if opts[:map]
- view = {}
- view['map'] = opts.delete(:map)
- if opts[:reduce]
- view['reduce'] = opts.delete(:reduce)
- opts[:reduce] = false
- end
- generated_design_doc['views'][method_name] = view
- else
- doc_keys = keys.collect{|k|"doc['#{k}']"}
- key_protection = doc_keys.join(' && ')
- key_emit = doc_keys.length == 1 ? "#{doc_keys.first}" : "[#{doc_keys.join(', ')}]"
- map_function = <<-JAVASCRIPT
- function(doc) {
- if (#{!ducktype ? "doc['couchrest-type'] == '#{type}' && " : ""}#{key_protection}) {
- emit(#{key_emit}, null);
- }
- }
- JAVASCRIPT
- generated_design_doc['views'][method_name] = {
- 'map' => map_function
- }
- end
- generated_design_doc['views'][method_name]['couchrest-defaults'] = opts
+ # if ducktype
+ # end
+ keys.push opts
+ self.design_doc.view_by(*keys)
self.design_doc_fresh = false
- method_name
end
+ # def view_by *keys
+ # opts = keys.pop if keys.last.is_a?(Hash)
+ # opts ||= {}
+ #
+ #
+ # type = self.to_s
+ #
+ # method_name = "by_#{keys.join('_and_')}"
+ # self.generated_design_doc ||= default_design_doc
+ # ducktype = opts.delete(:ducktype)
+ # if opts[:map]
+ # view = {}
+ # view['map'] = opts.delete(:map)
+ # if opts[:reduce]
+ # view['reduce'] = opts.delete(:reduce)
+ # opts[:reduce] = false
+ # end
+ # generated_design_doc['views'][method_name] = view
+ # else
+ # doc_keys = keys.collect{|k|"doc['#{k}']"}
+ # key_protection = doc_keys.join(' && ')
+ # key_emit = doc_keys.length == 1 ? "#{doc_keys.first}" : "[#{doc_keys.join(', ')}]"
+ # map_function = <<-JAVASCRIPT
+ # function(doc) {
+ # if (#{!ducktype ? "doc['couchrest-type'] == '#{type}' && " : ""}#{key_protection}) {
+ # emit(#{key_emit}, null);
+ # }
+ # }
+ # JAVASCRIPT
+ # generated_design_doc['views'][method_name] = {
+ # 'map' => map_function
+ # }
+ # end
+ # generated_design_doc['views'][method_name]['couchrest-defaults'] = opts
+ # self.design_doc_fresh = false
+ # method_name
+ # end
+
def method_missing m, *args
- if opts = has_view?(m)
+ if has_view?(m)
query = args.shift || {}
- view(m, opts.merge(query), *args)
+ view(m, query, *args)
else
super
end
@@ -314,15 +327,14 @@ module CouchRest
# returns stored defaults if the there is a view named this in the design doc
def has_view?(view)
view = view.to_s
- if generated_design_doc['views'][view]
- generated_design_doc['views'][view]["couchrest-defaults"]
- end
+ design_doc && design_doc['views'] && design_doc['views'][view]
end
- # Fetch the generated design doc. Could raise an error if the generated views have not been queried yet.
- def design_doc
- database.get("_design/#{design_doc_slug}")
- end
+ # # Fetch the generated design doc. Could raise an error if the generated
+ # # views have not been queried yet.
+ # def design_doc
+ # database.get("_design/#{design_doc_slug}")
+ # end
# Dispatches to any named view.
def view name, query={}, &block
@@ -331,8 +343,7 @@ module CouchRest
end
query[:raw] = true if query[:reduce]
raw = query.delete(:raw)
- view_name = "#{design_doc_slug}/#{name}"
- fetch_view_with_docs(view_name, query, raw, &block)
+ fetch_view_with_docs(name, query, raw, &block)
end
private
@@ -356,7 +367,7 @@ module CouchRest
def fetch_view view_name, opts, &block
retryable = true
begin
- database.view(view_name, opts, &block)
+ design_doc.view(view_name, opts, &block)
# the design doc could have been deleted by a rouge process
rescue RestClient::ResourceNotFound => e
if retryable
@@ -372,7 +383,7 @@ module CouchRest
def design_doc_slug
return design_doc_slug_cache if design_doc_slug_cache && design_doc_fresh
funcs = []
- generated_design_doc['views'].each do |name, view|
+ design_doc['views'].each do |name, view|
funcs << "#{name}/#{view['map']}#{view['reduce']}"
end
md5 = Digest::MD5.hexdigest(funcs.sort.join(''))
@@ -398,13 +409,15 @@ module CouchRest
did = "_design/#{design_doc_slug}"
saved = database.get(did) rescue nil
if saved
- generated_design_doc['views'].each do |name, view|
+ design_doc['views'].each do |name, view|
saved['views'][name] = view
end
database.save(saved)
else
- generated_design_doc['_id'] = did
- database.save(generated_design_doc)
+ design_doc['_id'] = did
+ design_doc.delete('_rev')
+ design_doc.database = database
+ design_doc.save
end
self.design_doc_fresh = true
end
@@ -416,16 +429,6 @@ module CouchRest
self.class.database
end
- # alias for self['_id']
- def id
- self['_id']
- end
-
- # alias for self['_rev']
- def rev
- self['_rev']
- end
-
# Takes a hash as argument, and applies the values by using writer methods
# for each key. Raises a NoMethodError if the corresponding methods are
# missing. In case of error, no attributes are changed.
@@ -439,63 +442,40 @@ module CouchRest
save
end
- # returns true if the document has never been saved
- def new_record?
- !rev
- end
+ # for compatibility with old-school frameworks
+ alias :new_record? :new_document?
- # Saves the document to the db using create or update. Also runs the :save
- # callbacks. Sets the _id and _rev fields based on
- # CouchDB's response.
- def save
- if new_record?
- create
+ # We override this to create the create and update callback opportunities.
+ # I think we should drop those and just have save. If you care, in your callback,
+ # check new_document?
+ def save actually=false
+ if actually
+ super()
else
- update
- end
+ if new_document?
+ create
+ else
+ update
+ end
+ end
end
-
- # Deletes the document from the database. Runs the :delete callbacks.
- # Removes the _id and _rev fields, preparing the
- # document to be saved to a new _id.
- def destroy
- result = database.delete self
- if result['ok']
- self['_rev'] = nil
- self['_id'] = nil
- end
- result['ok']
- end
-
- protected
-
- # Saves a document for the first time, after running the before(:create)
- # callbacks, and applying the unique_id.
- def create
- set_unique_id if respond_to?(:set_unique_id) # hack
- save_doc
- end
-
- # Saves the document and runs the :update callbacks.
+
def update
- save_doc
+ save :actually
+ end
+
+ def create
+ # can we use the callbacks for this?
+ set_unique_id if self.respond_to?(:set_unique_id)
+ save :actually
end
private
- def save_doc
- result = database.save self
- if result['ok']
- self['_id'] = result['id']
- self['_rev'] = result['rev']
- end
- result['ok']
- end
-
def apply_defaults
if self.class.default
self.class.default.each do |k,v|
- self[k.to_s] = v
+ self[k] = v
end
end
end
@@ -518,6 +498,8 @@ module CouchRest
end
include ::Extlib::Hook
+ # todo: drop create and update hooks...
+ # (use new_record? in callbacks if you care)
register_instance_hooks :save, :create, :update, :destroy
end # class Model
diff --git a/lib/couchrest/core/view.rb b/lib/couchrest/core/view.rb
new file mode 100644
index 0000000..b05dc40
--- /dev/null
+++ b/lib/couchrest/core/view.rb
@@ -0,0 +1,4 @@
+module CouchRest
+ class View
+ end
+end
\ No newline at end of file
diff --git a/spec/couchrest/core/database_spec.rb b/spec/couchrest/core/database_spec.rb
index a51ee26..5fb2e70 100644
--- a/spec/couchrest/core/database_spec.rb
+++ b/spec/couchrest/core/database_spec.rb
@@ -215,7 +215,7 @@ describe CouchRest::Database do
r2["lemons"].should == "from texas"
end
it "should use PUT with UUIDs" do
- CouchRest.should_receive(:put)
+ CouchRest.should_receive(:put).and_return({"ok" => true, "id" => "100", "rev" => "55"})
r = @db.save({'just' => ['another document']})
end
@@ -418,6 +418,9 @@ describe CouchRest::Database do
@db.delete doc
lambda{@db.get @docid}.should raise_error
end
+ it "should fail without an _id" do
+ lambda{@db.delete({"not"=>"a real doc"})}.should raise_error(ArgumentError)
+ end
end
it "should list documents" do
diff --git a/spec/couchrest/core/design_spec.rb b/spec/couchrest/core/design_spec.rb
new file mode 100644
index 0000000..c408383
--- /dev/null
+++ b/spec/couchrest/core/design_spec.rb
@@ -0,0 +1,131 @@
+require File.dirname(__FILE__) + '/../../spec_helper'
+
+describe CouchRest::Design do
+
+ describe "defining a view" do
+ it "should add a view to the design doc" do
+ @des = CouchRest::Design.new
+ method = @des.view_by :name
+ method.should == "by_name"
+ @des["views"]["by_name"].should_not be_nil
+ end
+ end
+
+ describe "with an unsaved view" do
+ before(:each) do
+ @des = CouchRest::Design.new
+ method = @des.view_by :name
+ end
+ it "should accept a name" do
+ @des.name = "mytest"
+ @des.name.should == "mytest"
+ end
+ it "should not save on view definition" do
+ @des.rev.should be_nil
+ end
+ it "should freak out on view access" do
+ lambda{@des.view :by_name}.should raise_error
+ end
+ end
+
+ describe "saving" do
+ before(:each) do
+ @des = CouchRest::Design.new
+ method = @des.view_by :name
+ @des.database = reset_test_db!
+ end
+ it "should fail without a name" do
+ lambda{@des.save}.should raise_error(ArgumentError)
+ end
+ it "should work with a name" do
+ @des.name = "myview"
+ @des.save
+ end
+ end
+
+ describe "when it's saved" do
+ before(:each) do
+ @db = reset_test_db!
+ @db.bulk_save([{"name" => "x"},{"name" => "y"}])
+ @des = CouchRest::Design.new
+ @des.database = @db
+ method = @des.view_by :name
+ end
+ it "should by queryable when it's saved" do
+ @des.name = "mydesign"
+ @des.save
+ res = @des.view :by_name
+ res["rows"][0]["key"].should == "x"
+ end
+ end
+
+ describe "from a saved document" do
+ before(:all) do
+ @db = reset_test_db!
+ @db.save({
+ "_id" => "_design/test",
+ "views" => {
+ "by_name" => {
+ "map" => "function(doc){if (doc.name) emit(doc.name, null)}"
+ }
+ }
+ })
+ @db.bulk_save([{"name" => "a"},{"name" => "b"}])
+ @des = @db.get "_design/test"
+ end
+ it "should be a Design" do
+ @des.should be_an_instance_of CouchRest::Design
+ end
+ it "should have a modifiable name" do
+ @des.name.should == "test"
+ @des.name = "supertest"
+ @des.id.should == "_design/supertest"
+ end
+ it "should by queryable" do
+ res = @des.view :by_name
+ res["rows"][0]["key"].should == "a"
+ end
+ end
+
+ describe "a view with default options" do
+ before(:all) do
+ @db = reset_test_db!
+ @des = CouchRest::Design.new
+ @des.name = "test"
+ method = @des.view_by :name, :descending => true
+ @des.database = @db
+ @des.save
+ @db.bulk_save([{"name" => "a"},{"name" => "z"}])
+ end
+ it "should save them" do
+ @d2 = @db.get(@des.id)
+ @d2["views"]["by_name"]["couchrest-defaults"].should == {"descending"=>true}
+ end
+ it "should use them" do
+ res = @des.view :by_name
+ res["rows"].first["key"].should == "z"
+ end
+ it "should override them" do
+ res = @des.view :by_name, :descending => false
+ res["rows"].first["key"].should == "a"
+ end
+ end
+
+ describe "a view with multiple keys" do
+ before(:all) do
+ @db = reset_test_db!
+ @des = CouchRest::Design.new
+ @des.name = "test"
+ method = @des.view_by :name, :age
+ @des.database = @db
+ @des.save
+ @db.bulk_save([{"name" => "a", "age" => 2},
+ {"name" => "a", "age" => 4},{"name" => "z", "age" => 9}])
+ end
+ it "should work" do
+ res = @des.view :by_name_and_age
+ res["rows"].first["key"].should == ["a",2]
+ end
+ end
+
+end
\ No newline at end of file
diff --git a/spec/couchrest/core/document_spec.rb b/spec/couchrest/core/document_spec.rb
new file mode 100644
index 0000000..31c7151
--- /dev/null
+++ b/spec/couchrest/core/document_spec.rb
@@ -0,0 +1,96 @@
+require File.dirname(__FILE__) + '/../../spec_helper'
+
+describe CouchRest::Document, "[]=" do
+ before(:each) do
+ @doc = CouchRest::Document.new
+ end
+ it "should work" do
+ @doc["enamel"].should == nil
+ @doc["enamel"] = "Strong"
+ @doc["enamel"].should == "Strong"
+ end
+ it "[]= should convert to string" do
+ @doc["enamel"].should == nil
+ @doc[:enamel] = "Strong"
+ @doc["enamel"].should == "Strong"
+ end
+ it "should read as a string" do
+ @doc[:enamel] = "Strong"
+ @doc[:enamel].should == "Strong"
+ end
+end
+
+describe CouchRest::Document, "new" do
+ before(:each) do
+ @doc = CouchRest::Document.new("key" => [1,2,3], :more => "values")
+ end
+ it "should create itself from a Hash" do
+ @doc["key"].should == [1,2,3]
+ @doc["more"].should == "values"
+ end
+ it "should not have rev and id" do
+ @doc.rev.should be_nil
+ @doc.id.should be_nil
+ end
+ it "should freak out when saving without a database" do
+ lambda{@doc.save}.should raise_error(ArgumentError)
+ end
+end
+
+# move to database spec
+describe CouchRest::Document, "saving using a database" do
+ before(:all) do
+ @doc = CouchRest::Document.new("key" => [1,2,3], :more => "values")
+ @db = reset_test_db!
+ @resp = @db.save(@doc)
+ end
+ it "should apply the database" do
+ @doc.database.should == @db
+ end
+ it "should get id and rev" do
+ @doc.id.should == @resp["id"]
+ @doc.rev.should == @resp["rev"]
+ end
+end
+
+describe "getting from a database" do
+ before(:all) do
+ @db = reset_test_db!
+ @resp = @db.save({
+ "key" => "value"
+ })
+ @doc = @db.get @resp['id']
+ end
+ it "should return a document" do
+ @doc.should be_an_instance_of(CouchRest::Document)
+ end
+ it "should have a database" do
+ @doc.database.should == @db
+ end
+ it "should be saveable and resavable" do
+ @doc["more"] = "keys"
+ @doc.save
+ @db.get(@resp['id'])["more"].should == "keys"
+ @doc["more"] = "these keys"
+ @doc.save
+ @db.get(@resp['id'])["more"].should == "these keys"
+ end
+end
+
+describe "destroying a document from a db" do
+ before(:all) do
+ @db = reset_test_db!
+ @resp = @db.save({
+ "key" => "value"
+ })
+ @doc = @db.get @resp['id']
+ end
+ it "should make it disappear" do
+ @doc.destroy
+ lambda{@db.get @resp['id']}.should raise_error
+ end
+ it "should error when there's no db" do
+ @doc = CouchRest::Document.new("key" => [1,2,3], :more => "values")
+ lambda{@doc.destroy}.should raise_error(ArgumentError)
+ end
+end
\ No newline at end of file
diff --git a/spec/couchrest/core/model_spec.rb b/spec/couchrest/core/model_spec.rb
index d350335..20ddbbf 100644
--- a/spec/couchrest/core/model_spec.rb
+++ b/spec/couchrest/core/model_spec.rb
@@ -16,6 +16,7 @@ end
class Question < CouchRest::Model
key_accessor :q, :a
+ couchrest_type = 'Question'
end
class Person < CouchRest::Model
@@ -213,7 +214,7 @@ describe CouchRest::Model do
@course["questions"][0].a[0].should == "beast"
end
end
-
+
describe "finding all instances of a model" do
before(:all) do
WithTemplate.new('important-field' => '1').save
@@ -285,6 +286,11 @@ describe CouchRest::Model do
Article.database.delete(@old) if @old
end
+ it "should be a new document" do
+ @art.should be_a_new_document
+ @art.title.should be_nil
+ end
+
it "should require the title" do
lambda{@art.save}.should raise_error
@art.title = 'This is the title'
@@ -390,10 +396,14 @@ describe CouchRest::Model do
written_at += 24 * 3600
end
end
+
+ it "should have a design doc" do
+ Article.design_doc["views"]["by_date"].should_not be_nil
+ end
- it "should create the design doc" do
- Article.by_date rescue nil
- doc = Article.design_doc
+ it "should save the design doc" do
+ Article.by_date #rescue nil
+ doc = Article.database.get Article.design_doc.id
doc['views']['by_date'].should_not be_nil
end
@@ -402,7 +412,7 @@ describe CouchRest::Model do
view['rows'].length.should == 4
end
- it "should return the matching objects (with descending)" do
+ it "should return the matching objects (with default argument :descending => true)" do
articles = Article.by_date
articles.collect{|a|a.title}.should == @titles.reverse
end
@@ -417,10 +427,9 @@ describe CouchRest::Model do
before(:all) do
Course.database.delete! rescue nil
@db = @cr.create_db(TESTDB) rescue nil
- Course.new(:title => 'aaa').save
- Course.new(:title => 'bbb').save
- Course.new(:title => 'ddd').save
- Course.new(:title => 'eee').save
+ %w{aaa bbb ddd eee}.each do |title|
+ Course.new(:title => title).save
+ end
end
it "should make the design doc upon first query" do
Course.by_title
@@ -442,13 +451,12 @@ describe CouchRest::Model do
courses = []
rs = Course.by_title # remove me
Course.view(:by_title) do |course|
- # puts "course"
courses << course
end
- # courses.should == 'x'
courses[0]["doc"]["title"].should =='aaa'
end
end
+
describe "a ducktype view" do
before(:all) do
@@ -527,6 +535,7 @@ describe CouchRest::Model do
end
end
+ # TODO: moved to Design, delete
describe "adding a view" do
before(:each) do
Article.by_date
@@ -544,6 +553,8 @@ describe CouchRest::Model do
Article.by_updated_at
newdocs = Article.database.documents :startkey => "_design/",
:endkey => "_design/\u9999"
+ # puts @design_docs.inspect
+ # puts newdocs.inspect
newdocs["rows"].length.should == @design_docs["rows"].length + 1
end
end
diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb
index 3cb4791..c6ff9c4 100644
--- a/spec/spec_helper.rb
+++ b/spec/spec_helper.rb
@@ -3,4 +3,12 @@ require File.dirname(__FILE__) + '/../lib/couchrest'
FIXTURE_PATH = File.dirname(__FILE__) + '/fixtures'
COUCHHOST = "http://localhost:5984"
-TESTDB = 'couchrest-test'
\ No newline at end of file
+TESTDB = 'couchrest-test'
+
+def reset_test_db!
+ cr = CouchRest.new(COUCHHOST)
+ db = cr.database(TESTDB)
+ db.delete! rescue nil
+ db = cr.create_db(TESTDB) rescue nin
+ db
+end
\ No newline at end of file