Issue #428: Modified test for Material Internal Consumption

ETP-60: The order of the instructions in the test testCountWithoutDocTypeConfigured was not correct, and the last change was added to the getTransactionsBatchCount function which also made use of the new Date()

src-test/src/org/openbravo/test/accounting/PostedNoDocConfigTest.java

@@ -91,12 +91,11 @@ public void testCountWithoutDocTypeConfigured() {
 
       createInternalConsumptionLine(internalConsumption);
 
-      TestCostingUtils.runCostingBackground();
-
       processInternalConsumption(internalConsumption);
 
       activeOrDeactiveMaterialManagementConsumptionTable(true);
 
+      TestCostingUtils.runCostingBackground();
 
       OBDal.getInstance().commitAndClose();
 

src/org/openbravo/costing/CostingBackground.java

@@ -244,14 +244,13 @@ private int getTransactionsBatchCount(List<String> orgsWithRule) {
         + "   and p.stocked = true"
         + "   and trxtype.reference.id = :refid"
         + "   and trxtype.searchKey = trx.movementType"
-        + "   and trx.transactionProcessDate <= :now"
+        + "   and trx.transactionProcessDate <= now()"
         + "   and trx.organization.id in (:orgs)";
 
     //@formatter:on
     Query<Long> trxQry = OBDal.getInstance().getSession().createQuery(where, Long.class);
 
     trxQry.setParameter("refid", CostAdjustmentUtils.MovementTypeRefID);
-    trxQry.setParameter("now", new Date());
     trxQry.setParameterList("orgs", orgsWithRule);
     trxQry.setMaxResults(1);
     return (trxQry.uniqueResult()).intValue();