dynamini 2.6.3 → 2.6.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7075482c77f8b6cec60de580a672fed23b646f51
-  data.tar.gz: 03b4480b94b8e8c312885339d06ed86fd59687ad
+  metadata.gz: aeef2ffc6acac0bce16107fb9ec281db9509bcb2
+  data.tar.gz: c0757f84ebc577b1ea555c61ded01733acb2ee9a
 SHA512:
-  metadata.gz: 49a1fff7890939d2cd39c8fb756c29cfd1c852155e90efa797f659d8488b26e720eedd7da0521734a047bb5e9f0ebf7e8c10cd090f8d2bed52337b55a1575b11
-  data.tar.gz: 1e0ece3e30f4c3d7acbc10a6f6a5ecf814f3b09efc4542776cc180c40a2c2e83a2f39fa33f9d3845f3a49155411fdd65e3282d287a47d786c2ae11111aa7c08d
+  metadata.gz: b9fbb688330e9ad1f1c0330f6f45e0e04eff3e2a9524d99d7e3602b2e232fb548e4b5056d14862099206c6cd77a628f26bcc71008f87a8d406d45f66a8652c4f
+  data.tar.gz: a610af70e44ce47b7cec2c80ee927300a9537ee85f52091342888280fce09b1c829b6f8ae3f3116fe3a1903964375bc56702991d8fb13b981063e1c22e6dce90

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    dynamini (2.6.3)
+    dynamini (2.6.4)
       activemodel (>= 3, < 5.0)
       aws-sdk (~> 2)
 

data/dynamini.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name = 'dynamini'
-  s.version = '2.6.3'
+  s.version = '2.6.4'
   s.summary = 'DynamoDB interface'
   s.description = 'Lightweight DynamoDB interface gem designed as
                    a drop-in replacement for ActiveRecord.

data/lib/dynamini/base.rb

@@ -5,6 +5,7 @@ require_relative 'dirty'
 require_relative 'increment'
 require_relative 'type_handler'
 require_relative 'adder'
+require_relative 'errors'
 
 module Dynamini
   # Core db interface class.
@@ -19,7 +20,6 @@ module Dynamini
     include Dynamini::TypeHandler
     include Dynamini::Adder
 
-
     attr_reader :attributes
     class_attribute :handles
 
@@ -114,7 +114,6 @@ module Dynamini
 
     def save!(options = {})
       run_callbacks :save do
-
         options[:validate] = true if options[:validate].nil?
 
         unless @changes.empty?

data/lib/dynamini/errors.rb

@@ -0,0 +1,4 @@
+module Dynamini
+  class RecordNotFound < StandardError
+  end
+end

data/lib/dynamini/querying.rb

@@ -5,13 +5,19 @@ module Dynamini
     def find(hash_value, range_value = nil)
       fail 'Range key cannot be blank.' if range_key && range_value.nil?
       response = client.get_item(table_name: table_name, key: create_key_hash(hash_value, range_value))
-      raise 'Item not found.' unless response.item
+
+      unless response.item
+        error_msg = "Couldn't find #{self} with '#{hash_key}'=#{hash_value}"
+        error_msg += " and '#{range_key}'=#{range_value}" if range_value
+        raise Dynamini::RecordNotFound, error_msg
+      end
+
       new(response.item.symbolize_keys, false)
     end
 
     def find_or_nil(hash_value, range_value = nil)
       find(hash_value, range_value)
-    rescue RuntimeError
+    rescue Dynamini::RecordNotFound
       nil
     end
 
@@ -25,7 +31,6 @@ module Dynamini
     def find_or_new(hash_value, range_value = nil)
       validate_query_values(hash_value, range_value)
 
-
       r = client.get_item(table_name: table_name, key: create_key_hash(hash_value, range_value))
       if r.item
         new(r.item.symbolize_keys, false)
@@ -108,4 +113,4 @@ module Dynamini
       fail 'Range key cannot be blank.' if range_key && range_value.nil?
     end
   end
-end
+end

data/spec/dynamini/base_spec.rb

@@ -256,7 +256,7 @@ describe Dynamini::Base do
       context 'when the item exists in the DB' do
         it 'should delete the item and return the item' do
           expect(model.delete).to eq(model)
-          expect { Dynamini::Base.find(model.id) }.to raise_error ('Item not found.')
+          expect { Dynamini::Base.find(model.id) }.to raise_error Dynamini::RecordNotFound
         end
       end
       context 'when the item does not exist in the DB' do

data/spec/dynamini/batch_operations_spec.rb

@@ -104,9 +104,9 @@ describe Dynamini::BatchOperations do
 
     it 'should delete all items in collection to the database' do
       subject.batch_delete(ids)
-      expect{ Dynamini::Base.find('4321') }.to raise_error(RuntimeError)
-      expect{ Dynamini::Base.find('4567') }.to raise_error(RuntimeError)
-      expect{ Dynamini::Base.find('7890') }.to raise_error(RuntimeError)
+      expect{ Dynamini::Base.find('4321') }.to raise_error(Dynamini::RecordNotFound)
+      expect{ Dynamini::Base.find('4567') }.to raise_error(Dynamini::RecordNotFound)
+      expect{ Dynamini::Base.find('7890') }.to raise_error(Dynamini::RecordNotFound)
     end
   end
 end

data/spec/dynamini/querying_spec.rb

@@ -37,7 +37,13 @@ describe Dynamini::Querying do
 
     context 'when the object does not exist' do
       it 'should raise an error' do
-        expect { Dynamini::Base.find('f') }.to raise_error 'Item not found.'
+        expect do
+          Dynamini::Base.find('1')
+        end.to raise_error Dynamini::RecordNotFound, "Couldn't find Dynamini::Base with 'id'=1"
+
+        expect do
+          TestClassWithRange.find('1', '2')
+        end.to raise_error Dynamini::RecordNotFound, "Couldn't find TestClassWithRange with 'foo'=1 and 'bar'=2"
       end
 
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: dynamini
 version: !ruby/object:Gem::Version
-  version: 2.6.3
+  version: 2.6.4
 platform: ruby
 authors:
 - Greg Ward
@@ -15,7 +15,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-09-27 00:00:00.000000000 Z
+date: 2016-09-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel
@@ -148,6 +148,7 @@ files:
 - lib/dynamini/client_interface.rb
 - lib/dynamini/configuration.rb
 - lib/dynamini/dirty.rb
+- lib/dynamini/errors.rb
 - lib/dynamini/global_id.rb
 - lib/dynamini/increment.rb
 - lib/dynamini/querying.rb
@@ -184,7 +185,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 2.4.6
 signing_key: 
 specification_version: 4
 summary: DynamoDB interface