vorpal 0.0.6.rc4 → 0.0.6

data/lib/vorpal/util/array_hash.rb

@@ -1,19 +1,17 @@
 module Vorpal
-
-# @private
-module ArrayHash
-  def add_to_hash(array_hash, key, values)
-    if array_hash[key].nil? || array_hash[key].empty?
-      array_hash[key] = []
+  # @private
+  module ArrayHash
+    def add_to_hash(array_hash, key, values)
+      if array_hash[key].nil? || array_hash[key].empty?
+        array_hash[key] = []
+      end
+      array_hash[key].concat(Array(values))
     end
-    array_hash[key].concat(Array(values))
-  end
 
-  def pop(array_hash)
-    key = array_hash.first.first
-    values = array_hash.delete(key)
-    [key, values]
+    def pop(array_hash)
+      key = array_hash.first.first
+      values = array_hash.delete(key)
+      [key, values]
+    end
   end
 end
-
-end

data/lib/vorpal/version.rb

@@ -1,3 +1,3 @@
 module Vorpal
-  VERSION = "0.0.6.rc4"
+  VERSION = "0.0.6"
 end

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

@@ -211,7 +211,7 @@ describe 'Aggregate Repository' do
     expect(tree.fissures.first.length).to eq 21
   end
 
-  describe "cycles" do
+  describe 'cycles' do
     it 'persists' do
      test_repository = configure_with_cycle
 
@@ -225,7 +225,7 @@ describe 'Aggregate Repository' do
     end
 
     it 'hydrates' do
-     test_repository = configure_with_cycle
+      test_repository = configure_with_cycle
 
       tree_db = TreeDB.create!
       BranchDB.create!(length: 50, tree_id: tree_db.id)
@@ -514,6 +514,20 @@ describe 'Aggregate Repository' do
     end
   end
 
+  describe 'load_all' do
+    it 'returns objects in the same order as the ids' do
+      test_repository = configure
+
+      tree_db1 = TreeDB.create!
+      tree_db2 = TreeDB.create!
+      tree_db3 = TreeDB.create!
+
+      trees = test_repository.load_all([tree_db2.id, tree_db1.id, tree_db3.id], Tree)
+
+      expect(trees.map(&:id)).to eq [tree_db2.id, tree_db1.id, tree_db3.id]
+    end
+  end
+
   describe 'destroy' do
     it 'removes the entity from the database' do
       test_repository = configure
@@ -581,6 +595,123 @@ describe 'Aggregate Repository' do
     end
   end
 
+  describe 'destroy_by_id' do
+    it 'removes the entity from the database' do
+      test_repository = configure
+
+      tree_db = TreeDB.create!
+
+      test_repository.destroy_all_by_id([tree_db.id], Tree)
+
+      expect(TreeDB.count).to eq 0
+    end
+  end
+
+  describe 'non-existent values' do
+    it 'load_all ignores ids that do not exist' do
+      test_repository = configure
+
+      results = test_repository.load_all([99], Tree)
+      expect(results).to eq []
+    end
+
+    it 'load_all throws an exception when given a nil id' do
+      test_repository = configure
+
+      expect {
+        test_repository.load_all([nil], Tree)
+      }.to raise_error(Vorpal::InvalidPrimaryKeyValue, "Nil primary key values are not allowed.")
+    end
+
+    it 'load throws an exception when given a nil id' do
+      test_repository = configure
+
+      expect {
+        test_repository.load(nil, Tree)
+      }.to raise_error(Vorpal::InvalidPrimaryKeyValue, "Nil primary key values are not allowed.")
+    end
+
+    it 'load_all ignores empty arrays' do
+      test_repository = configure
+
+      results = test_repository.load_all([], Tree)
+      expect(results).to eq []
+    end
+
+    it 'persist_all ignores empty arrays' do
+      test_repository = configure
+
+      results = test_repository.persist_all([])
+      expect(results).to eq []
+    end
+
+    it 'persist_all throws an exception when given a nil root' do
+      test_repository = configure
+      expect {
+        test_repository.persist_all([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
+      expect {
+        test_repository.persist(nil)
+      }.to raise_error(Vorpal::InvalidAggregateRoot, "Nil aggregate roots are not allowed.")
+    end
+
+    it 'destroy_all ignores empty arrays' do
+      test_repository = configure
+
+      results = test_repository.destroy_all([])
+      expect(results).to eq []
+    end
+
+    it 'destroy throws an exception when given a nil root' do
+      test_repository = configure
+
+      expect {
+        test_repository.destroy(nil)
+      }.to raise_error(Vorpal::InvalidAggregateRoot, "Nil aggregate roots are not allowed.")
+    end
+
+    it 'destroy_all throws an exception when given a nil root' do
+      test_repository = configure
+
+      expect {
+        test_repository.destroy_all([nil])
+      }.to raise_error(Vorpal::InvalidAggregateRoot, "Nil aggregate roots are not allowed.")
+    end
+
+    it 'destroy_all_by_id ignores empty arrays' do
+      test_repository = configure
+
+      results = test_repository.destroy_all_by_id([], Tree)
+      expect(results).to eq []
+    end
+
+    it 'destroy_all_by_id ignores ids that do not exist' do
+      test_repository = configure
+
+      test_repository.destroy_all_by_id([99], Tree)
+    end
+
+    it 'destroy_all_by_id throws an exception when given a nil id' do
+      test_repository = configure
+
+      expect {
+        test_repository.destroy_all_by_id([nil], Tree)
+      }.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
+
+      expect {
+        test_repository.destroy_by_id(nil, Tree)
+      }.to raise_error(Vorpal::InvalidPrimaryKeyValue, "Nil primary key values are not allowed.")
+    end
+  end
+
 private
 
   def configure_polymorphic_has_many

data/spec/vorpal/{performance_spec.rb → acceptance/performance_spec.rb}

@@ -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_all(trees) }
-      x.report("update") { test_repository.persist_all(trees) }
-      x.report("load") { ids = trees.map(&:id); test_repository.load_all(ids, Tree) }
-      x.report("destroy") { test_repository.destroy_all(trees) }
+      x.report('create') { test_repository.persist_all(trees) }
+      x.report('update') { test_repository.persist_all(trees) }
+      x.report('load') { ids = trees.map(&:id); test_repository.load_all(ids, Tree) }
+      x.report('destroy') { test_repository.destroy_all(trees) }
     end
   end
 

data/spec/vorpal/unit/loaded_objects_spec.rb

@@ -0,0 +1,22 @@
+require 'unit_spec_helper'
+
+require 'virtus'
+require 'vorpal/loaded_objects'
+require 'vorpal/configs'
+
+describe Vorpal::LoadedObjects do
+  class TestObject
+    include Virtus.model
+    attribute :id, Integer
+  end
+
+  it 'does not accept duplicate objects' do
+    object = TestObject.new(id: 22)
+    config = Vorpal::ClassConfig.new({domain_class: Object})
+
+    subject.add(config, [object, object])
+    subject.add(config, [object])
+
+    expect(subject.objects.values.flatten).to contain_exactly(object)
+  end
+end

data/vorpal.gemspec

@@ -20,6 +20,7 @@ Gem::Specification.new do |spec|
 
   spec.add_runtime_dependency "simple_serializer", "~> 1.0"
   spec.add_runtime_dependency "equalizer"
+  spec.add_runtime_dependency "activesupport"
 
   spec.add_development_dependency "rake", "~> 10.0"
   spec.add_development_dependency "rspec", "~> 3.0.0"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: vorpal
 version: !ruby/object:Gem::Version
-  version: 0.0.6.rc4
+  version: 0.0.6
 platform: ruby
 authors:
 - Sean Kirby
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-04-23 00:00:00.000000000 Z
+date: 2015-05-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: simple_serializer
@@ -38,6 +38,20 @@ dependencies:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: activesupport
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -153,11 +167,12 @@ files:
 - spec/helpers/db_helpers.rb
 - spec/integration_spec_helper.rb
 - spec/unit_spec_helper.rb
-- spec/vorpal/aggregate_repository_spec.rb
-- spec/vorpal/class_config_builder_spec.rb
-- spec/vorpal/configs_spec.rb
-- spec/vorpal/identity_map_spec.rb
-- spec/vorpal/performance_spec.rb
+- spec/vorpal/acceptance/aggregate_repository_spec.rb
+- spec/vorpal/acceptance/performance_spec.rb
+- spec/vorpal/unit/class_config_builder_spec.rb
+- spec/vorpal/unit/configs_spec.rb
+- spec/vorpal/unit/identity_map_spec.rb
+- spec/vorpal/unit/loaded_objects_spec.rb
 - vorpal.gemspec
 homepage: https://github.com/nulogy/vorpal
 licenses:
@@ -174,9 +189,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ! '>'
+  - - ! '>='
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubyforge_project: 
 rubygems_version: 2.4.5
@@ -187,8 +202,9 @@ test_files:
 - spec/helpers/db_helpers.rb
 - spec/integration_spec_helper.rb
 - spec/unit_spec_helper.rb
-- spec/vorpal/aggregate_repository_spec.rb
-- spec/vorpal/class_config_builder_spec.rb
-- spec/vorpal/configs_spec.rb
-- spec/vorpal/identity_map_spec.rb
-- spec/vorpal/performance_spec.rb
+- spec/vorpal/acceptance/aggregate_repository_spec.rb
+- spec/vorpal/acceptance/performance_spec.rb
+- spec/vorpal/unit/class_config_builder_spec.rb
+- spec/vorpal/unit/configs_spec.rb
+- spec/vorpal/unit/identity_map_spec.rb
+- spec/vorpal/unit/loaded_objects_spec.rb