From 6e9a5470496ec6fe357e14ec921c5a2d920ac592 Mon Sep 17 00:00:00 2001 From: hengsin Date: Tue, 27 Oct 2020 01:24:57 +0800 Subject: [PATCH] IDEMPIERE-4128 Matched invoice posting with wrong precision based on transaction currency instead of schema currency (#317) - minor unit testing improvement --- .../src/org/idempiere/test/base/MatchInvTest.java | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/org.idempiere.test/src/org/idempiere/test/base/MatchInvTest.java b/org.idempiere.test/src/org/idempiere/test/base/MatchInvTest.java index c877b5d30f..47c7f89073 100644 --- a/org.idempiere.test/src/org/idempiere/test/base/MatchInvTest.java +++ b/org.idempiere.test/src/org/idempiere/test/base/MatchInvTest.java @@ -679,11 +679,15 @@ public class MatchInvTest extends AbstractTestCase { MFactAcct fa = new MFactAcct(Env.getCtx(), id, getTrxName()); if (fa.getAccount_ID() == acctNIR.getAccount_ID()) { assertTrue(fa.getAmtAcctDr().compareTo(Env.ZERO) >= 0); - assertTrue(fa.getAmtAcctDr().compareTo(acctAmount) == 0, fa.getAmtAcctDr().toPlainString() + " != " + acctAmount.toPlainString()); + assertTrue(fa.getAmtAcctDr().toPlainString().compareTo(acctAmount.toPlainString()) == 0, fa.getAmtAcctDr().toPlainString() + " != " + acctAmount.toPlainString()); + // verify source amt and currency + assertTrue(fa.getC_Currency_ID() == japaneseYen.getC_Currency_ID()); assertTrue(fa.getAmtSourceDr().toPlainString().compareTo(acctSource.toPlainString()) == 0, fa.getAmtSourceDr().toPlainString() + " != " + acctSource.toPlainString()); } else if (fa.getAccount_ID() == acctInvClr.getAccount_ID()) { assertTrue(fa.getAmtAcctCr().compareTo(Env.ZERO) >= 0); - assertTrue(fa.getAmtAcctCr().compareTo(acctAmount) == 0, fa.getAmtAcctCr().toPlainString() + " != " + acctAmount.toPlainString()); + assertTrue(fa.getAmtAcctCr().toPlainString().compareTo(acctAmount.toPlainString()) == 0, fa.getAmtAcctCr().toPlainString() + " != " + acctAmount.toPlainString()); + // verify source amt and currency + assertTrue(fa.getC_Currency_ID() == japaneseYen.getC_Currency_ID()); assertTrue(fa.getAmtSourceCr().toPlainString().compareTo(acctSource.toPlainString()) == 0, fa.getAmtSourceCr().toPlainString() + " != " + acctSource.toPlainString()); } }