Etienne Pallier / labinvent

Commit ef8cb4cebb4810e0101ffb0a968ff82c8828d6df

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

GROS Bugfix tests : un problème très ancien résolu today !!! 

Dans certains cas, les tests utilisaient la BD de prod au lieu de la BD
de test, c'est maintenant terminé, non mais !

v4.105.19-3.7.9
Showing 12 changed files with 75 additions and 13 deletions   Show diff stats
Inline Side-by-side
CHANGES.txt
  Diff comments   View file @ef8cb4c
... ... @@ -120,6 +120,12 @@ Outre ces changements, voici d'autres changements importants :
120 120 ======= CHANGES =======
121 121  
122 122 -------
  123 +13/10/2020 v4.105.19-3.7.9 (EP)
  124 + - (b) GROS Bugfix tests : un problème très ancien résolu today !!!
  125 + Dans certains cas, les tests utilisaient la BD de prod au lieu de la BD de test,
  126 + c'est maintenant terminé, non mais !
  127 +
  128 +-------
123 129 12/10/2020 v4.105.18-3.7.9 (EP)
124 130 - (b) Bugfix tests + correction de parties de code qui généraient des erreurs NOTICE
125 131 - (b) Bugfix liste des projets, tri par colonnes
... ...
README.md
  Diff comments   View file @ef8cb4c
... ... @@ -42,8 +42,8 @@ Logiciel testé et validé sur les configurations suivantes :
42 42  
43 43 --------------------------------------------------------------------------------------------
44 44  
45   -Date: 12/10/2020
46   -Version: 4.105.18-3.7.9
  45 +Date: 13/10/2020
  46 +Version: 4.105.19-3.7.9
47 47  
48 48  
49 49 HISTORIQUE DES CHANGEMENTS DE VERSION : voir le fichier CHANGES.txt (ou la page web /pages/changes)
... ...
TESTS.sh
  Diff comments   View file @ef8cb4c
... ... @@ -20,7 +20,7 @@ sudo rm -rf tmp/cache/*
20 20  
21 21  
22 22 # D'abord on affiche la version courante
23   -./VERSION
  23 +#./VERSION
24 24  
25 25  
26 26 if [[ "$1" != "" ]] ; then
... ...
src/Model/Table/ConfigurationsTable.php
  Diff comments   View file @ef8cb4c
... ... @@ -24,7 +24,10 @@ class ConfigurationsTable extends AppTable
24 24 parent::initialize($config);
25 25  
26 26 $this->setTable('configurations');
27   - $this->setDisplayField('id');
  27 +
  28 + //$this->setDisplayField('id');
  29 + $this->setDisplayField('labNameShort');
  30 +
28 31 $this->setPrimaryKey('id');
29 32 }
30 33  
... ...
tests/TestCase/Controller/EmpruntsControllerTest.php
  Diff comments   View file @ef8cb4c
... ... @@ -18,20 +18,27 @@ class EmpruntsControllerTest extends General
18 18 * @var array
19 19 */
20 20 public $fixtures = [
  21 + 'app.configurations',
  22 +
21 23 'app.emprunts',
  24 +
22 25 'app.materiels',
  26 +
  27 + 'app.suivis',
  28 +
23 29 'app.sur_categories',
24 30 'app.categories',
25 31 'app.sous_categories',
  32 +
26 33 'app.groupes_thematiques',
27 34 'app.groupes_metiers',
  35 +
28 36 'app.fakeldapusers',
29 37 'app.users',
  38 +
30 39 'app.organismes',
31 40 'app.sites',
32 41 'app.documents',
33   - 'app.suivis',
34   - 'app.configurations',
35 42 'app.fournisseurs',
36 43 'app.unites'
37 44 ];
... ...
tests/TestCase/Controller/General.php
  Diff comments   View file @ef8cb4c
... ... @@ -123,6 +123,32 @@ class General extends TestCase {
123 123 }
124 124 */
125 125  
  126 + public function setUp() {
  127 +
  128 + // (EP202010)
  129 + // https://stackoverflow.com/questions/29748911/cakephp-connections-while-testing/45592461
  130 + /*
  131 + * Some tables tests were getting default db instead of test db for no reason.
  132 + * This solves the problem (we have a main test class from which we extend all tests, do some setting up. We put it there)
  133 + */
  134 + TableRegistry::clear();
  135 +
  136 + parent::setUp();
  137 + //$this->toto = 'titi';
  138 + // Check qu'on lit bien la table configurations de la BD de test !!
  139 + $config = TableRegistry::exists('Configurations') ? [] : [
  140 + 'className' => 'App\Model\Table\ConfigurationsTable'
  141 + ];
  142 + $confLabinvent = TableRegistry::getTableLocator()->get('Configurations', $config)->find()->first();
  143 + //assert($confLabinvent->ldap_authenticationType == 'uid', 'On ne lit pas la bonne configuration');
  144 + //debug($confLabinvent->ldap_authenticationType);
  145 + $this->assertTextEquals('uid', $confLabinvent->ldap_authenticationType, 'On ne lit pas la bonne configuration');
  146 + $this->assertTextNotEquals('samaccountname', $confLabinvent->ldap_authenticationType, 'On ne lit pas la bonne configuration');
  147 + $this->assertTextEquals(1, $confLabinvent->test, 'On ne lit pas la bonne configuration');
  148 + $this->assertTextEquals(800, $confLabinvent->prix_inventaire_administratif, 'On ne lit pas la bonne configuration');
  149 + $this->assertTextEquals('TEST', $confLabinvent->labNameShort, 'On ne lit pas la bonne configuration');
  150 + }
  151 +
126 152  
127 153 public function d($msg) {
128 154 if ($this->DEBUG) pr($msg);
... ...
tests/TestCase/Controller/MaterielsControllerTest.php
  Diff comments   View file @ef8cb4c
... ... @@ -57,10 +57,10 @@ class MaterielsControllerTest extends General {
57 57 * @var array
58 58 */
59 59 public $fixtures = [
60   - 'app.materiels',
61   -
62 60 'app.configurations',
63 61  
  62 + 'app.materiels',
  63 +
64 64 'app.sur_categories',
65 65 'app.categories',
66 66 'app.sous_categories',
... ... @@ -204,7 +204,9 @@ class MaterielsControllerTest extends General {
204 204 * @return void
205 205 */
206 206 public function setUp() {
  207 + //$this->toto = 'titi';
207 208 parent::setUp();
  209 +
208 210  
209 211 // Initialisation du tableau $this->is_authorized_action
210 212 ///$this->_setAuthorizations();
... ... @@ -213,7 +215,17 @@ class MaterielsControllerTest extends General {
213 215 'className' => 'App\Model\Table\MaterielsTable'
214 216 ];
215 217 $this->Materiels = TableRegistry::getTableLocator()->get('Materiels', $config);
216   -
  218 +
  219 + /* moved to class General
  220 + // Check qu'on lit bien la table configurations de la BD de test !!
  221 + $confLabinvent = TableRegistry::getTableLocator()->get('Configurations')->find()->first();
  222 + //assert($confLabinvent->ldap_authenticationType == 'uid', 'On ne lit pas la bonne configuration');
  223 + //debug($confLabinvent->ldap_authenticationType);
  224 + $this->assertTextEquals('uid', $confLabinvent->ldap_authenticationType, 'On ne lit pas la bonne configuration');
  225 + $this->assertTextNotEquals('samaccountname', $confLabinvent->ldap_authenticationType, 'On ne lit pas la bonne configuration');
  226 + $this->assertTextEquals('TEST', $confLabinvent->labNameShort, 'On ne lit pas la bonne configuration');
  227 + */
  228 +
217 229 /*
218 230 $config = TableRegistry::exists('Suivis') ? [] : [
219 231 'className' => 'App\Model\Table\SuivisTable'
... ... @@ -2008,6 +2020,8 @@ class MaterielsControllerTest extends General {
2008 2020 * @return void
2009 2021 */
2010 2022 public function testMaterielPanne() {
  2023 + //debug($this->toto);
  2024 + //$this->assertTextEquals('titi', $this->toto);
2011 2025 $this->authSuperAdmin();
2012 2026  
2013 2027 $this->get('/materiels/view/2');
... ...
tests/TestCase/Controller/PagesControllerTest.php
  Diff comments   View file @ef8cb4c
... ... @@ -29,9 +29,10 @@ class PagesControllerTest extends General
29 29 * @var array
30 30 */
31 31 public $fixtures = [
  32 + 'app.configurations',
  33 +
32 34 'app.fakeldapusers',
33 35 'app.users',
34   - 'app.configurations',
35 36 ];
36 37  
37 38 /*
... ...
tests/TestCase/Controller/ProjetsControllerTest.php
  Diff comments   View file @ef8cb4c
... ... @@ -18,6 +18,8 @@ class ProjetsControllerTest extends TestCase
18 18 * @var array
19 19 */
20 20 public $fixtures = [
  21 + 'app.configurations',
  22 +
21 23 /* Généré automatiquement par bake (majuscules) :
22 24 'app.Projets',
23 25 'app.GroupesThematiques',
... ...
tests/TestCase/Controller/SuivisControllerTest.php
  Diff comments   View file @ef8cb4c
... ... @@ -20,6 +20,8 @@ class SuivisControllerTest extends General
20 20 * @var array
21 21 */
22 22 public $fixtures = [
  23 + 'app.configurations',
  24 +
23 25 'app.suivis',
24 26 'app.materiels',
25 27 'app.sur_categories',
... ... @@ -33,7 +35,6 @@ class SuivisControllerTest extends General
33 35 'app.sites',
34 36 'app.documents',
35 37 'app.emprunts',
36   - 'app.configurations',
37 38 'app.type_suivis',
38 39 'app.type_documents',
39 40 'app.unites',
... ...
tests/TestCase/Controller/SurCategoriesControllerTest.php
  Diff comments   View file @ef8cb4c
... ... @@ -29,9 +29,10 @@ class SurCategoriesControllerTest extends General
29 29 * @var array
30 30 */
31 31 public $fixtures = [
  32 + 'app.configurations',
  33 +
32 34 'app.fakeldapusers',
33 35 'app.users',
34   - 'app.configurations',
35 36 'app.sur_categories',
36 37 'app.categories',
37 38 'app.sous_categories',
... ...
tests/TestCase/Controller/UsersControllerTest.php
  Diff comments   View file @ef8cb4c
... ... @@ -17,6 +17,8 @@ class UsersControllerTest extends General
17 17 * @var array
18 18 */
19 19 public $fixtures = [
  20 + 'app.configurations',
  21 +
20 22 'app.fakeldapusers',
21 23 'app.users',
22 24 'app.groupes_metiers',
... ... @@ -31,7 +33,6 @@ class UsersControllerTest extends General
31 33 'app.documents',
32 34 'app.suivis',
33 35 'app.type_suivis',
34   - 'app.configurations'
35 36 ];
36 37  
37 38 //@Implement
... ...