ledger-rest 2.0.3 → 3.0.0

data/spec/ledger-rest/api/transactions_spec.rb

@@ -0,0 +1,201 @@
+# -*- coding: utf-8 -*-
+require 'spec_helper'
+
+describe '/transactions' do
+  describe 'GET' do
+    let(:valid_json) do
+      [
+       {
+         'date' => '2013/01/01',
+         'payee' => 'Opening Balance',
+         'cleared' => true,
+         'posts' =>
+         [
+          {
+            'account' => 'Assets:Giro',
+            'amount' => 2000.0,
+            'commodity' => 'EUR'
+          }, {
+            'account' => 'Assets:Reimbursements:Hans Maulwurf',
+            'amount' => 19.0,
+            'commodity' => 'EUR'
+          }, {
+            'account' => 'Equity:Opening Balances',
+            'amount' => -2019.0,
+            'commodity' => 'EUR'
+          }
+         ],
+         'pending' => false
+       }, {
+         'date' => '2013/12/03',
+         'payee' => 'NaveenaPath',
+         'cleared' => false,
+         'posts' =>
+         [
+          {
+            'account' => 'Expenses:Restaurants',
+            'amount' => 9.0,
+            'commodity' => 'EUR'
+          }, {
+            'account' => 'Assets:Cash',
+            'amount' => -9.0,
+            'commodity' => 'EUR'
+          }
+         ],
+         'pending' => false
+       }, {
+         'date' => '2013/12/05',
+         'payee' => 'Shikgoo',
+         'cleared' => false,
+         'posts' =>
+         [
+          {
+            'account' => 'Expenses:Restaurants',
+            'amount' => 12.0,
+            'commodity' => 'EUR'
+          }, {
+            'account' => 'Liabilities:Max Mustermann',
+            'amount' => -12.0,
+            'commodity' => 'EUR'
+          }
+         ],
+         'pending' => false
+       }, {
+         'posts' =>
+         [
+          {
+            'account' => 'Assets:Reimbursements:Hans Maulwurf',
+            'amount' => 3.1,
+            'commodity' => 'EUR'
+          }, {
+            'account' => 'Assets:Cash',
+            'amount' => -3.1,
+            'commodity' => 'EUR'
+          }
+         ],
+         'note' => 'Sonnenblumenkernbrot',
+         'payee' => 'Bioladen Tegeler Straße',
+         'date' => '2013/12/10',
+         'cleared' => false,
+         'pending' => false
+       }, {
+         'effective_date' => '2013/12/02',
+         'posts' =>
+         [
+          {
+            'account' => 'Assets:Cash',
+            'amount' => 50.0,
+            'commodity' => 'EUR'
+          }, {
+            'account' => 'Assets:Giro',
+            'amount' => -50.0,
+            'commodity' => 'EUR'
+          }
+         ],
+         'payee' => 'Sparkasse',
+         'date' => '2013/12/01',
+         'cleared' => true,
+         'pending' => false
+       }, {
+         'date' => '2013/12/04',
+         'payee' => 'Trattoria La Vialla',
+         'cleared' => false,
+         'posts' =>
+         [
+          {
+            'account' => 'Expenses:Restaurants',
+            'amount' => 32.2,
+            'commodity' => 'EUR'
+          }, {
+            'account' => 'Assets:Cash',
+            'amount' => -32.2,
+            'commodity' => 'EUR'
+          }
+         ],
+         'pending' => false
+       }, {
+         'date' => '2013/12/06',
+         'payee' => 'Customer X',
+         'cleared' => false,
+         'posts' =>
+         [
+          {
+            'account' => 'Assets:Giro',
+            'amount' => 1200.0,
+            'commodity' => 'EUR'
+          }, {
+            'account' => 'Income:Invoice',
+            'amount' => -1200.0,
+            'commodity' => 'EUR'
+          }
+         ],
+         'pending' => false
+       }
+      ]
+    end
+
+    it 'returns all transactions' do
+      get '/transactions'
+      JSON.parse(last_response.body).should deep_eq valid_json
+    end
+  end
+
+  describe 'POST' do
+    let(:transaction) do
+      {
+        date: '2013/12/12',
+        cleared: true,
+        payee: 'New Payee',
+        postings:
+        [
+         {
+           account: 'Expenses:Restaurants',
+           amount: 11.0,
+           commodity: 'EUR'
+         },
+         {
+           account: 'Assets:Cash',
+           amount: -11.0,
+           commodity: 'EUR'
+         }
+        ]
+      }
+    end
+
+    let(:correct_response) do
+      { transaction: transaction }
+    end
+
+    it 'adds a new transaction to the append file' do
+      restore_file('spec/files/append.ledger') do
+        post '/transactions', transaction.to_json
+
+        last_response.status.should == 201
+        JSON.parse(last_response.body, symbolize_names: true).should deep_eq correct_response
+        puts File.read('spec/files/append.ledger').should == <<RESULT
+2013/12/03 NaveenaPath
+    Expenses:Restaurants                     9.00EUR
+    Assets:Cash
+
+2013/12/05 Shikgoo
+    Expenses:Restaurants                    12.00EUR
+    Liabilities:Max Mustermann
+    ; meta_info: Some interesting meta information
+
+2013/12/10 Bioladen Tegeler Straße
+    ; Sonnenblumenkernbrot
+    Assets:Reimbursements:Hans Maulwurf      3.10EUR
+    Assets:Cash
+
+2013/12/12 * New Payee
+    Expenses:Restaurants  11.00EUR
+    Assets:Cash  -11.00EUR
+RESULT
+      end
+    end
+  end
+
+  describe 'PUT' do
+    it 'updates a transaction'
+  end
+end

data/spec/ledger-rest/api/version_spec.rb

@@ -0,0 +1,13 @@
+# -*- coding: utf-8 -*-
+require 'spec_helper'
+
+describe '/version' do
+  it 'returns the ledger-rest version' do
+    get '/version'
+
+    JSON.parse(last_response.body).should == {
+      'version' => LedgerRest::VERSION,
+      'ledger-version' => LedgerRest::Ledger.version
+    }
+  end
+end

data/spec/ledger-rest/ledger/balance_spec.rb

@@ -0,0 +1,124 @@
+require 'spec_helper'
+
+describe LedgerRest::Ledger::Balance do
+  describe '::expand_accounts' do
+    let(:accounts) do
+      [
+       {
+         fullname: 'Expenses',
+         name: 'Expenses',
+         depth: 1,
+         total: '453.10EUR'
+       },
+       {
+         fullname: 'Expenses:Groceries:Bread',
+         name: 'Groceries:Bread',
+         depth: 3,
+         total: '3.10EUR'
+       },
+       {
+         fullname: 'Assets:Cash',
+         name: 'Assets:Cash',
+         depth: 1,
+         total: '200.00EUR'
+       }
+      ]
+    end
+
+    let(:expanded) do
+      [
+       { fullname: 'Expenses', name: 'Expenses', depth: 1, total: '453.10EUR' },
+       { fullname: 'Expenses:Groceries', name: 'Groceries', depth: 2, total: '3.10EUR' },
+       { fullname: 'Expenses:Groceries:Bread', name: 'Bread', depth: 3, total: '3.10EUR' },
+       { fullname: 'Assets', name: 'Assets', depth: 1, total: '200.00EUR' },
+       { fullname: 'Assets:Cash', name: 'Cash', depth: 2, total: '200.00EUR' }
+      ]
+    end
+
+    it 'expands accounts correctly' do
+      LedgerRest::Ledger::Balance.expand_accounts(accounts).should deep_eq expanded
+    end
+  end
+
+  describe '::wrap_accounts' do
+    let(:accounts) do
+      [
+       {
+         fullname: 'Expenses',
+         name: 'Expenses',
+         depth: 1,
+         total: '453.10EUR'
+       },
+       {
+         fullname: 'Expenses:Groceries',
+         name: 'Groceries',
+         depth: 2,
+         total: '3.10EUR'
+       },
+       {
+         fullname: 'Expenses:Groceries:Bread',
+         name: 'Bread',
+         depth: 3,
+         total: '3.10EUR'
+       },
+       {
+         fullname: 'Expenses:Flat',
+         name: 'Flat',
+         depth: 2,
+         total: '450.00EUR'
+       },
+       {
+         fullname: 'Assets',
+         name: 'Assets',
+         depth: 1,
+         total: '200.00EUR'
+       }
+      ]
+    end
+
+    let(:wrapped) do
+      [
+       {
+         fullname: 'Expenses',
+         name: 'Expenses',
+         depth: 1,
+         total: '453.10EUR',
+         accounts:
+         [
+          {
+            fullname: 'Expenses:Groceries',
+            name: 'Groceries',
+            depth: 2,
+            total: '3.10EUR',
+            accounts:
+            [
+             {
+               fullname: 'Expenses:Groceries:Bread',
+               name: 'Bread',
+               depth: 3,
+               total: '3.10EUR'
+             }
+            ]
+          },
+          {
+            fullname: 'Expenses:Flat',
+            name: 'Flat',
+            depth: 2,
+            total: '450.00EUR'
+          }
+         ]
+       },
+       {
+         fullname: 'Assets',
+         name: 'Assets',
+         depth: 1,
+         total: '200.00EUR'
+       }
+      ]
+    end
+
+    it 'wraps accounts correctly' do
+      LedgerRest::Ledger::Balance.wrap_accounts(accounts).should deep_eq wrapped
+    end
+  end
+end

data/spec/ledger-rest/ledger/parser_spec.rb

@@ -6,79 +6,83 @@ describe LedgerRest::Ledger::Parser do
     @parser = LedgerRest::Ledger::Parser.new
   end
 
-  context '#parse' do
-    subject {
-      LedgerRest::Ledger::Transaction.new(:date => "2012/03/01",
-                                          :effective_date => "2012/03/23",
-                                          :cleared => true,
-                                          :pending => false,
-                                          :code => "INV#23",
-                                          :payee => "me, myself and I",
-                                          :postings => [
-                                                        {
-                                                          :account => "Expenses:Imaginary",
-                                                          :amount => "€ 23",
-                                                          :per_unit_cost => "USD 2300",
-                                                          :actual_date => "2012/03/24",
-                                                          :effective_date => "2012/03/25"
-                                                        }, {
-                                                          :account => "Expenses:Magical",
-                                                          :amount => "€ 42",
-                                                          :posting_cost => "USD 23000000",
-                                                          :virtual => true
-                                                        }, {
-                                                          :account => "Assets:Mighty"
-                                                        }, {
-                                                          :comment => "This is a freeform comment"
-                                                        },
-                                                       ])
-    }
+  describe '#parse' do
+    subject do
+      LedgerRest::Ledger::Transaction.new(date: '2012/03/01',
+                                          effective_date: '2012/03/23',
+                                          cleared: true,
+                                          pending: false,
+                                          code: 'INV#23',
+                                          payee: 'me, myself and I',
+                                          postings:
+                                          [
+                                           {
+                                             account: 'Expenses:Imaginary',
+                                             amount: 23.0,
+                                             commodity: 'EUR',
+                                             per_unit_cost: 2300.0,
+                                             per_unit_commodity: 'USD',
+                                             actual_date: '2012/03/24',
+                                             effective_date: '2012/03/25'
+                                           }, {
+                                             account: 'Expenses:Magical',
+                                             amount: 42.0,
+                                             commodity: 'EUR',
+                                             posting_cost: 23000000.0,
+                                             posting_cost_commodity: 'USD',
+                                             virtual: true
+                                           }, {
+                                             account: 'Assets:Mighty'
+                                           }, {
+                                             comment: 'This is a freeform comment'
+                                           },
+                                          ])
+    end
 
     it 'should parse a to_ledger converted transaction into the same original hash' do
       @parser.parse(subject.to_ledger).should == subject
     end
-
   end
 
-  context '#parse_date' do
+  describe '#parse_date' do
     before :all do
       @ret = @parser.parse_date("2012/11/23 * Rest with\nAnd Stuff")
     end
 
-    it 'should return the parsed date' do
+    it 'returns the parsed date' do
       @ret[0].should == '2012/11/23'
     end
 
-    it 'should return the rest of the input' do
+    it 'returns the rest of the input' do
       @ret[1].should == " * Rest with\nAnd Stuff"
     end
   end
 
-  context '#parse_effective_date' do
+  describe '#parse_effective_date' do
     before :all do
       @ret = @parser.parse_effective_date("=2012/11/24 * Rest with\nAnd Stuff")
     end
 
-    it 'should return the parsed date' do
-      @ret[0].should == "2012/11/24"
+    it 'returns the parsed date' do
+      @ret[0].should == '2012/11/24'
     end
 
-    it 'should return the rest of the input' do
+    it 'returns the rest of the input' do
       @ret[1].should == " * Rest with\nAnd Stuff"
     end
   end
 
-  context '#parse_state' do
+  describe '#parse_state' do
     context 'given cleared transaction input' do
       before :all do
         @ret = @parser.parse_cleared(" * Rest with\nAnd Stuff")
       end
 
-      it 'should return true' do
+      it 'returns true' do
         @ret[0].should == true
       end
 
-      it 'should return the rest of the input' do
+      it 'returns the rest of the input' do
         @ret[1].should == "Rest with\nAnd Stuff"
       end
     end
@@ -88,27 +92,27 @@ describe LedgerRest::Ledger::Parser do
         @ret = @parser.parse_cleared("Rest with\nAnd Stuff")
       end
 
-      it 'should return false' do
+      it 'returns false' do
         @ret[0].should == false
       end
 
-      it 'should return the rest of the input' do
+      it 'returns the rest of the input' do
         @ret[1].should == "Rest with\nAnd Stuff"
       end
     end
   end
 
-  context '#parse_pending' do
+  describe '#parse_pending' do
     context 'given pending transaction input' do
       before :all do
         @ret = @parser.parse_pending(" ! Rest with\nAnd Stuff")
       end
 
-      it 'should return true' do
+      it 'returns true' do
         @ret[0].should == true
       end
 
-      it 'should return the rest of the input' do
+      it 'returns the rest of the input' do
         @ret[1].should == "Rest with\nAnd Stuff"
       end
     end
@@ -118,22 +122,22 @@ describe LedgerRest::Ledger::Parser do
         @ret = @parser.parse_pending("Rest with\nAnd Stuff")
       end
 
-      it 'should return false' do
+      it 'returns false' do
         @ret[0].should == false
       end
 
-      it 'should return the rest of the input' do
+      it 'returns the rest of the input' do
         @ret[1].should == "Rest with\nAnd Stuff"
       end
     end
   end
 
-  context '#parse_code' do
+  describe '#parse_code' do
     context 'given a transaction with white-spaced code' do
       subject { @parser.parse_code(" (#123) Rest with\nAnd Stuff") }
 
       its(:first) { should == '#123' }
-      its(:last) { should == "Rest with\nAnd Stuff"}
+      its(:last) { should == "Rest with\nAnd Stuff" }
     end
 
     context 'given a transaction with code' do
@@ -146,124 +150,124 @@ describe LedgerRest::Ledger::Parser do
     context 'given a transaction without code' do
       subject { @parser.parse_code("Rest with\nAnd Stuff") }
 
-      its(:first) { should == nil }
+      its(:first) { should be_nil }
       its(:last) { should == "Rest with\nAnd Stuff" }
     end
   end
 
-  context '#parse_payee' do
+  describe '#parse_payee' do
     context 'given an unstripped line' do
       subject { @parser.parse_payee("  Monsieur Le Payee\n  Some:Account  123EUR\n  Some:Other")}
 
-      its(:first) { should == "Monsieur Le Payee" }
-      its(:last) { should == "  Some:Account  123EUR\n  Some:Other"}
+      its(:first) { should == 'Monsieur Le Payee' }
+      its(:last) { should == "  Some:Account  123EUR\n  Some:Other" }
     end
 
     context 'given a stripped line' do
       context 'given an unstripped line' do
         subject { @parser.parse_payee("Monsieur Le Payee\n  Some:Account  123EUR\n  Some:Other")}
 
-        its(:first) { should == "Monsieur Le Payee" }
-        its(:last) { should == "  Some:Account  123EUR\n  Some:Other"}
+        its(:first) { should == 'Monsieur Le Payee' }
+        its(:last) { should == "  Some:Account  123EUR\n  Some:Other" }
       end
     end
   end
 
-  context '#parse_comments' do
+  describe '#parse_comments' do
     context 'given no comments' do
-      subject { @parser.parse_comments("  Assets:Some:Stuff  23EUR")}
+      subject { @parser.parse_comments('  Assets:Some:Stuff  23EUR')}
 
-      it 'should return all comments' do
-        subject[0].should == nil
+      it 'returns all comments' do
+        subject[0].should be_nil
       end
 
-      it 'should return the rest of the input' do
-        subject[1].should == "  Assets:Some:Stuff  23EUR"
+      it 'returns the rest of the input' do
+        subject[1].should == '  Assets:Some:Stuff  23EUR'
       end
     end
 
     context 'given one line of transaction comments' do
       subject { @parser.parse_comments("  ; ABC\n  Assets:Some:Stuff  23EUR")}
 
-      it 'should return all comments' do
+      it 'returns all comments' do
         subject[0].should == "ABC\n"
       end
 
-      it 'should return the rest of the input' do
-        subject[1].should == "  Assets:Some:Stuff  23EUR"
+      it 'returns the rest of the input' do
+        subject[1].should == '  Assets:Some:Stuff  23EUR'
       end
     end
 
     context 'given multiple lines of transaction comments' do
       subject { @parser.parse_comments("  ; ABC\n  ;DEF\n  Assets:Some:Stuff  23EUR")}
 
-      it 'should return all comments' do
+      it 'returns all comments' do
         subject[0].should == "ABC\nDEF\n"
       end
 
-      it 'should return the rest of the input' do
-        subject[1].should == "  Assets:Some:Stuff  23EUR"
+      it 'returns the rest of the input' do
+        subject[1].should == '  Assets:Some:Stuff  23EUR'
       end
     end
   end
 
-  context '#parse_account' do
+  describe '#parse_account' do
     context 'given normal' do
       subject { @parser.parse_account("  Assets:Some:Nice  200EUR\n  Assets:Account")}
 
-      it 'should return the account' do
-        subject[0].should == "Assets:Some:Nice"
+      it 'returns the account' do
+        subject[0].should == 'Assets:Some:Nice'
       end
 
-      it 'should not be virtual' do
+      it 'is not virtual' do
         subject[1].should == false
       end
 
-      it 'should not be balanced virtual' do
+      it 'is not balanced virtual' do
         subject[2].should == false
       end
 
-      it 'should return the rest of the input' do
+      it 'returns the rest of the input' do
         subject[3].should == "200EUR\n  Assets:Account"
       end
     end
 
     context 'given input without amount' do
-      subject { @parser.parse_account("  Assets:Some:Nice")}
+      subject { @parser.parse_account('  Assets:Some:Nice') }
 
-      it 'should return the account' do
-        subject[0].should == "Assets:Some:Nice"
+      it 'returns the account' do
+        subject[0].should == 'Assets:Some:Nice'
       end
 
-      it 'should not be virtual' do
+      it 'is not virtual' do
         subject[1].should == false
       end
 
-      it 'should not be balanced virtual' do
+      it 'is not balanced virtual' do
         subject[2].should == false
       end
 
-      it 'should return the rest of the input' do
-        subject[3].should == ""
+      it 'returns the rest of the input' do
+        subject[3].should == ''
       end
     end
 
     context 'given virtual' do
       subject { @parser.parse_account("  (Assets:Some:Nice)  200EUR\n  Assets:Account")}
 
-      it 'should return the account' do
-        subject[0].should == "Assets:Some:Nice"
+      it 'returns the account' do
+        subject[0].should == 'Assets:Some:Nice'
       end
 
-      it 'should not be virtual' do
+      it 'is not virtual' do
         subject[1].should == true
       end
 
-      it 'should not be balanced virtual' do
+      it 'is not balanced virtual' do
         subject[2].should == false
       end
 
-      it 'should return the rest of the input' do
+      it 'returns the rest of the input' do
         subject[3].should == "200EUR\n  Assets:Account"
       end
     end
@@ -271,92 +275,155 @@ describe LedgerRest::Ledger::Parser do
     context 'given balanced virtual' do
       subject { @parser.parse_account("  [Assets:Some:Nice]  200EUR\n  Assets:Account")}
 
-      it 'should return the account' do
-        subject[0].should == "Assets:Some:Nice"
+      it 'returns the account' do
+        subject[0].should == 'Assets:Some:Nice'
       end
 
-      it 'should not be virtual' do
+      it 'is not virtual' do
         subject[1].should == true
       end
 
-      it 'should not be balanced virtual' do
+      it 'is not balanced virtual' do
         subject[2].should == true
       end
 
-      it 'should return the rest of the input' do
+      it 'returns the rest of the input' do
         subject[3].should == "200EUR\n  Assets:Account"
       end
     end
   end
 
-  context '#parse_amount' do
+  describe '#parse_amount_parts' do
+    context 'given "23.00EUR"' do
+      subject { @parser.parse_amount_parts('23.00EUR') }
+
+      it 'returns the value' do
+        subject[0].should == 23.0
+      end
+
+      it 'returns the commodity' do
+        subject[1].should == 'EUR'
+      end
+    end
+
+    context 'given "23EUR"' do
+      subject { @parser.parse_amount_parts('23EUR') }
+
+      it 'returns the value' do
+        subject[0].should == 23.0
+      end
+
+      it 'returns the commodity' do
+        subject[1].should == 'EUR'
+      end
+    end
+
+    context 'given "USD23"' do
+      subject { @parser.parse_amount_parts('USD23') }
+
+      it 'returns the value' do
+        subject[0].should == 23.0
+      end
+
+      it 'returns the commodity' do
+        subject[1].should == 'USD'
+      end
+    end
+
+    context 'given "USD23.00"' do
+      subject { @parser.parse_amount_parts('USD23') }
+
+      it 'returns the value' do
+        subject[0].should == 23.0
+      end
+
+      it 'returns the commodity' do
+        subject[1].should == 'USD'
+      end
+    end
+
+    context 'given "€ 23.00"' do
+      subject { @parser.parse_amount_parts('€ 23.00') }
+
+      it 'returns the value' do
+        subject[0].should == 23.0
+      end
+
+      it 'returns the commodity' do
+        subject[1].should == '€'
+      end
+    end
+  end
+
+  describe '#parse_amount' do
     context 'given "23.00EUR"' do
-      subject { @parser.parse_amount("23.00EUR") }
+      subject { @parser.parse_amount('23.00EUR') }
 
-      it 'should return amount and commodity' do
-        subject[0].should == "23.00EUR"
+      it 'returns amount and commodity' do
+        subject[0].should == '23.00EUR'
       end
 
-      it 'should return no posting_cost' do
-        subject[1].should == nil
+      it 'returns no posting_cost' do
+        subject[1].should be_nil
       end
 
-      it 'should return no per_unit_cost' do
-        subject[2].should == nil
+      it 'returns no per_unit_cost' do
+        subject[2].should be_nil
       end
     end
 
     context 'given "25 AAPL @ 10.00EUR"' do
-      subject { @parser.parse_amount("25 AAPL @ 10.00EUR") }
+      subject { @parser.parse_amount('25 AAPL @ 10.00EUR') }
 
-      it 'should return amount and commodity' do
-        subject[0].should == "25 AAPL"
+      it 'returns amount and commodity' do
+        subject[0].should == '25 AAPL'
       end
 
-      it 'should return correct posting_cost' do
-        subject[1].should == "10.00EUR"
+      it 'returns correct posting_cost' do
+        subject[1].should == '10.00EUR'
       end
 
-      it 'should return no per_unit_cost' do
-        subject[2].should == nil
+      it 'returns no per_unit_cost' do
+        subject[2].should be_nil
       end
     end
 
     context 'given "30Liters @@ 1.64EUR"' do
-      subject { @parser.parse_amount("30Liters @@ 1.64EUR") }
+      subject { @parser.parse_amount('30Liters @@ 1.64EUR') }
 
-      it 'should return amount and commodity' do
-        subject[0].should == "30Liters"
+      it 'returns amount and commodity' do
+        subject[0].should == '30Liters'
       end
 
-      it 'should return no posting_cost' do
-        subject[1].should == nil
+      it 'returns no posting_cost' do
+        subject[1].should be_nil
       end
 
-      it 'should return correct per_unit_cost' do
-        subject[2].should == "1.64EUR"
+      it 'returns correct per_unit_cost' do
+        subject[2].should == '1.64EUR'
       end
     end
   end
 
-  context '#parse_posting' do
+  describe '#parse_posting' do
     context 'given posting with comment' do
       subject { @parser.parse_posting("  Assets:Test:Account  123EUR\n  ; Some comment") }
 
-      it 'should have parsed correctly' do
+      it 'has parsed correctly' do
         subject.should == {
-          :account => "Assets:Test:Account",
-          :amount => "123EUR"
+          account: 'Assets:Test:Account',
+          amount: 123.0,
+          commodity: 'EUR'
         }
       end
     end
 
     context 'given source posting' do
-      subject { @parser.parse_posting("  Assets:Test:Account") }
+      subject { @parser.parse_posting('  Assets:Test:Account') }
 
-      it 'should have parsed correctly' do
+      it 'has parsed correctly' do
         subject.should == {
-          :account => "Assets:Test:Account"
+          account: 'Assets:Test:Account'
         }
       end
     end