Skip to content

Commit

Permalink
Fix test failing due to assuming a different default locale
Browse files Browse the repository at this point in the history
TransactionTest.testCreateAutoBalanceSplit assumed a default locale with
EUR commodity. The default commodity is set in new Transaction objects,
which might change depending on the environment where the tests are run.
Now we set it explicitly in the tests.
  • Loading branch information
rivaldi8 committed Sep 13, 2017
1 parent a0d8d10 commit 2b0e03e
Showing 1 changed file with 2 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -80,6 +80,7 @@ public void settingUID_shouldSetTransactionUidOfSplits(){
@Test
public void testCreateAutoBalanceSplit() {
Transaction transactionCredit = new Transaction("Transaction with more credit");
transactionCredit.setCommodity(Commodity.getInstance("EUR"));
Split creditSplit = new Split(new Money("1", "EUR"), "test-account");
creditSplit.setType(TransactionType.CREDIT);
transactionCredit.addSplit(creditSplit);
Expand All @@ -93,6 +94,7 @@ public void testCreateAutoBalanceSplit() {


Transaction transactionDebit = new Transaction("Transaction with more debit");
transactionDebit.setCommodity(Commodity.getInstance("EUR"));
Split debitSplit = new Split(new Money("1", "EUR"), "test-account");
debitSplit.setType(TransactionType.DEBIT);
transactionDebit.addSplit(debitSplit);
Expand Down

0 comments on commit 2b0e03e

Please sign in to comment.