Renforcement tests

- on s'assure de bien lire les configs de test (et non pas les configs "nominales") v5.5.4-3.10.1

Renforcement tests
- on s'assure de bien lire les configs de test (et non pas les configs "nominales") v5.5.4-3.10.1
Etienne Pallier
1 parent 779b03b8
Showing 7 changed files with 31 additions and 8 deletions Show diff stats
.gitignore
CHANGELOG
README.md
config/.gitignore
config/app_labinvent_mandatory_fields_test.yml
tests/TestCase/Controller/General.php
tests/TestCase/Controller/MaterielsControllerTest.php
@@ -2,6 +2,9 @@
 #newfolderempty/*
+# vieux fichiers
+OLD/*
+
 *.36
 *.37
 *.37.*
@@ -484,6 +484,10 @@ rien ?
 ======= CHANGES =======
 -------
+09/12/2021	v5.5.4-3.10.1
+	- (i) Renforcement des tests : on s'assure de bien lire les configs de test (et non pas les configs "nominales")
+
+-------
 06/12/2021	v5.5.3-3.10.1
 	- (i) Bugfix pour php5 dans src/Shell/DbShell.php
 	- (i) update gitignore (ignorer config IRAP)
@@ -52,8 +52,8 @@ Logiciel testé et validé sur les configurations suivantes :
 --------------------------------------------------------------------------------------------
-Date: 06/12/2021	
-Version: v5.5.3-3.10.1
+Date: 09/12/2021	
+Version: v5.5.4-3.10.1
 HISTORIQUE DES CHANGEMENTS DE VERSION : voir le fichier CHANGES.txt (ou la page web /pages/changes)
+/LABOS/*
 /app.php.ORIG
 app_labinvent_mandatory_fields.php
 app_labinvent_mandatory_fields.yml
@@ -3,7 +3,7 @@
 MANDATORY_AND_READONLY_FIELDS:
     GENERAL: { 
-        fieldset_comment: Paramètres généraux,
+        fieldset_comment: Paramètres généraux _TEST_,
         HAS_ORDER_BUTTON: { selected: '1', comment: 'Ajouter un bouton COMMANDE sur la fiche matériel ?' } }
     UNEDITABLE_FIELDS: { 
@@ -153,11 +153,15 @@ class General extends TestCase {
         $this->assertTextEquals('TEST', $confLabinvent->labNameShort, 'On ne lit pas la bonne configuration');
         // On lit la config de test pour les champs materiels
+        // => moved to config/bootstrap avec _IN_TEST_MODE
         /*
         #Configure::config('my_yaml_engine', new YamlConfig());
         $filetoload = CORE_PATH . 'config' . DS . CONFIG_MATERIEL_FIELDS_FILE_NAME.'.test.yml';
         Configure::load($filetoload, 'my_yaml_engine');
         */
+        // On check la config pour voir si on lit bien la config de test
+        $general_fieldset_comment = Configure::read('MANDATORY_AND_READONLY_FIELDS.GENERAL.fieldset_comment');
+        $this->assertTextEquals('Paramètres généraux _TEST_', $general_fieldset_comment, 'On ne lit pas le bon fichier de configuration des champs materiels obligatoires');
     }
@@ -2076,13 +2076,18 @@ class MaterielsControllerTest extends General {
      * @return void
      */
     public function testMaterielPanne() {
+        $id=2;
         //debug($this->toto);
         //$this->assertTextEquals('titi', $this->toto);
+
         $this->authSuperAdmin();
-        $this->get('/materiels/view/2');
+        $m = $this->Materiels->get($id); // ou Cake\Datasource\Exception\RecordNotFoundException
+        $this->assertNull($m->hors_service);
+        
+        $this->get("/materiels/view/$id");
         $this->assertResponseNotContains("(HORS SERVICE)", "Le matériel est hors-service par défaut.");
-
+        
         $data = [
             'designation' => 'Test 6',
             'sur_categorie_id' => 1,
@@ -2098,11 +2103,17 @@ class MaterielsControllerTest extends General {
             'nom_modificateur' => 'Jean Administration',
             'nom_responsable' => 'Jacques Utilisateur',
             'email_responsable' => 'Jacques.Utilisateur@irap.omp.eu',
-            'hors_service' => 1
+            //'hors_service' => 1
+            'hors_service' => true
         ];
-        $this->post('/materiels/edit/2', $data);
-        $this->get('/materiels/view/2');
+        $this->post("/materiels/edit/$id", $data);
+        
+        $m = $this->Materiels->get($id); // ou Cake\Datasource\Exception\RecordNotFoundException
+        //$this->assertEquals(2, $m->hors_service, "La modification du matériel ne s'est pas bien passée.");
+        $this->assertTrue($m->hors_service);
+        
+        $this->get("/materiels/view/$id");
         $this->assertResponseContains("(HORS SERVICE)", "Le matériel n'est pas hors-service comme demandé.");
     }