vorpal 0.0.7.rc2 → 0.0.7.rc3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8b6b4c26a538b1aeac920b566ce3b80e0b04e4fe
-  data.tar.gz: 88d0f3a57930e3a8587d89d085293ac9d032fc58
+  metadata.gz: 9f2563f7c44050a5908ff5d2d3d08e9d35a343db
+  data.tar.gz: 569a5eb41e1278bb6b28f8e49eab416e4b38cfa5
 SHA512:
-  metadata.gz: f0190b9231a1ecce12aec8c62a92345e62ae98470f488401f8e0947ddf37bba7573263a9b78b79635958d85614b9249ab952526542cfe48a081c1d6de179ff3d
-  data.tar.gz: 32e2963bc4cb79a56de8499fff09b439475f0371d6b4d6f2cfaa8bb87d5b431070965adfcf708e45546db846063418abcabd5e9e64ed8af4054e9c45fbe37be7
+  metadata.gz: e45f9c3942c256d2fcda1c360dda4e51da9d777d8381ab7bd0dfa1fcaf6db6901a3699ab4d9110d7f850556c43fc7a345af6406eda03c7748702e69c09a06269
+  data.tar.gz: e24ffc7f449a88ad55a9af19cb4a043f389ae34e3788f466b7354a34eda83f3846562d72efda14493ed8995e05bee9a8c5c9d0d21a73e827e4d07210768ab058

data/README.md

@@ -123,22 +123,22 @@ module TreeRepository
       belongs_to :tree
     end
   end
-  @repository = engine.repository_for(Tree)
+  @mapper = engine.mapper_for(Tree)
 
   def find(tree_id)
-    @repository.load_one(@repository.db_class.where(id: tree_id).first)
+    @mapper.query.where(id: tree_id).load_one
   end
 
   def save(tree)
-    @repository.persist(tree)
+    @mapper.persist(tree)
   end
 
   def destroy(tree)
-    @repository.destroy(tree)
+    @mapper.destroy(tree)
   end
 
   def destroy_by_id(tree_id)
-    @repository.destroy_by_id(tree_id)
+    @mapper.destroy_by_id(tree_id)
   end
 end
 ```
@@ -209,8 +209,7 @@ For example:
 
 ```ruby
   def find_all
-    ids = @repository.db_class(Tree).pluck(:id) # use an AR query to determine the aggregate ids
-    @repository.load_all(ids, Tree) # use the repository to load all the aggregates
+    @mapper.query.load_all # use the mapper to load all the aggregates
   end
 ```
 

data/lib/vorpal/{aggregate_repository.rb → aggregate_mapper.rb}

@@ -1,7 +1,7 @@
 require 'vorpal/identity_map'
 
 module Vorpal
-  class AggregateRepository
+  class AggregateMapper
     # @private
     def initialize(domain_class, engine)
       @domain_class = domain_class
@@ -85,7 +85,6 @@ module Vorpal
     end
 
     def query
-      # db_class.unscoped.extending(ArelQueryMethods.new(self))
       @engine.query(@domain_class)
     end
   end

data/lib/vorpal/configuration.rb

@@ -6,7 +6,7 @@ module Vorpal
 
     # Configures and creates a {Engine} instance.
     #
-    # @param options [Hash] Global configuration options for the repository instance.
+    # @param options [Hash] Global configuration options for the engine instance.
     # @option options [Object] :db_driver (Object that will be used to interact with the DB.)
     #  Must be duck-type compatible with {DbDriver}.
     #

data/lib/vorpal/db_driver.rb

@@ -68,8 +68,8 @@ module Vorpal
     # Builds a composable query object (e.g. ActiveRecord::Relation) with Vorpal methods mixed in.
     #
     # @param class_config [ClassConfig] Config of the entity whose db representations should be returned.
-    def query(class_config)
-      class_config.db_class.unscoped.extending(ArelQueryMethods.new(self))
+    def query(class_config, aggregate_mapper)
+      class_config.db_class.unscoped.extending(ArelQueryMethods.new(aggregate_mapper))
     end
 
     private
@@ -88,30 +88,30 @@ module Vorpal
   end
 
   class ArelQueryMethods < Module
-    def initialize(repository)
-      @repository = repository
+    def initialize(mapper)
+      @mapper = mapper
     end
 
     def extended(descendant)
       super
       descendant.extend(Methods)
-      descendant.vorpal_aggregate_repository = @repository
+      descendant.vorpal_aggregate_mapper = @mapper
     end
 
     # Methods in this module will appear on any composable
     module Methods
-      attr_writer :vorpal_aggregate_repository
+      attr_writer :vorpal_aggregate_mapper
 
-      # See {AggregateRepository#load_many}.
+      # See {AggregateMapper#load_many}.
       def load_many
         db_roots = self.all
-        @vorpal_aggregate_repository.load_many(db_roots)
+        @vorpal_aggregate_mapper.load_many(db_roots)
       end
 
-      # See {AggregateRepository#load_one}.
+      # See {AggregateMapper#load_one}.
       def load_one
         db_root = self.first
-        @vorpal_aggregate_repository.load_one(db_root)
+        @vorpal_aggregate_mapper.load_one(db_root)
       end
     end
   end

data/lib/vorpal/engine.rb

@@ -2,7 +2,7 @@ require 'vorpal/identity_map'
 require 'vorpal/aggregate_utils'
 require 'vorpal/db_loader'
 require 'vorpal/db_driver'
-require 'vorpal/aggregate_repository'
+require 'vorpal/aggregate_mapper'
 
 module Vorpal
   class Engine
@@ -12,16 +12,16 @@ module Vorpal
       @configs = master_config
     end
 
-    # Creates a repository for saving/updating/loading/destroying an aggregate to/from
+    # Creates a mapper for saving/updating/loading/destroying an aggregate to/from
     # the DB. It is possible to use the methods directly on the {Engine}.
     #
     # @param domain_class [Class] Class of the root of the aggregate.
-    # @return [AggregateRepository] Repository suitable for mapping a single aggregate.
-    def repository_for(domain_class)
-      AggregateRepository.new(domain_class, self)
+    # @return [AggregateMapper] Mapper suitable for mapping a single aggregate.
+    def mapper_for(domain_class)
+      AggregateMapper.new(domain_class, self)
     end
 
-    # Try to use {AggregateRepository#persist} instead.
+    # Try to use {AggregateMapper#persist} instead.
     def persist(roots)
       roots = wrap(roots)
       return roots if roots.empty?
@@ -46,12 +46,12 @@ module Vorpal
       end
     end
 
-    # Try to use {AggregateRepository#load_one} instead.
+    # Try to use {AggregateMapper#load_one} instead.
     def load_one(db_root, domain_class, identity_map)
       load_many(Array(db_root), domain_class, identity_map).first
     end
 
-    # Try to use {AggregateRepository#load_many} instead.
+    # Try to use {AggregateMapper#load_many} instead.
     def load_many(db_roots, domain_class, identity_map)
       raise InvalidAggregateRoot, 'Nil aggregate roots are not allowed.' if db_roots.any?(&:nil?)
 
@@ -62,7 +62,7 @@ module Vorpal
       db_roots.map { |db_object| identity_map.get(db_object) }
     end
 
-    # Try to use {AggregateRepository#destroy} instead.
+    # Try to use {AggregateMapper#destroy} instead.
     def destroy(roots)
       roots = wrap(roots)
       return roots if roots.empty?
@@ -72,7 +72,7 @@ module Vorpal
       roots
     end
 
-    # Try to use {AggregateRepository#destroy_by_id} instead.
+    # Try to use {AggregateMapper#destroy_by_id} instead.
     def destroy_by_id(ids, domain_class)
       ids = wrap(ids)
       raise InvalidPrimaryKeyValue, 'Nil primary key values are not allowed.' if ids.any?(&:nil?)
@@ -84,13 +84,13 @@ module Vorpal
       ids
     end
 
-    # Try to use {AggregateRepository#db_class} instead.
+    # Try to use {AggregateMapper#db_class} instead.
     def db_class(domain_class)
       @configs.config_for(domain_class).db_class
     end
 
     def query(domain_class)
-      @db_driver.query(@configs.config_for(domain_class))
+      @db_driver.query(@configs.config_for(domain_class), mapper_for(domain_class))
     end
 
     private

data/lib/vorpal/version.rb

@@ -1,3 +1,3 @@
 module Vorpal
-  VERSION = "0.0.7.rc2"
+  VERSION = "0.0.7.rc3"
 end

data/lib/vorpal.rb

@@ -22,7 +22,7 @@ require "vorpal/configuration"
 #   end
 # end
 #
-# repository = engine.repository_for(Tree)
+# mapper = engine.mapper_for(Tree)
 # ```
 module Vorpal
   extend Vorpal::Configuration

data/spec/vorpal/acceptance/{aggregate_repository_spec.rb → aggregate_mapper_spec.rb}

@@ -3,7 +3,7 @@ require 'vorpal'
 require 'virtus'
 require 'activerecord-import/base'
 
-describe 'Aggregate Repository' do
+describe 'AggregateMapper' do
 
   # for testing polymorphic associations
   class Bug
@@ -60,20 +60,20 @@ describe 'Aggregate Repository' do
 
   describe 'new records' do
     it 'saves attributes' do
-      test_repository = configure
+      test_mapper = configure
 
       tree = Tree.new(name: 'backyard tree')
-      test_repository.persist(tree)
+      test_mapper.persist(tree)
 
       tree_db = TreeDB.first
       expect(tree_db.name).to eq 'backyard tree'
     end
 
     it 'sets the id when first saved' do
-      test_repository = configure
+      test_mapper = configure
 
       tree = Tree.new()
-      test_repository.persist(tree)
+      test_mapper.persist(tree)
 
       expect(tree.id).to_not be nil
 
@@ -82,12 +82,12 @@ describe 'Aggregate Repository' do
     end
 
     it 'saves AR::Base objects' do
-      test_repository = configure
+      test_mapper = configure
 
       fissure = Fissure.new(length: 21)
       tree = Tree.new(fissures: [fissure])
 
-      test_repository.persist(tree)
+      test_mapper.persist(tree)
 
       expect(Fissure.first.length).to eq 21
     end
@@ -96,7 +96,7 @@ describe 'Aggregate Repository' do
   describe 'on error' do
     it 'nils ids of new objects' do
       db_driver = Vorpal::DbDriver.new
-      test_repository = configure(db_driver: db_driver)
+      test_mapper = configure(db_driver: db_driver)
 
       tree_db = TreeDB.create!
 
@@ -106,7 +106,7 @@ describe 'Aggregate Repository' do
       tree = Tree.new(id: tree_db.id, fissures: [fissure])
 
       expect {
-        test_repository.persist(tree)
+        test_mapper.persist(tree)
       }.to raise_error(Exception)
 
       expect(fissure.id).to eq nil
@@ -116,112 +116,112 @@ describe 'Aggregate Repository' do
 
   describe 'existing records' do
     it 'updates attributes' do
-      test_repository = configure
+      test_mapper = configure
 
       tree = Tree.new(name: 'little tree')
-      test_repository.persist(tree)
+      test_mapper.persist(tree)
 
       tree.name = 'big tree'
-      test_repository.persist(tree)
+      test_mapper.persist(tree)
 
       tree_db = TreeDB.first
       expect(tree_db.name).to eq 'big tree'
     end
 
     it 'does not change the id on update' do
-      test_repository = configure
+      test_mapper = configure
 
       tree = Tree.new
-      test_repository.persist(tree)
+      test_mapper.persist(tree)
 
       original_id = tree.id
 
       tree.name = 'change it'
-      test_repository.persist(tree)
+      test_mapper.persist(tree)
 
       expect(tree.id).to eq original_id
     end
 
     it 'does not create additional records' do
-      test_repository = configure
+      test_mapper = configure
 
       tree = Tree.new
-      test_repository.persist(tree)
+      test_mapper.persist(tree)
 
       tree.name = 'change it'
-      test_repository.persist(tree)
+      test_mapper.persist(tree)
 
       expect(TreeDB.count).to eq 1
     end
 
     it 'removes orphans' do
-      test_repository = configure
+      test_mapper = configure
 
       tree_db = TreeDB.create!
       BranchDB.create!(tree_id: tree_db.id)
 
       tree = Tree.new(id: tree_db.id, branches: [])
 
-      test_repository.persist(tree)
+      test_mapper.persist(tree)
 
       expect(BranchDB.count).to eq 0
     end
 
     it 'does not remove orphans from unowned associations' do
-      test_repository = configure_unowned
+      test_mapper = configure_unowned
 
       tree_db = TreeDB.create!
       BranchDB.create!(tree_id: tree_db.id)
 
       tree = Tree.new(id: tree_db.id, branches: [])
 
-      test_repository.persist(tree)
+      test_mapper.persist(tree)
 
       expect(BranchDB.count).to eq 1
     end
   end
 
   it 'copies attributes to domain' do
-    test_repository = configure
+    test_mapper = configure
 
     tree_db = TreeDB.create! name: 'tree name'
-    tree = test_repository.load_one(tree_db)
+    tree = test_mapper.load_one(tree_db)
 
     expect(tree.id).to eq tree_db.id
     expect(tree.name).to eq 'tree name'
   end
 
   it 'hydrates ActiveRecord::Base associations' do
-    test_repository = configure
+    test_mapper = configure
 
     tree_db = TreeDB.create!
     Fissure.create! length: 21, tree_id: tree_db.id
 
-    tree = test_repository.load_one(tree_db)
+    tree = test_mapper.load_one(tree_db)
 
     expect(tree.fissures.first.length).to eq 21
   end
 
   describe 'cycles' do
     it 'persists' do
-     test_repository = configure_with_cycle
+     test_mapper = configure_with_cycle
 
       tree = Tree.new
       long_branch = Branch.new(length: 100, tree: tree)
       tree.branches << long_branch
 
-      test_repository.persist(tree)
+      test_mapper.persist(tree)
 
       expect(TreeDB.count).to eq 1
     end
 
     it 'hydrates' do
-      test_repository = configure_with_cycle
+      test_mapper = configure_with_cycle
 
       tree_db = TreeDB.create!
       BranchDB.create!(length: 50, tree_id: tree_db.id)
 
-      tree = test_repository.load_one(tree_db)
+      tree = test_mapper.load_one(tree_db)
 
       expect(tree).to be tree.branches.first.tree
     end
@@ -229,7 +229,7 @@ describe 'Aggregate Repository' do
 
   describe 'recursive associations' do
     it 'persists' do
-      test_repository = configure_recursive
+      test_mapper = configure_recursive
 
       tree = Tree.new
       long_branch = Branch.new(length: 100, tree: tree)
@@ -237,19 +237,19 @@ describe 'Aggregate Repository' do
       short_branch = Branch.new(length: 50, tree: tree)
       long_branch.branches << short_branch
 
-      test_repository.persist(tree)
+      test_mapper.persist(tree)
 
       expect(BranchDB.count).to eq 2
     end
 
     it 'hydrates' do
-      test_repository = configure_recursive
+      test_mapper = configure_recursive
 
       tree_db = TreeDB.create!
       long_branch = BranchDB.create!(length: 100, tree_id: tree_db.id)
       BranchDB.create!(length: 50, branch_id: long_branch.id)
 
-      tree = test_repository.load_one(tree_db)
+      tree = test_mapper.load_one(tree_db)
 
       expect(tree.branches.first.branches.first.length).to eq 50
     end
@@ -257,68 +257,68 @@ describe 'Aggregate Repository' do
 
   describe 'belongs_to associations' do
     it 'saves attributes' do
-      test_repository = configure
+      test_mapper = configure
       trunk = Trunk.new(length: 12)
       tree = Tree.new(trunk: trunk)
 
-      test_repository.persist(tree)
+      test_mapper.persist(tree)
 
       trunk_db = TrunkDB.first
       expect(trunk_db.length).to eq 12
     end
 
     it 'saves foreign keys' do
-      test_repository = configure
+      test_mapper = configure
       trunk = Trunk.new
       tree = Tree.new(trunk: trunk)
 
-      test_repository.persist(tree)
+      test_mapper.persist(tree)
 
       tree_db = TreeDB.first
       expect(tree_db.trunk_id).to eq trunk.id
     end
 
     it 'updating does not create additional rows' do
-      test_repository = configure
+      test_mapper = configure
       trunk = Trunk.new
       tree = Tree.new(trunk: trunk)
 
-      test_repository.persist(tree)
+      test_mapper.persist(tree)
 
       trunk.length = 21
 
-      expect{ test_repository.persist(tree) }.to_not change{ TrunkDB.count }
+      expect{ test_mapper.persist(tree) }.to_not change{ TrunkDB.count }
     end
 
     it 'only saves entities that are owned' do
-      test_repository = configure_unowned
+      test_mapper = configure_unowned
 
       trunk = Trunk.new
       tree = Tree.new(trunk: trunk)
 
-      test_repository.persist(tree)
+      test_mapper.persist(tree)
 
       expect(TrunkDB.count).to eq 0
     end
 
     it 'hydrates' do
-      test_repository = configure
+      test_mapper = configure
       trunk_db = TrunkDB.create!(length: 21)
       tree_db = TreeDB.create!(trunk_id: trunk_db.id)
 
-      new_tree = test_repository.load_one(tree_db)
+      new_tree = test_mapper.load_one(tree_db)
       expect(new_tree.trunk.length).to eq 21
     end
   end
 
   describe 'has_many associations' do
     it 'saves' do
-      test_repository = configure
+      test_mapper = configure
       tree = Tree.new
       tree.branches << Branch.new(length: 100)
       tree.branches << Branch.new(length: 3)
 
-      test_repository.persist(tree)
+      test_mapper.persist(tree)
 
       branches = BranchDB.all
       expect(branches.size).to eq 2
@@ -327,51 +327,51 @@ describe 'Aggregate Repository' do
     end
 
     it 'saves foreign keys' do
-      test_repository = configure
+      test_mapper = configure
       tree = Tree.new
       tree.branches << Branch.new(length: 100)
 
-      test_repository.persist(tree)
+      test_mapper.persist(tree)
 
       branches = BranchDB.all
       expect(branches.first.tree_id).to eq tree.id
     end
 
     it 'updates' do
-      test_repository = configure
+      test_mapper = configure
       tree = Tree.new
       long_branch = Branch.new(length: 100)
       tree.branches << long_branch
 
-      test_repository.persist(tree)
+      test_mapper.persist(tree)
 
       long_branch.length = 120
 
-      test_repository.persist(tree)
+      test_mapper.persist(tree)
 
       branches = BranchDB.all
       expect(branches.first.length).to eq 120
     end
 
     it 'only saves entities that are owned' do
-      test_repository = configure_unowned
+      test_mapper = configure_unowned
 
       tree = Tree.new
       long_branch = Branch.new(length: 100)
       tree.branches << long_branch
 
-      test_repository.persist(tree)
+      test_mapper.persist(tree)
 
       expect(BranchDB.count).to eq 0
     end
 
     it 'hydrates' do
-      test_repository = configure
+      test_mapper = configure
 
       tree_db = TreeDB.create!
       BranchDB.create!(length: 50, tree_id: tree_db.id)
 
-      tree = test_repository.load_one(tree_db)
+      tree = test_mapper.load_one(tree_db)
 
       expect(tree.branches.first.length).to eq 50
     end
@@ -379,42 +379,42 @@ describe 'Aggregate Repository' do
 
   describe 'has_one associations' do
     it 'saves' do
-      test_repository = configure_has_one
+      test_mapper = configure_has_one
       tree = Tree.new(name: 'big tree')
       trunk = Trunk.new(tree: tree)
 
-      test_repository.persist(trunk)
+      test_mapper.persist(trunk)
 
       expect(TreeDB.first.name).to eq 'big tree'
     end
 
     it 'saves foreign keys' do
-      test_repository = configure_has_one
+      test_mapper = configure_has_one
       tree = Tree.new(name: 'big tree')
       trunk = Trunk.new(tree: tree)
 
-      test_repository.persist(trunk)
+      test_mapper.persist(trunk)
 
       expect(TreeDB.first.trunk_id).to eq trunk.id
     end
 
     it 'only saves entities that are owned' do
-      test_repository = configure_unowned_has_one
+      test_mapper = configure_unowned_has_one
       tree = Tree.new
       trunk = Trunk.new(tree: tree)
 
-      test_repository.persist(trunk)
+      test_mapper.persist(trunk)
 
       expect(TreeDB.count).to eq 0
     end
 
     it 'hydrates' do
-      test_repository = configure_has_one
+      test_mapper = configure_has_one
 
       trunk_db = TrunkDB.create!
       TreeDB.create!(name: 'big tree', trunk_id: trunk_db.id)
 
-      trunk = test_repository.load_one(trunk_db)
+      trunk = test_mapper.load_one(trunk_db)
 
       expect(trunk.tree.name).to eq 'big tree'
     end
@@ -422,7 +422,7 @@ describe 'Aggregate Repository' do
 
   describe 'polymorphic associations' do
     it 'saves with has_manys' do
-      test_repository = configure_polymorphic_has_many
+      test_mapper = configure_polymorphic_has_many
       trunk = Trunk.new
       branch = Branch.new
       tree = Tree.new(trunk: trunk, branches: [branch])
@@ -432,50 +432,50 @@ describe 'Aggregate Repository' do
       branch_bug = Bug.new
       branch.bugs << branch_bug
 
-      test_repository.persist(tree)
+      test_mapper.persist(tree)
 
       expect(BugDB.find(trunk_bug.id).lives_on_type).to eq Trunk.name
       expect(BugDB.find(branch_bug.id).lives_on_type).to eq Branch.name
     end
 
     it 'restores with has_manys' do
-      test_repository = configure_polymorphic_has_many
+      test_mapper = configure_polymorphic_has_many
 
       trunk_db = TrunkDB.create!
       tree_db = TreeDB.create!(trunk_id: trunk_db.id)
       BugDB.create!(name: 'trunk bug', lives_on_id: trunk_db.id, lives_on_type: Trunk.name)
       BugDB.create!(name: 'not a trunk bug!', lives_on_id: trunk_db.id, lives_on_type: 'some other table')
 
-      tree = test_repository.load_one(tree_db)
+      tree = test_mapper.load_one(tree_db)
 
       expect(tree.trunk.bugs.map(&:name)).to eq ['trunk bug']
     end
 
     it 'saves with belongs_tos' do
-      test_repository = configure_polymorphic_belongs_to
+      test_mapper = configure_polymorphic_belongs_to
 
       trunk_bug = Bug.new(lives_on: Trunk.new)
       branch_bug = Bug.new(lives_on: Branch.new)
 
-      test_repository.persist([trunk_bug, branch_bug])
+      test_mapper.persist([trunk_bug, branch_bug])
 
       expect(BugDB.find(trunk_bug.id).lives_on_type).to eq Trunk.name
       expect(BugDB.find(branch_bug.id).lives_on_type).to eq Branch.name
     end
 
     it 'saves associations to unowned entities via belongs_to' do
-      test_repository = configure_unowned_polymorphic_belongs_to
+      test_mapper = configure_unowned_polymorphic_belongs_to
       trunk = Trunk.new
 
       trunk_bug = Bug.new(lives_on: trunk)
 
-      test_repository.persist(trunk_bug)
+      test_mapper.persist(trunk_bug)
 
       expect(BugDB.find(trunk_bug.id).lives_on_type).to eq Trunk.name
     end
 
     it 'restores with belongs_tos' do
-      test_repository = configure_polymorphic_belongs_to
+      test_mapper = configure_polymorphic_belongs_to
 
       # makes sure that we are using the fk_type to discriminate against
       # two entities with the same primary key value
@@ -488,19 +488,19 @@ describe 'Aggregate Repository' do
       branch_db.save!
       branch_bug_db = BugDB.create!(lives_on_id: branch_db.id, lives_on_type: Branch.name)
 
-      trunk_bug, branch_bug = test_repository.load_many([trunk_bug_db, branch_bug_db])
+      trunk_bug, branch_bug = test_mapper.load_many([trunk_bug_db, branch_bug_db])
 
       expect(trunk_bug.lives_on.length).to eq 99
       expect(branch_bug.lives_on.length).to eq 5
     end
 
     it 'restores active record objects' do
-      test_repository = configure_ar_polymorphic_belongs_to
+      test_mapper = configure_ar_polymorphic_belongs_to
 
       swamp = Swamp.create!
       tree_db = TreeDB.create!(environment_id: swamp.id, environment_type: Swamp.name)
 
-      tree = test_repository.load_one(tree_db)
+      tree = test_mapper.load_one(tree_db)
 
       expect(tree.environment).to eq swamp
     end
@@ -508,23 +508,23 @@ describe 'Aggregate Repository' do
 
   describe 'arel' do
     it 'loads many' do
-      test_repository = configure
+      test_mapper = configure
 
       TreeDB.create!
       tree_db = TreeDB.create!
 
-      trees = test_repository.query.where(id: tree_db.id).load_many
+      trees = test_mapper.query.where(id: tree_db.id).load_many
 
       expect(trees.map(&:id)).to eq [tree_db.id]
     end
 
     it 'loads one' do
-      test_repository = configure
+      test_mapper = configure
 
       TreeDB.create!
       tree_db = TreeDB.create!
 
-      trees = test_repository.query.where(id: tree_db.id).load_one
+      trees = test_mapper.query.where(id: tree_db.id).load_one
 
       expect(trees.id).to eq tree_db.id
     end
@@ -532,24 +532,24 @@ describe 'Aggregate Repository' do
 
   describe 'load_many' do
     it 'maps given db objects' do
-      test_repository = configure
+      test_mapper = configure
 
       TreeDB.create!
       tree_db = TreeDB.create!
 
-      trees = test_repository.load_many([tree_db])
+      trees = test_mapper.load_many([tree_db])
 
       expect(trees.map(&:id)).to eq [tree_db.id]
     end
 
     it 'only returns roots' do
-      test_repository = configure
+      test_mapper = configure
 
       TreeDB.create!
       tree_db = TreeDB.create!
       BranchDB.create!(tree_id: tree_db.id)
 
-      trees = test_repository.load_many([tree_db])
+      trees = test_mapper.load_many([tree_db])
 
       expect(trees.map(&:id)).to eq [tree_db.id]
     end
@@ -557,66 +557,66 @@ describe 'Aggregate Repository' do
 
   describe 'destroy' do
     it 'removes the entity from the database' do
-      test_repository = configure
+      test_mapper = configure
 
       tree_db = TreeDB.create!
 
-      test_repository.destroy([Tree.new(id: tree_db.id)])
+      test_mapper.destroy([Tree.new(id: tree_db.id)])
 
       expect(TreeDB.count).to eq 0
     end
 
     it 'removes has many children from the database' do
-      test_repository = configure
+      test_mapper = configure
 
       tree_db = TreeDB.create!
       BranchDB.create!(tree_id: tree_db.id)
 
-      test_repository.destroy(Tree.new(id: tree_db.id))
+      test_mapper.destroy(Tree.new(id: tree_db.id))
 
       expect(BranchDB.count).to eq 0
     end
 
     it 'removes belongs to children from the database' do
-      test_repository = configure
+      test_mapper = configure
 
       trunk_db = TrunkDB.create!
       tree_db = TreeDB.create!(trunk_id: trunk_db.id)
 
-      test_repository.destroy(Tree.new(id: tree_db.id))
+      test_mapper.destroy(Tree.new(id: tree_db.id))
 
       expect(TrunkDB.count).to eq 0
     end
 
     it 'removes AR children from the database' do
-      test_repository = configure
+      test_mapper = configure
 
       tree_db = TreeDB.create!
       Fissure.create!(tree_id: tree_db.id)
 
-      test_repository.destroy(Tree.new(id: tree_db.id))
+      test_mapper.destroy(Tree.new(id: tree_db.id))
 
       expect(Fissure.count).to eq 0
     end
 
     it 'leaves unowned belongs to children in the database' do
-      test_repository = configure_unowned
+      test_mapper = configure_unowned
 
       trunk_db = TrunkDB.create!
       tree_db = TreeDB.create!(trunk_id: trunk_db.id)
 
-      test_repository.destroy(Tree.new(id: tree_db.id))
+      test_mapper.destroy(Tree.new(id: tree_db.id))
 
       expect(TrunkDB.count).to eq 1
     end
 
     it 'leaves unowned has many children in the database' do
-      test_repository = configure_unowned
+      test_mapper = configure_unowned
 
       tree_db = TreeDB.create!
       BranchDB.create!(tree_id: tree_db.id)
 
-      test_repository.destroy(Tree.new(id: tree_db.id))
+      test_mapper.destroy(Tree.new(id: tree_db.id))
 
       expect(BranchDB.count).to eq 1
     end
@@ -624,11 +624,11 @@ describe 'Aggregate Repository' do
 
   describe 'destroy_by_id' do
     it 'removes the entity from the database' do
-      test_repository = configure
+      test_mapper = configure
 
       tree_db = TreeDB.create!
 
-      test_repository.destroy_by_id([tree_db.id])
+      test_mapper.destroy_by_id([tree_db.id])
 
       expect(TreeDB.count).to eq 0
     end
@@ -636,97 +636,97 @@ describe 'Aggregate Repository' do
 
   describe 'non-existent values' do
     it 'load_many returns an empty array when given an empty array' do
-      test_repository = configure
+      test_mapper = configure
 
-      results = test_repository.load_many([])
+      results = test_mapper.load_many([])
       expect(results).to eq []
     end
 
     it 'load_many throws an exception when given a nil db_object' do
-      test_repository = configure
+      test_mapper = configure
 
       expect {
-        test_repository.load_many([nil])
+        test_mapper.load_many([nil])
       }.to raise_error(Vorpal::InvalidAggregateRoot, "Nil aggregate roots are not allowed.")
     end
 
     it 'load_one returns nil when given nil' do
-      test_repository = configure
+      test_mapper = configure
 
-      result = test_repository.load_one(nil)
+      result = test_mapper.load_one(nil)
       expect(result).to eq nil
     end
 
     it 'persist ignores empty arrays' do
-      test_repository = configure
+      test_mapper = configure
 
-      results = test_repository.persist([])
+      results = test_mapper.persist([])
       expect(results).to eq []
     end
 
     it 'persist throws an exception when given a collection with a nil root' do
-      test_repository = configure
+      test_mapper = configure
       expect {
-        test_repository.persist([nil])
+        test_mapper.persist([nil])
       }.to raise_error(Vorpal::InvalidAggregateRoot, "Nil aggregate roots are not allowed.")
     end
 
     it 'persist throws an exception when given a nil root' do
-      test_repository = configure
+      test_mapper = configure
       expect {
-        test_repository.persist(nil)
+        test_mapper.persist(nil)
       }.to raise_error(Vorpal::InvalidAggregateRoot, "Nil aggregate roots are not allowed.")
     end
 
     it 'destroy ignores empty arrays' do
-      test_repository = configure
+      test_mapper = configure
 
-      results = test_repository.destroy([])
+      results = test_mapper.destroy([])
       expect(results).to eq []
     end
 
     it 'destroy throws an exception when given a nil root' do
-      test_repository = configure
+      test_mapper = configure
 
       expect {
-        test_repository.destroy(nil)
+        test_mapper.destroy(nil)
       }.to raise_error(Vorpal::InvalidAggregateRoot, "Nil aggregate roots are not allowed.")
     end
 
     it 'destroy throws an exception when given a collection with a nil root' do
-      test_repository = configure
+      test_mapper = configure
 
       expect {
-        test_repository.destroy([nil])
+        test_mapper.destroy([nil])
       }.to raise_error(Vorpal::InvalidAggregateRoot, "Nil aggregate roots are not allowed.")
     end
 
     it 'destroy_by_id ignores empty arrays' do
-      test_repository = configure
+      test_mapper = configure
 
-      results = test_repository.destroy_by_id([])
+      results = test_mapper.destroy_by_id([])
       expect(results).to eq []
     end
 
     it 'destroy_by_id ignores ids that do not exist' do
-      test_repository = configure
+      test_mapper = configure
 
-      test_repository.destroy_by_id([99])
+      test_mapper.destroy_by_id([99])
     end
 
     it 'destroy_by_id throws an exception when given a collection with a nil id' do
-      test_repository = configure
+      test_mapper = configure
 
       expect {
-        test_repository.destroy_by_id([nil])
+        test_mapper.destroy_by_id([nil])
       }.to raise_error(Vorpal::InvalidPrimaryKeyValue, "Nil primary key values are not allowed.")
     end
 
     it 'destroy_by_id throws an exception when given a nil id' do
-      test_repository = configure
+      test_mapper = configure
 
       expect {
-        test_repository.destroy_by_id(nil)
+        test_mapper.destroy_by_id(nil)
       }.to raise_error(Vorpal::InvalidPrimaryKeyValue, "Nil primary key values are not allowed.")
     end
   end
@@ -755,7 +755,7 @@ private
         attributes :name
       end
     end
-    engine.repository_for(Tree)
+    engine.mapper_for(Tree)
   end
 
   def configure_polymorphic_belongs_to
@@ -773,7 +773,7 @@ private
         attributes :length
       end
     end
-    engine.repository_for(Bug)
+    engine.mapper_for(Bug)
   end
 
   def configure_ar_polymorphic_belongs_to
@@ -785,7 +785,7 @@ private
 
       map Swamp, to: Swamp
     end
-    engine.repository_for(Tree)
+    engine.mapper_for(Tree)
   end
 
   def configure_unowned_polymorphic_belongs_to
@@ -803,7 +803,7 @@ private
         attributes :length
       end
     end
-    engine.repository_for(Bug)
+    engine.mapper_for(Bug)
   end
 
   def configure_unowned
@@ -822,7 +822,7 @@ private
         attributes :length
       end
     end
-    engine.repository_for(Tree)
+    engine.mapper_for(Tree)
   end
 
   def configure_recursive
@@ -837,7 +837,7 @@ private
         has_many :branches
       end
     end
-    engine.repository_for(Tree)
+    engine.mapper_for(Tree)
   end
 
   def configure_with_cycle
@@ -852,7 +852,7 @@ private
         has_many :branches
       end
     end
-    engine.repository_for(Tree)
+    engine.mapper_for(Tree)
   end
   
   def configure(options={})
@@ -874,7 +874,7 @@ private
 
       map Fissure, to: Fissure
     end
-    engine.repository_for(Tree)
+    engine.mapper_for(Tree)
   end
 
   def configure_has_one
@@ -888,7 +888,7 @@ private
         attributes :name
       end
     end
-    engine.repository_for(Trunk)
+    engine.mapper_for(Trunk)
   end
 
   def configure_unowned_has_one
@@ -902,6 +902,6 @@ private
         attributes :name
       end
     end
-    engine.repository_for(Trunk)
+    engine.mapper_for(Trunk)
   end
 end

data/spec/vorpal/performance/performance_spec.rb

@@ -74,7 +74,7 @@ describe 'performance' do
     TrunkDB = defineAr('trunks_perf')
   end
 
-  let(:test_repository) { build_repository }
+  let(:test_mapper) { build_mapper }
 
   # Vorpal 0.0.5:
   #               user     system      total        real
@@ -92,10 +92,10 @@ describe 'performance' do
   it 'benchmarks all operations' do
     trees = build_trees(1000)
     Benchmark.bm(7) do |x|
-      x.report('create') { test_repository.persist(trees) }
-      x.report('update') { test_repository.persist(trees) }
-      x.report('load') { test_repository.load_many(TreeDB.where(id: trees.map(&:id)).all) }
-      x.report('destroy') { test_repository.destroy(trees) }
+      x.report('create') { test_mapper.persist(trees) }
+      x.report('update') { test_mapper.persist(trees) }
+      x.report('load') { test_mapper.load_many(TreeDB.where(id: trees.map(&:id)).all) }
+      x.report('destroy') { test_mapper.destroy(trees) }
     end
   end
 
@@ -104,39 +104,39 @@ describe 'performance' do
 
     puts 'starting persistence benchmark'
     puts Benchmark.measure {
-      test_repository.persist(trees)
+      test_mapper.persist(trees)
     }
   end
 
   it 'updates aggregates quickly' do
     trees = build_trees(1000)
 
-    test_repository.persist(trees)
+    test_mapper.persist(trees)
 
     puts 'starting update benchmark'
     puts Benchmark.measure {
-      test_repository.persist(trees)
+      test_mapper.persist(trees)
     }
   end
 
   it 'loads aggregates quickly' do
     trees = build_trees(1000)
-    test_repository.persist(trees)
+    test_mapper.persist(trees)
     ids = trees.map(&:id)
 
     puts 'starting loading benchmark'
     puts Benchmark.measure {
-      test_repository.load_many(TreeDB.where(id: ids).all)
+      test_mapper.load_many(TreeDB.where(id: ids).all)
     }
   end
 
   it 'destroys aggregates quickly' do
     trees = build_trees(1000)
-    test_repository.persist(trees)
+    test_mapper.persist(trees)
 
     puts 'starting destruction benchmark'
     puts Benchmark.measure {
-      test_repository.destroy(trees)
+      test_mapper.destroy(trees)
     }
   end
 
@@ -162,7 +162,7 @@ describe 'performance' do
     bug_home.bugs = [bug]
   end
 
-  def build_repository
+  def build_mapper
     engine = Vorpal.define do
       map Tree do
         attributes :name
@@ -188,6 +188,6 @@ describe 'performance' do
         belongs_to :lives_on, fk: :lives_on_id, fk_type: :lives_on_type, child_classes: [Trunk, Branch]
       end
     end
-    engine.repository_for(Tree)
+    engine.mapper_for(Tree)
   end
 end

data/spec/vorpal/unit/db_loader_spec.rb

@@ -65,7 +65,7 @@ describe Vorpal::DbLoader do
   #   p loaded_objects.all_objects
   #   # expect(loaded_objects.all_objects.size).to eq(2)
   #
-  #   repo = Vorpal::AggregateRepository.new(driver, master_config)
+  #   repo = Vorpal::AggregateMapper.new(driver, master_config)
   #   post = repo.load(post_db.id, Post)
   #   p post
   #   expect(post.comments.size).to eq(1)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: vorpal
 version: !ruby/object:Gem::Version
-  version: 0.0.7.rc2
+  version: 0.0.7.rc3
 platform: ruby
 authors:
 - Sean Kirby
@@ -152,7 +152,7 @@ files:
 - README.md
 - Rakefile
 - lib/vorpal.rb
-- lib/vorpal/aggregate_repository.rb
+- lib/vorpal/aggregate_mapper.rb
 - lib/vorpal/aggregate_traversal.rb
 - lib/vorpal/aggregate_utils.rb
 - lib/vorpal/config_builder.rb
@@ -169,7 +169,7 @@ files:
 - spec/helpers/db_helpers.rb
 - spec/integration_spec_helper.rb
 - spec/unit_spec_helper.rb
-- spec/vorpal/acceptance/aggregate_repository_spec.rb
+- spec/vorpal/acceptance/aggregate_mapper_spec.rb
 - spec/vorpal/performance/performance_spec.rb
 - spec/vorpal/unit/config_builder_spec.rb
 - spec/vorpal/unit/configs_spec.rb
@@ -205,7 +205,7 @@ test_files:
 - spec/helpers/db_helpers.rb
 - spec/integration_spec_helper.rb
 - spec/unit_spec_helper.rb
-- spec/vorpal/acceptance/aggregate_repository_spec.rb
+- spec/vorpal/acceptance/aggregate_mapper_spec.rb
 - spec/vorpal/performance/performance_spec.rb
 - spec/vorpal/unit/config_builder_spec.rb
 - spec/vorpal/unit/configs_spec.rb