plutus 0.5.2 → 0.7.0

data/spec/controllers/transactions_controller_spec.rb

@@ -1,25 +1,28 @@
 require 'spec_helper'
 
-describe TransactionsController do
+module Plutus
+  describe TransactionsController do
+    # Run these tests if you enable routing in your rails app. See README
 
-  def mock_transaction(stubs={})
-    @mock_transaction ||= mock_model(Transaction, stubs)
-  end
-  
-  describe "GET index" do
-    it "assigns all transactions as @transactions" do
-      Transaction.stub(:find).with(:all).and_return([mock_transaction])
-      get :index
-      assigns[:transactions].should == [mock_transaction]
-    end
-  end
-  
-  describe "GET show" do
-    it "assigns the requested transaction as @transaction" do
-      Transaction.stub(:find).with("37").and_return(mock_transaction)
-      get :show, :id => "37"
-      assigns[:transaction].should equal(mock_transaction)
-    end
-  end  
+    #def mock_transaction(stubs={})
+      #@mock_transaction ||= mock_model(Transaction, stubs)
+    #end
+
+    #describe "GET index" do
+      #it "assigns all transactions as @transactions" do
+        #Transaction.stub(:find).with(:all).and_return([mock_transaction])
+        #get :index
+        #assigns[:transactions].should == [mock_transaction]
+      #end
+    #end
 
+    #describe "GET show" do
+      #it "assigns the requested transaction as @transaction" do
+        #Transaction.stub(:find).with("37").and_return(mock_transaction)
+        #get :show, :id => "37"
+        #assigns[:transaction].should equal(mock_transaction)
+      #end
+    #end
+
+  end
 end

data/spec/factories/account_factory.rb

@@ -1,29 +1,35 @@
-Factory.define :account do |account|
-  account.name 'factory name'
-  account.contra false
-end
+FactoryGirl.define do
+  factory :account, :class => Plutus::Account do |account|
+    account.name
+    account.contra false
+  end
 
-Factory.define :asset do |account|
-  account.name 'factory name'
-  account.contra false
-end
+  factory :asset, :class => Plutus::Asset do |account|
+    account.name
+    account.contra false
+  end
 
-Factory.define :equity do |account|
-  account.name 'factory name'
-  account.contra false
-end
+  factory :equity, :class => Plutus::Equity do |account|
+    account.name
+    account.contra false
+  end
 
-Factory.define :expense do |account|
-  account.name 'factory name'
-  account.contra false
-end
+  factory :expense, :class => Plutus::Expense do |account|
+    account.name
+    account.contra false
+  end
 
-Factory.define :liability do |account|
-  account.name 'factory name'
-  account.contra false
-end
+  factory :liability, :class => Plutus::Liability do |account|
+    account.name
+    account.contra false
+  end
+
+  factory :revenue, :class => Plutus::Revenue do |account|
+    account.name
+    account.contra false
+  end
 
-Factory.define :revenue do |account|
-  account.name 'factory name'
-  account.contra false
-end
+  sequence :name do |n|
+    "Factory Name #{n}"
+  end
+end

data/spec/factories/amount_factory.rb

@@ -0,0 +1,19 @@
+FactoryGirl.define do
+  factory :amount, :class => Plutus::Amount do |amount|
+    amount.amount BigDecimal.new('473')
+    amount.association :transaction, :factory => :transaction_with_credit_and_debit
+    amount.association :account, :factory => :asset
+  end
+
+  factory :credit_amount, :class => Plutus::CreditAmount do |credit_amount|
+    credit_amount.amount BigDecimal.new('473')
+    credit_amount.association :transaction, :factory => :transaction_with_credit_and_debit
+    credit_amount.association :account, :factory => :revenue
+  end
+
+  factory :debit_amount, :class => Plutus::DebitAmount do |debit_amount|
+    debit_amount.amount BigDecimal.new('473')
+    debit_amount.association :transaction, :factory => :transaction_with_credit_and_debit
+    debit_amount.association :account, :factory => :asset
+  end
+end

data/spec/factories/transaction_factory.rb

@@ -1,6 +1,11 @@
-Factory.define :transaction do |transaction|
-  transaction.description 'factory description'
-  transaction.credit_account {|credit_account| credit_account.association(:asset)}
-  transaction.debit_account {|debit_account| debit_account.association(:revenue)}
-  transaction.amount BigDecimal.new('100')
-end
+FactoryGirl.define do
+  factory :transaction, :class => Plutus::Transaction do |transaction|
+    transaction.description 'factory description'
+    factory :transaction_with_credit_and_debit, :class => Plutus::Transaction do |transaction_cd|
+      transaction_cd.after_build do |t|
+        t.credit_amounts << FactoryGirl.build(:credit_amount, :transaction => t)
+        t.debit_amounts << FactoryGirl.build(:debit_amount, :transaction => t)
+      end
+    end
+  end
+end

data/spec/models/account_spec.rb

@@ -1,43 +1,64 @@
 require 'spec_helper'
 
-describe Account do
-  
-  it "should not allow creating an account without a subtype" do
-    account = Factory.build(:account)
-    account.should_not be_valid
-  end
-  
-  it "should not have a balance method" do
-    lambda{Account.balance}.should raise_error(NoMethodError)
-  end
-  
-  it "should have a trial balance" do
-    Account.should respond_to(:trial_balance)
-    Account.trial_balance.should be_kind_of(BigDecimal)
-  end
-  
-  it "should report a trial balance of 0 with correct transactions" do
-    # credit accounts    
-    liability = Factory(:liability)
-    equity = Factory(:equity)
-    revenue = Factory(:revenue)
-    contra_asset = Factory(:asset, :contra => true)
-    contra_expense = Factory(:expense, :contra => true)
-
-    # debit accounts
-    asset = Factory(:asset)
-    expense = Factory(:expense)
-    contra_liability = Factory(:liability, :contra => true)
-    contra_equity = Factory(:equity, :contra => true)
-    contra_revenue = Factory(:revenue, :contra => true)
-    
-    Factory(:transaction, :credit_account =>  liability, :debit_account => asset, :amount => 100000)
-    Factory(:transaction, :credit_account =>  equity, :debit_account => expense, :amount => 1000)
-    Factory(:transaction, :credit_account =>  revenue, :debit_account => contra_liability, :amount => 40404)
-    Factory(:transaction, :credit_account =>  contra_asset, :debit_account => contra_equity, :amount => 2)
-    Factory(:transaction, :credit_account =>  contra_expense, :debit_account => contra_revenue, :amount => 333)
-
-    Account.trial_balance.should == 0
-  end
+module Plutus
+  describe Account do
+
+    it "should not allow creating an account without a subtype" do
+      account = FactoryGirl.build(:account)
+      account.should_not be_valid
+    end
+
+    it "should be unique per name" do
+      FactoryGirl.create(:account, :name => "Test1", :type => "Plutus::Asset")
+      account = FactoryGirl.build(:account, :name => "Test1", :type => "Plutus::Asset")
+      account.should_not be_valid
+      account.errors[:name].should == ["has already been taken"]
+    end
+
+    it "should not have a balance method" do
+      lambda{Account.balance}.should raise_error(NoMethodError)
+    end
 
-end
+    it "should have a trial balance" do
+      Account.should respond_to(:trial_balance)
+      Account.trial_balance.should be_kind_of(BigDecimal)
+    end
+
+    it "should report a trial balance of 0 with correct transactions" do
+      # credit accounts
+      liability = FactoryGirl.create(:liability)
+      equity = FactoryGirl.create(:equity)
+      revenue = FactoryGirl.create(:revenue)
+      contra_asset = FactoryGirl.create(:asset, :contra => true)
+      contra_expense = FactoryGirl.create(:expense, :contra => true)
+      # credit amounts
+      ca1 = FactoryGirl.build(:credit_amount, :account => liability, :amount => 100000)
+      ca2 = FactoryGirl.build(:credit_amount, :account => equity, :amount => 1000)
+      ca3 = FactoryGirl.build(:credit_amount, :account => revenue, :amount => 40404)
+      ca4 = FactoryGirl.build(:credit_amount, :account => contra_asset, :amount => 2)
+      ca5 = FactoryGirl.build(:credit_amount, :account => contra_expense, :amount => 333)
+
+      # debit accounts
+      asset = FactoryGirl.create(:asset)
+      expense = FactoryGirl.create(:expense)
+      contra_liability = FactoryGirl.create(:liability, :contra => true)
+      contra_equity = FactoryGirl.create(:equity, :contra => true)
+      contra_revenue = FactoryGirl.create(:revenue, :contra => true)
+      # debit amounts
+      da1 = FactoryGirl.build(:debit_amount, :account => asset, :amount => 100000)
+      da2 = FactoryGirl.build(:debit_amount, :account => expense, :amount => 1000)
+      da3 = FactoryGirl.build(:debit_amount, :account => contra_liability, :amount => 40404)
+      da4 = FactoryGirl.build(:debit_amount, :account => contra_equity, :amount => 2)
+      da5 = FactoryGirl.build(:debit_amount, :account => contra_revenue, :amount => 333)
+
+      FactoryGirl.create(:transaction, :credit_amounts => [ca1], :debit_amounts => [da1])
+      FactoryGirl.create(:transaction, :credit_amounts => [ca2], :debit_amounts => [da2]) 
+      FactoryGirl.create(:transaction, :credit_amounts => [ca3], :debit_amounts => [da3])
+      FactoryGirl.create(:transaction, :credit_amounts => [ca4], :debit_amounts => [da4])
+      FactoryGirl.create(:transaction, :credit_amounts => [ca5], :debit_amounts => [da5])
+
+      Account.trial_balance.should == 0
+    end
+
+  end
+end

data/spec/models/amount_spec.rb

@@ -0,0 +1,12 @@
+require 'spec_helper'
+
+module Plutus
+  describe Amount do
+
+    it "should not allow creating an amount without a subtype" do
+      amount = FactoryGirl.build(:amount)
+      amount.should_not be_valid
+    end
+
+  end
+end

data/spec/models/asset_spec.rb

@@ -1,46 +1,53 @@
 require 'spec_helper'
 
-describe Asset do
-  
-  it "should allow creating an asset account" do
-    asset = Factory(:asset)
-  end
-  
-  it "should report a balance for the asset account" do
-    asset = Factory(:asset)
-    asset.balance.should be_kind_of(BigDecimal)
-  end
-  
-  it "should report a balance for the class of accounts" do
-    Asset.should respond_to(:balance)
-    Asset.balance.should be_kind_of(BigDecimal)
-  end
+module Plutus
+  describe Asset do
 
-  it "should not report a trial balance" do
-    lambda{Asset.trial_balance}.should raise_error(NoMethodError)
-  end
+    it "should allow creating an asset account" do
+      asset = FactoryGirl.create(:asset)
+    end
 
-  it "should not be valid without a name" do
-    asset = Factory.build(:asset, :name => nil)
-    asset.should_not be_valid
-  end
-  
-  it "should have many credit transactions" do
-    asset = Factory(:asset)
-    asset.should respond_to(:credit_transactions)
-  end
+    it "should report a balance for the asset account" do
+      asset = FactoryGirl.create(:asset)
+      FactoryGirl.create(:debit_amount, :account => asset)
+      asset.balance.should > 0
+      asset.balance.should be_kind_of(BigDecimal)
+    end
 
-  it "should have many debit transactions" do
-    asset = Factory(:asset)
-    asset.should respond_to(:debit_transactions)
-  end
+    it "should report a balance for the class of accounts" do
+      asset = FactoryGirl.create(:asset)
+      FactoryGirl.create(:debit_amount, :account => asset)
+      Asset.should respond_to(:balance)
+      Asset.balance.should > 0
+      Asset.balance.should be_kind_of(BigDecimal)
+    end
+
+    it "should not report a trial balance" do
+      lambda{Asset.trial_balance}.should raise_error(NoMethodError)
+    end
+
+    it "should not be valid without a name" do
+      asset = FactoryGirl.build(:asset, :name => nil)
+      asset.should_not be_valid
+    end
+
+    it "should have many credit transactions" do
+      asset = Factory(:asset)
+      asset.should respond_to(:credit_transactions)
+    end
+
+    it "should have many debit transactions" do
+      asset = Factory(:asset)
+      asset.should respond_to(:debit_transactions)
+    end
+
+    it "a contra account should reverse the normal balance" do
+      contra_asset = Factory(:asset, :contra => true)
+      # the odd amount below is because factories create an asset debit_amount
+      FactoryGirl.create(:credit_amount, :account => contra_asset, :amount => 473) 
+      contra_asset.balance.should > 0
+      Asset.balance.should == 0 
+    end
 
-  it "a contra account should reverse the normal balance" do
-    asset = Factory(:asset)
-    contra_asset = Factory(:asset, :contra => true)
-    transaction = Factory(:transaction, :credit_account => contra_asset, :debit_account => asset, :amount => 1000)
-    contra_asset.balance.should > 0
-    Asset.balance.should == 0
   end
-    
-end
+end

data/spec/models/credit_amount_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+
+module Plutus
+  describe CreditAmount do
+
+    it "should be invalid without an amount" do
+      credit_amount = FactoryGirl.build(:credit_amount, :amount => nil)
+      credit_amount.should_not be_valid
+    end
+
+    it "should not be valid without a transaction" do
+      credit_amount = FactoryGirl.build(:credit_amount, :transaction => nil)
+      credit_amount.should_not be_valid
+    end
+
+    it "should not be valid without an account" do
+      credit_amount = FactoryGirl.build(:credit_amount, :account => nil)
+      credit_amount.should_not be_valid
+    end
+
+  end
+end

data/spec/models/debit_amount_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+
+module Plutus
+  describe DebitAmount do
+
+    it "should be invalid without an amount" do
+      debit_amount = FactoryGirl.build(:debit_amount, :amount => nil)
+      debit_amount.should_not be_valid
+    end
+
+    it "should not be valid without a transaction" do
+      debit_amount = FactoryGirl.build(:debit_amount, :transaction => nil)
+      debit_amount.should_not be_valid
+    end
+
+    it "should not be valid without an account" do
+      debit_amount = FactoryGirl.build(:debit_amount, :account => nil)
+      debit_amount.should_not be_valid
+    end
+
+  end
+end

data/spec/models/equity_spec.rb

@@ -1,46 +1,54 @@
 require 'spec_helper'
 
-describe Equity do
-  
-  it "should allow creating a equity account" do
-    equity = Factory(:equity)
-  end
-  
-  it "should report a balance for the equity account" do
-    equity = Factory(:equity)
-    equity.balance.should be_kind_of(BigDecimal)
-  end
-  
-  it "should report a balance for the class of accounts" do
-    Equity.should respond_to(:balance)
-    Equity.balance.should be_kind_of(BigDecimal)
-  end
-  
-  it "should not report a trial balance" do
-    lambda{Equity.trial_balance}.should raise_error(NoMethodError)
-  end
-  
-  it "should not be valid without a name" do
-    equity = Factory.build(:equity, :name => nil)
-    equity.should_not be_valid
-  end  
-  
-  it "should have many credit transactions" do
-    equity = Factory(:equity)
-    equity.should respond_to(:credit_transactions)
-  end
+module Plutus
+  describe Equity do
+
+    it "should allow creating an equity account" do
+      equity = FactoryGirl.create(:equity)
+    end
+
+    it "should report a balance for the equity account" do
+      equity = FactoryGirl.create(:equity)
+      FactoryGirl.create(:credit_amount, :account => equity)
+      equity.balance.should > 0
+      equity.balance.should be_kind_of(BigDecimal)
+    end
 
-  it "should have many debit transactions" do
-    equity = Factory(:equity)
-    equity.should respond_to(:debit_transactions)
-  end  
-  
-  it "a contra account should reverse the normal balance" do
-    equity = Factory(:equity)
-    contra_equity = Factory(:equity, :contra => true)
-    transaction = Factory(:transaction, :credit_account => equity, :debit_account => contra_equity, :amount => 1000)
-    contra_equity.balance.should > 0
-    Equity.balance.should == 0
-  end  
-
-end
+    it "should report a balance for the class of accounts" do
+      equity = FactoryGirl.create(:equity)
+      FactoryGirl.create(:credit_amount, :account => equity)
+      Equity.should respond_to(:balance)
+      Equity.balance.should > 0
+      Equity.balance.should be_kind_of(BigDecimal)
+    end
+
+    it "should not report a trial balance" do
+      lambda{Equity.trial_balance}.should raise_error(NoMethodError)
+    end
+
+    it "should not be valid without a name" do
+      equity = FactoryGirl.build(:equity, :name => nil)
+      equity.should_not be_valid
+    end
+
+    it "should have many credit transactions" do
+      equity = Factory(:equity)
+      equity.should respond_to(:credit_transactions)
+    end
+
+    it "should have many debit transactions" do
+      equity = Factory(:equity)
+      equity.should respond_to(:debit_transactions)
+    end
+
+    it "a contra account should reverse the normal balance" do
+      equity = FactoryGirl.create(:equity)
+      contra_equity = Factory(:equity, :contra => true)
+      FactoryGirl.create(:credit_amount, :account => equity)
+      FactoryGirl.create(:debit_amount, :account => contra_equity)
+      contra_equity.balance.should > 0
+      Equity.balance.should == 0
+    end
+
+  end
+end