vorpal 0.0.7 → 0.1.0.rc1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 813b5c4649667d1289ba85df9114c3badab734ed
-  data.tar.gz: e1e1deb869ccc9b6577b87f9122c056ff0803d62
+  metadata.gz: 17ab11df546bfd8e01294e22adea87aca049266c
+  data.tar.gz: 1e25a0a08f20f54b277c05e2e90b8bc0d825e85e
 SHA512:
-  metadata.gz: 9b7692c298e0f63b8bf1723d2846899e8ac62adc9434e1f01c234e2c027c579752e2bf8d2e95d58c93df9a6d28243b4c9a00f694222608ea825dc3cdad08d7ef
-  data.tar.gz: 5e5a6b4280a7a94e8f1c1cc0564fb1718af715e3390819caa477f3a99db3f401e15255d37f6f0dfaae48b49eb037a753a4271442e01bcb9c7a5233cef4c5c7f5
+  metadata.gz: 5c6f183685c1b5fbd7a89926556a545a20ef87aa604184d5a90223f234780a5a63dd0d69a7e72c864b7dfe79c1fad917f51cb0a8785496400e28d481d7356a27
+  data.tar.gz: 70c6768539ac3214339352902e752ae3178eb109d6cb1f791f62719ba18473cc65a7d950435dac40e936e854d4c06b6a4e1050a95edbe39aa7c190810ad2eb89

data/.rspec

@@ -0,0 +1 @@
+--colour

data/README.md

@@ -237,8 +237,8 @@ For example:
 
 1. Start a PostgreSQL server.
 2. Either:
-  * Create a DB user called `vorpal` with password `pass` and a DB called `vorpal_test`. OR:
-  * Modify `spec/integration_spec_helper.rb`.
+  * Create a DB user called `vorpal` with password `pass`. OR:
+  * Modify `spec/helpers/db_helpers.rb`.
 3. Run `rake` from the terminal.
 
 ## Contributors

data/lib/vorpal/config_builder.rb

@@ -87,21 +87,13 @@ module Vorpal
 
     # @private
     def build_db_class
-      # Module#parent comes from 'active_support/core_ext/module/introspection'
-      parent_module = @domain_class.parent
-
-      return parent_module.const_get(db_class_name) if parent_module.const_defined?(db_class_name)
-
-      db_class = @db_driver.build_db_class(table_name)
-      parent_module.const_set(db_class_name, db_class)
-
-      db_class
+      @db_driver.build_db_class(table_name)
     end
 
     private
 
     def db_class_name
-      @domain_class.name.split('::').last + 'DB'
+      ActiveSupport::Inflector.demodulize(@domain_class.name) + 'DB'
     end
 
     def build_class_config
@@ -125,11 +117,13 @@ module Vorpal
     end
 
     def foreign_key(name)
-      ActiveSupport::Inflector.underscore(name.to_s) + '_id'
+      ActiveSupport::Inflector.foreign_key(name.to_s)
     end
 
     def child_class(association_name)
-      ActiveSupport::Inflector.constantize(ActiveSupport::Inflector.classify(association_name.to_s))
+      # Module#parent comes from 'active_support/core_ext/module/introspection'
+      parent_module = @domain_class.parent
+      parent_module.const_get(ActiveSupport::Inflector.classify(association_name.to_s))
     end
 
     def build_has_ones

data/lib/vorpal/configuration.rb

@@ -21,8 +21,9 @@ module Vorpal
     #
     # @param domain_class [Class] Type of the domain model to be mapped
     # @param options [Hash] Configure how to map the domain model
-    # @option options [String] :to (Class with the same name as the domain class with a 'DB' appended.)
+    # @option options [String] :to
     #   Class of the ActiveRecord object that will map this domain class to the DB.
+    #   Optional, if one is not specified, it will be generated.
     # @option options [Object] :serializer (map the {ConfigBuilder#attributes} directly)
     #   Object that will convert the domain objects into a hash.
     #

data/lib/vorpal/db_driver.rb

@@ -60,7 +60,24 @@ module Vorpal
     # @param table_name [String] Name of the DB table the DB class should interface with.
     # @return [Class] ActiveRecord::Base Class
     def build_db_class(table_name)
-      db_class = Class.new(ActiveRecord::Base)
+      db_class = Class.new(ActiveRecord::Base) do
+        # This is overridden for two reasons:
+        # 1) For anonymous classes, #name normally returns nil. Class names in Ruby come from the
+        #   name of the constant they are assigned to.
+        # 2) Because the default implementation for Class#name for anonymous classes is very, very
+        #   slow. https://bugs.ruby-lang.org/issues/11119
+        # Remove this override once #2 has been fixed!
+        def self.name
+          @name ||= "Vorpal Generated ActiveRecord::Base Class for #{table_name}. Object ID: #{object_id}"
+        end
+
+        # Overridden because, like #name, the default implementation for anonymous classes is very,
+        # very slow.
+        def self.to_s
+          name
+        end
+      end
+
       db_class.table_name = table_name
       db_class
     end

data/lib/vorpal/engine.rb

@@ -3,6 +3,7 @@ require 'vorpal/aggregate_utils'
 require 'vorpal/db_loader'
 require 'vorpal/db_driver'
 require 'vorpal/aggregate_mapper'
+require 'vorpal/exceptions'
 
 module Vorpal
   class Engine
@@ -233,9 +234,4 @@ module Vorpal
       objects.each { |object| object.id = nil }
     end
   end
-
-  class InvalidPrimaryKeyValue < StandardError
-  end
-  class InvalidAggregateRoot < StandardError
-  end
 end

data/lib/vorpal/exceptions.rb

@@ -0,0 +1,5 @@
+module Vorpal
+  class InvalidPrimaryKeyValue < StandardError; end
+
+  class InvalidAggregateRoot < StandardError; end
+end

data/lib/vorpal/identity_map.rb

@@ -28,6 +28,7 @@ module Vorpal
     def key(key_object)
       return nil unless key_object
       raise "Cannot put entity '#{key_object.inspect}' into IdentityMap without an id." if key_object.id.nil?
+      raise "Cannot put entity '#{key_object.inspect}' into IdentityMap without a Class with a name." if key_object.class.name.nil?
       [key_object.id, key_object.class.name]
     end
   end

data/lib/vorpal/version.rb

@@ -1,3 +1,3 @@
 module Vorpal
-  VERSION = "0.0.7"
+  VERSION = "0.1.0.rc1"
 end

data/spec/helpers/db_helpers.rb

@@ -1,8 +1,41 @@
 module DbHelpers
+  module_function
+
+  CONNECTION_SETTINGS = {
+    adapter: 'postgresql',
+    host: 'localhost',
+    database: 'vorpal_test',
+    min_messages: 'error',
+    # Change the following to reflect your database settings
+    # username: 'vorpal',
+    # password: 'pass',
+  }
+
+  def ensure_database_exists
+    test_database_name = CONNECTION_SETTINGS.fetch(:database)
+    if !db_exists?(test_database_name)
+      db_connection.create_database(test_database_name)
+    end
+  end
+
+  def db_exists?(db_name)
+    ActiveRecord::Base.establish_connection(CONNECTION_SETTINGS.merge(database: 'template1'))
+
+    return db_connection.exec_query("SELECT 1 from pg_database WHERE datname='#{db_name}';").present?
+  end
+
+  def db_connection
+    ActiveRecord::Base.connection
+  end
+
+  def establish_connection
+    ActiveRecord::Base.establish_connection(CONNECTION_SETTINGS)
+  end
+
   # when you change a table's columns, set force to true to re-generate the table in the DB
   def define_table(table_name, columns, force)
-    if !ActiveRecord::Base.connection.table_exists?(table_name) || force
-      ActiveRecord::Base.connection.create_table(table_name, force: true) do |t|
+    if !db_connection.table_exists?(table_name) || force
+      db_connection.create_table(table_name, force: true) do |t|
         columns.each do |name, type|
           t.send(type, name)
         end

data/spec/helpers/profile_helpers.rb

@@ -0,0 +1,26 @@
+require 'ruby-prof'
+
+# In order to use these helpers do the following:
+# 1) Add `spec.add_development_dependency "ruby-prof"` to the vorpal.gemspec file.
+# 2) Do a `bundle update`
+# 3) Add `require 'helpers/profile_helpers'` to the spec where you wish to profile.
+module ProfileHelpers
+  module_function
+
+  # Runs a block a given number of times and outputs the profiling results in a 'Call Tree'
+  # format suitable for display by a tool like KCacheGrind.
+  #
+  # - Installing QCacheGrind on OSX: http://nickology.com/2014/04/16/view-xdebug-cachegrind-files-on-mac-os/
+  def output_callgrind(description, times=1, &block)
+    RubyProf.measure_mode = RubyProf::PROCESS_TIME
+    RubyProf.start
+
+    times.times(&block)
+
+    result = RubyProf.stop
+    printer = RubyProf::CallTreePrinter.new(result)
+    File.open("#{description}_#{DateTime.now.strftime("%FT%H:%M:%S%z")}.callgrind", "w") do |file|
+      printer.print(file)
+    end
+  end
+end

data/spec/integration_spec_helper.rb

@@ -1,18 +1,10 @@
 require 'active_record'
 require 'pg'
-
-# Change the following to reflect your database settings
-ActiveRecord::Base.establish_connection(
-  adapter:  'postgresql',
-  host:     'localhost',
-  database: 'vorpal_test',
-  username: 'vorpal',
-  password: 'pass',
-  min_messages: 'error'
-)
-
 require 'helpers/db_helpers'
 
+DbHelpers.ensure_database_exists
+DbHelpers.establish_connection
+
 RSpec.configure do |config|
   config.include DbHelpers
 

data/spec/vorpal/acceptance/aggregate_mapper_spec.rb

@@ -65,7 +65,7 @@ describe 'AggregateMapper' do
       tree = Tree.new(name: 'backyard tree')
       test_mapper.persist(tree)
 
-      tree_db = TreeDB.first
+      tree_db = db_class_for(Tree, test_mapper).first
       expect(tree_db.name).to eq 'backyard tree'
     end
 
@@ -77,7 +77,7 @@ describe 'AggregateMapper' do
 
       expect(tree.id).to_not be nil
 
-      tree_db = TreeDB.first
+      tree_db = db_class_for(Tree, test_mapper).first
       expect(tree_db.id).to eq tree.id
     end
 
@@ -98,7 +98,7 @@ describe 'AggregateMapper' do
       db_driver = Vorpal::DbDriver.new
       test_mapper = configure(db_driver: db_driver)
 
-      tree_db = TreeDB.create!
+      tree_db = db_class_for(Tree, test_mapper).create!
 
       expect(db_driver).to receive(:update).and_raise('not so good')
 
@@ -124,7 +124,7 @@ describe 'AggregateMapper' do
       tree.name = 'big tree'
       test_mapper.persist(tree)
 
-      tree_db = TreeDB.first
+      tree_db = db_class_for(Tree, test_mapper).first
       expect(tree_db.name).to eq 'big tree'
     end
 
@@ -151,40 +151,40 @@ describe 'AggregateMapper' do
       tree.name = 'change it'
       test_mapper.persist(tree)
 
-      expect(TreeDB.count).to eq 1
+      expect(db_class_for(Tree, test_mapper).count).to eq 1
     end
 
     it 'removes orphans' do
       test_mapper = configure
 
-      tree_db = TreeDB.create!
-      BranchDB.create!(tree_id: tree_db.id)
+      tree_db = db_class_for(Tree, test_mapper).create!
+      db_class_for(Branch, test_mapper).create!(tree_id: tree_db.id)
 
       tree = Tree.new(id: tree_db.id, branches: [])
 
       test_mapper.persist(tree)
 
-      expect(BranchDB.count).to eq 0
+      expect(db_class_for(Branch, test_mapper).count).to eq 0
     end
 
     it 'does not remove orphans from unowned associations' do
       test_mapper = configure_unowned
 
-      tree_db = TreeDB.create!
-      BranchDB.create!(tree_id: tree_db.id)
+      tree_db = db_class_for(Tree, test_mapper).create!
+      db_class_for(Branch, test_mapper).create!(tree_id: tree_db.id)
 
       tree = Tree.new(id: tree_db.id, branches: [])
 
       test_mapper.persist(tree)
 
-      expect(BranchDB.count).to eq 1
+      expect(db_class_for(Branch, test_mapper).count).to eq 1
     end
   end
 
   it 'copies attributes to domain' do
     test_mapper = configure
 
-    tree_db = TreeDB.create! name: 'tree name'
+    tree_db = db_class_for(Tree, test_mapper).create! name: 'tree name'
     tree = test_mapper.load_one(tree_db)
 
     expect(tree.id).to eq tree_db.id
@@ -194,7 +194,7 @@ describe 'AggregateMapper' do
   it 'hydrates ActiveRecord::Base associations' do
     test_mapper = configure
 
-    tree_db = TreeDB.create!
+    tree_db = db_class_for(Tree, test_mapper).create!
     Fissure.create! length: 21, tree_id: tree_db.id
 
     tree = test_mapper.load_one(tree_db)
@@ -212,14 +212,14 @@ describe 'AggregateMapper' do
 
       test_mapper.persist(tree)
 
-      expect(TreeDB.count).to eq 1
+      expect(db_class_for(Tree, test_mapper).count).to eq 1
     end
 
     it 'hydrates' do
       test_mapper = configure_with_cycle
 
-      tree_db = TreeDB.create!
-      BranchDB.create!(length: 50, tree_id: tree_db.id)
+      tree_db = db_class_for(Tree, test_mapper).create!
+      db_class_for(Branch, test_mapper).create!(length: 50, tree_id: tree_db.id)
 
       tree = test_mapper.load_one(tree_db)
 
@@ -239,15 +239,15 @@ describe 'AggregateMapper' do
 
       test_mapper.persist(tree)
 
-      expect(BranchDB.count).to eq 2
+      expect(db_class_for(Branch, test_mapper).count).to eq 2
     end
 
     it 'hydrates' do
       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_db = db_class_for(Tree, test_mapper).create!
+      long_branch = db_class_for(Branch, test_mapper).create!(length: 100, tree_id: tree_db.id)
+      db_class_for(Branch, test_mapper).create!(length: 50, branch_id: long_branch.id)
 
       tree = test_mapper.load_one(tree_db)
 
@@ -263,7 +263,7 @@ describe 'AggregateMapper' do
 
       test_mapper.persist(tree)
 
-      trunk_db = TrunkDB.first
+      trunk_db = db_class_for(Trunk, test_mapper).first
       expect(trunk_db.length).to eq 12
     end
 
@@ -274,7 +274,7 @@ describe 'AggregateMapper' do
 
       test_mapper.persist(tree)
 
-      tree_db = TreeDB.first
+      tree_db = db_class_for(Tree, test_mapper).first
       expect(tree_db.trunk_id).to eq trunk.id
     end
 
@@ -287,7 +287,7 @@ describe 'AggregateMapper' do
 
       trunk.length = 21
 
-      expect{ test_mapper.persist(tree) }.to_not change{ TrunkDB.count }
+      expect{ test_mapper.persist(tree) }.to_not change{ db_class_for(Trunk, test_mapper).count }
     end
 
     it 'only saves entities that are owned' do
@@ -298,13 +298,13 @@ describe 'AggregateMapper' do
 
       test_mapper.persist(tree)
 
-      expect(TrunkDB.count).to eq 0
+      expect(db_class_for(Trunk, test_mapper).count).to eq 0
     end
 
     it 'hydrates' do
       test_mapper = configure
-      trunk_db = TrunkDB.create!(length: 21)
-      tree_db = TreeDB.create!(trunk_id: trunk_db.id)
+      trunk_db = db_class_for(Trunk, test_mapper).create!(length: 21)
+      tree_db = db_class_for(Tree, test_mapper).create!(trunk_id: trunk_db.id)
 
       new_tree = test_mapper.load_one(tree_db)
       expect(new_tree.trunk.length).to eq 21
@@ -320,7 +320,7 @@ describe 'AggregateMapper' do
 
       test_mapper.persist(tree)
 
-      branches = BranchDB.all
+      branches = db_class_for(Branch, test_mapper).all
       expect(branches.size).to eq 2
       expect(branches.first.length).to eq 100
       expect(branches.second.length).to eq 3
@@ -333,7 +333,7 @@ describe 'AggregateMapper' do
 
       test_mapper.persist(tree)
 
-      branches = BranchDB.all
+      branches = db_class_for(Branch, test_mapper).all
       expect(branches.first.tree_id).to eq tree.id
     end
 
@@ -349,7 +349,7 @@ describe 'AggregateMapper' do
 
       test_mapper.persist(tree)
 
-      branches = BranchDB.all
+      branches = db_class_for(Branch, test_mapper).all
       expect(branches.first.length).to eq 120
     end
 
@@ -362,14 +362,14 @@ describe 'AggregateMapper' do
 
       test_mapper.persist(tree)
 
-      expect(BranchDB.count).to eq 0
+      expect(db_class_for(Branch, test_mapper).count).to eq 0
     end
 
     it 'hydrates' do
       test_mapper = configure
 
-      tree_db = TreeDB.create!
-      BranchDB.create!(length: 50, tree_id: tree_db.id)
+      tree_db = db_class_for(Tree, test_mapper).create!
+      db_class_for(Branch, test_mapper).create!(length: 50, tree_id: tree_db.id)
 
       tree = test_mapper.load_one(tree_db)
 
@@ -385,7 +385,7 @@ describe 'AggregateMapper' do
 
       test_mapper.persist(trunk)
 
-      expect(TreeDB.first.name).to eq 'big tree'
+      expect(db_class_for(Tree, test_mapper).first.name).to eq 'big tree'
     end
 
     it 'saves foreign keys' do
@@ -395,7 +395,7 @@ describe 'AggregateMapper' do
 
       test_mapper.persist(trunk)
 
-      expect(TreeDB.first.trunk_id).to eq trunk.id
+      expect(db_class_for(Tree, test_mapper).first.trunk_id).to eq trunk.id
     end
 
     it 'only saves entities that are owned' do
@@ -405,14 +405,14 @@ describe 'AggregateMapper' do
 
       test_mapper.persist(trunk)
 
-      expect(TreeDB.count).to eq 0
+      expect(db_class_for(Tree, test_mapper).count).to eq 0
     end
 
     it 'hydrates' do
       test_mapper = configure_has_one
 
-      trunk_db = TrunkDB.create!
-      TreeDB.create!(name: 'big tree', trunk_id: trunk_db.id)
+      trunk_db = db_class_for(Trunk, test_mapper).create!
+      db_class_for(Tree, test_mapper).create!(name: 'big tree', trunk_id: trunk_db.id)
 
       trunk = test_mapper.load_one(trunk_db)
 
@@ -434,17 +434,17 @@ describe 'AggregateMapper' do
 
       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
+      expect(db_class_for(Bug, test_mapper).find(trunk_bug.id).lives_on_type).to eq Trunk.name
+      expect(db_class_for(Bug, test_mapper).find(branch_bug.id).lives_on_type).to eq Branch.name
     end
 
     it 'restores with has_manys' do
       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')
+      trunk_db = db_class_for(Trunk, test_mapper).create!
+      tree_db = db_class_for(Tree, test_mapper).create!(trunk_id: trunk_db.id)
+      db_class_for(Bug, test_mapper).create!(name: 'trunk bug', lives_on_id: trunk_db.id, lives_on_type: Trunk.name)
+      db_class_for(Bug, test_mapper).create!(name: 'not a trunk bug!', lives_on_id: trunk_db.id, lives_on_type: 'some other table')
 
       tree = test_mapper.load_one(tree_db)
 
@@ -459,8 +459,8 @@ describe 'AggregateMapper' do
 
       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
+      expect(db_class_for(Bug, test_mapper).find(trunk_bug.id).lives_on_type).to eq Trunk.name
+      expect(db_class_for(Bug, test_mapper).find(branch_bug.id).lives_on_type).to eq Branch.name
     end
 
     it 'saves associations to unowned entities via belongs_to' do
@@ -471,7 +471,7 @@ describe 'AggregateMapper' do
 
       test_mapper.persist(trunk_bug)
 
-      expect(BugDB.find(trunk_bug.id).lives_on_type).to eq Trunk.name
+      expect(db_class_for(Bug, test_mapper).find(trunk_bug.id).lives_on_type).to eq Trunk.name
     end
 
     it 'restores with belongs_tos' do
@@ -479,14 +479,14 @@ describe 'AggregateMapper' do
 
       # makes sure that we are using the fk_type to discriminate against
       # two entities with the same primary key value
-      trunk_db = TrunkDB.new(length: 99)
+      trunk_db = db_class_for(Trunk, test_mapper).new(length: 99)
       trunk_db.id = 99
       trunk_db.save!
-      trunk_bug_db = BugDB.create!(lives_on_id: trunk_db.id, lives_on_type: Trunk.name)
-      branch_db = BranchDB.new(length: 5)
+      trunk_bug_db = db_class_for(Bug, test_mapper).create!(lives_on_id: trunk_db.id, lives_on_type: Trunk.name)
+      branch_db = db_class_for(Branch, test_mapper).new(length: 5)
       branch_db.id = 99
       branch_db.save!
-      branch_bug_db = BugDB.create!(lives_on_id: branch_db.id, lives_on_type: Branch.name)
+      branch_bug_db = db_class_for(Bug, test_mapper).create!(lives_on_id: branch_db.id, lives_on_type: Branch.name)
 
       trunk_bug, branch_bug = test_mapper.load_many([trunk_bug_db, branch_bug_db])
 
@@ -498,7 +498,7 @@ describe 'AggregateMapper' do
       test_mapper = configure_ar_polymorphic_belongs_to
 
       swamp = Swamp.create!
-      tree_db = TreeDB.create!(environment_id: swamp.id, environment_type: Swamp.name)
+      tree_db = db_class_for(Tree, test_mapper).create!(environment_id: swamp.id, environment_type: Swamp.name)
 
       tree = test_mapper.load_one(tree_db)
 
@@ -510,8 +510,8 @@ describe 'AggregateMapper' do
     it 'loads many' do
       test_mapper = configure
 
-      TreeDB.create!
-      tree_db = TreeDB.create!
+      db_class_for(Tree, test_mapper).create!
+      tree_db = db_class_for(Tree, test_mapper).create!
 
       trees = test_mapper.query.where(id: tree_db.id).load_many
 
@@ -521,8 +521,8 @@ describe 'AggregateMapper' do
     it 'loads one' do
       test_mapper = configure
 
-      TreeDB.create!
-      tree_db = TreeDB.create!
+      db_class_for(Tree, test_mapper).create!
+      tree_db = db_class_for(Tree, test_mapper).create!
 
       trees = test_mapper.query.where(id: tree_db.id).load_one
 
@@ -534,8 +534,8 @@ describe 'AggregateMapper' do
     it 'maps given db objects' do
       test_mapper = configure
 
-      TreeDB.create!
-      tree_db = TreeDB.create!
+      db_class_for(Tree, test_mapper).create!
+      tree_db = db_class_for(Tree, test_mapper).create!
 
       trees = test_mapper.load_many([tree_db])
 
@@ -545,9 +545,9 @@ describe 'AggregateMapper' do
     it 'only returns roots' do
       test_mapper = configure
 
-      TreeDB.create!
-      tree_db = TreeDB.create!
-      BranchDB.create!(tree_id: tree_db.id)
+      db_class_for(Tree, test_mapper).create!
+      tree_db = db_class_for(Tree, test_mapper).create!
+      db_class_for(Branch, test_mapper).create!(tree_id: tree_db.id)
 
       trees = test_mapper.load_many([tree_db])
 
@@ -559,39 +559,39 @@ describe 'AggregateMapper' do
     it 'removes the entity from the database' do
       test_mapper = configure
 
-      tree_db = TreeDB.create!
+      tree_db = db_class_for(Tree, test_mapper).create!
 
       test_mapper.destroy([Tree.new(id: tree_db.id)])
 
-      expect(TreeDB.count).to eq 0
+      expect(db_class_for(Tree, test_mapper).count).to eq 0
     end
 
     it 'removes has many children from the database' do
       test_mapper = configure
 
-      tree_db = TreeDB.create!
-      BranchDB.create!(tree_id: tree_db.id)
+      tree_db = db_class_for(Tree, test_mapper).create!
+      db_class_for(Branch, test_mapper).create!(tree_id: tree_db.id)
 
       test_mapper.destroy(Tree.new(id: tree_db.id))
 
-      expect(BranchDB.count).to eq 0
+      expect(db_class_for(Branch, test_mapper).count).to eq 0
     end
 
     it 'removes belongs to children from the database' do
       test_mapper = configure
 
-      trunk_db = TrunkDB.create!
-      tree_db = TreeDB.create!(trunk_id: trunk_db.id)
+      trunk_db = db_class_for(Trunk, test_mapper).create!
+      tree_db = db_class_for(Tree, test_mapper).create!(trunk_id: trunk_db.id)
 
       test_mapper.destroy(Tree.new(id: tree_db.id))
 
-      expect(TrunkDB.count).to eq 0
+      expect(db_class_for(Trunk, test_mapper).count).to eq 0
     end
 
     it 'removes AR children from the database' do
       test_mapper = configure
 
-      tree_db = TreeDB.create!
+      tree_db = db_class_for(Tree, test_mapper).create!
       Fissure.create!(tree_id: tree_db.id)
 
       test_mapper.destroy(Tree.new(id: tree_db.id))
@@ -602,23 +602,23 @@ describe 'AggregateMapper' do
     it 'leaves unowned belongs to children in the database' do
       test_mapper = configure_unowned
 
-      trunk_db = TrunkDB.create!
-      tree_db = TreeDB.create!(trunk_id: trunk_db.id)
+      trunk_db = db_class_for(Trunk, test_mapper).create!
+      tree_db = db_class_for(Tree, test_mapper).create!(trunk_id: trunk_db.id)
 
       test_mapper.destroy(Tree.new(id: tree_db.id))
 
-      expect(TrunkDB.count).to eq 1
+      expect(db_class_for(Trunk, test_mapper).count).to eq 1
     end
 
     it 'leaves unowned has many children in the database' do
       test_mapper = configure_unowned
 
-      tree_db = TreeDB.create!
-      BranchDB.create!(tree_id: tree_db.id)
+      tree_db = db_class_for(Tree, test_mapper).create!
+      db_class_for(Branch, test_mapper).create!(tree_id: tree_db.id)
 
       test_mapper.destroy(Tree.new(id: tree_db.id))
 
-      expect(BranchDB.count).to eq 1
+      expect(db_class_for(Branch, test_mapper).count).to eq 1
     end
   end
 
@@ -626,11 +626,11 @@ describe 'AggregateMapper' do
     it 'removes the entity from the database' do
       test_mapper = configure
 
-      tree_db = TreeDB.create!
+      tree_db = db_class_for(Tree, test_mapper).create!
 
       test_mapper.destroy_by_id([tree_db.id])
 
-      expect(TreeDB.count).to eq 0
+      expect(db_class_for(Tree, test_mapper).count).to eq 0
     end
   end
 
@@ -733,6 +733,10 @@ describe 'AggregateMapper' do
 
 private
 
+  def db_class_for(clazz, mapper)
+    mapper.engine.mapper_for(clazz).db_class
+  end
+
   def configure_polymorphic_has_many
     engine = Vorpal.define do
       map Tree do

data/spec/vorpal/performance/performance_spec.rb

@@ -3,6 +3,7 @@ require 'vorpal'
 require 'virtus'
 require 'activerecord-import/base'
 
+module Performance
 describe 'performance' do
 
   class Bug
@@ -62,19 +63,12 @@ describe 'performance' do
 
   before(:all) do
     define_table('branches_perf', {length: :decimal, tree_id: :integer, branch_id: :integer}, false)
-    BranchDB = defineAr('branches_perf')
-
     define_table('bugs_perf', {name: :text, lives_on_id: :integer, lives_on_type: :string}, false)
-    BugDB = defineAr('bugs_perf')
-
     define_table('trees_perf', {name: :text, trunk_id: :integer}, false)
-    TreeDB = defineAr('trees_perf')
-
     define_table('trunks_perf', {length: :decimal}, false)
-    TrunkDB = defineAr('trunks_perf')
   end
 
-  let(:test_mapper) { build_mapper }
+  let(:tree_mapper) { build_mapper }
 
   # Vorpal 0.0.5:
   #               user     system      total        real
@@ -92,10 +86,10 @@ describe 'performance' do
   it 'benchmarks all operations' do
     trees = build_trees(1000)
     Benchmark.bm(7) do |x|
-      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) }
+      x.report('create') { tree_mapper.persist(trees) }
+      x.report('update') { tree_mapper.persist(trees) }
+      x.report('load') { tree_mapper.query.where(id: trees.map(&:id)).load_many }
+      x.report('destroy') { tree_mapper.destroy(trees) }
     end
   end
 
@@ -104,39 +98,39 @@ describe 'performance' do
 
     puts 'starting persistence benchmark'
     puts Benchmark.measure {
-      test_mapper.persist(trees)
+      tree_mapper.persist(trees)
     }
   end
 
   it 'updates aggregates quickly' do
     trees = build_trees(1000)
 
-    test_mapper.persist(trees)
+    tree_mapper.persist(trees)
 
     puts 'starting update benchmark'
     puts Benchmark.measure {
-      test_mapper.persist(trees)
+      tree_mapper.persist(trees)
     }
   end
 
   it 'loads aggregates quickly' do
     trees = build_trees(1000)
-    test_mapper.persist(trees)
+    tree_mapper.persist(trees)
     ids = trees.map(&:id)
 
     puts 'starting loading benchmark'
     puts Benchmark.measure {
-      test_mapper.load_many(TreeDB.where(id: ids).all)
+      tree_mapper.query.where(id: ids).load_many
     }
   end
 
   it 'destroys aggregates quickly' do
     trees = build_trees(1000)
-    test_mapper.persist(trees)
+    tree_mapper.persist(trees)
 
     puts 'starting destruction benchmark'
     puts Benchmark.measure {
-      test_mapper.destroy(trees)
+      tree_mapper.destroy(trees)
     }
   end
 
@@ -164,26 +158,26 @@ describe 'performance' do
 
   def build_mapper
     engine = Vorpal.define do
-      map Tree do
+      map Tree, table_name: "trees_perf" do
         attributes :name
         belongs_to :trunk
         has_many :branches
       end
 
-      map Trunk do
+      map Trunk, table_name: "trunks_perf" do
         attributes :length
         has_one :tree
         has_many :bugs, fk: :lives_on_id, fk_type: :lives_on_type
       end
 
-      map Branch do
+      map Branch, table_name: "branches_perf" do
         attributes :length
         belongs_to :tree
         has_many :bugs, fk: :lives_on_id, fk_type: :lives_on_type
         has_many :branches
       end
 
-      map Bug do
+      map Bug, table_name: "bugs_perf" do
         attributes :name
         belongs_to :lives_on, fk: :lives_on_id, fk_type: :lives_on_type, child_classes: [Trunk, Branch]
       end
@@ -191,3 +185,4 @@ describe 'performance' do
     engine.mapper_for(Tree)
   end
 end
+end

data/spec/vorpal/unit/config_builder_spec.rb

@@ -29,31 +29,6 @@ describe Vorpal::ConfigBuilder do
     end
   end
 
-  describe 'build_db_class' do
-    it 'sets the class name' do
-      driver = instance_double(Vorpal::DbDriver)
-      new_class = Class.new
-      expect(driver).to receive(:build_db_class).and_return(new_class)
-
-      builder = Vorpal::ConfigBuilder.new(A::B::C::Test, {}, driver)
-
-      builder.build_db_class
-
-      expect(A::B::C::TestDB).to eq(new_class)
-    end
-
-    it 'does not redefine constants' do
-      stub_const('A::B::C::TestDB', 1)
-
-      builder = Vorpal::ConfigBuilder.new(A::B::C::Test, {}, nil)
-
-      db_class = builder.build_db_class
-
-      expect(A::B::C::TestDB).to eq(1)
-      expect(db_class).to eq(1)
-    end
-  end
-
   private
 
   module A

data/spec/vorpal/unit/identity_map_spec.rb

@@ -5,7 +5,7 @@ require 'vorpal/identity_map'
 describe Vorpal::IdentityMap do
   let(:map) { Vorpal::IdentityMap.new }
 
-  it 'does not rely on the key objects implementation of hash and eql?' do
+  it "does not rely on the key object's implementation of hash and eql?" do
     funny1 = build_funny_entity(1)
     map.set(funny1, 'funny 1')
 
@@ -15,12 +15,23 @@ describe Vorpal::IdentityMap do
     expect(map.get(funny1)).to eq('funny 1')
   end
 
-  it 'raises an exception when the key object does not have an id set' do
+  it "raises an exception when the key object does not have an id set" do
     entity = build_entity(nil)
 
     expect { map.set(entity, 'something') }.to raise_error(/Cannot put entity/)
   end
 
+  it 'raises an exception when the key object extends a class with no name (such as anonymous classes)' do
+    anonymous_class = Class.new do
+      attr_accessor :id
+    end
+
+    entity = anonymous_class.new
+    entity.id = 1
+
+    expect { map.set(entity, 'something') }.to raise_error(/Cannot put entity/)
+  end
+
   def build_entity(id)
     entity = Entity.new
     entity.id = id

data/vorpal.gemspec

@@ -24,7 +24,7 @@ Gem::Specification.new do |spec|
 
   spec.add_development_dependency "rake", "~> 10.0"
   spec.add_development_dependency "rspec", "~> 3.0"
-  spec.add_development_dependency "activerecord", "~> 4.0.0"
+  spec.add_development_dependency "activerecord", "~> 4.0"
   spec.add_development_dependency "pg", "~> 0.17.0"
   spec.add_development_dependency "virtus", "~> 1.0"
   spec.add_development_dependency "activerecord-import", "~> 0.10.0"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: vorpal
 version: !ruby/object:Gem::Version
-  version: 0.0.7
+  version: 0.1.0.rc1
 platform: ruby
 authors:
 - Sean Kirby
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-01-14 00:00:00.000000000 Z
+date: 2016-07-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: simple_serializer
@@ -86,14 +86,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 4.0.0
+        version: '4.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 4.0.0
+        version: '4.0'
 - !ruby/object:Gem::Dependency
   name: pg
   requirement: !ruby/object:Gem::Requirement
@@ -145,6 +145,7 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".gitignore"
+- ".rspec"
 - ".ruby-version"
 - ".yardopts"
 - CHANGELOG.md
@@ -162,12 +163,14 @@ files:
 - lib/vorpal/db_driver.rb
 - lib/vorpal/db_loader.rb
 - lib/vorpal/engine.rb
+- lib/vorpal/exceptions.rb
 - lib/vorpal/identity_map.rb
 - lib/vorpal/loaded_objects.rb
 - lib/vorpal/util/array_hash.rb
 - lib/vorpal/util/hash_initialization.rb
 - lib/vorpal/version.rb
 - spec/helpers/db_helpers.rb
+- spec/helpers/profile_helpers.rb
 - spec/integration_spec_helper.rb
 - spec/unit_spec_helper.rb
 - spec/vorpal/acceptance/aggregate_mapper_spec.rb
@@ -193,9 +196,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubyforge_project: 
 rubygems_version: 2.4.8
@@ -204,6 +207,7 @@ specification_version: 4
 summary: Separate your domain model from your persistence mechanism.
 test_files:
 - spec/helpers/db_helpers.rb
+- spec/helpers/profile_helpers.rb
 - spec/integration_spec_helper.rb
 - spec/unit_spec_helper.rb
 - spec/vorpal/acceptance/aggregate_mapper_spec.rb