minimapper 0.9.0 → 0.10.0

data/Gemfile

@@ -1,7 +1,7 @@
 source 'https://rubygems.org'
 
 # We don't require active_record to use minimapper, only to
-# use minimapper/mapper/ar. We do require it for the tests though :)
+# use minimapper/mapper. We do require it for the tests though :)
 #
 # Also locked to below rails 4.0 as our specs use rails 3 features. Need to look into
 # if there are any issues with rails 4.

data/README.md

@@ -98,7 +98,7 @@ old_id = user.id
 user_mapper.delete(user)
 p user.id                        # => nil
 p user_mapper.find_by_id(old_id) # => nil
-# user_mapper.find(old_id)       # raises Minimapper::EntityNotFound
+# user_mapper.find(old_id)       # raises ActiveRecord::RecordNotFound
 # user_mapper.delete_all
 # user_mapper.delete_by_id(1)
 
@@ -285,29 +285,6 @@ When you do `mapper.delete(entity)`, it will use [ActiveRecord's `delete`](http:
 
 (FIXME?: Should we support `destroy` instead or as well?)
 
-### Exceptions and status 404
-
-Code like `UserMapper.find(123)` will raise `Minimapper::EntityNotFound` if there's no such record.
-
-Rails treats the similar `ActiveRecord::RecordNotFound` exception as a 404 error, using that status code with the error backtrace in development and test, and showing the 404 page in production.
-
-If you want the same for Minimapper, you can add a `config/initializers/minimapper.rb` containing:
-
-``` ruby
-# For 404 status with the dev/test backtrace, and a 404 page in production.
-ActionDispatch::ExceptionWrapper.rescue_responses.merge!(
-  "Minimapper::EntityNotFound" => :not_found
-)
-```
-
-You may also want to ignore those errors in your error logger. With [Honeybadger](https://www.honeybadger.io), you do:
-
-``` ruby
-Honeybadger.configure do |config|
-  config.ignore << "Minimapper::EntityNotFound"
-end
-```
-
 ### Custom entity class
 
 [Minimapper::Entity](https://github.com/joakimk/minimapper/blob/master/lib/minimapper/entity.rb) adds some convenience methods for when a model is used within a Rails application. If you don't need that you can just include the core API from the [Minimapper::Entity::Core](https://github.com/joakimk/minimapper/blob/master/lib/minimapper/entity/core.rb) module (or implement your own version that behaves like [Minimapper::Entity::Core](https://github.com/joakimk/minimapper/blob/master/lib/minimapper/entity/core.rb)).
@@ -327,8 +304,7 @@ Robert "Uncle Bob" Martin:
 
 ### Apps
 
-* About 4 people at [Barsoom](http://barsoom.se/) are currently working full time on building a rails app that uses minimapper. We'll be extending minimapper as we go.
-* The deploy status app that minimapper was extracted from: [https://github.com/joakimk/deployer](https://github.com/joakimk/deployer)
+* [Barsoom](http://barsoom.se/) has built an app which at the time of writing has 16 tables mapped with minimapper. Most additions to minimapper comes from this app.
 
 ## Contributing
 

data/lib/minimapper/mapper.rb

@@ -1,6 +1,4 @@
 module Minimapper
-  EntityNotFound = Class.new(StandardError)
-
   class Mapper
     attr_accessor :repository
 
@@ -9,16 +7,18 @@ module Minimapper
     def create(entity)
       record = record_class.new
 
-      copy_attributes_to_record(record, entity)
-      validate_record_and_copy_errors_to_entity(record, entity)
-
-      if entity.valid?
-        record.save!
-        entity.mark_as_persisted
-        entity.id = record.id
-        entity.id
-      else
-        false
+      with_save_hooks(entity, record) do
+        copy_attributes_to_record(record, entity)
+        validate_record_and_copy_errors_to_entity(record, entity)
+
+        if entity.valid?
+          record.save!
+          entity.mark_as_persisted
+          entity.id = record.id
+          entity.id
+        else
+          false
+        end
       end
     end
 
@@ -44,6 +44,10 @@ module Minimapper
       entity_for(record_class.order("id ASC").last)
     end
 
+    def reload(entity)
+      find(entity.id)
+    end
+
     def count
       record_class.count
     end
@@ -53,14 +57,16 @@ module Minimapper
     def update(entity)
       record = record_for(entity)
 
-      copy_attributes_to_record(record, entity)
-      validate_record_and_copy_errors_to_entity(record, entity)
+      with_save_hooks(entity, record) do
+        copy_attributes_to_record(record, entity)
+        validate_record_and_copy_errors_to_entity(record, entity)
 
-      if entity.valid?
-        record.save!
-        true
-      else
-        false
+        if entity.valid?
+          record.save!
+          true
+        else
+          false
+        end
       end
     end
 
@@ -110,8 +116,7 @@ module Minimapper
     end
 
     def find_record_safely(id)
-      find_record(id) ||
-        raise(EntityNotFound, :id => id)
+      record_class.find(id)
     end
 
     def find_record(id)
@@ -119,8 +124,7 @@ module Minimapper
     end
 
     def record_for(entity)
-      (entity.id && record_class.find_by_id(entity.id)) ||
-        raise(EntityNotFound, entity.inspect)
+      record_class.find(entity.id)
     end
 
     def entities_for(records, klass = entity_class)
@@ -144,9 +148,34 @@ module Minimapper
       end
     end
 
+    def with_save_hooks(entity, record)
+      creating = record.new_record?
+      before_save(entity, record)
+      result = yield
+
+      if result
+        after_save(entity, record)
+
+        if creating
+          after_create(entity, record)
+        end
+      end
+
+      result
+    end
+
     # Hooks
 
     def after_find(entity, record)
     end
+
+    def before_save(entity, record)
+    end
+
+    def after_save(entity, record)
+    end
+
+    def after_create(entity, record)
+    end
   end
 end

data/lib/minimapper/version.rb

@@ -1,3 +1,3 @@
 module Minimapper
-  VERSION = "0.9.0"
+  VERSION = "0.10.0"
 end

data/spec/mapper_spec.rb

@@ -25,9 +25,79 @@ describe Minimapper::Mapper do
   let(:mapper) { ProjectMapper.new }
   let(:entity_class) { Project }
 
-  include_examples :mapper
+  it "can set and get repository" do
+    mapper.repository = :repository_instance
+    mapper.repository.should == :repository_instance
+  end
 
   describe "#create" do
+    it "sets an id on the entity" do
+      entity1 = build_valid_entity
+      entity1.id.should be_nil
+      mapper.create(entity1)
+      entity1.id.should > 0
+
+      entity2 = build_valid_entity
+      mapper.create(entity2)
+      entity2.id.should == entity1.id + 1
+    end
+
+    it "marks the entity as persisted" do
+      entity1 = build_valid_entity
+      entity1.should_not be_persisted
+      mapper.create(entity1)
+      entity1.should be_persisted
+    end
+
+    it "returns the id" do
+      id = mapper.create(build_valid_entity)
+      id.should be_kind_of(Fixnum)
+      id.should > 0
+    end
+
+    it "does not store by reference" do
+      entity = build_valid_entity
+      mapper.create(entity)
+      mapper.last.object_id.should_not == entity.object_id
+      mapper.last.attributes[:name].should == "test"
+    end
+
+    it "validates the record before saving" do
+      entity = entity_class.new
+      def entity.valid?
+        false
+      end
+      mapper.create(entity).should be_false
+    end
+
+    it "calls before_save and after_save on the mapper" do
+      entity = build_valid_entity
+      record = ProjectMapper::Record.new
+      ProjectMapper::Record.stub(:new => record)
+      mapper.should_receive(:before_save).with(entity, record)
+      mapper.should_receive(:after_save).with(entity, record)
+      mapper.create(entity)
+    end
+
+    it "calls after_create on the mapper" do
+      entity = build_valid_entity
+      record = ProjectMapper::Record.new
+      ProjectMapper::Record.stub(:new => record)
+      mapper.should_receive(:after_create).with(entity, record)
+      mapper.create(entity)
+    end
+
+    it "does not call after_save or after_create if the save fails" do
+      entity = entity_class.new
+      def entity.valid?
+        false
+      end
+      mapper.should_receive(:before_save)
+      mapper.should_not_receive(:after_save)
+      mapper.should_not_receive(:after_create)
+      mapper.create(entity)
+    end
+
     it "does not include protected attributes" do
       # because it leads to exceptions when mass_assignment_sanitizer is set to strict
       entity = build_entity(:visible => true, :name => "Joe")
@@ -65,15 +135,226 @@ describe Minimapper::Mapper do
       mapper.create(new_entity)
       new_entity.should be_valid
     end
+  end
 
-    def build_entity(attributes)
-      entity = Project.new
-      entity.attributes = attributes
-      entity
+  describe "#find" do
+    it "returns an entity matching the id" do
+      entity = build_valid_entity
+      mapper.create(entity)
+      found_entity = mapper.find(entity.id)
+      found_entity.attributes[:name].should == "test"
+      found_entity.id.should == entity.id
+      found_entity.should be_kind_of(Minimapper::Entity::Core)
+    end
+
+    it "supports string ids" do
+      entity = build_valid_entity
+      mapper.create(entity)
+      mapper.find(entity.id.to_s)
+    end
+
+    it "does not return the same instance" do
+      entity = build_valid_entity
+      mapper.create(entity)
+      mapper.find(entity.id).object_id.should_not == entity.object_id
+      mapper.find(entity.id).object_id.should_not == mapper.find(entity.id).object_id
+    end
+
+    it "calls after_find on the mapper" do
+      entity = build_valid_entity
+      mapper.create(entity)
+      mapper.should_receive(:after_find)
+      found_entity = mapper.find(entity.id)
+    end
+
+    it "returns an entity marked as persisted" do
+      entity = build_valid_entity
+      mapper.create(entity)
+      found_entity = mapper.find(entity.id)
+      found_entity.should be_persisted
+    end
+
+    it "fails when an entity can not be found" do
+      lambda { mapper.find(-1) }.should raise_error(ActiveRecord::RecordNotFound)
+    end
+  end
+
+  describe "#find_by_id" do
+    it "returns an entity matching the id" do
+      entity = build_valid_entity
+      mapper.create(entity)
+      found_entity = mapper.find_by_id(entity.id)
+      found_entity.attributes[:name].should == "test"
+      found_entity.id.should == entity.id
+      found_entity.should be_kind_of(Minimapper::Entity::Core)
+    end
+
+    it "supports string ids" do
+      entity = build_valid_entity
+      mapper.create(entity)
+      mapper.find_by_id(entity.id.to_s)
+    end
+
+    it "does not return the same instance" do
+      entity = build_valid_entity
+      mapper.create(entity)
+      mapper.find_by_id(entity.id).object_id.should_not == entity.object_id
+      mapper.find_by_id(entity.id).object_id.should_not == mapper.find_by_id(entity.id).object_id
+    end
+
+    it "returns nil when an entity can not be found" do
+      mapper.find_by_id(-1).should be_nil
+    end
+  end
+
+  describe "#all" do
+    it "returns all entities in undefined order" do
+      first_created_entity = build_valid_entity
+      second_created_entity = build_valid_entity
+      mapper.create(first_created_entity)
+      mapper.create(second_created_entity)
+      all_entities = mapper.all
+      all_entities.map(&:id).should include(first_created_entity.id)
+      all_entities.map(&:id).should include(second_created_entity.id)
+      all_entities.first.should be_kind_of(Minimapper::Entity::Core)
+    end
+
+    it "does not return the same instances" do
+      entity = build_valid_entity
+      mapper.create(entity)
+      mapper.all.first.object_id.should_not == entity.object_id
+      mapper.all.first.object_id.should_not == mapper.all.first.object_id
+    end
+  end
+
+  describe "#first" do
+    it "returns the first entity" do
+      first_created_entity = build_valid_entity
+      mapper.create(first_created_entity)
+      mapper.create(build_valid_entity)
+      mapper.first.id.should == first_created_entity.id
+      mapper.first.should be_kind_of(entity_class)
+    end
+
+    it "does not return the same instance" do
+      entity = build_valid_entity
+      mapper.create(entity)
+      mapper.first.object_id.should_not == entity.object_id
+      mapper.first.object_id.should_not == mapper.first.object_id
+    end
+
+    it "returns nil when there is no entity" do
+      mapper.first.should be_nil
+    end
+  end
+
+  describe "#last" do
+    it "returns the last entity" do
+      last_created_entity = build_valid_entity
+      mapper.create(build_valid_entity)
+      mapper.create(last_created_entity)
+      mapper.last.id.should == last_created_entity.id
+      mapper.last.should be_kind_of(entity_class)
+    end
+
+    it "does not return the same instance" do
+      entity = build_valid_entity
+      mapper.create(entity)
+      mapper.last.object_id.should_not == entity.object_id
+      mapper.last.object_id.should_not == mapper.last.object_id
+    end
+
+    it "returns nil when there is no entity" do
+      mapper.last.should be_nil
+    end
+  end
+
+  describe "#reload" do
+    it "reloads the given record" do
+      entity = build_entity(:email => "foo@example.com")
+      mapper.create(entity)
+      entity.attributes[:email] = "test@example.com"
+      mapper.reload(entity)
+      entity.attributes[:email] = "foo@example.com"
+      mapper.reload(entity).object_id.should_not == entity.object_id
+    end
+  end
+
+  describe "#count" do
+    it "returns the number of entities" do
+      mapper.create(build_valid_entity)
+      mapper.create(build_valid_entity)
+      mapper.count.should == 2
     end
   end
 
   describe "#update" do
+    it "updates" do
+      entity = build_valid_entity
+      mapper.create(entity)
+
+      entity.attributes = { :name => "Updated" }
+      mapper.last.attributes[:name].should == "test"
+
+      mapper.update(entity)
+      mapper.last.id.should == entity.id
+      mapper.last.attributes[:name].should == "Updated"
+    end
+
+    it "does not update and returns false when the entity isn't valid" do
+      entity = build_valid_entity
+      mapper.create(entity)
+
+      def entity.valid?
+        false
+      end
+
+      mapper.update(entity).should be_false
+      mapper.last.attributes[:name].should == "test"
+    end
+
+    it "calls before_save and after_save on the mapper" do
+      entity = build_valid_entity
+      mapper.create(entity)
+
+      record = ProjectMapper::Record.last
+      ProjectMapper::Record.stub(:find_by_id => record)
+
+      mapper.should_receive(:before_save).with(entity, record)
+      mapper.should_receive(:after_save).with(entity, record)
+      mapper.update(entity)
+    end
+
+    it "does not call after_create" do
+      entity = build_valid_entity
+      mapper.create(entity)
+
+      record = ProjectMapper::Record.last
+      ProjectMapper::Record.stub(:find_by_id => record)
+
+      mapper.should_receive(:after_save)
+      mapper.should_not_receive(:after_create)
+      mapper.update(entity)
+    end
+
+    it "returns true" do
+      entity = build_valid_entity
+      mapper.create(entity)
+      mapper.update(entity).should == true
+    end
+
+    it "fails when the entity does not have an id" do
+      entity = build_valid_entity
+      lambda { mapper.update(entity) }.should raise_error(ActiveRecord::RecordNotFound)
+    end
+
+    it "fails when the entity no longer exists" do
+      entity = build_valid_entity
+      mapper.create(entity)
+      mapper.delete_all
+      lambda { mapper.update(entity) }.should raise_error(ActiveRecord::RecordNotFound)
+    end
+
     it "does not include protected attributes" do
       entity = Project.new
       mapper.create(entity)
@@ -117,11 +398,73 @@ describe Minimapper::Mapper do
       mapper.update(new_entity)
       new_entity.should be_valid
     end
+  end
 
-    def build_entity(attributes)
-      entity = Project.new
-      entity.attributes = attributes
-      entity
+  describe "#delete" do
+    it "removes the entity" do
+      entity = build_valid_entity
+      removed_entity_id = entity.id
+      mapper.create(entity)
+      mapper.create(build_valid_entity)
+      mapper.delete(entity)
+      mapper.all.size.should == 1
+      mapper.first.id.should_not == removed_entity_id
+    end
+
+    it "marks the entity as no longer persisted" do
+      entity = build_valid_entity
+      mapper.create(entity)
+      entity.should be_persisted
+      mapper.delete(entity)
+      entity.should_not be_persisted
+    end
+
+    it "fails when the entity does not have an id" do
+      entity = entity_class.new
+      lambda { mapper.delete(entity) }.should raise_error(ActiveRecord::RecordNotFound)
     end
+
+    it "fails when the entity can not be found" do
+      entity = entity_class.new
+      entity.id = -1
+      lambda { mapper.delete(entity) }.should raise_error(ActiveRecord::RecordNotFound)
+    end
+  end
+
+  describe "#delete_by_id" do
+    it "removes the entity" do
+      entity = build_valid_entity
+      mapper.create(entity)
+      mapper.create(build_valid_entity)
+      mapper.delete_by_id(entity.id)
+      mapper.all.size.should == 1
+      mapper.first.id.should_not == entity.id
+    end
+
+    it "fails when an entity can not be found" do
+      lambda { mapper.delete_by_id(-1) }.should raise_error(ActiveRecord::RecordNotFound)
+    end
+  end
+
+  describe "#delete_all" do
+    it "empties the mapper" do
+      mapper.create(build_valid_entity)
+      mapper.delete_all
+      mapper.all.should == []
+    end
+  end
+
+  private
+
+  def build_valid_entity
+    entity = entity_class.new
+    entity.attributes = { :name => 'test' }
+    entity
+  end
+
+  def build_entity(attributes)
+    entity = Project.new
+    entity.attributes = attributes
+    entity
   end
 end

data/spec/spec_helper.rb

@@ -2,7 +2,6 @@ require "active_record"
 require "minimapper"
 
 ROOT = File.expand_path(File.join(File.dirname(__FILE__), "..")) unless defined?(ROOT)
-Dir[File.join(ROOT, "spec/support/shared_examples/*.rb")].each { |f| require f }
 
 require File.join(ROOT, "spec/support/database_setup")
 

data/unit/spec_helper.rb

@@ -4,7 +4,5 @@ $: << File.join(ROOT, "unit")
 
 require "minimapper"
 
-Dir[File.join(ROOT, "spec/support/shared_examples/*.rb")].each { |f| require f }
-
 RSpec.configure do |config|
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: minimapper
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 0.10.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-10-04 00:00:00.000000000 Z
+date: 2013-10-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70096190311420 !ruby/object:Gem::Requirement
+  requirement: &70186737281960 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,7 +21,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70096190311420
+  version_requirements: *70186737281960
 description: A minimalistic way of separating your models from ActiveRecord.
 email:
 - joakim.kolsjo@gmail.com
@@ -54,7 +54,6 @@ files:
 - spec/mapper_spec.rb
 - spec/spec_helper.rb
 - spec/support/database_setup.rb
-- spec/support/shared_examples/mapper.rb
 - unit/entity/attributes_spec.rb
 - unit/entity/convert_spec.rb
 - unit/entity/core_spec.rb
@@ -77,7 +76,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2287659495149187599
+      hash: -374043645528492869
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -86,10 +85,10 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2287659495149187599
+      hash: -374043645528492869
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.5
+rubygems_version: 1.8.6
 signing_key: 
 specification_version: 3
 summary: A minimalistic way of separating your models from ActiveRecord.
@@ -97,4 +96,3 @@ test_files:
 - spec/mapper_spec.rb
 - spec/spec_helper.rb
 - spec/support/database_setup.rb
-- spec/support/shared_examples/mapper.rb

data/spec/support/shared_examples/mapper.rb

@@ -1,295 +0,0 @@
-shared_examples :mapper do
-  # expects mapper and entity_class to be defined
-
-  it "can set and get repository" do
-    mapper.repository = :repository_instance
-    mapper.repository.should == :repository_instance
-  end
-
-  describe "#create" do
-    it "sets an id on the entity" do
-      entity1 = build_valid_entity
-      entity1.id.should be_nil
-      mapper.create(entity1)
-      entity1.id.should > 0
-
-      entity2 = build_valid_entity
-      mapper.create(entity2)
-      entity2.id.should == entity1.id + 1
-    end
-
-    it "marks the entity as persisted" do
-      entity1 = build_valid_entity
-      entity1.should_not be_persisted
-      mapper.create(entity1)
-      entity1.should be_persisted
-    end
-
-    it "returns the id" do
-      id = mapper.create(build_valid_entity)
-      id.should be_kind_of(Fixnum)
-      id.should > 0
-    end
-
-    it "does not store by reference" do
-      entity = build_valid_entity
-      mapper.create(entity)
-      mapper.last.object_id.should_not == entity.object_id
-      mapper.last.attributes[:name].should == "test"
-    end
-
-    it "validates the record before saving" do
-      entity = entity_class.new
-      def entity.valid?
-        false
-      end
-      mapper.create(entity).should be_false
-    end
-  end
-
-  describe "#find" do
-    it "returns an entity matching the id" do
-      entity = build_valid_entity
-      mapper.create(entity)
-      found_entity = mapper.find(entity.id)
-      found_entity.attributes[:name].should == "test"
-      found_entity.id.should == entity.id
-      found_entity.should be_kind_of(Minimapper::Entity::Core)
-    end
-
-    it "supports string ids" do
-      entity = build_valid_entity
-      mapper.create(entity)
-      mapper.find(entity.id.to_s)
-    end
-
-    it "does not return the same instance" do
-      entity = build_valid_entity
-      mapper.create(entity)
-      mapper.find(entity.id).object_id.should_not == entity.object_id
-      mapper.find(entity.id).object_id.should_not == mapper.find(entity.id).object_id
-    end
-
-    it "calls after_find on the mapper" do
-      entity = build_valid_entity
-      mapper.create(entity)
-      mapper.should_receive(:after_find)
-      found_entity = mapper.find(entity.id)
-    end
-
-    it "returns an entity marked as persisted" do
-      entity = build_valid_entity
-      mapper.create(entity)
-      found_entity = mapper.find(entity.id)
-      found_entity.should be_persisted
-    end
-
-    it "fails when an entity can not be found" do
-      lambda { mapper.find(-1) }.should raise_error(Minimapper::EntityNotFound)
-    end
-  end
-
-  describe "#find_by_id" do
-    it "returns an entity matching the id" do
-      entity = build_valid_entity
-      mapper.create(entity)
-      found_entity = mapper.find_by_id(entity.id)
-      found_entity.attributes[:name].should == "test"
-      found_entity.id.should == entity.id
-      found_entity.should be_kind_of(Minimapper::Entity::Core)
-    end
-
-    it "supports string ids" do
-      entity = build_valid_entity
-      mapper.create(entity)
-      mapper.find_by_id(entity.id.to_s)
-    end
-
-    it "does not return the same instance" do
-      entity = build_valid_entity
-      mapper.create(entity)
-      mapper.find_by_id(entity.id).object_id.should_not == entity.object_id
-      mapper.find_by_id(entity.id).object_id.should_not == mapper.find_by_id(entity.id).object_id
-    end
-
-    it "returns nil when an entity can not be found" do
-      mapper.find_by_id(-1).should be_nil
-    end
-  end
-
-  describe "#all" do
-    it "returns all entities in undefined order" do
-      first_created_entity = build_valid_entity
-      second_created_entity = build_valid_entity
-      mapper.create(first_created_entity)
-      mapper.create(second_created_entity)
-      all_entities = mapper.all
-      all_entities.map(&:id).should include(first_created_entity.id)
-      all_entities.map(&:id).should include(second_created_entity.id)
-      all_entities.first.should be_kind_of(Minimapper::Entity::Core)
-    end
-
-    it "does not return the same instances" do
-      entity = build_valid_entity
-      mapper.create(entity)
-      mapper.all.first.object_id.should_not == entity.object_id
-      mapper.all.first.object_id.should_not == mapper.all.first.object_id
-    end
-  end
-
-  describe "#first" do
-    it "returns the first entity" do
-      first_created_entity = build_valid_entity
-      mapper.create(first_created_entity)
-      mapper.create(build_valid_entity)
-      mapper.first.id.should == first_created_entity.id
-      mapper.first.should be_kind_of(entity_class)
-    end
-
-    it "does not return the same instance" do
-      entity = build_valid_entity
-      mapper.create(entity)
-      mapper.first.object_id.should_not == entity.object_id
-      mapper.first.object_id.should_not == mapper.first.object_id
-    end
-
-    it "returns nil when there is no entity" do
-      mapper.first.should be_nil
-    end
-  end
-
-  describe "#last" do
-    it "returns the last entity" do
-      last_created_entity = build_valid_entity
-      mapper.create(build_valid_entity)
-      mapper.create(last_created_entity)
-      mapper.last.id.should == last_created_entity.id
-      mapper.last.should be_kind_of(entity_class)
-    end
-
-    it "does not return the same instance" do
-      entity = build_valid_entity
-      mapper.create(entity)
-      mapper.last.object_id.should_not == entity.object_id
-      mapper.last.object_id.should_not == mapper.last.object_id
-    end
-
-    it "returns nil when there is no entity" do
-      mapper.last.should be_nil
-    end
-  end
-
-  describe "#count" do
-    it "returns the number of entities" do
-      mapper.create(build_valid_entity)
-      mapper.create(build_valid_entity)
-      mapper.count.should == 2
-    end
-  end
-
-  describe "#update" do
-    it "updates" do
-      entity = build_valid_entity
-      mapper.create(entity)
-
-      entity.attributes = { :name => "Updated" }
-      mapper.last.attributes[:name].should == "test"
-
-      mapper.update(entity)
-      mapper.last.id.should == entity.id
-      mapper.last.attributes[:name].should == "Updated"
-    end
-
-    it "does not update and returns false when the entity isn't valid" do
-      entity = build_valid_entity
-      mapper.create(entity)
-
-      def entity.valid?
-        false
-      end
-
-      mapper.update(entity).should be_false
-      mapper.last.attributes[:name].should == "test"
-    end
-
-    it "returns true" do
-      entity = build_valid_entity
-      mapper.create(entity)
-      mapper.update(entity).should == true
-    end
-
-    it "fails when the entity does not have an id" do
-      entity = build_valid_entity
-      lambda { mapper.update(entity) }.should raise_error(Minimapper::EntityNotFound)
-    end
-
-    it "fails when the entity no longer exists" do
-      entity = build_valid_entity
-      mapper.create(entity)
-      mapper.delete_all
-      lambda { mapper.update(entity) }.should raise_error(Minimapper::EntityNotFound)
-    end
-  end
-
-  describe "#delete" do
-    it "removes the entity" do
-      entity = build_valid_entity
-      removed_entity_id = entity.id
-      mapper.create(entity)
-      mapper.create(build_valid_entity)
-      mapper.delete(entity)
-      mapper.all.size.should == 1
-      mapper.first.id.should_not == removed_entity_id
-    end
-
-    it "marks the entity as no longer persisted" do
-      entity = build_valid_entity
-      mapper.create(entity)
-      entity.should be_persisted
-      mapper.delete(entity)
-      entity.should_not be_persisted
-    end
-
-    it "fails when the entity does not have an id" do
-      entity = entity_class.new
-      lambda { mapper.delete(entity) }.should raise_error(Minimapper::EntityNotFound)
-    end
-
-    it "fails when the entity can not be found" do
-      entity = entity_class.new
-      entity.id = -1
-      lambda { mapper.delete(entity) }.should raise_error(Minimapper::EntityNotFound)
-    end
-  end
-
-  describe "#delete_by_id" do
-    it "removes the entity" do
-      entity = build_valid_entity
-      mapper.create(entity)
-      mapper.create(build_valid_entity)
-      mapper.delete_by_id(entity.id)
-      mapper.all.size.should == 1
-      mapper.first.id.should_not == entity.id
-    end
-
-    it "fails when an entity can not be found" do
-      lambda { mapper.delete_by_id(-1) }.should raise_error(Minimapper::EntityNotFound)
-    end
-  end
-
-  describe "#delete_all" do
-    it "empties the mapper" do
-      mapper.create(build_valid_entity)
-      mapper.delete_all
-      mapper.all.should == []
-    end
-  end
-
-  private
-
-  def build_valid_entity
-    entity = entity_class.new
-    entity.attributes = { :name => 'test' }
-    entity
-  end
-end