Commit 7d7b451fa72c86d84cd5c7510169f0456f22675a

Authored by Etienne Pallier
1 parent e659572f
Exists in master and in 1 other branch dev

Bugfix tests : suppression erreur Notice sur type documents (et suivis)

=> corrigée (fixtures)

v5.5.5-3.10.1
@@ -484,6 +484,12 @@ rien ? @@ -484,6 +484,12 @@ rien ?
484 ======= CHANGES ======= 484 ======= CHANGES =======
485 485
486 ------- 486 -------
  487 +10/12/2021 v5.5.5-3.10.1
  488 + - (i) Bugfix tests : erreur Notice sur le type des documents (et types suivis) => corrigée (fixtures)
  489 + => plus d'erreur Notice !!
  490 + => suite de tests enfin clean !
  491 +
  492 +-------
487 09/12/2021 v5.5.4-3.10.1 493 09/12/2021 v5.5.4-3.10.1
488 - (i) Renforcement des tests : on s'assure de bien lire les configs de test (et non pas les configs "nominales") 494 - (i) Renforcement des tests : on s'assure de bien lire les configs de test (et non pas les configs "nominales")
489 495
@@ -52,8 +52,8 @@ Logiciel testé et validé sur les configurations suivantes : @@ -52,8 +52,8 @@ Logiciel testé et validé sur les configurations suivantes :
52 52
53 -------------------------------------------------------------------------------------------- 53 --------------------------------------------------------------------------------------------
54 54
55 -Date: 09/12/2021  
56 -Version: v5.5.4-3.10.1 55 +Date: 10/12/2021
  56 +Version: v5.5.5-3.10.1
57 57
58 58
59 HISTORIQUE DES CHANGEMENTS DE VERSION : voir le fichier CHANGES.txt (ou la page web /pages/changes) 59 HISTORIQUE DES CHANGEMENTS DE VERSION : voir le fichier CHANGES.txt (ou la page web /pages/changes)
config/app.default.php
@@ -215,10 +215,15 @@ return [ @@ -215,10 +215,15 @@ return [
215 //'errorLevel' => E_ALL, 215 //'errorLevel' => E_ALL,
216 //'errorLevel' => E_ALL & ~E_DEPRECATED, 216 //'errorLevel' => E_ALL & ~E_DEPRECATED,
217 217
218 - // - Pour branche dev : sans les deprecated errors 218 + // - Pour branche dev (en mode development) : sans les deprecated errors
219 //'errorLevel' => E_ALL & ~E_USER_DEPRECATED, 219 //'errorLevel' => E_ALL & ~E_USER_DEPRECATED,
  220 +
  221 + // - Pour les TESTS afin d'eviter les messages deprecated et les warnings...
  222 + //'errorLevel' => E_ALL & ~E_USER_DEPRECATED & ~E_WARNING,
  223 + // - ... et aussi les erreurs Notice
  224 + //'errorLevel' => E_ALL & ~E_USER_DEPRECATED & ~E_WARNING & ~E_NOTICE,
220 225
221 - // - Pour branche master : Sans les deprecated ni les Warnings 226 + // - Pour branche master (en mode production) : Sans les deprecated ni les Warnings
222 'errorLevel' => E_ALL & ~E_USER_DEPRECATED & ~E_DEPRECATED & ~E_WARNING, 227 'errorLevel' => E_ALL & ~E_USER_DEPRECATED & ~E_DEPRECATED & ~E_WARNING,
223 228
224 // 3.7 229 // 3.7
tests/Fixture/DocumentsFixture.php
@@ -55,8 +55,8 @@ class DocumentsFixture extends TestFixture @@ -55,8 +55,8 @@ class DocumentsFixture extends TestFixture
55 //'id' => 1, 55 //'id' => 1,
56 'nom' => 'Lorem ipsum dolor sit amet', 56 'nom' => 'Lorem ipsum dolor sit amet',
57 'type_doc' => 'Lorem ipsum dolor ', 57 'type_doc' => 'Lorem ipsum dolor ',
58 - //'type_document_id' => 1  
59 - 'type_document_id' => null, 58 + //'type_document_id' => null,
  59 + 'type_document_id' => 1,
60 'materiel_id' => 3, 60 'materiel_id' => 3,
61 'suivi_id' => 1, 61 'suivi_id' => 1,
62 ], 62 ],
tests/Fixture/TypeDocumentsFixture.php
@@ -40,7 +40,35 @@ class TypeDocumentsFixture extends TestFixture @@ -40,7 +40,35 @@ class TypeDocumentsFixture extends TestFixture
40 public $records = [ 40 public $records = [
41 [ 41 [
42 //'id' => 1, 42 //'id' => 1,
43 - 'nom' => 'Lorem ipsum dolor sit amet' 43 + //'nom' => 'Lorem ipsum dolor sit amet'
  44 + 'nom' => 'DEVIS'
44 ], 45 ],
  46 + [
  47 + //'id' => 2,
  48 + 'nom' => 'BC (Bon de Commande)'
  49 + ],
  50 + [
  51 + //'id' => 3,
  52 + 'nom' => 'BL (Bon de Livraison)'
  53 + ],
  54 + [
  55 + //'id' => 4,
  56 + 'nom' => 'FACTURE'
  57 + ],
  58 + /*
  59 + ('DEVIS'),
  60 + ('BC (Bon de Commande)'),
  61 + ('BL (Bon de Livraison)'),
  62 + ('FACTURE'),
  63 + ('Photo'),
  64 + ('Documentation technique'),
  65 + ('Spécifications'),
  66 + ('CR de maintenance'),
  67 + ("CR d'étalonnage"),
  68 + ('Suivi de panne'),
  69 + ('Certificat de garantie'),
  70 + ('Certificat de conformité'),
  71 + ("Manuel d'utilisation");
  72 + */
45 ]; 73 ];
46 } 74 }
tests/Fixture/TypeSuivisFixture.php
@@ -40,7 +40,22 @@ class TypeSuivisFixture extends TestFixture @@ -40,7 +40,22 @@ class TypeSuivisFixture extends TestFixture
40 public $records = [ 40 public $records = [
41 [ 41 [
42 //'id' => 1, 42 //'id' => 1,
43 - 'nom' => 'Lorem ipsum dolor sit amet' 43 + //'nom' => 'Lorem ipsum dolor sit amet'
  44 + 'nom' => 'Etalonnage',
44 ], 45 ],
  46 + [
  47 + //'id' => 2,
  48 + 'nom' => 'Maintenance',
  49 + ],
  50 + [
  51 + //'id' => 3,
  52 + 'nom' => 'Réparation',
  53 + ],
  54 + /*
  55 + ('Etalonnage', NULL, 1, 0),
  56 + ('Maintenance', NULL, 0, 0),
  57 + ('Réparation', NULL, 0, 0),
  58 + ('Vérification métrologique', NULL, 1, 1);
  59 + */
45 ]; 60 ];
46 } 61 }
tests/TestCase/Controller/General.php
@@ -84,7 +84,7 @@ class General extends TestCase { @@ -84,7 +84,7 @@ class General extends TestCase {
84 ['USER'], 84 ['USER'],
85 ['RESP'], 85 ['RESP'],
86 ['ADMIN'], 86 ['ADMIN'],
87 - ['SUPER'] 87 + ['SUPER'],
88 ]; 88 ];
89 89
90 const PROFILES = AppController::PROFILES; 90 const PROFILES = AppController::PROFILES;
@@ -301,12 +301,12 @@ class General extends TestCase { @@ -301,12 +301,12 @@ class General extends TestCase {
301 ]; 301 ];
302 */ 302 */
303 $roles4 = [ 303 $roles4 = [
  304 + /*
  305 + */
304 'user', 306 'user',
305 'resp', 307 'resp',
306 'admin', 308 'admin',
307 'super', 309 'super',
308 - /*  
309 - */  
310 ]; 310 ];
311 //$actions = ['delete']; 311 //$actions = ['delete'];
312 //$actions = ['edit']; 312 //$actions = ['edit'];
tests/TestCase/Controller/MaterielsControllerTest.php
@@ -609,6 +609,8 @@ class MaterielsControllerTest extends General { @@ -609,6 +609,8 @@ class MaterielsControllerTest extends General {
609 */ 609 */
610 public function testMat10ReadOneAs($role) 610 public function testMat10ReadOneAs($role)
611 { 611 {
  612 + $id = 3;
  613 +
612 //$this->setUp(); 614 //$this->setUp();
613 615
614 //$this->authSuperAdmin(); 616 //$this->authSuperAdmin();
@@ -617,7 +619,12 @@ class MaterielsControllerTest extends General { @@ -617,7 +619,12 @@ class MaterielsControllerTest extends General {
617 $myrole = $this->getUserRole(); 619 $myrole = $this->getUserRole();
618 $this->assertEquals('Super Administrateur', $myrole); 620 $this->assertEquals('Super Administrateur', $myrole);
619 */ 621 */
620 - $this->get('/materiels/view/3'); 622 + /*
  623 + $m = $this->Materiels->get($id);
  624 + debug($m);
  625 + */
  626 +
  627 + $this->get("/materiels/view/$id");
621 $this->assertResponseContains("Test 3", "Le matériel retourné n'est pas celui demandé."); 628 $this->assertResponseContains("Test 3", "Le matériel retourné n'est pas celui demandé.");
622 $this->assertResponseContains('alt="QrCode', "Le QRCode n'est pas sur la vue matériel."); 629 $this->assertResponseContains('alt="QrCode', "Le QRCode n'est pas sur la vue matériel.");
623 $this->assertResponseContains("Suivi(s) du matériel (1)", "Le nb de suivis liés au matériel est incorrect."); 630 $this->assertResponseContains("Suivi(s) du matériel (1)", "Le nb de suivis liés au matériel est incorrect.");