dynamini 1.10.2 → 1.10.4

data/spec/dynamini/dirty_spec.rb

@@ -0,0 +1,144 @@
+require 'spec_helper'
+
+describe Dynamini::Dirty do
+
+  let(:model_attributes) {
+    {
+        name: 'Widget',
+        price: 9.99,
+        id: 'abcd1234',
+        hash_key: '009'
+    }
+  }
+  let(:dirty_model) { Dynamini::Base.new(model_attributes) }
+  let(:model) { Dynamini::Base.new(model_attributes, false) }
+
+
+  describe '.new' do
+
+
+    it 'should append all initial attrs to @changed, including hash_key' do
+      expect(dirty_model.changed).to eq(model_attributes.keys.map(&:to_s).delete_if { |k, v| k == 'id' })
+    end
+
+    it 'should not include the primary key in the changes' do
+      expect(dirty_model.changes[:id]).to be_nil
+    end
+  end
+
+  describe '#__was' do
+
+    context 'nonexistent attribute' do
+      it 'should raise an error' do
+        expect { Dynamini::Base.new.thing_was }.to raise_error ArgumentError
+      end
+    end
+
+    context 'after saving' do
+      it 'should clear all _was values' do
+        model = Dynamini::Base.new
+        model.new_val = 'new'
+        model.save
+        expect(model.new_val_was).to eq('new')
+      end
+    end
+
+    context 'new record' do
+
+      subject(:model) { Dynamini::Base.new(baz: 'baz') }
+      it { is_expected.to respond_to(:baz_was) }
+
+      context 'handled attribute with default' do
+        it 'should return the default value' do
+          Dynamini::Base.handle(:num, :integer, default: 2)
+          expect(model.num_was).to eq(2)
+        end
+      end
+
+      context 'handled attribute with no default' do
+        it 'should return nil' do
+          Dynamini::Base.handle(:num, :integer)
+          expect(model.num_was).to be_nil
+        end
+      end
+
+      context 'newly assigned attribute' do
+        it 'should return nil' do
+          model.new_attribute = 'hello'
+          expect(model.new_attribute_was).to be_nil
+        end
+      end
+    end
+
+    context 'previously saved record' do
+      subject(:model) { Dynamini::Base.new({baz: 'baz', nil_val: nil}, false) }
+      context 'unchanged attribute' do
+        it 'should return the current value' do
+          expect(model.baz_was).to eq('baz')
+        end
+      end
+
+      context 'newly assigned attribute or attribute changed from explicit nil' do
+        it 'should return nil' do
+          model.nil_val = 'no longer nil'
+          model.new_val = 'new'
+          expect(model.nil_val_was).to be_nil
+          expect(model.new_val_was).to be_nil
+        end
+      end
+
+      context 'attribute changed from value to value' do
+        it 'should return the old value' do
+          model.baz = 'baz2'
+          expect(model.baz_was).to eq('baz')
+        end
+      end
+    end
+  end
+
+  describe '#changes' do
+    it 'should not return the hash key or range key' do
+      Dynamini::Base.set_range_key(:range_key)
+      model.instance_variable_set(:@changes, {id: 'test_hash_key', range_key: "test_range_key"})
+      expect(model.changes).to eq({})
+      Dynamini::Base.set_range_key(nil)
+    end
+
+    context 'no change detected' do
+      it 'should return an empty hash' do
+        expect(model.changes).to eq({})
+      end
+    end
+
+    context 'attribute changed' do
+      before { model.price = 1 }
+      it 'should include the changed attribute' do
+        expect(model.changes['price']).to eq([9.99, 1])
+      end
+    end
+
+    context 'attribute created' do
+      before { model.foo = 'bar' }
+      it 'should include the created attribute' do
+        expect(model.changes['foo']).to eq([nil, 'bar'])
+      end
+    end
+
+    context 'attribute changed twice' do
+      before do
+        model.foo = 'bar'
+        model.foo = 'baz'
+      end
+      it 'should only include one copy of the changed attribute' do
+        expect(model.changes['foo']).to eq(['bar', 'baz'])
+      end
+    end
+  end
+
+  describe '#changed' do
+    it 'should stringify the keys of changes' do
+      allow(model).to receive(:changes).and_return({'price' => [1, 2], 'name' => ['a', 'b']})
+      expect(model.changed).to eq(['price', 'name'])
+    end
+  end
+end

data/spec/dynamini/increment_spec.rb

@@ -0,0 +1,88 @@
+require 'spec_helper'
+
+describe Dynamini::Increment do
+
+  let(:model_attributes) {
+    {
+        name: 'Widget',
+        price: 9.99,
+        id: 'abcd1234',
+        hash_key: '009'
+    }
+  }
+
+  subject(:model) { Dynamini::Base.create!(model_attributes) }
+
+  describe '.increment!' do
+    context 'when incrementing a nil value' do
+      it 'should save' do
+        expect(model.class.client).to receive(:update_item).with(
+                                          table_name: 'bases',
+                                          key: {id: model_attributes[:id]},
+                                          attribute_updates: hash_including(
+                                              "foo" => {
+                                                  value: 5,
+                                                  action: 'ADD'
+                                              }
+                                          )
+                                      )
+        model.increment!(foo: 5)
+      end
+      it 'should update the value' do
+        model.increment!(foo: 5)
+        expect(Dynamini::Base.find('abcd1234').foo.to_i).to eq 5
+      end
+    end
+    context 'when incrementing a numeric value' do
+      it 'should save' do
+        expect(model).to receive(:read_attribute).and_return(9.99)
+        expect(model.class.client).to receive(:update_item).with(
+                                          table_name: 'bases',
+                                          key: {id: model_attributes[:id]},
+                                          attribute_updates: hash_including(
+                                              "price" => {
+                                                  value: 5,
+                                                  action: 'ADD'
+                                              }
+                                          )
+                                      )
+        model.increment!(price: 5)
+
+      end
+      it 'should sum the values' do
+        expect(model).to receive(:read_attribute).and_return(9.99)
+        model.increment!(price: 5)
+        expect(Dynamini::Base.find('abcd1234').price).to eq 14.99
+      end
+    end
+    context 'when incrementing a non-numeric value' do
+      it 'should raise an error and not save' do
+        expect(model).to receive(:read_attribute).and_return('hello')
+        expect { model.increment!(price: 5) }.to raise_error(StandardError)
+      end
+    end
+    context 'when incrementing with a non-numeric value' do
+      it 'should raise an error and not save' do
+        expect { model.increment!(foo: 'bar') }.to raise_error(StandardError)
+      end
+    end
+    context 'when incrementing multiple values' do
+      it 'should create/sum both values' do
+        allow(model).to receive(:read_attribute).and_return(9.99)
+        model.increment!(price: 5, baz: 6)
+        found_model = Dynamini::Base.find('abcd1234')
+        expect(found_model.price).to eq 14.99
+        expect(found_model.baz).to eq 6
+      end
+    end
+    context 'when incrementing a new record' do
+      it 'should save the record and init the values and timestamps' do
+        Dynamini::Base.new(id: 1, foo: 'bar').increment!(baz: 1)
+        found_model = Dynamini::Base.find(1)
+        expect(found_model.baz).to eq 1
+        expect(found_model.created_at).to_not be_nil
+        expect(found_model.updated_at).to_not be_nil
+      end
+    end
+  end
+end

data/spec/dynamini/type_handler_spec.rb

@@ -0,0 +1,79 @@
+require 'spec_helper'
+
+describe Dynamini::TypeHandler do
+  describe '.handle' do
+
+    class HandledClass < Dynamini::Base;
+    end
+
+    context 'when reading the handled attirubte' do
+      before { HandledClass.handle :price, :integer, default: 9 }
+      it 'should return the proper format' do
+        object = HandledClass.new(price: "1")
+        expect(object.price).to eq(1)
+      end
+      it 'should return the default value if not assigned' do
+        object = HandledClass.new
+        expect(object.price).to eq(9)
+      end
+      it 'should return an array with formated item if handled' do
+        object = HandledClass.new(price: ["1", "2"])
+        expect(object.price).to eq([1, 2])
+      end
+    end
+
+    context 'when writing the handled attribute' do
+      before { HandledClass.handle :price, :float, default: 9 }
+      it 'should convert the value to handled format' do
+        object = HandledClass.new(price: "1")
+        expect(object.attributes[:price]).to eq(1.0)
+      end
+    end
+
+  end
+
+  class HandleModel < Dynamini::Base
+    handle :price, :float, default: 10
+    handle :start_date, :time
+    handle :int_list, :integer
+    handle :sym_list, :symbol
+  end
+
+  let(:handle_model) { HandleModel.new }
+
+  it 'should create getters and setters' do
+    expect(handle_model).to_not receive(:method_missing)
+    handle_model.price = 1
+    handle_model.price
+  end
+
+  it 'should retrieve price as a float' do
+    handle_model.price = '5.2'
+    expect(handle_model.price).to be_a(Float)
+  end
+
+  it 'should default price to 0 if not set' do
+    expect(handle_model.price).to eq 10
+  end
+
+  it 'should store times as floats' do
+    handle_model.start_date = Time.now
+    expect(handle_model.attributes[:start_date]).to be_a(Float)
+    expect(handle_model.attributes[:start_date] > 1_000_000_000).to be_truthy
+    expect(handle_model.start_date).to be_a(Time)
+  end
+
+  it 'should reject bad data' do
+    expect { handle_model.int_list = {a: 1} }.to raise_error NoMethodError
+  end
+
+  it 'should save casted arrays' do
+    handle_model.int_list = [12, 24, 48]
+    expect(handle_model.int_list).to eq([12, 24, 48])
+  end
+
+  it 'should retrieve casted arrays' do
+    handle_model.sym_list = ['foo', 'bar', 'baz']
+    expect(handle_model.sym_list).to eq([:foo, :bar, :baz])
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: dynamini
 version: !ruby/object:Gem::Version
-  version: 1.10.2
+  version: 1.10.4
 platform: ruby
 authors:
 - Greg Ward
@@ -15,110 +15,110 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-01-07 00:00:00.000000000 Z
+date: 2015-01-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '3'
-    - - <
+    - - "<"
       - !ruby/object:Gem::Version
         version: '5.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '3'
-    - - <
+    - - "<"
       - !ruby/object:Gem::Version
         version: '5.0'
 - !ruby/object:Gem::Dependency
   name: aws-sdk
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '3'
 - !ruby/object:Gem::Dependency
   name: pry
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: fuubar
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2'
 - !ruby/object:Gem::Dependency
   name: guard-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: guard-shell
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 description: |-
@@ -130,9 +130,9 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- .rspec
-- .travis.yml
+- ".gitignore"
+- ".rspec"
+- ".travis.yml"
 - Gemfile
 - Gemfile.lock
 - Guardfile
@@ -144,13 +144,21 @@ files:
 - lib/dynamini.rb
 - lib/dynamini/base.rb
 - lib/dynamini/batch_operations.rb
+- lib/dynamini/client_interface.rb
 - lib/dynamini/configuration.rb
+- lib/dynamini/dirty.rb
+- lib/dynamini/increment.rb
 - lib/dynamini/querying.rb
 - lib/dynamini/test_client.rb
+- lib/dynamini/type_handler.rb
 - spec/dynamini/base_spec.rb
 - spec/dynamini/batch_operations_spec.rb
+- spec/dynamini/client_interface_spec.rb
+- spec/dynamini/dirty_spec.rb
+- spec/dynamini/increment_spec.rb
 - spec/dynamini/querying_spec.rb
 - spec/dynamini/test_client_spec.rb
+- spec/dynamini/type_handler_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/47colborne/dynamini
 licenses:
@@ -162,23 +170,27 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.4.6
 signing_key: 
 specification_version: 4
 summary: DynamoDB interface
 test_files:
 - spec/dynamini/base_spec.rb
 - spec/dynamini/batch_operations_spec.rb
+- spec/dynamini/client_interface_spec.rb
+- spec/dynamini/dirty_spec.rb
+- spec/dynamini/increment_spec.rb
 - spec/dynamini/querying_spec.rb
 - spec/dynamini/test_client_spec.rb
+- spec/dynamini/type_handler_spec.rb
 - spec/spec_helper.rb