keepr 0.3.2 → 0.4.0

data/spec/keepr/contact_export_spec.rb

@@ -1,9 +1,9 @@
 require 'spec_helper'
 
 describe Keepr::ContactExport do
-  let!(:account_1000)  { FactoryGirl.create :account, :kind => :asset,     :number => 1000,  :name => 'Kasse'      }
-  let!(:account_10000) { FactoryGirl.create :account, :kind => :creditor,  :number => 10000, :name => 'Meyer GmbH' }
-  let!(:account_70000) { FactoryGirl.create :account, :kind => :debtor,    :number => 70000, :name => 'Schulze AG' }
+  let!(:account_1000)  { FactoryBot.create :account, kind: :asset,     number: 1000,  name: 'Kasse'      }
+  let!(:account_10000) { FactoryBot.create :account, kind: :creditor,  number: 10000, name: 'Meyer GmbH' }
+  let!(:account_70000) { FactoryBot.create :account, kind: :debtor,    number: 70000, name: 'Schulze AG' }
 
   let(:scope) { Keepr::Account.all }
 

data/spec/keepr/cost_center_spec.rb

@@ -1,15 +1,15 @@
 require 'spec_helper'
 
 describe Keepr::CostCenter do
-  let(:cost_center) { FactoryGirl.create(:cost_center) }
-  let(:account)     { FactoryGirl.create(:account, :number => 8400, :kind => :revenue) }
+  let(:cost_center) { FactoryBot.create(:cost_center) }
+  let(:account)     { FactoryBot.create(:account, number: 8400, kind: :revenue) }
 
   it 'should have postings' do
-    posting = Keepr::Posting.create! :amount            => 10,
-                                     :side              => 'debit',
-                                     :keepr_account     => account,
-                                     :keepr_cost_center => cost_center,
-                                     :keepr_journal_id  => 42
+    posting = Keepr::Posting.create! amount: 10,
+                                     side: 'debit',
+                                     keepr_account: account,
+                                     keepr_cost_center: cost_center,
+                                     keepr_journal_id: 42
 
     expect(cost_center.keepr_postings).to eq([posting])
   end

data/spec/keepr/group_spec.rb

@@ -3,13 +3,13 @@ require 'spec_helper'
 describe Keepr::Group do
   describe 'validations' do
     it "should allow is_result for liability" do
-      group = Keepr::Group.new(:is_result => true, :target => :liability, :name => 'foo')
+      group = Keepr::Group.new(is_result: true, target: :liability, name: 'foo')
       expect(group).to be_valid
     end
 
     [ :asset, :profit_and_loss ].each do |target|
       it "should not allow is_result for #{target}" do
-        group = Keepr::Group.new(:is_result => true, :target => target, :name => 'foo')
+        group = Keepr::Group.new(is_result: true, target: target, name: 'foo')
         expect(group).not_to be_valid
         expect(group.errors.added? :base, :liability_needed_for_result).to eq(true)
       end
@@ -18,8 +18,8 @@ describe Keepr::Group do
 
   describe :get_from_parent do
     it 'should preset parent' do
-      root = FactoryGirl.create :group, :target => :asset
-      child = root.children.create! :name => 'Bar'
+      root = FactoryBot.create :group, target: :asset
+      child = root.children.create! name: 'Bar'
 
       expect(child.target).to eq('asset')
     end
@@ -27,8 +27,8 @@ describe Keepr::Group do
 
   describe :keepr_accounts do
     it 'should not destroy if there are accounts' do
-      group = FactoryGirl.create :group
-      account = FactoryGirl.create :account, :number => 1000, :keepr_group => group
+      group = FactoryBot.create :group
+      account = FactoryBot.create :account, number: 1000, keepr_group: group
 
       expect { group.destroy }.to_not change { Keepr::Group.count }
       expect(group.destroy).to eq(false)
@@ -36,7 +36,7 @@ describe Keepr::Group do
     end
 
     it 'should destroy if there are no accounts' do
-      group = FactoryGirl.create :group
+      group = FactoryBot.create :group
 
       expect { group.destroy }.to change { Keepr::Group.count }.by(-1)
     end
@@ -44,37 +44,37 @@ describe Keepr::Group do
 
   describe :keepr_postings do
     # Simple asset group hierarchy
-    let(:group_1)     { FactoryGirl.create :group, :target => :asset }
-    let(:group_1_1)   { FactoryGirl.create :group, :target => :asset, :parent => group_1 }
-    let(:group_1_1_1) { FactoryGirl.create :group, :target => :asset, :parent => group_1_1 }
+    let(:group_1)     { FactoryBot.create :group, target: :asset }
+    let(:group_1_1)   { FactoryBot.create :group, target: :asset, parent: group_1 }
+    let(:group_1_1_1) { FactoryBot.create :group, target: :asset, parent: group_1_1 }
 
     # Group for P&L accounts
-    let(:group_2)     { FactoryGirl.create :group, :target => :profit_and_loss }
+    let(:group_2)     { FactoryBot.create :group, target: :profit_and_loss }
 
     # Group for balance result
-    let(:group_result){ FactoryGirl.create :group, :target => :liability, :is_result => true }
+    let(:group_result){ FactoryBot.create :group, target: :liability, is_result: true }
 
     # Accounts
-    let(:account_1a)  { FactoryGirl.create :account, :number => '0001', :keepr_group => group_1_1_1 }
-    let(:account_1b)  { FactoryGirl.create :account, :number => '0011', :keepr_group => group_1_1_1 }
-    let(:account_1c)  { FactoryGirl.create :account, :number => '0111', :keepr_group => group_1_1_1 }
+    let(:account_1a)  { FactoryBot.create :account, number: '0001', keepr_group: group_1_1_1 }
+    let(:account_1b)  { FactoryBot.create :account, number: '0011', keepr_group: group_1_1_1 }
+    let(:account_1c)  { FactoryBot.create :account, number: '0111', keepr_group: group_1_1_1 }
 
-    let(:account_2)   { FactoryGirl.create :account, :number => '8400', :keepr_group => group_2, :kind => :revenue }
+    let(:account_2)   { FactoryBot.create :account, number: '8400', keepr_group: group_2, kind: :revenue }
 
     # Journals
-    let!(:journal1)   { Keepr::Journal.create! :keepr_postings_attributes => [
-                          { :keepr_account => account_1a, :amount => 100.99, :side => 'debit' },
-                          { :keepr_account => account_2,  :amount => 100.99, :side => 'credit' }
+    let!(:journal1)   { Keepr::Journal.create! keepr_postings_attributes: [
+                          { keepr_account: account_1a, amount: 100.99, side: 'debit' },
+                          { keepr_account: account_2,  amount: 100.99, side: 'credit' }
                         ]
                       }
-    let!(:journal2)   { Keepr::Journal.create! :keepr_postings_attributes => [
-                          { :keepr_account => account_1b, :amount => 100.99, :side => 'debit' },
-                          { :keepr_account => account_2, :amount => 100.99, :side => 'credit' }
+    let!(:journal2)   { Keepr::Journal.create! keepr_postings_attributes: [
+                          { keepr_account: account_1b, amount: 100.99, side: 'debit' },
+                          { keepr_account: account_2, amount: 100.99, side: 'credit' }
                         ]
                       }
-    let!(:journal3)   { Keepr::Journal.create! :keepr_postings_attributes => [
-                          { :keepr_account => account_1c, :amount => 100.99, :side => 'debit' },
-                          { :keepr_account => account_2,  :amount => 100.99, :side => 'credit' }
+    let!(:journal3)   { Keepr::Journal.create! keepr_postings_attributes: [
+                          { keepr_account: account_1c, amount: 100.99, side: 'debit' },
+                          { keepr_account: account_2,  amount: 100.99, side: 'credit' }
                         ]
                       }
 

data/spec/keepr/journal_export_spec.rb

@@ -1,19 +1,19 @@
 require 'spec_helper'
 
 describe Keepr::JournalExport do
-  let(:ust) { Keepr::Tax.create! :name => 'USt19', :value => 19.0, :keepr_account => account_1776 }
-  let(:vst) { Keepr::Tax.create! :name => 'VSt19', :value => 19.0, :keepr_account => account_1576 }
+  let(:ust) { Keepr::Tax.create! name: 'USt19', value: 19.0, keepr_account: account_1776 }
+  let(:vst) { Keepr::Tax.create! name: 'VSt19', value: 19.0, keepr_account: account_1576 }
 
-  let(:account_1000)  { FactoryGirl.create :account, :number => 1000, :kind => :asset     }
-  let(:account_1200)  { FactoryGirl.create :account, :number => 1200, :kind => :asset     }
-  let(:account_1576)  { FactoryGirl.create :account, :number => 1576, :kind => :asset     }
-  let(:account_1776)  { FactoryGirl.create :account, :number => 1776, :kind => :liability }
-  let(:account_1600)  { FactoryGirl.create :account, :number => 1600, :kind => :liability }
-  let(:account_1718)  { FactoryGirl.create :account, :number => 1718, :kind => :liability, :keepr_tax => ust }
-  let(:account_4920)  { FactoryGirl.create :account, :number => 4920, :kind => :expense,   :keepr_tax => vst }
-  let(:account_8400)  { FactoryGirl.create :account, :number => 8400, :kind => :revenue,   :keepr_tax => ust }
+  let(:account_1000)  { FactoryBot.create :account, number: 1000, kind: :asset     }
+  let(:account_1200)  { FactoryBot.create :account, number: 1200, kind: :asset     }
+  let(:account_1576)  { FactoryBot.create :account, number: 1576, kind: :asset     }
+  let(:account_1776)  { FactoryBot.create :account, number: 1776, kind: :liability }
+  let(:account_1600)  { FactoryBot.create :account, number: 1600, kind: :liability }
+  let(:account_1718)  { FactoryBot.create :account, number: 1718, kind: :liability, keepr_tax: ust }
+  let(:account_4920)  { FactoryBot.create :account, number: 4920, kind: :expense,   keepr_tax: vst }
+  let(:account_8400)  { FactoryBot.create :account, number: 8400, kind: :revenue,   keepr_tax: ust }
 
-  let(:account_10000) { FactoryGirl.create :account, :number => 10000, :kind => :debtor   }
+  let(:account_10000) { FactoryBot.create :account, number: 10000, kind: :debtor   }
 
   let(:scope) { Keepr::Journal.reorder(:number) }
 
@@ -51,12 +51,12 @@ describe Keepr::JournalExport do
 
     context "Journal without tax" do
       let!(:journal_without_tax) do
-        Keepr::Journal.create! :number                    => 'BELEG-1',
-                               :subject                   => 'Geldautomat',
-                               :date                      => Date.new(2016,06,23),
-                               :keepr_postings_attributes => [
-                                 { :keepr_account => account_1000, :amount => 105, :side => 'debit' },
-                                 { :keepr_account => account_1200, :amount => 105, :side => 'credit' }
+        Keepr::Journal.create! number: 'BELEG-1',
+                               subject: 'Geldautomat',
+                               date: Date.new(2016,06,23),
+                               keepr_postings_attributes: [
+                                 { keepr_account: account_1000, amount: 105, side: 'debit' },
+                                 { keepr_account: account_1200, amount: 105, side: 'credit' }
                                ]
       end
 
@@ -75,13 +75,13 @@ describe Keepr::JournalExport do
 
     context "Journal with tax" do
       let!(:journal_with_tax) do
-        Keepr::Journal.create! :number                    => 'BELEG-2',
-                               :subject                   => 'Telefonrechnung',
-                               :date                      => Date.new(2016,06,24),
-                               :keepr_postings_attributes => [
-                                 { :keepr_account => account_4920, :amount =>  8.40, :side => 'debit' },
-                                 { :keepr_account => account_1576, :amount =>  1.60, :side => 'debit' },
-                                 { :keepr_account => account_1600, :amount => 10.00, :side => 'credit' }
+        Keepr::Journal.create! number: 'BELEG-2',
+                               subject: 'Telefonrechnung',
+                               date: Date.new(2016,06,24),
+                               keepr_postings_attributes: [
+                                 { keepr_account: account_4920, amount:  8.40, side: 'debit' },
+                                 { keepr_account: account_1576, amount:  1.60, side: 'debit' },
+                                 { keepr_account: account_1600, amount: 10.00, side: 'credit' }
                                ]
       end
 
@@ -108,16 +108,16 @@ describe Keepr::JournalExport do
 
     context "Journal with debtor" do
       let!(:journal_with_debtor) do
-        Keepr::Journal.create! :number                    => 'BELEG-3',
-                               :subject                   => 'Warenverkauf mit Anzahlung',
-                               :date                      => Date.new(2016,06,25),
-                               :keepr_postings_attributes => [
-                                 { :keepr_account => account_10000, :amount => 4760.00, :side => 'debit'  },
-                                 { :keepr_account => account_1718,  :amount => 1000.00, :side => 'debit'  },
-                                 { :keepr_account => account_1776,  :amount =>  190.00, :side => 'debit'  },
-
-                                 { :keepr_account => account_8400,  :amount => 5000.00, :side => 'credit' },
-                                 { :keepr_account => account_1776,  :amount =>  950.00, :side => 'credit' }
+        Keepr::Journal.create! number: 'BELEG-3',
+                               subject: 'Warenverkauf mit Anzahlung',
+                               date: Date.new(2016,06,25),
+                               keepr_postings_attributes: [
+                                 { keepr_account: account_10000, amount: 4760.00, side: 'debit'  },
+                                 { keepr_account: account_1718,  amount: 1000.00, side: 'debit'  },
+                                 { keepr_account: account_1776,  amount:  190.00, side: 'debit'  },
+
+                                 { keepr_account: account_8400,  amount: 5000.00, side: 'credit' },
+                                 { keepr_account: account_1776,  amount:  950.00, side: 'credit' }
                                ]
       end
 

data/spec/keepr/journal_spec.rb

@@ -1,26 +1,26 @@
 require 'spec_helper'
 
 describe Keepr::Journal do
-  let!(:account_1000) { FactoryGirl.create(:account, :number => 1000, :kind => :asset) }
-  let!(:account_1200) { FactoryGirl.create(:account, :number => 1200, :kind => :asset) }
-  let!(:account_1210) { FactoryGirl.create(:account, :number => 1210, :kind => :asset) }
-  let!(:account_4910) { FactoryGirl.create(:account, :number => 4910, :kind => :expense) }
-  let!(:account_4920) { FactoryGirl.create(:account, :number => 4920, :kind => :expense) }
-  let!(:account_1576) { FactoryGirl.create(:account, :number => 1576, :kind => :asset) }
-  let!(:account_1600) { FactoryGirl.create(:account, :number => 1600, :kind => :liability) }
+  let!(:account_1000) { FactoryBot.create(:account, number: 1000, kind: :asset) }
+  let!(:account_1200) { FactoryBot.create(:account, number: 1200, kind: :asset) }
+  let!(:account_1210) { FactoryBot.create(:account, number: 1210, kind: :asset) }
+  let!(:account_4910) { FactoryBot.create(:account, number: 4910, kind: :expense) }
+  let!(:account_4920) { FactoryBot.create(:account, number: 4920, kind: :expense) }
+  let!(:account_1576) { FactoryBot.create(:account, number: 1576, kind: :asset) }
+  let!(:account_1600) { FactoryBot.create(:account, number: 1600, kind: :liability) }
 
   let :simple_journal do
-    Keepr::Journal.create :keepr_postings_attributes => [
-                                { :keepr_account => account_1000, :amount => 100.99, :side => 'debit' },
-                                { :keepr_account => account_1200, :amount => 100.99, :side => 'credit' }
+    Keepr::Journal.create keepr_postings_attributes: [
+                                { keepr_account: account_1000, amount: 100.99, side: 'debit' },
+                                { keepr_account: account_1200, amount: 100.99, side: 'credit' }
                               ]
   end
 
   let :complex_journal do
-    Keepr::Journal.create :keepr_postings_attributes => [
-                                { :keepr_account => account_4920, :amount =>  8.40, :side => 'debit' },
-                                { :keepr_account => account_1576, :amount =>  1.60, :side => 'debit' },
-                                { :keepr_account => account_1600, :amount => 10.00, :side => 'credit' }
+    Keepr::Journal.create keepr_postings_attributes: [
+                                { keepr_account: account_4920, amount:  8.40, side: 'debit' },
+                                { keepr_account: account_1576, amount:  1.60, side: 'debit' },
+                                { keepr_account: account_1600, amount: 10.00, side: 'credit' }
                                ]
   end
 
@@ -34,7 +34,7 @@ describe Keepr::Journal do
     context 'with date given' do
       it 'should not modify the date' do
         old_date = Date.new(2013,10,1)
-        expect(Keepr::Journal.new(:date => old_date).date).to eq(old_date)
+        expect(Keepr::Journal.new(date: old_date).date).to eq(old_date)
       end
     end
   end
@@ -47,41 +47,41 @@ describe Keepr::Journal do
 
     it 'should accept journal with postings marked for destruction' do
       complex_journal.keepr_postings.first.mark_for_destruction
-      complex_journal.keepr_postings.build :keepr_account => account_4910, :amount => 8.4, :side => 'debit'
+      complex_journal.keepr_postings.build keepr_account: account_4910, amount: 8.4, side: 'debit'
 
       expect(complex_journal).to be_valid
     end
 
     it 'should fail for journal with only one posting' do
-      journal = Keepr::Journal.create :keepr_postings_attributes => [
-                                  { :keepr_account => account_4920, :amount => 8.40, :side => 'debit' }
+      journal = Keepr::Journal.create keepr_postings_attributes: [
+                                  { keepr_account: account_4920, amount: 8.40, side: 'debit' }
                                 ]
       expect(journal).not_to be_valid
       expect(journal.errors.added? :base, :account_missing).to eq(true)
     end
 
     it 'should fail for booking the same account twice' do
-      journal = Keepr::Journal.create :keepr_postings_attributes => [
-                                  { :keepr_account => account_1000, :amount => 10, :side => 'debit' },
-                                  { :keepr_account => account_1000, :amount => 10, :side => 'credit' }
+      journal = Keepr::Journal.create keepr_postings_attributes: [
+                                  { keepr_account: account_1000, amount: 10, side: 'debit' },
+                                  { keepr_account: account_1000, amount: 10, side: 'credit' }
                                 ]
       expect(journal).not_to be_valid
       expect(journal.errors.added? :base, :account_missing).to eq(true)
     end
 
     it 'should fail for unbalanced journal' do
-      journal = Keepr::Journal.create :keepr_postings_attributes => [
-                                  { :keepr_account => account_1000, :amount => 10, :side => 'debit' },
-                                  { :keepr_account => account_1200, :amount => 10, :side => 'debit' }
+      journal = Keepr::Journal.create keepr_postings_attributes: [
+                                  { keepr_account: account_1000, amount: 10, side: 'debit' },
+                                  { keepr_account: account_1200, amount: 10, side: 'debit' }
                                 ]
       expect(journal).not_to be_valid
       expect(journal.errors.added? :base, :amount_mismatch).to eq(true)
     end
 
     it 'should fail for nil amount' do
-      journal = Keepr::Journal.create :keepr_postings_attributes => [
-                                  { :keepr_account => account_1000, :amount => 10,  :side => 'debit' },
-                                  { :keepr_account => account_1200, :amount => nil, :side => 'credit' }
+      journal = Keepr::Journal.create keepr_postings_attributes: [
+                                  { keepr_account: account_1000, amount: 10,  side: 'debit' },
+                                  { keepr_account: account_1200, amount: nil, side: 'credit' }
                                 ]
       expect(journal).not_to be_valid
       expect(journal.errors.added? :base, :amount_mismatch).to eq(true)
@@ -90,11 +90,11 @@ describe Keepr::Journal do
 
   describe :permanent do
     before :each do
-      simple_journal.update_attributes! :permanent => true
+      simple_journal.update_attributes! permanent: true
     end
 
     it "should not allow update" do
-      expect(simple_journal.update_attributes :subject => 'foo').to eq(false)
+      expect(simple_journal.update_attributes subject: 'foo').to eq(false)
       expect(simple_journal.errors.added? :base, :changes_not_allowed).to eq(true)
     end
 

data/spec/keepr/posting_spec.rb

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 describe Keepr::Posting do
-  let!(:account_1000) { FactoryGirl.create(:account, :number => 1000, :kind => :asset) }
+  let!(:account_1000) { FactoryBot.create(:account, number: 1000, kind: :asset) }
 
   describe 'side/amount' do
     it 'should handle empty object' do
@@ -11,7 +11,7 @@ describe Keepr::Posting do
     end
 
     it 'should set credit amount' do
-      posting = Keepr::Posting.new :amount => 10, :side => 'credit'
+      posting = Keepr::Posting.new amount: 10, side: 'credit'
 
       expect(posting).to be_credit
       expect(posting.amount).to eq(10)
@@ -19,7 +19,7 @@ describe Keepr::Posting do
     end
 
     it 'should set debit amount' do
-      posting = Keepr::Posting.new :amount => 10, :side => 'debit'
+      posting = Keepr::Posting.new amount: 10, side: 'debit'
 
       expect(posting).to be_debit
       expect(posting.amount).to eq(10)
@@ -39,7 +39,7 @@ describe Keepr::Posting do
     end
 
     it 'should change to credit' do
-      posting = Keepr::Posting.new :amount => 10, :side => 'debit'
+      posting = Keepr::Posting.new amount: 10, side: 'debit'
       posting.side = 'credit'
 
       expect(posting).to be_credit
@@ -47,7 +47,7 @@ describe Keepr::Posting do
     end
 
     it 'should change to debit' do
-      posting = Keepr::Posting.new :amount => 10, :side => 'credit'
+      posting = Keepr::Posting.new amount: 10, side: 'credit'
       posting.side = 'debit'
 
       expect(posting).to be_debit
@@ -55,21 +55,21 @@ describe Keepr::Posting do
     end
 
     it 'should default to debit' do
-      posting = Keepr::Posting.new :amount => 10
+      posting = Keepr::Posting.new amount: 10
 
       expect(posting).to be_debit
       expect(posting.amount).to eq(10)
     end
 
     it 'should handle string amount' do
-      posting = Keepr::Posting.new :amount => '0.5'
+      posting = Keepr::Posting.new amount: '0.5'
 
       expect(posting).to be_debit
       expect(posting.amount).to eq(0.5)
     end
 
     it 'should recognized saved debit posting' do
-      posting = Keepr::Posting.create!(:amount => 10, :side => 'debit', :keepr_account => account_1000, :keepr_journal_id => 42)
+      posting = Keepr::Posting.create!(amount: 10, side: 'debit', keepr_account: account_1000, keepr_journal_id: 42)
       posting.reload
 
       expect(posting).to be_debit
@@ -77,7 +77,7 @@ describe Keepr::Posting do
     end
 
     it 'should recognized saved credit posting' do
-      posting = Keepr::Posting.create!(:amount => 10, :side => 'credit', :keepr_account => account_1000, :keepr_journal_id => 42)
+      posting = Keepr::Posting.create!(amount: 10, side: 'credit', keepr_account: account_1000, keepr_journal_id: 42)
       posting.reload
 
       expect(posting).to be_credit
@@ -86,20 +86,20 @@ describe Keepr::Posting do
 
     it 'should fail for negative amount' do
       expect {
-        Keepr::Posting.new(:amount => -10)
+        Keepr::Posting.new(amount: -10)
       }.to raise_error(ArgumentError)
     end
 
     it 'should fail for unknown side' do
       expect {
-        Keepr::Posting.new(:side => 'foo')
+        Keepr::Posting.new(side: 'foo')
       }.to raise_error(ArgumentError)
     end
   end
 
   describe 'scopes' do
-    let!(:debit_posting) { Keepr::Posting.create!(:amount => 10, :side => 'debit', :keepr_account => account_1000, :keepr_journal_id => 42) }
-    let!(:credit_posting) { Keepr::Posting.create!(:amount => 10, :side => 'credit', :keepr_account => account_1000, :keepr_journal_id => 42) }
+    let!(:debit_posting) { Keepr::Posting.create!(amount: 10, side: 'debit', keepr_account: account_1000, keepr_journal_id: 42) }
+    let!(:credit_posting) { Keepr::Posting.create!(amount: 10, side: 'credit', keepr_account: account_1000, keepr_journal_id: 42) }
 
     it 'should filter' do
       expect(account_1000.keepr_postings.debits).to eq([debit_posting])
@@ -108,16 +108,16 @@ describe Keepr::Posting do
   end
 
   describe 'cost_center handling' do
-    let!(:cost_center) { FactoryGirl.create(:cost_center) }
-    let!(:account_8400) { FactoryGirl.create(:account, :number => 8400, :kind => :revenue) }
+    let!(:cost_center) { FactoryBot.create(:cost_center) }
+    let!(:account_8400) { FactoryBot.create(:account, number: 8400, kind: :revenue) }
 
     it "should allow cost_center" do
-      posting = Keepr::Posting.new :keepr_account => account_8400, :amount => 100, :keepr_cost_center => cost_center
+      posting = Keepr::Posting.new keepr_account: account_8400, amount: 100, keepr_cost_center: cost_center
       expect(posting).to be_valid
     end
 
     it "should not allow cost_center" do
-      posting = Keepr::Posting.new :keepr_account => account_1000, :amount => 100, :keepr_cost_center => cost_center
+      posting = Keepr::Posting.new keepr_account: account_1000, amount: 100, keepr_cost_center: cost_center
       expect(posting).to_not be_valid
     end
   end