Bugfix pas mal de tests...

v5.2.10-3.7.9

Bugfix pas mal de tests...
v5.2.10-3.7.9
Etienne Pallier
1 parent a87adffc
Showing 6 changed files with 80 additions and 30 deletions Show diff stats
CHANGELOG
README.md
src/Controller/MaterielsController.php
tests/Fixture/MaterielsFixture.php
tests/TestCase/Controller/MaterielsControllerTest.php
tests/TestCase/Controller/PagesControllerTest.php
@@ -352,6 +352,23 @@ Commencer à implémenter le nouveau workflow v5 :
 ======= TODO =======
+	*) TESTS !!!!
+	
+	*) Validation multiple ne marche plus
+	
+	*) Les tests doivent se faire avec la config par défaut (et non pas celle spécifique du labo)
+	
+	*) Ajouter test Commande (avec devis joint !)
+	
+	*)	
+	TBO ou ARCHIVED (= validated) : ne pas autoriser edit ou delete des docs attachés
+	Autoriser certaines choses pour TBO :
+	- edit (mais pas delete)
+	- print label
+	- add doc attaché
+	- (pas emprunt ni suivi)
+	 
+	
 	*)
 	champ "budgets" à rendre optionnel dans schéma BD...
@@ -360,9 +377,7 @@ Commencer à implémenter le nouveau workflow v5 :
 	Facture jointe (si > 10K)
-	*)
-	TESTS !!!!
-	
+
 	*)
@@ -399,6 +414,10 @@ Commencer à implémenter le nouveau workflow v5 :
 ======= CHANGES =======
 -------
+14/09/2021	v5.2.10-3.7.9
+	- Bugfix pas mal de tests...
+	
+-------
 14/09/2021	v5.2.9-3.7.9
 	- Bugfix Ajout materiel
@@ -53,7 +53,7 @@ Logiciel testé et validé sur les configurations suivantes :
 --------------------------------------------------------------------------------------------
 Date: 14/09/2021	
-Version: v5.2.9-3.7.9
+Version: v5.2.10-3.7.9
@@ -3060,7 +3060,7 @@ class MaterielsController extends AppController {
             // Si au moins un champ obligatoire est nul ou vide => ERROR
             foreach ($mandatoryFields as $fname => $fnicename) {
-                // Champs obligatoires à ignorer : Champs virtuels ou ne pouvant être exigés si tôt
+                // Champs obligatoires à ignorer : Champs virtuels ou alors ne pouvant être exigés si tôt
                 if ( in_array($fname, ['DEVIS', 'etiquette', 'numero_inventaire_organisme']) ) continue;
                 $fval = $materiel->$fname;
@@ -3078,6 +3078,7 @@ class MaterielsController extends AppController {
                          * de bootstrap, ou bootsrap-ui, ou cakephp..., à surveiller donc)
                          */ 
                         $msgError1 = "Pour valider un matériel, le champ suivant ne doit pas être vide : ".$fnicename.' du matériel';
+                        debug($msgError1);
                         $this->Flash->error($msgError1);
                         /* MARCHE PAS POURQUOI ?
                         $materiel->setError($field, 'Ce champ ne doit pas être vide');
@@ -881,7 +881,8 @@ class MaterielsFixture extends TestFixture {
             'organisme_id' => 1,
             'site_id' => 1, // Roche
             'metrologie' => 0,
-            'fournisseur_id' => 1
+            'fournisseur_id' => 1,
+            'nom_user' => 'JP',
         ],
         // matos VALIDATED owned by anybody
@@ -163,6 +163,7 @@ class MaterielsControllerTest extends General {
         'gestionnaire_id' => 3, // user id 3 est un profil Administration
         'organisme_id' => 1,
         'eotp' => 'budget1, budget2',
+        'budgets' => 'bud1, bud2',
     ];
     /*
@@ -1864,8 +1865,14 @@ class MaterielsControllerTest extends General {
         $this->post('/materiels/status-validated/11');
         $this->get('/materiels/view/11');
+        /*
         $this->assertResponseNotContains('CREATED', "La validation du materiel ne se fait pas correctement (1)");
         $this->assertResponseContains('VALIDATED', "La validation du materiel ne se fait pas correctement (2)");
+        $this->assertResponseNotContains('CRÉÉ - à valider', "La validation du materiel ne se fait pas correctement (1)");
+        $this->assertResponseContains('VALIDÉ - livré & payé', "La validation du materiel ne se fait pas correctement (2)");
+        */
+        $this->assertResponseNotContains($this->Materiels->getNiceStatus('CREATED'), "La validation du materiel ne se fait pas correctement (1)");
+        $this->assertResponseContains($this->Materiels->getNiceStatus('VALIDATED'), "La validation du materiel ne se fait pas correctement (2)");
     }
     /**
@@ -1880,7 +1887,9 @@ class MaterielsControllerTest extends General {
         $this->post('/materiels/status-tobearchived/12');
         $this->get('/materiels/view/12');
-        $this->assertResponseContains('TOBEARCHIVED', "La demande d'archivage du materiel ne se fait pas correctement.");
+        //$this->assertResponseContains('TOBEARCHIVED', "La demande d'archivage du materiel ne se fait pas correctement.");
+        $this->assertResponseContains($this->Materiels->getNiceStatus('TOBEARCHIVED'), "La demande d'archivage du materiel ne se fait pas correctement");
+        
     }
     /**
@@ -1945,26 +1954,33 @@ class MaterielsControllerTest extends General {
             'what' => 'CREATED', 
             11 => '1', 
             12 => '1', 
-            13 => '1'
+            13 => '1',
         ]);
         $this->get('/materiels/view/11');
-        $this->assertResponseContains('VALIDATED', "(11) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
-        $this->assertResponseNotContains('CREATED', "(11) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
-        $this->assertResponseNotContains('TOBEARCHIVED', "(11) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
-        $this->assertResponseNotContains('ARCHIVED', "(11) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
+        
+        $statusC = $this->Materiels->getNiceStatus('CREATED');
+        $statusV = $this->Materiels->getNiceStatus('VALIDATED');
+        $statusTBA = $this->Materiels->getNiceStatus('TOBEARCHIVED');
+        $statusA = $this->Materiels->getNiceStatus('ARCHIVED');
+        
+        //$this->assertResponseContains('VALIDATED', "(11) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
+        $this->assertResponseContains($statusV, "(11) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
+        $this->assertResponseNotContains($statusC, "(11) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
+        $this->assertResponseNotContains($statusTBA, "(11) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
+        $this->assertResponseNotContains($statusA, "(11) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
         $this->get('/materiels/view/12');
-        $this->assertResponseContains('VALIDATED', "(12) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
-        $this->assertResponseNotContains('CREATED', "(12) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
-        $this->assertResponseNotContains('TOBEARCHIVED', "La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
-        $this->assertResponseNotContains('ARCHIVED', "La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
+        $this->assertResponseContains($statusV, "(12) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
+        $this->assertResponseNotContains($statusC, "(12) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
+        $this->assertResponseNotContains($statusTBA, "La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
+        $this->assertResponseNotContains($statusA, "La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
         $this->get('/materiels/view/13');
-        $this->assertResponseContains('VALIDATED', "(13) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
-        $this->assertResponseNotContains('CREATED', "(13) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
-        $this->assertResponseNotContains('TOBEARCHIVED', "La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
-        $this->assertResponseNotContains('ARCHIVED', "La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
+        $this->assertResponseContains($statusV, "(13) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
+        $this->assertResponseNotContains($statusC, "(13) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
+        $this->assertResponseNotContains($statusTBA, "La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
+        $this->assertResponseNotContains($statusA, "La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
     }
@@ -2007,15 +2023,16 @@ class MaterielsControllerTest extends General {
         $this->post('/materiels/status-validated/11');
         $this->get('/materiels/view/11');
-        $this->assertResponseContains('CREATED', "La validation du materiel se fait avec un profil utilisateur.");
-
+        //$this->assertResponseContains('CREATED', "La validation du materiel se fait avec un profil utilisateur.");
+        $this->assertResponseContains($this->Materiels->getNiceStatus('CREATED'), "La validation du materiel se fait avec un profil utilisateur.");
+        
         $this->post('/materiels/status-tobearchived/12');
         $this->get('/materiels/view/12');
-        $this->assertResponseContains('VALIDATED', "La demande d'archivage du materiel se fait avec un profil utilisateur.");
+        $this->assertResponseContains($this->Materiels->getNiceStatus('VALIDATED'), "La demande d'archivage du materiel se fait avec un profil utilisateur.");
         $this->post('/materiels/status-archived/13');
         $this->get('/materiels/view/13');
-        $this->assertResponseContains('TOBEARCHIVED', "L'archivage du materiel se fait avec un profil utilisateur.");
+        $this->assertResponseContains($this->Materiels->getNiceStatus('TOBEARCHIVED'), "L'archivage du materiel se fait avec un profil utilisateur.");
     }
     /**
@@ -2028,15 +2045,15 @@ class MaterielsControllerTest extends General {
         $this->post('/materiels/status-validated/11');
         $this->get('/materiels/view/11');
-        $this->assertResponseContains('VALIDATED', "La validation du materiel ne se fait pas correctement avec un profil admin+.");
+        $this->assertResponseContains($this->Materiels->getNiceStatus('VALIDATED'), "La validation du materiel ne se fait pas correctement avec un profil admin+.");
         $this->post('/materiels/status-tobearchived/12');
         $this->get('/materiels/view/12');
-        $this->assertResponseContains('TOBEARCHIVED', "La demande d'archivage du materiel ne se fait pas correctement avec un profil admin+.");
+        $this->assertResponseContains($this->Materiels->getNiceStatus('TOBEARCHIVED'), "La demande d'archivage du materiel ne se fait pas correctement avec un profil admin+.");
         $this->post('/materiels/status-archived/13');
         $this->get('/materiels/view/13');
-        $this->assertResponseContains('ARCHIVED', "L'archivage du materiel ne se fait pas correctement avec un profil admin+.");
+        $this->assertResponseContains($this->Materiels->getNiceStatus('ARCHIVED'), "L'archivage du materiel ne se fait pas correctement avec un profil admin+.");
     }
     /**
@@ -2438,7 +2455,15 @@ class MaterielsControllerTest extends General {
             'edit' => 'Éditer', // 'Editer ce matériel'
             'delete' => 'Supprimer',
             'add_by_copy' => 'Copier',
-            'statusCreated' => 'Dévalider',
+            
+            /* Problème : cette action change de nom selon le statut actuel du materiel
+             * Ca peut etre "Invalider", ou "Annuler Sortie", ou "Désarchiver"...
+             * Donc, on utilise plutot le nom interne de l'action dans le code html : status-created 
+             */
+            //'statusCreated' => 'Dévalider',
+            //'statusCreated' => 'Invalider',
+            'statusCreated' => 'status-created',
+            
             'statusValidated' => 'Valider',
             'statusTobearchived' => "Demander sortie",
             'statusArchived' => 'Sortie inventaire',
@@ -2614,7 +2639,8 @@ class MaterielsControllerTest extends General {
                 $expected_flash_message .= 'supprimé';
                 break;
             case 'statusCreated': 
-                $expected_flash_message .= 'rétrogradé au statut CREATED'; 
+                //$expected_flash_message .= 'rétrogradé au statut CREATED';
+                $expected_flash_message .= "rétrogradé au statut 'CRÉÉ'"; 
                 break;
             case 'statusValidated': 
                 $expected_flash_message .= 'validé'; 
@@ -74,6 +74,7 @@ class PagesControllerTest extends General
 	public function testPage10AccessHomeAsAdmin() { $this->_testPageAccessHomeAs('ADMIN'); }
 	//public function testPage10AccessHomeAsAdminPlus() { $this->_testPageAccessHomeAs('ADMINP'); }
 	public function testPage10AccessHomeAsSuperAdmin() { $this->_testPageAccessHomeAs('SUPER'); }
+	
 	private function _testPageAccessHomeAs($role) {
 	    //$this->authUser();
 	    $this->authAs($role);
@@ -92,7 +93,9 @@ class PagesControllerTest extends General
             case 'RESP': break;
             case 'ADMIN':
                 $this->assertResponseContains('Administration');
-                $this->assertResponseContains('Voir les matériels à valider');
+                //$this->assertResponseContains('Statistiques');
+                //$this->assertResponseContains('Voir mes matériels');
+                $this->assertResponseContains('Voir les matériels à Valider');
             break;
             //case 'ADMINP': break;
             case 'SUPER':