Solucionados problemas de tests con República Dominicana.

Author: Carlos Garcia

Core/Model/Subcuenta.php

@@ -206,7 +206,7 @@ public static function tableName(): string
 
     public function test(): bool
     {
-        $this->saldo = round($this->debe - $this->haber, FS_NF0);
+        $this->saldo = Tools::round($this->debe - $this->haber);
 
         // escape html
         foreach (['codcuenta', 'codsubcuenta', 'descripcion', 'codcuentaesp'] as $field) {
@@ -216,17 +216,22 @@ public function test(): bool
         $this->codsubcuenta = empty($this->idsubcuenta) ? $this->transformCodsubcuenta($this->codsubcuenta) : $this->codsubcuenta;
         $this->descripcion = Tools::noHtml($this->descripcion);
         if (strlen($this->descripcion) < 1 || strlen($this->descripcion) > 255) {
-            Tools::log()->warning(
-                'invalid-column-lenght',
-                ['%column%' => 'descripcion', '%min%' => '1', '%max%' => '255']
-            );
+            Tools::log()->warning('invalid-column-lenght', [
+                '%column%' => 'descripcion',
+                '%min%' => '1',
+                '%max%' => '255'
+            ]);
             return false;
         }
 
         // check exercise
         $exercise = $this->getExercise();
         if (false === $this->disable_additional_test && strlen($this->codsubcuenta) !== $exercise->longsubcuenta) {
-            Tools::log()->warning('account-length-error', ['%code%' => $this->codsubcuenta]);
+            Tools::log()->warning('account-length-error', [
+                '%code%' => $this->codsubcuenta,
+                '%length%' => $exercise->longsubcuenta,
+                '%exercise%' => $exercise->id()
+            ]);
             return false;
         }
 

Test/Core/Model/EjercicioCierreTest.php

@@ -21,6 +21,7 @@
 
 use FacturaScripts\Core\Base\DataBase;
 use FacturaScripts\Core\Base\DataBase\DataBaseWhere;
+use FacturaScripts\Core\DataSrc\Paises;
 use FacturaScripts\Core\Lib\Accounting\AccountingPlanImport;
 use FacturaScripts\Core\Lib\Accounting\ClosingToAcounting;
 use FacturaScripts\Core\Model\Almacen;
@@ -59,7 +60,7 @@ public function testCloseExercise(): void
         // obtenemos el almacén por defecto
         $almacen = new Almacen();
         $where = [new DataBaseWhere('idempresa', $empresa->idempresa)];
-        $this->assertTrue($almacen->loadFromCode('', $where));
+        $this->assertTrue($almacen->loadWhere($where));
 
         // creamos el ejercicio para 2020
         $ejercicio = new Ejercicio();
@@ -71,7 +72,7 @@ public function testCloseExercise(): void
         $this->assertTrue($ejercicio->save());
 
         // copiamos el plan contable
-        $filePath = FS_FOLDER . '/Core/Data/Codpais/ESP/defaultPlan.csv';
+        $filePath = FS_FOLDER . '/Core/Data/Codpais/' . Paises::default()->codpais . '/defaultPlan.csv';
         $planImport = new AccountingPlanImport();
         $planImport->importCSV($filePath, $ejercicio->codejercicio);
 
@@ -90,10 +91,11 @@ public function testCloseExercise(): void
             'copySubAccounts' => true
         ];
         $closing = new ClosingToAcounting();
+        $this->assertTrue($ejercicio->reload());
         $this->assertTrue($closing->exec($ejercicio, $data));
 
         // comprobamos que el ejercicio está cerrado
-        $this->assertTrue($ejercicio->loadFromCode($ejercicio->codejercicio));
+        $this->assertTrue($ejercicio->reload());
         $this->assertFalse($ejercicio->isOpened());
 
         // comprobamos que todas las subcuentas del ejercicio anterior tienen saldo 0
@@ -133,7 +135,7 @@ public function testCloseExerciseWithMissingEntries(): void
         // obtenemos el almacén por defecto
         $almacen = new Almacen();
         $where = [new DataBaseWhere('idempresa', $empresa->idempresa)];
-        $this->assertTrue($almacen->loadFromCode('', $where));
+        $this->assertTrue($almacen->loadWhere($where));
 
         // creamos el ejercicio para 2026
         $ejercicio = new Ejercicio();
@@ -145,7 +147,7 @@ public function testCloseExerciseWithMissingEntries(): void
         $this->assertTrue($ejercicio->save());
 
         // copiamos el plan contable
-        $filePath = FS_FOLDER . '/Core/Data/Codpais/ESP/defaultPlan.csv';
+        $filePath = FS_FOLDER . '/Core/Data/Codpais/' . Paises::default()->codpais . '/defaultPlan.csv';
         $planImport = new AccountingPlanImport();
         $planImport->importCSV($filePath, $ejercicio->codejercicio);
 

Test/Traits/DefaultSettingsTrait.php

@@ -21,6 +21,7 @@
 
 use FacturaScripts\Core\Base\DataBase\DataBaseWhere;
 use FacturaScripts\Core\DataSrc\Ejercicios;
+use FacturaScripts\Core\DataSrc\Paises;
 use FacturaScripts\Core\Lib\Accounting\AccountingPlanImport;
 use FacturaScripts\Core\Model\Almacen;
 use FacturaScripts\Core\Model\Cuenta;
@@ -33,8 +34,7 @@ trait DefaultSettingsTrait
     protected static function installAccountingPlan(): void
     {
         // ¿Existe el archivo del plan contable?
-        $codpais = Tools::settings('default', 'codpais', 'ESP');
-        $filePath = FS_FOLDER . '/Core/Data/Codpais/' . $codpais . '/defaultPlan.csv';
+        $filePath = FS_FOLDER . '/Core/Data/Codpais/' . Paises::default()->codpais . '/defaultPlan.csv';
         if (false === file_exists($filePath)) {
             return;
         }
@@ -70,8 +70,7 @@ protected static function removeTaxRegularization(): void
 
     protected static function setDefaultSettings(): void
     {
-        $codpais = Tools::settings('default', 'codpais', 'ESP');
-        $fileContent = file_get_contents(FS_FOLDER . '/Core/Data/Codpais/' . $codpais . '/default.json');
+        $fileContent = file_get_contents(FS_FOLDER . '/Core/Data/Codpais/' . Paises::default()->codpais . '/default.json');
         $defaultValues = json_decode($fileContent, true) ?? [];
         foreach ($defaultValues as $group => $values) {
             foreach ($values as $key => $value) {