This is an automated email from the ASF dual-hosted git repository.

jleroux pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git

commit cb1d2b94002fc3aac9afbbb9a194a06bd7258a55
Author: Jacques Le Roux <jacques.le.r...@les7arts.com>
AuthorDate: Fri Apr 5 11:01:58 2024 +0200

    Improved: Fix Accounting Groovy tests (OFBIZ-12995)
    
    Fixes 2 priority 2 Codenarc issues
---
 .../ofbiz/accounting/accounting/AutoAcctgTransPurchaseTests.groovy      | 2 +-
 .../apache/ofbiz/accounting/accounting/AutoAcctgTransSalesTests.groovy  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/applications/accounting/src/main/groovy/org/apache/ofbiz/accounting/accounting/AutoAcctgTransPurchaseTests.groovy
 
b/applications/accounting/src/main/groovy/org/apache/ofbiz/accounting/accounting/AutoAcctgTransPurchaseTests.groovy
index 1581682d0a..8d34dc1632 100644
--- 
a/applications/accounting/src/main/groovy/org/apache/ofbiz/accounting/accounting/AutoAcctgTransPurchaseTests.groovy
+++ 
b/applications/accounting/src/main/groovy/org/apache/ofbiz/accounting/accounting/AutoAcctgTransPurchaseTests.groovy
@@ -178,7 +178,7 @@ class AutoAcctgTransPurchaseTests extends OFBizTestCase {
         }
     }
 
-    void checkEntriesBalance(List<GenericValue> acctgTransEntryList) {
+    private void checkEntriesBalance(List<GenericValue> acctgTransEntryList) {
         BigDecimal debitTotal = new BigDecimal('0')
         BigDecimal creditTotal = new BigDecimal('0')
 
diff --git 
a/applications/accounting/src/main/groovy/org/apache/ofbiz/accounting/accounting/AutoAcctgTransSalesTests.groovy
 
b/applications/accounting/src/main/groovy/org/apache/ofbiz/accounting/accounting/AutoAcctgTransSalesTests.groovy
index 0411880a1e..130f2b4b22 100644
--- 
a/applications/accounting/src/main/groovy/org/apache/ofbiz/accounting/accounting/AutoAcctgTransSalesTests.groovy
+++ 
b/applications/accounting/src/main/groovy/org/apache/ofbiz/accounting/accounting/AutoAcctgTransSalesTests.groovy
@@ -225,7 +225,7 @@ class AutoAcctgTransSalesTests extends OFBizTestCase {
         }
     }
 
-    void checkEntriesBalance(List<GenericValue> acctgTransEntryList) {
+    private void checkEntriesBalance(List<GenericValue> acctgTransEntryList) {
         BigDecimal debitTotal = new BigDecimal('0')
         BigDecimal creditTotal = new BigDecimal('0')
 

Reply via email to