horza 0.1.2 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: aa0c514a09b3c9cc5c381187de5123e83d6bbc20
-  data.tar.gz: d6fb63bc973ed1cca3b703d8eceb5390163a5a0e
+  metadata.gz: 6313d7a6f90f6db3f51ed94b16b0de3498f2209f
+  data.tar.gz: eb2ecb68bb20c23db8032fe660a1b401b3b71fcf
 SHA512:
-  metadata.gz: d59d0f2c0ab53696247e2144f34541a3a60074731e1431cea21ea73d3da6beab250a2cd0a57f55a52f75d695e0d49ccc3d22c8f570d0c9d86787e87073203371
-  data.tar.gz: 8970bb56079f409f998e36a5d97192d5cec75f04a0ee05ac87611def55a51458b4c92b1c0901e64c25a746249f8b16dfb72777150e145e6f03db49676428349b
+  metadata.gz: c73d0a6d329c3326d8161ab9358407d6aa253f0137d2d5bb8afbd60301189b85aa7965aa5189ed95b39855243bf8fefb19ab7d0f288e5a27fb5a47c79175f322
+  data.tar.gz: 91988862e8911b424c7feeedcdba37b4bdd09dcfdcdf2c7885311df15ebb9548ba9037562f2b4f18e17355a3d97f2ecc66b9fa336df7062b703b37b36420ef58

data/lib/horza/adapters/abstract_adapter.rb

@@ -5,7 +5,7 @@ module Horza
 
       class << self
         def expected_errors
-          not_implemented_error
+          [::Horza::Errors::RecordNotFound, ::Horza::Errors::RecordInvalid]
         end
 
         def context_for_entity(entity)
@@ -26,7 +26,7 @@ module Horza
       end
 
       def get(options = {})
-        get!(options = {})
+        get!(options)
       rescue *self.class.expected_errors
       end
 
@@ -35,7 +35,7 @@ module Horza
       end
 
       def find_first(options = {})
-        find_first!(options = {})
+        find_first!(options)
       rescue *self.class.expected_errors
       end
 
@@ -47,6 +47,33 @@ module Horza
         not_implemented_error
       end
 
+      def create(options = {})
+        create!(options)
+      rescue *self.class.expected_errors
+      end
+
+      def create!(options = {})
+        not_implemented_error
+      end
+
+      def delete(id)
+        delete!(id)
+      rescue *self.class.expected_errors
+      end
+
+      def delete!(id)
+        not_implemented_error
+      end
+
+      def update(id, options = {})
+        update!(id, options)
+      rescue *self.class.expected_errors
+      end
+
+      def update!(id, options = {})
+        not_implemented_error
+      end
+
       def ancestors(options = {})
         not_implemented_error
       end

data/lib/horza/adapters/active_record.rb

@@ -4,10 +4,6 @@ module Horza
       INVALID_ANCESTRY_MSG = 'Invalid relation. Ensure that the plurality of your associations is correct.'
 
       class << self
-        def expected_errors
-          [::ActiveRecord::RecordNotFound]
-        end
-
         def context_for_entity(entity)
           entity_context_map[entity]
         end
@@ -19,16 +15,47 @@ module Horza
 
       def get!(id)
         entity_class(@context.find(id).attributes)
+      rescue ::ActiveRecord::RecordNotFound => e
+        raise Horza::Errors::RecordNotFound.new(e)
       end
 
       def find_first!(options = {})
         entity_class(base_query(options).first!.attributes)
+      rescue ::ActiveRecord::RecordNotFound => e
+        raise Horza::Errors::RecordNotFound.new(e)
       end
 
       def find_all(options = {})
         entity_class(base_query(options))
       end
 
+      def create!(options = {})
+        record = @context.new(options)
+        record.save!
+        entity_class(record.attributes)
+      rescue ::ActiveRecord::RecordInvalid => e
+        raise Horza::Errors::RecordInvalid.new(e)
+      end
+
+      def update!(id, options = {})
+        record = @context.find(id)
+        record.assign_attributes(options)
+        record.save!
+        record
+      rescue ::ActiveRecord::RecordNotFound => e
+        raise Horza::Errors::RecordNotFound.new(e)
+      rescue ::ActiveRecord::RecordInvalid => e
+        raise Horza::Errors::RecordInvalid.new(e)
+      end
+
+      def delete!(id)
+        record = @context.find(id)
+        record.destroy!
+        true
+      rescue ::ActiveRecord::RecordNotFound => e
+        raise Horza::Errors::RecordNotFound.new(e)
+      end
+
       def ancestors(options = {})
         result = walk_family_tree(@context.find(options[:id]), options)
 

data/lib/horza/errors.rb

@@ -14,5 +14,11 @@ module Horza
 
     class CannotGetHashFromCollection < StandardError
     end
+
+    class RecordInvalid < StandardError
+    end
+
+    class RecordNotFound < StandardError
+    end
   end
 end

data/spec/active_record_spec.rb

@@ -9,6 +9,7 @@ else
     ActiveRecord::Schema.define(:version => 0) do
       create_table(:employers, force: true) {|t| t.string :name }
       create_table(:users, force: true) {|t| t.string :first_name; t.string :last_name; t.references :employer; }
+      create_table(:customers, force: true) {|t| t.string :first_name; t.string :last_name; }
       create_table(:sports_cars, force: true) {|t| t.string :make; t.references :employer; }
       create_table(:dummy_models, force: true) {|t| t.string :key }
       create_table(:other_dummy_models, force: true) {|t| t.string :key }
@@ -25,6 +26,10 @@ else
       belongs_to :employer
     end
 
+    class Customer < ActiveRecord::Base
+      validates :last_name, presence: true
+    end
+
     class SportsCar < ActiveRecord::Base
       belongs_to :employer
     end
@@ -43,6 +48,7 @@ describe Horza do
   let(:last_name) { 'Turner' }
   let(:adapter) { :active_record }
   let(:user_adapter) { Horza.adapter.new(HorzaSpec::User) }
+  let(:customer_adapter) { Horza.adapter.new(HorzaSpec::Customer) }
   let(:employer_adapter) { Horza.adapter.new(HorzaSpec::Employer) }
   let(:sports_car_adapter) { Horza.adapter.new(HorzaSpec::SportsCar) }
 
@@ -71,7 +77,7 @@ describe Horza do
 
       context 'when user does not exist' do
         it 'throws error' do
-          expect { user_adapter.get!(999) }.to raise_error ActiveRecord::RecordNotFound
+          expect { user_adapter.get!(999) }.to raise_error Horza::Errors::RecordNotFound
         end
       end
     end
@@ -101,7 +107,7 @@ describe Horza do
 
       context 'when user does not exist' do
         it 'throws error' do
-          expect { user_adapter.find_first!(last_name: last_name) }.to raise_error ActiveRecord::RecordNotFound
+          expect { user_adapter.find_first!(last_name: last_name) }.to raise_error Horza::Errors::RecordNotFound
         end
       end
     end
@@ -132,6 +138,108 @@ describe Horza do
       end
     end
 
+    describe '#create' do
+      context 'when parameters are valid' do
+        it 'creates the record' do
+          expect { customer_adapter.create(last_name: last_name) }.to change(HorzaSpec::Customer, :count).by(1)
+        end
+
+        it 'returns the entity' do
+          expect(customer_adapter.create(last_name: last_name).last_name).to eq last_name
+        end
+      end
+
+      context 'when parameters are invalid' do
+        it 'does not create the record' do
+          expect { customer_adapter.create }.to change(HorzaSpec::Customer, :count).by(0)
+        end
+
+        it 'returns nil' do
+          expect(customer_adapter.create).to be nil
+        end
+      end
+    end
+
+    describe '#create!' do
+      context 'when parameters are invalid' do
+        it 'throws error' do
+          expect { customer_adapter.create! }.to raise_error Horza::Errors::RecordInvalid
+        end
+      end
+    end
+
+    describe '#update' do
+      let(:customer) { HorzaSpec::Customer.create(last_name: last_name) }
+
+      context 'when parameters are valid' do
+        let(:updated_last_name) { 'Smith' }
+
+        it 'returns the updated entity' do
+          expect(customer_adapter.update(customer.id, last_name: updated_last_name).last_name).to eq updated_last_name
+        end
+      end
+
+      context 'when parameters are invalid' do
+        it 'returns nil' do
+          expect(customer_adapter.update(customer.id, last_name: nil)).to be nil
+        end
+      end
+
+      context 'when record does not exist' do
+        it 'returns nil' do
+          expect(customer_adapter.update(999)).to be nil
+        end
+      end
+    end
+
+    describe '#update!' do
+      let(:customer) { HorzaSpec::Customer.create(last_name: last_name) }
+
+      context 'when parameters are invalid' do
+        it 'throws error' do
+          expect { customer_adapter.update!(customer.id, last_name: nil) }.to raise_error Horza::Errors::RecordInvalid
+        end
+      end
+
+      context 'when record does not exist' do
+        it 'returns nil' do
+          expect { customer_adapter.update!(999) }.to raise_error Horza::Errors::RecordNotFound
+        end
+      end
+    end
+
+    describe '#delete' do
+      let!(:customer) { HorzaSpec::Customer.create(last_name: last_name) }
+
+      context 'when record exists' do
+        let(:updated_last_name) { 'Smith' }
+
+        it 'destroys the record' do
+          expect { customer_adapter.delete(customer.id) }.to change(HorzaSpec::Customer, :count).by(-1)
+        end
+
+        it 'returns true' do
+          expect(customer_adapter.delete(customer.id)).to be true
+        end
+      end
+
+      context 'when record does not exist' do
+        it 'returns nil' do
+          expect(customer_adapter.delete(999)).to be nil
+        end
+      end
+    end
+
+    describe '#delete!' do
+      let(:customer) { HorzaSpec::Customer.create(last_name: last_name) }
+
+      context 'when record does not exist' do
+        it 'returns nil' do
+          expect { customer_adapter.delete!(999) }.to raise_error Horza::Errors::RecordNotFound
+        end
+      end
+    end
+
     context '#ancestors' do
       context 'direct relation' do
         let(:employer) { HorzaSpec::Employer.create }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: horza
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.2.0
 platform: ruby
 authors:
 - Blake Turner
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-05-27 00:00:00.000000000 Z
+date: 2015-05-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hashie