minimapper 0.3.0 → 0.4.0

data/.travis.yml

@@ -1,5 +1,8 @@
 language: ruby
 rvm:
+  # 2.0 rubies
+  - 2.0.0
+
   # 1.9 rubies
   - 1.9.3
   - 1.9.2

data/Gemfile

@@ -1,7 +1,7 @@
 source 'https://rubygems.org'
 
 # We don't require active_record to use minimapper, only to
-# use minimapper/ar. We do require it for the tests though :)
+# use minimapper/mapper/ar. We do require it for the tests though :)
 gem "activerecord"
 gem "rspec"
 

data/README.md

@@ -57,7 +57,7 @@ You can use the mappers like this (<strong>it's runnable, try copy and pasting i
 require "rubygems"
 require "minimapper"
 require "minimapper/entity"
-require "minimapper/memory"
+require "minimapper/mapper/memory"
 
 class User
   include Minimapper::Entity
@@ -66,7 +66,7 @@ class User
   validates :name, :presence => true
 end
 
-class UserMapper < Minimapper::Memory
+class UserMapper < Minimapper::Mapper::Memory
 end
 
 ## Creating
@@ -89,7 +89,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::Common::CanNotFindEntity
+# user_mapper.find(old_id)       # raises Minimapper::Mapper::CanNotFindEntity
 # user_mapper.delete_all
 # user_mapper.delete_by_id(1)
 
@@ -120,7 +120,7 @@ This is not directly runnable like the previous example, it requires ActiveRecor
 When you do need to use an ORM like ActiveRecord however, it now has the same API as your in-memory persistence (thanks to the [shared tests](https://github.com/joakimk/minimapper/blob/master/spec/support/shared_examples/mapper.rb) which define how a mapper is supposed to behave).
 
 ``` ruby
-require "minimapper/ar"
+require "minimapper/mapper/ar"
 
 module AR
   class UserMapper < Minimapper::AR
@@ -136,6 +136,24 @@ mapper = AR::UserMapper.new
 mapper.create(user)
 ```
 
+### Uniqueness validations and other DB validations
+
+Validations on uniqueness can't be implemented on the entity, because they need to access the database.
+
+Therefore, the ActiveRecord mapper will copy over any record errors to the entity when attempting to create or update.
+
+You would add these validations to the record itself, like:
+
+``` ruby
+class User < ActiveRecord::Base
+  validates :email, :uniqueness => true
+end
+```
+
+Note that calling `valid?` on the entity will not access the database. Errors copied over from the record will remain until the next attempt to create or update.
+
+So an entity that wouldn't be unique in the database will be `valid?` before you attempt to create it. And after you attempt to create it, the entity will not be `valid?` even after assigning a new value, until you attempt to create it again.
+
 ### Custom queries
 
 You can write custom queries like this:
@@ -143,7 +161,7 @@ You can write custom queries like this:
 ``` ruby
 # Memory implementation
 module Memory
-  class ProjectMapper < Minimapper::Memory
+  class ProjectMapper < Minimapper::Mapper::Memory
     def waiting_for_review
       all.find_all { |p| p.waiting_for_review }.sort_by(&:id).reverse
     end
@@ -173,6 +191,19 @@ end
 
 It gets simpler to maintain if you use shared tests to test both implementations. For inspiration, see the [shared tests](https://github.com/joakimk/minimapper/blob/master/spec/support/shared_examples/mapper.rb) used to test minimapper.
 
+`entity_for` returns nil for nil.
+
+It takes an optional second argument if you want a different entity class than the mapper's:
+
+```
+class ProjectMapper < Minimapper::AR
+  def owner_of(project)
+    owner_record = find(project).owner
+    entity_for(owner_record, User)
+  end
+end
+```
+
 ### Typed attributes
 
 If you specify type, minimapper will attempt to convert into that type. Supported types: Integer and DateTime (:integer and :date_time).
@@ -287,7 +318,6 @@ You need mysql and postgres installed (but they do not have to be running) to be
 I won't implement anything that isn't actually used. But here are some ideas for things that might make it into minimapper someday if there is a need for it.
 
 * Provide a hook to convert attributes between entities and the backing models (when your entity attributes and db-schema isn't a one-to-one match).
-* Copy validation errors back from the mapper to the entity (for example if you do uniqueness validation in a backing ActiveRecord-model).
 
 ## Credits and license
 

data/lib/minimapper/entity/attributes.rb

@@ -15,6 +15,7 @@ module Minimapper
         attributes [ *opts ]
       end
 
+      # Compatibility with certain Rails plugins, like Traco.
       def column_names
         @entity_columns.map(&:name).map(&:to_s)
       end

data/lib/minimapper/entity/core.rb

@@ -15,7 +15,15 @@ module Minimapper
       end
 
       def valid?
-        true
+        mapper_errors.empty?
+      end
+
+      def mapper_errors
+        @mapper_errors ||= []
+      end
+
+      def mapper_errors=(list)
+        @mapper_errors = list
       end
 
       private

data/lib/minimapper/entity/rails.rb

@@ -1,3 +1,7 @@
+# Include this in your entity models for
+# Ruby on Rails conveniences, like being
+# able to use them in forms.
+
 require "active_model"
 
 module Minimapper
@@ -7,6 +11,22 @@ module Minimapper
         klass.class_eval do
           extend  ActiveModel::Naming
           include ActiveModel::Validations
+
+          # Must be later than ActiveModel::Validations so
+          # it can call it with super.
+          include ValidationsWithMapperErrors
+        end
+      end
+
+      module ValidationsWithMapperErrors
+        def valid?
+          super
+
+          mapper_errors.each do |a, v|
+            errors.add(a, v)
+          end
+
+          errors.empty?
         end
       end
 

data/lib/minimapper/entity.rb

@@ -19,7 +19,11 @@ module Minimapper
     def self.included(klass)
       klass.send(:include, Minimapper::Entity::Rails)
       klass.send(:extend, Minimapper::Entity::Attributes)
-      klass.attributes([ :id, :integer ], [ :created_at, :date_time ], [ :updated_at, :date_time ])
+      klass.attributes(
+        [ :id, :integer ],
+        [ :created_at, :date_time ],
+        [ :updated_at, :date_time ]
+      )
     end
   end
 end

data/lib/minimapper/mapper/ar.rb

@@ -0,0 +1,138 @@
+require "minimapper/mapper/common"
+
+module Minimapper
+  module Mapper
+    class AR
+      include Common
+
+      # Create
+
+      def create(entity)
+        record = record_class.new
+        validate_record_and_copy_errors_to_entity(record, entity)
+
+        if entity.valid?
+          record.save!
+          entity.id = record.id
+          entity.id
+        else
+          false
+        end
+      end
+
+      # Read
+
+      def find(id)
+        entity_for(find_record_safely(id))
+      end
+
+      def find_by_id(id)
+        entity_for(find_record(id))
+      end
+
+      def all
+        entities_for record_class.all
+      end
+
+      def first
+        entity_for(record_class.order("id ASC").first)
+      end
+
+      def last
+        entity_for(record_class.order("id ASC").last)
+      end
+
+      def count
+        record_class.count
+      end
+
+      # Update
+
+      def update(entity)
+        record = record_for(entity)
+        validate_record_and_copy_errors_to_entity(record, entity)
+
+        if entity.valid?
+          record.save!
+          true
+        else
+          false
+        end
+      end
+
+      # Delete
+
+      def delete(entity)
+        delete_by_id(entity.id)
+        entity.id = nil
+      end
+
+      def delete_by_id(id)
+        find_record_safely(id).delete
+      end
+
+      def delete_all
+        record_class.delete_all
+      end
+
+      private
+
+      # NOTE: Don't memoize the classes or code reloading will break in rails apps.
+
+      # Will attempt to use AR:Project as the record class
+      # when the mapper class name is AR::ProjectMapper
+      def record_class
+        self.class.name.sub(/Mapper$/, '').constantize
+      end
+
+      # Will attempt to use Project as the enity class when
+      # the mapper class name is AR::ProjectMapper
+      def entity_class
+        ("::" + self.class.name.split('::').last.sub(/Mapper$/, '')).constantize
+      end
+
+      def accessible_attributes(entity)
+        entity.attributes.reject { |k, v| protected_attributes.include?(k.to_s) }
+      end
+
+      def protected_attributes
+        record_class.protected_attributes
+      end
+
+      def validate_record_and_copy_errors_to_entity(record, entity)
+        record.attributes = accessible_attributes(entity)
+        record.valid?
+        entity.mapper_errors = record.errors.map { |k, v| [k, v] }
+      end
+
+      def find_record_safely(id)
+        find_record(id) ||
+          raise(CanNotFindEntity, :id => id)
+      end
+
+      def find_record(id)
+        id && record_class.find_by_id(id)
+      end
+
+      def record_for(entity)
+        (entity.id && record_class.find_by_id(entity.id)) ||
+          raise(CanNotFindEntity, entity.inspect)
+      end
+
+      def entities_for(records)
+        records.map { |record| entity_for(record) }
+      end
+
+      def entity_for(record, klass = entity_class)
+        if record
+          entity = klass.new
+          entity.id = record.id
+          entity.attributes = record.attributes.symbolize_keys
+          entity
+        else
+          nil
+        end
+      end
+    end
+  end
+end

data/lib/minimapper/mapper/common.rb

@@ -0,0 +1,9 @@
+module Minimapper
+  module Mapper
+    CanNotFindEntity = Class.new(StandardError)
+
+    module Common
+      attr_accessor :repository
+    end
+  end
+end

data/lib/minimapper/mapper/memory.rb

@@ -0,0 +1,94 @@
+require 'minimapper/mapper/common'
+
+module Minimapper
+  module Mapper
+    class Memory
+      include Common
+
+      def initialize
+        @store = []
+        @last_id = 0
+      end
+
+      # Create
+      def create(entity)
+        if entity.valid?
+          entity.id = next_id
+          store.push(entity.dup)
+          last_id
+        else
+          false
+        end
+      end
+
+      # Read
+      def find(id)
+        find_internal_safely(id).dup
+      end
+
+      def find_by_id(id)
+        entity = find_internal(id)
+        entity && entity.dup
+      end
+
+      def all
+        store.map { |entity| entity.dup }
+      end
+
+      def first
+        store.first && store.first.dup
+      end
+
+      def last
+        store.last && store.last.dup
+      end
+
+      def count
+        all.size
+      end
+
+      # Update
+      def update(entity)
+        if entity.valid?
+          known_entity = find_internal_safely(entity.id)
+          known_entity.attributes = entity.attributes
+          true
+        else
+          false
+        end
+      end
+
+      # Delete
+      def delete(entity)
+        delete_by_id(entity.id)
+        entity.id = nil
+      end
+
+      def delete_by_id(id)
+        entity = find_internal_safely(id)
+        store.delete(entity)
+      end
+
+      def delete_all
+        store.clear
+      end
+
+      private
+
+      def find_internal_safely(id)
+        find_internal(id) ||
+          raise(CanNotFindEntity, :id => id)
+      end
+
+      def find_internal(id)
+        id && store.find { |e| e.id == id.to_i }
+      end
+
+      def next_id
+        @last_id += 1
+      end
+
+      attr_reader :store, :last_id
+    end
+  end
+end

data/lib/minimapper/version.rb

@@ -1,3 +1,3 @@
 module Minimapper
-  VERSION = "0.3.0"
+  VERSION = "0.4.0"
 end

data/spec/mapper/ar_spec.rb

@@ -0,0 +1,135 @@
+require "spec_helper"
+require "minimapper/entity/core"
+require "minimapper/mapper/ar"
+
+class TestEntity
+  include Minimapper::Entity::Core
+end
+
+class TestMapper < Minimapper::Mapper::AR
+  private
+
+  def entity_class
+    TestEntity
+  end
+
+  def record_class
+    Record
+  end
+
+  class Record < ActiveRecord::Base
+    attr_protected :visible
+
+    validates :email,
+      :uniqueness => true,
+      :allow_nil => true
+
+    self.table_name = :projects
+    self.mass_assignment_sanitizer = :strict
+  end
+end
+
+describe Minimapper::Mapper::AR do
+  let(:mapper) { TestMapper.new }
+  let(:entity_class) { TestEntity }
+
+  include_examples :mapper
+
+  describe "#create" do
+    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")
+      mapper.create(entity)
+
+      stored_entity = mapper.find(entity.id)
+      stored_entity.attributes[:visible].should be_nil
+      stored_entity.attributes[:name].should == "Joe"
+
+      entity = TestEntity.new
+      entity.attributes = { :visible => true, :name => "Joe" }
+      TestMapper::Record.stub(:protected_attributes => [])
+      lambda { mapper.create(entity) }.should raise_error(ActiveModel::MassAssignmentSecurity::Error)
+    end
+
+    it "copies record validation errors to entity" do
+      old_entity = build_entity(:email => "joe@example.com")
+      mapper.create(old_entity)
+      old_entity.mapper_errors.should == []
+
+      new_entity = build_entity(:email => "joe@example.com")
+      mapper.create(new_entity)
+      new_entity.mapper_errors.should == [ [:email, "has already been taken"] ]
+    end
+
+    it "can revalidate on record validation errors" do
+      old_entity = build_entity(:email => "joe@example.com")
+      mapper.create(old_entity)
+
+      new_entity = build_entity(:email => "joe@example.com")
+      mapper.create(new_entity)
+      new_entity.mapper_errors.should == [ [:email, "has already been taken"] ]
+
+      new_entity.attributes = { :email => "something.else@example.com" }
+      mapper.create(new_entity)
+      new_entity.should be_valid
+    end
+
+    def build_entity(attributes)
+      entity = TestEntity.new
+      entity.attributes = attributes
+      entity
+    end
+  end
+
+  describe "#update" do
+    it "does not include protected attributes" do
+      entity = TestEntity.new
+      mapper.create(entity)
+
+      entity.attributes = { :visible => true, :name => "Joe" }
+      mapper.update(entity)
+      stored_entity = mapper.find(entity.id)
+      stored_entity.attributes[:visible].should be_nil
+      stored_entity.attributes[:name].should == "Joe"
+
+      TestMapper::Record.stub(:protected_attributes => [])
+      lambda { mapper.update(entity) }.should raise_error(ActiveModel::MassAssignmentSecurity::Error)
+    end
+
+    it "copies record validation errors to entity" do
+      old_entity = build_entity(:email => "joe@example.com")
+      mapper.create(old_entity)
+
+      new_entity = TestEntity.new
+      mapper.create(new_entity)
+      new_entity.mapper_errors.should == []
+
+      new_entity.attributes = { :email => "joe@example.com" }
+      mapper.update(new_entity)
+      new_entity.mapper_errors.should == [ [:email, "has already been taken"] ]
+    end
+
+    it "can revalidate on record validation errors" do
+      old_entity = build_entity(:email => "joe@example.com")
+      mapper.create(old_entity)
+
+      new_entity = TestEntity.new
+      mapper.create(new_entity)
+      new_entity.mapper_errors.should == []
+
+      new_entity.attributes = { :email => "joe@example.com" }
+      mapper.update(new_entity)
+      new_entity.mapper_errors.should == [ [:email, "has already been taken"] ]
+
+      new_entity.attributes = { :email => "something.else@example.com" }
+      mapper.update(new_entity)
+      new_entity.should be_valid
+    end
+
+    def build_entity(attributes)
+      entity = TestEntity.new
+      entity.attributes = attributes
+      entity
+    end
+  end
+end

data/spec/support/database_setup.rb

@@ -45,7 +45,7 @@ end
 silence_stream(STDOUT) do
   ActiveRecord::Schema.define(:version => 0) do
     create_table :projects, :force => true do |t|
-      t.string :name
+      t.string :name, :email
       t.boolean :visible
     end
   end

data/spec/support/shared_examples/mapper.rb

@@ -64,7 +64,7 @@ shared_examples :mapper do
     end
 
     it "fails when an entity can not be found" do
-      lambda { mapper.find(-1) }.should raise_error(Minimapper::Common::CanNotFindEntity)
+      lambda { mapper.find(-1) }.should raise_error(Minimapper::Mapper::CanNotFindEntity)
     end
   end
 
@@ -199,14 +199,14 @@ shared_examples :mapper do
 
     it "fails when the entity does not have an id" do
       entity = build_valid_entity
-      lambda { mapper.update(entity) }.should raise_error(Minimapper::Common::CanNotFindEntity)
+      lambda { mapper.update(entity) }.should raise_error(Minimapper::Mapper::CanNotFindEntity)
     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::Common::CanNotFindEntity)
+      lambda { mapper.update(entity) }.should raise_error(Minimapper::Mapper::CanNotFindEntity)
     end
   end
 
@@ -231,13 +231,13 @@ shared_examples :mapper do
 
     it "fails when the entity does not have an id" do
       entity = entity_class.new
-      lambda { mapper.delete(entity) }.should raise_error(Minimapper::Common::CanNotFindEntity)
+      lambda { mapper.delete(entity) }.should raise_error(Minimapper::Mapper::CanNotFindEntity)
     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::Common::CanNotFindEntity)
+      lambda { mapper.delete(entity) }.should raise_error(Minimapper::Mapper::CanNotFindEntity)
     end
   end
 
@@ -252,7 +252,7 @@ shared_examples :mapper do
     end
 
     it "fails when an entity can not be found" do
-      lambda { mapper.delete_by_id(-1) }.should raise_error(Minimapper::Common::CanNotFindEntity)
+      lambda { mapper.delete_by_id(-1) }.should raise_error(Minimapper::Mapper::CanNotFindEntity)
     end
   end
 

data/unit/entity/core_spec.rb

@@ -27,14 +27,43 @@ describe Minimapper::Entity::Core do
     entity.attributes.should == { :one => 11 }
   end
 
-  it "returns true for valid?" do
-    entity = BasicEntity.new
-    entity.should be_valid
-  end
-
   it "responds to id" do
     entity = BasicEntity.new
     entity.id = 10
     entity.id.should == 10
   end
+
+  describe "#mapper_errors" do
+    it "defaults to an empty array" do
+      entity = BasicEntity.new
+      entity.mapper_errors.should == []
+    end
+
+    it "can be changed" do
+      entity = BasicEntity.new
+      entity.mapper_errors = [ [:one, "bad"] ]
+      entity.mapper_errors.should == [ [:one, "bad"] ]
+    end
+  end
+
+  describe "#mapper_errors=" do
+    it "makes it invalid if present" do
+      entity = BasicEntity.new
+      entity.mapper_errors = [ [:one, "bad"] ]
+      entity.valid?.should be_false
+    end
+  end
+
+  describe "#valid?" do
+    it "is true without errors" do
+      entity = BasicEntity.new
+      entity.valid?.should be_true
+    end
+
+    it "is false with errors" do
+      entity = BasicEntity.new
+      entity.mapper_errors = [ [:one, "bad"] ]
+      entity.valid?.should be_false
+    end
+  end
 end

data/unit/entity/rails_spec.rb

@@ -50,4 +50,15 @@ describe Minimapper::Entity::Rails do
     entity.name = "Joe"
     entity.should be_valid
   end
+
+  describe "#mapper_errors=" do
+    it "adds an error to the errors collection" do
+      entity = RailsEntity.new
+      entity.name = "Joe"
+      entity.should be_valid
+      entity.mapper_errors = [ [:name, "must be unique"] ]
+      entity.should_not be_valid
+      entity.errors[:name].should == ["must be unique"]
+    end
+  end
 end

data/unit/{memory_spec.rb → mapper/memory_spec.rb}

@@ -1,11 +1,11 @@
-require 'minimapper/memory'
+require 'minimapper/mapper/memory'
 require 'minimapper/entity/core'
 
 class BasicEntity
   include Minimapper::Entity::Core
 end
 
-describe Minimapper::Memory do
+describe Minimapper::Mapper::Memory do
   let(:mapper) { described_class.new }
   let(:entity_class) { BasicEntity }
 

data/unit/repository_spec.rb

@@ -1,8 +1,8 @@
 require 'minimapper/repository'
-require 'minimapper/memory'
+require 'minimapper/mapper/memory'
 
 module Test
-  class ProjectMapper < Minimapper::Memory
+  class ProjectMapper < Minimapper::Mapper::Memory
   end
 end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: minimapper
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-03-14 00:00:00.000000000 Z
+date: 2013-03-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70114985880520 !ruby/object:Gem::Requirement
+  requirement: &70108741422080 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,7 +21,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70114985880520
+  version_requirements: *70108741422080
 description: A minimalistic way of separating your models from ORMs like ActiveRecord.
 email:
 - joakim.kolsjo@gmail.com
@@ -37,8 +37,6 @@ files:
 - README.md
 - Rakefile
 - lib/minimapper.rb
-- lib/minimapper/ar.rb
-- lib/minimapper/common.rb
 - lib/minimapper/entity.rb
 - lib/minimapper/entity/attributes.rb
 - lib/minimapper/entity/convert.rb
@@ -46,13 +44,15 @@ files:
 - lib/minimapper/entity/convert/to_integer.rb
 - lib/minimapper/entity/core.rb
 - lib/minimapper/entity/rails.rb
-- lib/minimapper/memory.rb
+- lib/minimapper/mapper/ar.rb
+- lib/minimapper/mapper/common.rb
+- lib/minimapper/mapper/memory.rb
 - lib/minimapper/repository.rb
 - lib/minimapper/version.rb
 - minimapper.gemspec
 - script/ci
 - script/turbux_rspec
-- spec/ar_spec.rb
+- spec/mapper/ar_spec.rb
 - spec/spec_helper.rb
 - spec/support/database_setup.rb
 - spec/support/shared_examples/mapper.rb
@@ -60,7 +60,7 @@ files:
 - unit/entity/core_spec.rb
 - unit/entity/rails_spec.rb
 - unit/entity_spec.rb
-- unit/memory_spec.rb
+- unit/mapper/memory_spec.rb
 - unit/repository_spec.rb
 - unit/spec_helper.rb
 homepage: ''
@@ -77,7 +77,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1267216802408816283
+      hash: 3446705768728463359
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -86,7 +86,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1267216802408816283
+      hash: 3446705768728463359
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.5
@@ -94,7 +94,7 @@ signing_key:
 specification_version: 3
 summary: A minimalistic way of separating your models from ORMs like ActiveRecord.
 test_files:
-- spec/ar_spec.rb
+- spec/mapper/ar_spec.rb
 - spec/spec_helper.rb
 - spec/support/database_setup.rb
 - spec/support/shared_examples/mapper.rb

data/lib/minimapper/ar.rb

@@ -1,118 +0,0 @@
-require "minimapper/common"
-
-module Minimapper
-  class AR
-    include Common
-
-    # Create
-    def create(entity)
-      if entity.valid?
-        entity.id = record_class.create!(accessible_attributes(entity)).id
-      else
-        false
-      end
-    end
-
-    # Read
-    def find(id)
-      entity_for(find_record_safely(id))
-    end
-
-    def find_by_id(id)
-      entity_for(find_record(id))
-    end
-
-    def all
-      entities_for record_class.all
-    end
-
-    def first
-      entity_for(record_class.order("id ASC").first)
-    end
-
-    def last
-      entity_for(record_class.order("id ASC").last)
-    end
-
-    def count
-      record_class.count
-    end
-
-    # Update
-    def update(entity)
-      if entity.valid?
-        record_for(entity).update_attributes!(accessible_attributes(entity))
-        true
-      else
-        false
-      end
-    end
-
-    # Delete
-    def delete(entity)
-      delete_by_id(entity.id)
-      entity.id = nil
-    end
-
-    def delete_by_id(id)
-      find_record_safely(id).delete
-    end
-
-    def delete_all
-      record_class.delete_all
-    end
-
-    private
-
-    # NOTE: Don't memoize the classes or code reloading will break in rails apps.
-
-    # Will attempt to use AR:Project as the record class
-    # when the mapper class name is AR::ProjectMapper
-    def record_class
-      self.class.name.gsub(/Mapper/, '').constantize
-    end
-
-    # Will attempt to use Project as the enity class when
-    # the mapper class name is AR::ProjectMapper
-    def entity_class
-      ("::" + self.class.name.split('::').last.gsub(/Mapper/, '')).constantize
-    end
-
-    def accessible_attributes(entity)
-      entity.attributes.reject { |k, v| protected_attributes.include?(k.to_s) }
-    end
-
-    def protected_attributes
-      record_class.protected_attributes
-    end
-
-    def find_record_safely(id)
-      find_record(id) ||
-        raise(Common::CanNotFindEntity, :id => id)
-    end
-
-    def find_record(id)
-      id && record_class.find_by_id(id)
-    end
-
-    def record_for(entity)
-      (entity.id && record_class.find_by_id(entity.id)) ||
-        raise(Common::CanNotFindEntity, entity.inspect)
-    end
-
-    def entities_for(records)
-      records.map { |record| entity_for(record) }
-    end
-
-    def entity_for(record)
-      if record
-        entity = entity_class.new
-        entity.id = record.id
-        entity.attributes = record.attributes.symbolize_keys
-        entity
-      else
-        nil
-      end
-    end
-  end
-end

data/lib/minimapper/common.rb

@@ -1,7 +0,0 @@
-module Minimapper
-  module Common
-    CanNotFindEntity            = Class.new(StandardError)
-
-    attr_accessor :repository
-  end
-end

data/lib/minimapper/memory.rb

@@ -1,92 +0,0 @@
-require 'minimapper/common'
-
-module Minimapper
-  class Memory
-    include Common
-
-    def initialize
-      @store = []
-      @last_id = 0
-    end
-
-    # Create
-    def create(entity)
-      if entity.valid?
-        entity.id = next_id
-        store.push(entity.dup)
-        last_id
-      else
-        false
-      end
-    end
-
-    # Read
-    def find(id)
-      find_internal_safely(id).dup
-    end
-
-    def find_by_id(id)
-      entity = find_internal(id)
-      entity && entity.dup
-    end
-
-    def all
-      store.map { |entity| entity.dup }
-    end
-
-    def first
-      store.first && store.first.dup
-    end
-
-    def last
-      store.last && store.last.dup
-    end
-
-    def count
-      all.size
-    end
-
-    # Update
-    def update(entity)
-      if entity.valid?
-        known_entity = find_internal_safely(entity.id)
-        known_entity.attributes = entity.attributes
-        true
-      else
-        false
-      end
-    end
-
-    # Delete
-    def delete(entity)
-      delete_by_id(entity.id)
-      entity.id = nil
-    end
-
-    def delete_by_id(id)
-      entity = find_internal_safely(id)
-      store.delete(entity)
-    end
-
-    def delete_all
-      store.clear
-    end
-
-    private
-
-    def find_internal_safely(id)
-      find_internal(id) ||
-        raise(Common::CanNotFindEntity, :id => id)
-    end
-
-    def find_internal(id)
-      id && store.find { |e| e.id == id.to_i }
-    end
-
-    def next_id
-      @last_id += 1
-    end
-
-    attr_reader :store, :last_id
-  end
-end

data/spec/ar_spec.rb

@@ -1,67 +0,0 @@
-require "spec_helper"
-require "minimapper/entity/core"
-require "minimapper/ar"
-
-class TestEntity
-  include Minimapper::Entity::Core
-end
-
-class TestMapper < Minimapper::AR
-  private
-
-  def entity_class
-    TestEntity
-  end
-
-  def record_class
-    Record
-  end
-
-  class Record < ActiveRecord::Base
-    attr_protected :visible
-
-    self.table_name = :projects
-    self.mass_assignment_sanitizer = :strict
-  end
-end
-
-describe Minimapper::AR do
-  let(:mapper) { TestMapper.new }
-  let(:entity_class) { TestEntity }
-
-  include_examples :mapper
-
-  describe "#create" do
-    it "does not include protected attributes" do
-      # because it leads to exceptions when mass_assignment_sanitizer is set to strict
-      entity = TestEntity.new
-      entity.attributes = { :visible => true, :name => "Joe" }
-      mapper.create(entity)
-
-      stored_entity = mapper.find(entity.id)
-      stored_entity.attributes[:visible].should be_nil
-      stored_entity.attributes[:name].should == "Joe"
-
-      entity = TestEntity.new
-      entity.attributes = { :visible => true, :name => "Joe" }
-      TestMapper::Record.stub(:protected_attributes => [])
-      lambda { mapper.create(entity) }.should raise_error(ActiveModel::MassAssignmentSecurity::Error)
-    end
-  end
-
-  describe "#update" do
-    it "does not include protected attributes" do
-      entity = TestEntity.new
-      mapper.create(entity)
-
-      entity.attributes = { :visible => true, :name => "Joe" }
-      mapper.update(entity)
-      stored_entity = mapper.find(entity.id)
-      stored_entity.attributes[:visible].should be_nil
-      stored_entity.attributes[:name].should == "Joe"
-
-      TestMapper::Record.stub(:protected_attributes => [])
-      lambda { mapper.update(entity) }.should raise_error(ActiveModel::MassAssignmentSecurity::Error)
-    end
-  end
-end