double_double 0.2.5 → 0.2.6

data/.travis.yml

@@ -2,6 +2,5 @@ language: ruby
 rvm:
   - 1.9.2
   - 1.9.3
-  - rbx-19mode
   - ruby-head
-script: rake
+script: rake

data/lib/double_double/account.rb

@@ -79,8 +79,9 @@ module DoubleDouble
       def side_balance(is_debit, hash)
         a = is_debit ? DoubleDouble::DebitAmount.scoped : DoubleDouble::CreditAmount.scoped
         a = a.where(account_id: self.id)
-        a = a.by_context(hash[:context])     if hash.has_key? :context
-        a = a.by_accountee(hash[:accountee]) if hash.has_key? :accountee
+        a = a.by_context(hash[:context])                   if hash.has_key? :context
+        a = a.by_accountee(hash[:accountee])               if hash.has_key? :accountee
+        a = a.by_transaction_type(hash[:transaction_type]) if hash.has_key? :transaction_type
         Money.new(a.sum(:amount_cents))
       end
       # The balance method that derived Accounts utilize.

data/lib/double_double/amount.rb

@@ -14,8 +14,9 @@ module DoubleDouble
     belongs_to :accountee,  polymorphic: true
     belongs_to :context,    polymorphic: true
     
-    scope :by_accountee, ->(a) { where(accountee_id: a.id, accountee_type: a.class.base_class) }
-    scope :by_context,   ->(c) { where(context_id:   c.id, context_type:   c.class.base_class) }
+    scope :by_accountee,        ->(a) { where(accountee_id: a.id, accountee_type: a.class.base_class) }
+    scope :by_context,          ->(c) { where(context_id:   c.id, context_type:   c.class.base_class) }
+    scope :by_transaction_type, ->(t) { joins(:transaction).where(double_double_transactions: {transaction_type_id: t}) }
 
     validates_presence_of :type, :transaction, :account
     validates :amount_cents, numericality: {greater_than: 0}

data/lib/double_double/version.rb

@@ -1,3 +1,3 @@
 module DoubleDouble
-  VERSION = "0.2.5"
+  VERSION = "0.2.6"
 end

data/spec/models/transaction_spec.rb

@@ -114,6 +114,22 @@ module DoubleDouble
       t.errors['base'].should == ['The credit and debit amounts are not equal']
     end
 
+    describe 'transaction reversing' do
+      it "should negate the same non-reversed transaction" do
+        args_normal = {description: 'reverse test',
+          debits:  [{account: 'Cash_11', amount: 10}],
+          credits: [{account: 'Loan_12', amount:  9},
+                    {account: 'Loan_12', amount:  1}]}
+        args_reversed = args_normal.merge({reversed: true})
+        Transaction.create!(args_normal)
+        Account.named('Cash_11').balance.should eq(10)
+        Account.named('Loan_12').balance.should eq(10)
+        Transaction.create!(args_reversed)
+        Account.named('Cash_11').balance.should eq(0)
+        Account.named('Loan_12').balance.should eq(0)
+      end
+    end
+
     describe 'transaction_types' do
       it 'should create a Transaction with a TransactionType of Unassigned if none is passed in' do
         t = Transaction.build(
@@ -136,6 +152,29 @@ module DoubleDouble
       end
     end
 
+    describe 'transaction_types, when multiple types exist together' do
+      it 'should segment based on transaction type' do
+        DoubleDouble::Liability.create!(name:'hotdogs', number: 1015)
+        DoubleDouble::Liability.create!(name:'junk',    number: 1016)
+        TransactionType.create!(description: 'ketchup')
+        TransactionType.create!(description: 'onions')
+        Transaction.create!(
+          description: 'processed ketchup',
+          transaction_type: TransactionType.of(:ketchup),
+          debits:  [{account: 'junk',    amount: 60, context: @campaign1, accountee: @user1}], 
+          credits: [{account: 'hotdogs', amount: 60, context: @campaign1, accountee: @user1}])
+        DoubleDouble::Account.named('hotdogs').credits_balance({context: @campaign1, accountee: @user1, transaction_type: TransactionType.of(:ketchup)}).should == 60
+        DoubleDouble::Account.named('hotdogs').credits_balance({context: @campaign1, accountee: @user1, transaction_type: TransactionType.of(:onions)}).should == 0
+        Transaction.create!(
+          description: 'processed onions',
+          transaction_type: TransactionType.of(:onions),
+          debits:  [{account: 'junk',    amount: 5, context: @campaign1, accountee: @user1}], 
+          credits: [{account: 'hotdogs', amount: 5, context: @campaign1, accountee: @user1}])
+        DoubleDouble::Account.named('hotdogs').credits_balance({context: @campaign1, accountee: @user1, transaction_type: TransactionType.of(:ketchup)}).should == 60
+        DoubleDouble::Account.named('hotdogs').credits_balance({context: @campaign1, accountee: @user1, transaction_type: TransactionType.of(:onions)}).should == 5
+      end
+    end
+
     describe 'amount accountee references' do
       it 'should allow a Transaction to be built describing the accountee in the hash' do
         Transaction.create!(

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: double_double
 version: !ruby/object:Gem::Version
-  version: 0.2.5
+  version: 0.2.6
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-02-06 00:00:00.000000000 Z
+date: 2013-02-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: money