samlown-couchrest 0.35

data/spec/couchrest/more/extended_doc_spec.rb

@@ -0,0 +1,797 @@
+# encoding: utf-8
+
+require File.expand_path("../../../spec_helper", __FILE__)
+require File.join(FIXTURE_PATH, 'more', 'article')
+require File.join(FIXTURE_PATH, 'more', 'course')
+require File.join(FIXTURE_PATH, 'more', 'cat')
+
+describe "ExtendedDocument" do
+  
+  class WithDefaultValues < CouchRest::ExtendedDocument
+    use_database TEST_SERVER.default_database
+    property :preset, :type => 'Object', :default => {:right => 10, :top_align => false}
+    property :set_by_proc, :default => Proc.new{Time.now}, :cast_as => 'Time'
+    property :tags, :type => ['String'], :default => []
+    property :read_only_with_default, :default => 'generic', :read_only => true
+    property :default_false, :type => 'Boolean', :default => false
+    property :name
+    timestamps!
+  end
+  
+  class WithCallBacks < CouchRest::ExtendedDocument
+    include ::CouchRest::Validation
+    use_database TEST_SERVER.default_database
+    property :name
+    property :run_before_validate
+    property :run_after_validate
+    property :run_before_save
+    property :run_after_save
+    property :run_before_create
+    property :run_after_create
+    property :run_before_update
+    property :run_after_update
+    
+    before_validate do |object|
+      object.run_before_validate = true
+    end
+    after_validate do |object| 
+      object.run_after_validate = true
+    end
+    before_save do |object| 
+      object.run_before_save = true
+    end
+    after_save do |object| 
+      object.run_after_save = true
+    end
+    before_create do |object| 
+      object.run_before_create = true
+    end
+    after_create do |object| 
+      object.run_after_create = true
+    end
+    before_update do |object| 
+      object.run_before_update = true
+    end
+    after_update do |object| 
+      object.run_after_update = true
+    end
+    
+    property :run_one
+    property :run_two
+    property :run_three
+    
+    before_save :run_one_method, :run_two_method do |object| 
+      object.run_three = true
+    end
+    def run_one_method
+      self.run_one = true
+    end
+    def run_two_method
+      self.run_two = true
+    end
+    
+    attr_accessor :run_it
+    property :conditional_one
+    property :conditional_two
+    
+    before_save :conditional_one_method, :conditional_two_method, :if => proc { self.run_it }
+    def conditional_one_method
+      self.conditional_one = true
+    end
+    def conditional_two_method
+      self.conditional_two = true
+    end
+  end
+  
+  class WithTemplateAndUniqueID < CouchRest::ExtendedDocument
+    use_database TEST_SERVER.default_database
+    unique_id do |model|
+      model['important-field']
+    end
+    property :preset, :default => 'value'
+    property :has_no_default
+  end
+
+  class WithGetterAndSetterMethods < CouchRest::ExtendedDocument
+    use_database TEST_SERVER.default_database
+    
+    property :other_arg
+    def arg
+      other_arg
+    end
+
+    def arg=(value)
+      self.other_arg = "foo-#{value}"
+    end
+  end
+
+  class WithAfterInitializeMethod < CouchRest::ExtendedDocument
+    use_database TEST_SERVER.default_database
+    
+    property :some_value
+
+    def after_initialize
+      self.some_value ||= "value"
+    end
+
+  end
+
+  
+  before(:each) do
+    @obj = WithDefaultValues.new
+  end
+  
+  describe "instance database connection" do
+    it "should use the default database" do
+      @obj.database.name.should == 'couchrest-test'
+    end
+    
+    it "should override the default db" do
+      @obj.database = TEST_SERVER.database!('couchrest-extendedmodel-test')
+      @obj.database.name.should == 'couchrest-extendedmodel-test'
+      @obj.database.delete!
+    end
+  end
+  
+  describe "a new model" do
+    it "should be a new document" do
+      @obj = Basic.new
+      @obj.rev.should be_nil
+      @obj.should be_new
+      @obj.should be_new_document
+      @obj.should be_new_record
+    end
+
+    it "should not failed on a nil value in argument" do
+      @obj = Basic.new(nil)
+      @obj.should == { 'couchrest-type' => 'Basic' }
+    end
+  end
+  
+  describe "creating a new document" do
+    it "should instantialize and save a document" do
+      article = Article.create(:title => 'my test')
+      article.title.should == 'my test'
+      article.should_not be_new
+    end 
+    
+    it "should trigger the create callbacks" do
+      doc = WithCallBacks.create(:name => 'my other test') 
+      doc.run_before_create.should be_true
+      doc.run_after_create.should be_true
+      doc.run_before_save.should be_true
+      doc.run_after_save.should be_true
+    end
+  end
+  
+  describe "update attributes without saving" do
+    before(:each) do
+      a = Article.get "big-bad-danger" rescue nil
+      a.destroy if a
+      @art = Article.new(:title => "big bad danger")
+      @art.save
+    end
+    it "should work for attribute= methods" do
+      @art['title'].should == "big bad danger"
+      @art.update_attributes_without_saving('date' => Time.now, :title => "super danger")
+      @art['title'].should == "super danger"
+    end
+    it "should silently ignore _id" do
+      @art.update_attributes_without_saving('_id' => 'foobar')
+      @art['_id'].should_not == 'foobar'
+    end
+    it "should silently ignore _rev" do
+      @art.update_attributes_without_saving('_rev' => 'foobar')
+      @art['_rev'].should_not == 'foobar'
+    end
+    it "should silently ignore created_at" do
+      @art.update_attributes_without_saving('created_at' => 'foobar')
+      @art['created_at'].should_not == 'foobar'
+    end
+    it "should silently ignore updated_at" do
+      @art.update_attributes_without_saving('updated_at' => 'foobar')
+      @art['updated_at'].should_not == 'foobar'
+    end
+    it "should also work using attributes= alias" do
+      @art.respond_to?(:attributes=).should be_true
+      @art.attributes = {'date' => Time.now, :title => "something else"}
+      @art['title'].should == "something else"
+    end
+    
+    it "should flip out if an attribute= method is missing" do
+      lambda {
+        @art.update_attributes_without_saving('slug' => "new-slug", :title => "super danger")        
+      }.should raise_error
+    end
+    
+    it "should not change other attributes if there is an error" do
+      lambda {
+        @art.update_attributes_without_saving('slug' => "new-slug", :title => "super danger")        
+      }.should raise_error
+      @art['title'].should == "big bad danger"
+    end
+  end
+  
+  describe "update attributes" do
+    before(:each) do
+      a = Article.get "big-bad-danger" rescue nil
+      a.destroy if a
+      @art = Article.new(:title => "big bad danger")
+      @art.save
+    end
+    it "should save" do
+      @art['title'].should == "big bad danger"
+      @art.update_attributes('date' => Time.now, :title => "super danger")
+      loaded = Article.get(@art.id)
+      loaded['title'].should == "super danger"
+    end
+  end
+  
+  describe "with default" do
+    it "should have the default value set at initalization" do
+      @obj.preset.should == {:right => 10, :top_align => false}
+    end
+
+    it "should have the default false value explicitly assigned" do
+      @obj.default_false.should == false
+    end
+    
+    it "should automatically call a proc default at initialization" do
+      @obj.set_by_proc.should be_an_instance_of(Time)
+      @obj.set_by_proc.should == @obj.set_by_proc
+      @obj.set_by_proc.should < Time.now
+    end
+    
+    it "should let you overwrite the default values" do
+      obj = WithDefaultValues.new(:preset => 'test')
+      obj.preset = 'test'
+    end
+    
+    it "should work with a default empty array" do
+      obj = WithDefaultValues.new(:tags => ['spec'])
+      obj.tags.should == ['spec']
+    end
+    
+    it "should set default value of read-only property" do
+      obj = WithDefaultValues.new
+      obj.read_only_with_default.should == 'generic'
+    end
+  end
+  
+  describe "a doc with template values (CR::Model spec)" do
+    before(:all) do
+      WithTemplateAndUniqueID.all.map{|o| o.destroy(true)}
+      WithTemplateAndUniqueID.database.bulk_delete
+      @tmpl = WithTemplateAndUniqueID.new
+      @tmpl2 = WithTemplateAndUniqueID.new(:preset => 'not_value', 'important-field' => '1')
+    end
+    it "should have fields set when new" do
+      @tmpl.preset.should == 'value'
+    end
+    it "shouldn't override explicitly set values" do
+      @tmpl2.preset.should == 'not_value'
+    end
+    it "shouldn't override existing documents" do
+      @tmpl2.save
+      tmpl2_reloaded = WithTemplateAndUniqueID.get(@tmpl2.id)
+      @tmpl2.preset.should == 'not_value'
+      tmpl2_reloaded.preset.should == 'not_value'
+    end
+  end
+  
+  describe "getting a model" do
+    before(:all) do
+      @art = Article.new(:title => 'All About Getting')
+      @art.save
+    end
+    it "should load and instantiate it" do
+      foundart = Article.get @art.id
+      foundart.title.should == "All About Getting"
+    end
+    
+    it "should return nil if `get` is used and the document doesn't exist" do
+      foundart = Article.get 'matt aimonetti'
+      foundart.should be_nil
+    end                     
+    
+    it "should raise an error if `get!` is used and the document doesn't exist" do
+       lambda{foundart = Article.get!('matt aimonetti')}.should raise_error
+    end
+  end
+
+  describe "getting a model with a subobjects array" do
+    before(:all) do
+      course_doc = {
+        "title" => "Metaphysics 200",
+        "questions" => [
+          {
+            "q" => "Carve the ___ of reality at the ___.",
+            "a" => ["beast","joints"]
+          },{
+            "q" => "Who layed the smack down on Leibniz's Law?",
+            "a" => "Willard Van Orman Quine"
+          }
+        ]
+      }
+      r = Course.database.save_doc course_doc
+      @course = Course.get r['id']
+    end
+    it "should load the course" do
+      @course.title.should == "Metaphysics 200"
+    end
+    it "should instantiate them as such" do
+      @course["questions"][0].a[0].should == "beast"
+    end
+  end
+  
+  describe "finding all instances of a model" do
+    before(:all) do
+      WithTemplateAndUniqueID.design_doc_fresh = false
+      WithTemplateAndUniqueID.all.map{|o| o.destroy(true)}
+      WithTemplateAndUniqueID.database.bulk_delete
+      WithTemplateAndUniqueID.new('important-field' => '1').save
+      WithTemplateAndUniqueID.new('important-field' => '2').save
+      WithTemplateAndUniqueID.new('important-field' => '3').save
+      WithTemplateAndUniqueID.new('important-field' => '4').save
+    end
+    it "should make the design doc" do
+      WithTemplateAndUniqueID.all
+      d = WithTemplateAndUniqueID.design_doc
+      d['views']['all']['map'].should include('WithTemplateAndUniqueID')
+    end
+    it "should find all" do
+      rs = WithTemplateAndUniqueID.all 
+      rs.length.should == 4
+    end
+  end
+  
+  describe "counting all instances of a model" do
+    before(:each) do
+      @db = reset_test_db!
+      WithTemplateAndUniqueID.design_doc_fresh = false
+    end
+    
+    it ".count should return 0 if there are no docuemtns" do
+      WithTemplateAndUniqueID.count.should == 0
+    end
+    
+    it ".count should return the number of documents" do
+      WithTemplateAndUniqueID.new('important-field' => '1').save
+      WithTemplateAndUniqueID.new('important-field' => '2').save
+      WithTemplateAndUniqueID.new('important-field' => '3').save
+      
+      WithTemplateAndUniqueID.count.should == 3
+    end
+  end
+  
+  describe "finding the first instance of a model" do
+    before(:each) do      
+      @db = reset_test_db!
+      WithTemplateAndUniqueID.design_doc_fresh = false
+      WithTemplateAndUniqueID.new('important-field' => '1').save
+      WithTemplateAndUniqueID.new('important-field' => '2').save
+      WithTemplateAndUniqueID.new('important-field' => '3').save
+      WithTemplateAndUniqueID.new('important-field' => '4').save
+    end
+    it "should make the design doc" do
+      WithTemplateAndUniqueID.all
+      d = WithTemplateAndUniqueID.design_doc
+      d['views']['all']['map'].should include('WithTemplateAndUniqueID')
+    end
+    it "should find first" do
+      rs = WithTemplateAndUniqueID.first
+      rs['important-field'].should == "1"
+    end
+    it "should return nil if no instances are found" do
+      WithTemplateAndUniqueID.all.each {|obj| obj.destroy }
+      WithTemplateAndUniqueID.first.should be_nil
+    end
+  end
+  
+  describe "getting a model with a subobject field" do
+    before(:all) do
+      course_doc = {
+        "title" => "Metaphysics 410",
+        "professor" => {
+          "name" => ["Mark", "Hinchliff"]
+        },
+        "ends_at" => "2008/12/19 13:00:00 +0800"
+      }
+      r = Course.database.save_doc course_doc
+      @course = Course.get r['id']
+    end
+    it "should load the course" do
+      @course["professor"]["name"][1].should == "Hinchliff"
+    end
+    it "should instantiate the professor as a person" do
+      @course['professor'].last_name.should == "Hinchliff"
+    end
+    it "should instantiate the ends_at as a Time" do
+      @course['ends_at'].should == Time.parse("2008/12/19 13:00:00 +0800")
+    end
+  end
+  
+  describe "timestamping" do
+    before(:each) do
+      oldart = Article.get "saving-this" rescue nil
+      oldart.destroy if oldart
+      @art = Article.new(:title => "Saving this")
+      @art.save
+    end
+    
+    it "should define the updated_at and created_at getters and set the values" do
+      @obj.save
+      obj = WithDefaultValues.get(@obj.id)
+      obj.should be_an_instance_of(WithDefaultValues)
+      obj.created_at.should be_an_instance_of(Time)
+      obj.updated_at.should be_an_instance_of(Time)
+      obj.created_at.to_s.should == @obj.updated_at.to_s
+    end 
+    it "should set the time on create" do
+      (Time.now - @art.created_at).should < 2
+      foundart = Article.get @art.id
+      foundart.created_at.should == foundart.updated_at
+    end
+    it "should set the time on update" do
+      @art.save
+      @art.created_at.should < @art.updated_at
+    end
+  end
+  
+  describe "basic saving and retrieving" do
+    it "should work fine" do
+      @obj.name = "should be easily saved and retrieved"
+      @obj.save
+      saved_obj = WithDefaultValues.get(@obj.id)
+      saved_obj.should_not be_nil
+    end
+    
+    it "should parse the Time attributes automatically" do
+      @obj.name = "should parse the Time attributes automatically"
+      @obj.set_by_proc.should be_an_instance_of(Time)
+      @obj.save
+      @obj.set_by_proc.should be_an_instance_of(Time)
+      saved_obj = WithDefaultValues.get(@obj.id)
+      saved_obj.set_by_proc.should be_an_instance_of(Time)
+    end
+  end
+  
+  describe "saving a model" do
+    before(:all) do
+      @sobj = Basic.new
+      @sobj.save.should == true
+    end
+    
+    it "should save the doc" do
+      doc = Basic.get(@sobj.id)
+      doc['_id'].should == @sobj.id
+    end
+    
+    it "should be set for resaving" do
+      rev = @obj.rev
+      @sobj['another-key'] = "some value"
+      @sobj.save
+      @sobj.rev.should_not == rev
+    end
+    
+    it "should set the id" do
+      @sobj.id.should be_an_instance_of(String)
+    end
+    
+    it "should set the type" do
+      @sobj['couchrest-type'].should == 'Basic'
+    end
+    
+    describe "save!" do
+      
+      before(:each) do
+        @sobj = Card.new(:first_name => "Marcos", :last_name => "Tapajós")
+      end
+      
+      it "should return true if save the document" do
+        @sobj.save!.should == true
+      end
+      
+      it "should raise error if don't save the document" do
+        @sobj.first_name = nil
+        lambda {  @sobj.save!.should == true }.should raise_error(RuntimeError)
+      end
+
+    end
+    
+    
+  end
+  
+  describe "saving a model with a unique_id configured" do
+    before(:each) do
+      @art = Article.new
+      @old = Article.database.get('this-is-the-title') rescue nil
+      Article.database.delete_doc(@old) if @old
+    end
+    
+    it "should be a new document" do
+      @art.should be_new
+      @art.title.should be_nil
+    end
+    
+    it "should require the title" do
+      lambda{@art.save}.should raise_error
+      @art.title = 'This is the title'
+      @art.save.should == true
+    end
+    
+    it "should not change the slug on update" do
+      @art.title = 'This is the title'
+      @art.save.should == true
+      @art.title = 'new title'
+      @art.save.should == true
+      @art.slug.should == 'this-is-the-title'
+    end
+    
+    it "should raise an error when the slug is taken" do
+      @art.title = 'This is the title'
+      @art.save.should == true
+      @art2 = Article.new(:title => 'This is the title!')
+      lambda{@art2.save}.should raise_error
+    end
+    
+    it "should set the slug" do
+      @art.title = 'This is the title'
+      @art.save.should == true
+      @art.slug.should == 'this-is-the-title'
+    end
+    
+    it "should set the id" do
+      @art.title = 'This is the title'
+      @art.save.should == true
+      @art.id.should == 'this-is-the-title'
+    end
+  end
+
+  describe "saving a model with a unique_id lambda" do
+    before(:each) do
+      @templated = WithTemplateAndUniqueID.new
+      @old = WithTemplateAndUniqueID.get('very-important') rescue nil
+      @old.destroy if @old
+    end
+    
+    it "should require the field" do
+      lambda{@templated.save}.should raise_error
+      @templated['important-field'] = 'very-important'
+      @templated.save.should == true
+    end
+    
+    it "should save with the id" do
+      @templated['important-field'] = 'very-important'
+      @templated.save.should == true
+      t = WithTemplateAndUniqueID.get('very-important')
+      t.should == @templated
+    end
+    
+    it "should not change the id on update" do
+      @templated['important-field'] = 'very-important'
+      @templated.save.should == true
+      @templated['important-field'] = 'not-important'
+      @templated.save.should == true
+      t = WithTemplateAndUniqueID.get('very-important')
+      t.should == @templated
+    end
+    
+    it "should raise an error when the id is taken" do
+      @templated['important-field'] = 'very-important'
+      @templated.save.should == true
+      lambda{WithTemplateAndUniqueID.new('important-field' => 'very-important').save}.should raise_error
+    end
+    
+    it "should set the id" do
+      @templated['important-field'] = 'very-important'
+      @templated.save.should == true
+      @templated.id.should == 'very-important'
+    end
+  end
+  
+  describe "destroying an instance" do
+    before(:each) do
+      @dobj = Basic.new
+      @dobj.save.should == true
+    end
+    it "should return true" do
+      result = @dobj.destroy
+      result.should == true
+    end
+    it "should be resavable" do
+      @dobj.destroy
+      @dobj.rev.should be_nil
+      @dobj.id.should be_nil
+      @dobj.save.should == true 
+    end
+    it "should make it go away" do
+      @dobj.destroy
+      lambda{Basic.get!(@dobj.id)}.should raise_error
+    end
+  end
+  
+  
+  describe "callbacks" do
+    
+    before(:each) do
+      @doc = WithCallBacks.new
+    end
+    
+    
+    describe "validate" do
+      it "should run before_validate before validating" do
+        @doc.run_before_validate.should be_nil
+        @doc.should be_valid
+        @doc.run_before_validate.should be_true
+      end
+      it "should run after_validate after validating" do
+        @doc.run_after_validate.should be_nil
+        @doc.should be_valid
+        @doc.run_after_validate.should be_true
+      end
+    end
+    describe "save" do
+      it "should run the after filter after saving" do
+        @doc.run_after_save.should be_nil
+        @doc.save.should be_true
+        @doc.run_after_save.should be_true
+      end
+      it "should run the grouped callbacks before saving" do
+        @doc.run_one.should be_nil
+        @doc.run_two.should be_nil
+        @doc.run_three.should be_nil
+        @doc.save.should be_true
+        @doc.run_one.should be_true
+        @doc.run_two.should be_true
+        @doc.run_three.should be_true
+      end
+      it "should not run conditional callbacks" do
+        @doc.run_it = false
+        @doc.save.should be_true
+        @doc.conditional_one.should be_nil
+        @doc.conditional_two.should be_nil
+      end
+      it "should run conditional callbacks" do
+        @doc.run_it = true
+        @doc.save.should be_true
+        @doc.conditional_one.should be_true
+        @doc.conditional_two.should be_true
+      end
+    end
+    describe "create" do
+      it "should run the before save filter when creating" do
+        @doc.run_before_save.should be_nil
+        @doc.create.should_not be_nil
+        @doc.run_before_save.should be_true
+      end
+      it "should run the before create filter" do
+        @doc.run_before_create.should be_nil
+        @doc.create.should_not be_nil
+        @doc.create
+        @doc.run_before_create.should be_true
+      end
+      it "should run the after create filter" do
+        @doc.run_after_create.should be_nil
+        @doc.create.should_not be_nil
+        @doc.create
+        @doc.run_after_create.should be_true
+      end
+    end
+    describe "update" do
+      
+      before(:each) do
+        @doc.save
+      end      
+      it "should run the before update filter when updating an existing document" do
+        @doc.run_before_update.should be_nil
+        @doc.update
+        @doc.run_before_update.should be_true
+      end
+      it "should run the after update filter when updating an existing document" do
+        @doc.run_after_update.should be_nil
+        @doc.update
+        @doc.run_after_update.should be_true
+      end
+      it "should run the before update filter when saving an existing document" do
+        @doc.run_before_update.should be_nil
+        @doc.save
+        @doc.run_before_update.should be_true
+      end
+      
+    end
+  end
+
+  describe "getter and setter methods" do
+    it "should try to call the arg= method before setting :arg in the hash" do
+      @doc = WithGetterAndSetterMethods.new(:arg => "foo")
+      @doc['arg'].should be_nil
+      @doc[:arg].should be_nil
+      @doc.other_arg.should == "foo-foo"
+    end
+  end
+
+  describe "initialization" do
+    it "should call after_initialize method if available" do
+      @doc = WithAfterInitializeMethod.new
+      @doc['some_value'].should eql('value')
+    end
+  end
+  
+  describe "recursive validation on an extended document" do
+    before :each do
+      reset_test_db!
+      @cat = Cat.new(:name => 'Sockington')
+    end
+    
+    it "should not save if a nested casted model is invalid" do
+      @cat.favorite_toy = CatToy.new
+      @cat.should_not be_valid
+      @cat.save.should be_false
+      lambda{@cat.save!}.should raise_error
+    end
+    
+    it "should save when nested casted model is valid" do
+      @cat.favorite_toy = CatToy.new(:name => 'Squeaky')
+      @cat.should be_valid
+      @cat.save.should be_true
+      lambda{@cat.save!}.should_not raise_error
+    end
+    
+    it "should not save when nested collection contains an invalid casted model" do
+      @cat.toys = [CatToy.new(:name => 'Feather'), CatToy.new]
+      @cat.should_not be_valid
+      @cat.save.should be_false
+      lambda{@cat.save!}.should raise_error
+    end
+    
+    it "should save when nested collection contains valid casted models" do
+      @cat.toys = [CatToy.new(:name => 'feather'), CatToy.new(:name => 'ball-o-twine')]
+      @cat.should be_valid
+      @cat.save.should be_true
+      lambda{@cat.save!}.should_not raise_error
+    end
+    
+    it "should not fail if the nested casted model doesn't have validation" do
+      Cat.property :trainer, :cast_as => 'Person'
+      Cat.validates_presence_of :name
+      cat = Cat.new(:name => 'Mr Bigglesworth')
+      cat.trainer = Person.new
+      cat.trainer.validatable?.should be_false
+      cat.should be_valid
+      cat.save.should be_true
+    end
+  end
+
+  describe "searching the contents of an extended document" do
+    before :each do
+      @db = reset_test_db!
+
+      names = ["Fuzzy", "Whiskers", "Mr Bigglesworth", "Sockington", "Smitty", "Sammy", "Samson", "Simon"]
+      names.each { |name| Cat.create(:name => name) }
+
+      search_function = { 'defaults' => {'store' => 'no', 'index' => 'analyzed_no_norms'},
+          'index' => "function(doc) { ret = new Document(); ret.add(doc['name'], {'field':'name'}); return ret; }" }
+      @db.save_doc({'_id' => '_design/search', 'fulltext' => {'cats' => search_function}})
+    end
+
+    it "should be able to paginate through a large set of search results" do
+      if couchdb_lucene_available?
+        names = []
+        Cat.paginated_each(:design_doc => "search", :view_name => "cats",
+             :q => 'name:S*', :search => true, :include_docs => true, :per_page => 3) do |cat|
+           cat.should_not be_nil
+           names << cat.name
+        end
+
+        names.size.should == 5
+        names.should include('Sockington')
+        names.should include('Smitty')
+        names.should include('Sammy')
+        names.should include('Samson')
+        names.should include('Simon')
+      end
+    end
+  end
+
+end