Grosse amelioration des tests (refactorisation, généralisation)

Etienne Pallier
1 parent d3e31702
Showing 26 changed files with 651 additions and 295 deletions Show diff stats
src/Controller/AppController.php
src/Controller/CategoriesController.php
src/Controller/DocumentsController.php
src/Controller/EmpruntsController.php
src/Controller/FichemetrologiquesController.php
src/Controller/FormulesController.php
src/Controller/FournisseursController.php
src/Controller/GroupesMetiersController.php
src/Controller/GroupesThematiquesController.php
src/Controller/MaterielsController.php
src/Controller/OrganismesController.php
src/Controller/PagesController.php
src/Controller/SitesController.php
src/Controller/SousCategoriesController.php
src/Controller/SuivisController.php
src/Controller/SurCategoriesController.php
src/Controller/TypeDocumentsController.php
src/Controller/TypeSuivisController.php
src/Controller/UnitesController.php
tests/Fixture/ConfigurationsFixture.php
@@ -40,8 +40,9 @@ class AppController extends Controller {
     const PROFILE_ADMINPLUS = 4;
     const PROFILE_SUPERADMIN = 5;
-    private $allProfiles = [
-   		'Utilisateur' => self::PROFILE_USER,
+    //private $allProfiles = [
+    const PROFILES = [
+        'Utilisateur' => self::PROFILE_USER,
    		'Responsable' => self::PROFILE_RESPONSABLE,
    		'Administration' => self::PROFILE_ADMIN,
    		'Administration Plus' => self::PROFILE_ADMINPLUS,
@@ -49,6 +50,12 @@ class AppController extends Controller {
     ];
+    public static function getRoleLevel($role) {
+        //return $this->allProfiles[$role];  
+        //debug("role is" .$role);
+        return self::PROFILES[$role];
+    }
+    
     /**
      * Initialization hook method.
      * Use this method to add common initialization code like loading components.
@@ -96,43 +103,42 @@ class AppController extends Controller {
         // ACL : Par défaut refuser
         return false;
     }
-
-    function userHasRole($roleDefine) {
+        
+    public function userHasRole($expectedRole, $ORMORE=false) {
         $configuration = $this->confLabinvent;
         $role = TableRegistry::get('Users')->find()
         	->where(['username' => $this->LdapAuth->user($configuration->authentificationType_ldap)[0]])
+        	//->where(['username' => $this->LdapAuth->user('cn')[0]])
         	->first()['role'];
-        $isAuthorized = false;
-
-        switch ($roleDefine) {
+        	if (! $ORMORE) return ($role == $expectedRole);
+        	return ($this->getRoleLevel($role) >= $this->getRoleLevel($expectedRole));
+        	/*
+        //$hasRole = false;
+        	switch ($expectedRole) {
             case 'Super Administrateur' :
-                if (in_array($role, ['Super Administrateur']))
-                    $isAuthorized = true;
+                return (in_array($role, ['Super Administrateur']));
                 break;
-
             case 'Administration Plus' :
-                if (in_array($role, ['Administration Plus', 'Super Administrateur']))
-                    $isAuthorized = true;
+                return (in_array($role, ['Administration Plus', 'Super Administrateur']));
                 break;
-
             case 'Administration' :
-                if (in_array($role, ['Administration', 'Administration Plus', 'Super Administrateur' ]))
-                    $isAuthorized = true;
+                return (in_array($role, ['Administration', 'Administration Plus', 'Super Administrateur' ]));
                 break;
             case 'Responsable' :
-                if (in_array($role, ['Responsable', 'Administration', 'Administration Plus', 'Super Administrateur']))
-                    $isAuthorized = true;
+                return (in_array($role, ['Responsable', 'Administration', 'Administration Plus', 'Super Administrateur']));
                 break;
-
             case 'Utilisateur' :
-                if (in_array($role, ['Utilisateur', 'Responsable', 'Administration', 'Administration Plus', 'Super Administrateur']))
-                    $isAuthorized = true;
+                return (in_array($role, ['Utilisateur', 'Responsable', 'Administration', 'Administration Plus', 'Super Administrateur']));
                 break;
         }
-        return $isAuthorized;
+        return $false;
+        */
     }
-
+    public function userHasRoleAtLeast($expectedRole) {
+        return $this->userHasRole($expectedRole, true);
+    }
+        
     /**
      * {@inheritdoc}
      *
@@ -171,8 +177,9 @@ class AppController extends Controller {
     	$this->set('PROFILE_RESPONSABLE', self::PROFILE_RESPONSABLE);
        	$this->set('PROFILE_ADMINPLUS', self::PROFILE_ADMINPLUS);
        	$this->set('PROFILE_SUPERADMIN', self::PROFILE_SUPERADMIN);
-       	$this->set('allProfiles', $this->allProfiles);
-    	
+       	//$this->set('allProfiles', $this->allProfiles);
+       	$this->set('allProfiles', self::PROFILES);
+       	
     	if (!array_key_exists('_serialize', $this->viewVars) && in_array($this->response->type(), ['application/json', 'application/xml' ]))
             $this->set('_serialize', true);
@@ -191,7 +198,8 @@ class AppController extends Controller {
             $role = 'Utilisateur';
         $this->set('role', $role);
-        $profile = $this->allProfiles["$role"];
+        //$profile = $this->allProfiles["$role"];
+        $profile = self::PROFILES["$role"];
         $this->set('profile', $profile);
         $USER_IS_UTILISATEUR = ($profile == self::PROFILE_USER);
@@ -40,7 +40,7 @@ class CategoriesController extends AppController
 			return true;
 		}
-		if($this->userHasRole('Administration Plus')) {
+		if($this->userHasRoleAtLeast('Administration Plus')) {
 			if($action != 'delete') return true;
 		}
@@ -34,7 +34,7 @@ class DocumentsController extends AppController {
 			'admission',
 			'sortie'
 		])) {
-			if ($this->userHasRole('Administration')) {
+			if ($this->userHasRoleAtLeast('Administration')) {
 				return true;
 			}
 		}
@@ -43,7 +43,7 @@ class DocumentsController extends AppController {
 			'delete',
 			'edit'
 		])) {
-			if ($this->userHasRole('Administration')) {
+			if ($this->userHasRoleAtLeast('Administration')) {
 				return true;
 			}
 			$u = TableRegistry::get('Users')->find()->where([
@@ -30,7 +30,7 @@ class EmpruntsController extends AppController
 		$action = $this->request->getAttribute('params')['action'];
-		if($this->userHasRole('Responsable')) return true;
+		if($this->userHasRoleAtLeast('Responsable')) return true;
 		//Pour un "utilisateur"
 		if (in_array($action, ['edit', 'delete'])) {
@@ -29,8 +29,8 @@ class FichemetrologiquesController extends AppController
 		$action = $this->request->getAttribute('params')['action'];
-		if($this->userHasRole('Administration')) return true;
-		if($this->userHasRole('Responsable') && $action == 'fichemetro') return true;
+		if($this->userHasRoleAtLeast('Administration')) return true;
+		if($this->userHasRoleAtLeast('Responsable') && $action == 'fichemetro') return true;
 		//Pour un "utilisateur"
 		if (in_array($action, ['edit', 'delete'])) {
 			$id = (int)$this->request->getAttribute('params')['pass'][0];
 <?php
 namespace App\Controller;
-use App\Controller\AppController;
 use Cake\ORM\TableRegistry;
 /**
@@ -26,7 +25,7 @@ class FormulesController extends AppController
 		$action = $this->request->getAttribute('params')['action'];
-		if($this->userHasRole('Administration')) return true;
+		if($this->userHasRoleAtLeast('Administration')) return true;
 		//Pour un "utilisateur"
 		if (in_array($action, ['edit', 'delete'])) {
@@ -34,7 +34,7 @@ class FournisseursController extends AppController
 			return true;
 		}
-		if($this->userHasRole('Administration Plus')) {
+		if($this->userHasRoleAtLeast('Administration Plus')) {
 			if($action != 'delete') return true;
 		}
@@ -35,7 +35,7 @@ class GroupesMetiersController extends AppController
 			return true;
 		}
-		if($this->userHasRole('Administration Plus')) {
+		if($this->userHasRoleAtLeast('Administration Plus')) {
 			if($action != 'delete') return true;
 		}
@@ -36,7 +36,7 @@ class GroupesThematiquesController extends AppController
 			return true;
 		}
-		if($this->userHasRole('Administration Plus')) {
+		if($this->userHasRoleAtLeast('Administration Plus')) {
 			if($action != 'delete') return true;
 		}
@@ -17,7 +17,7 @@ class MaterielsController extends AppController {
 	const TOBEARCHIVED = 3;
 	const ARCHIVED = 4;
-	private $allStatus = [
+	const allStatus = [
 		 'CREATED' => self::CREATED,
 		 'VALIDATED' => self::VALIDATED,
 		 'TOBEARCHIVED' => self::TOBEARCHIVED,
@@ -102,7 +102,7 @@ class MaterielsController extends AppController {
 						return true;
 					} else if ($role == 'Responsable' && $this->isRespGroup($id, $user[$configuration->authentificationType_ldap][0])) {
 						return true;
-					} else if ($this->userHasRole('Administration')) {
+					} else if ($this->userHasRoleAtLeast('Administration')) {
 						return true;
 					}
 					*/
@@ -119,7 +119,7 @@ class MaterielsController extends AppController {
 							break;
 					}
 				}
-				if ($this->userHasRole('Administration Plus')) {
+				if ($this->userHasRoleAtLeast('Administration Plus')) {
 					return true;
 				}
 				break;
@@ -142,7 +142,7 @@ class MaterielsController extends AppController {
 						return true;
 					} else if ($role == 'Responsable' && $this->isRespGroup($id, $user[$configuration->authentificationType_ldap][0])) {
 						return true;
-					} else if ($this->userHasRole('Administration')) {
+					} else if ($this->userHasRoleAtLeast('Administration')) {
 						return true;
 					}
 					*/
@@ -164,7 +164,7 @@ class MaterielsController extends AppController {
 			case 'statusCreated': // de-validation d'un materiel (repasse à CREATED)
 			    $id = (int) $this->request->getAttribute('params')['pass'][0];
 			    if ($role == 'Responsable' && $this->isRespGroup($id, $user[$configuration->authentificationType_ldap][0])) return true;
-			    else if ($this->userHasRole('Administration')) return true;
+			    else if ($this->userHasRoleAtLeast('Administration')) return true;
 			    break;
 			case 'statusValidated':
@@ -173,7 +173,7 @@ class MaterielsController extends AppController {
 					if ($this->isCreated($id)) {
 						return true;
 					}
-				} else if ($this->userHasRole('Administration')) {
+				} else if ($this->userHasRoleAtLeast('Administration')) {
 					if ($this->isCreated($id)) {
 						return true;
 					}
@@ -186,7 +186,7 @@ class MaterielsController extends AppController {
 					if ($this->isValidated($id)) {
 						return true;
 					}
-				} else if ($this->userHasRole('Administration')) {
+				} else if ($this->userHasRoleAtLeast('Administration')) {
 					if ($this->isValidated($id)) {
 						return true;
 					}
@@ -195,7 +195,7 @@ class MaterielsController extends AppController {
 			case 'statusArchived':
 				$id = (int) $this->request->getAttribute('params')['pass'][0];
-				if ($this->userHasRole('Administration')) {
+				if ($this->userHasRoleAtLeast('Administration')) {
 					if ($this->isToBeArchived($id)) {
 						if ($role == 'Super Administrateur') {
 							return false;
@@ -207,7 +207,7 @@ class MaterielsController extends AppController {
 				break;
 			case 'export':
-				if ($this->userHasRole('Responsable')) {
+				if ($this->userHasRoleAtLeast('Responsable')) {
 					return true;
 				}
 				break;
@@ -226,7 +226,7 @@ class MaterielsController extends AppController {
 			case 'setLabelIsPlacedOrNotPlaced':
 			case 'setLabelIsNotPlaced':
 			case 'printLabelRuban':
-				if ($this->userHasRole('Administration')) {
+				if ($this->userHasRoleAtLeast('Administration')) {
 					return true;
 				}
 				break;
@@ -432,7 +432,8 @@ class MaterielsController extends AppController {
 		$IS_VALIDATED = ( $materiel->status == 'VALIDATED' );
 		$IS_TOBEARCHIVED = ( $materiel->status == 'TOBEARCHIVED' );
 		$IS_ARCHIVED = ( $materiel->status == 'ARCHIVED' );
-		$status = $this->allStatus[$materiel->status];
+		//$status = $this->allStatus[$materiel->status];
+		$status = self::allStatus[$materiel->status];
 		$this->set('IS_CREATED', $IS_CREATED);
 		$this->set('IS_VALIDATED', $IS_VALIDATED);
 		$this->set('IS_TOBEARCHIVED', $IS_TOBEARCHIVED);
@@ -39,7 +39,7 @@ class OrganismesController extends AppController
 			return true;
 		}
-		if($this->userHasRole('Administration Plus')) {
+		if($this->userHasRoleAtLeast('Administration Plus')) {
 			if($action != 'delete') return true;
 		}
@@ -28,7 +28,37 @@ use Cake\ORM\TableRegistry;
  */
 class PagesController extends AppController
 {
-
+    
+    /**
+     * @param $user
+     * @return boolean
+     *
+     * Give authorization for materiels
+     */
+    /*
+    // (EP) TODO: ameliorer ca avec des variables globales IS_VALIDATED, IS_ADMIN, ...
+    public function isAuthorized($user) {
+        $path = func_get_args();
+        if($path[0] === null) {
+            $path[0] = '';
+        }
+        $page = $subpage = null;
+        if (!empty($path[0])) {
+            $page = $path[0];
+        }
+        if (!empty($path[1])) {
+            $subpage = $path[1];
+        }
+        if ($page == 'tools') {
+            // Autoriser seulement à partir du role ADMIN et +
+            if ($this->userHasRoleAtLeast('Utilisateur')) {
+                return false;
+            }
+        }
+        return true;
+    }
+    */
+    
     /**
      * Displays a view
      *
@@ -39,8 +69,7 @@ class PagesController extends AppController
     public function display()
     {
-        
-    	$configuration = $this->confLabinvent;
+        $configuration = $this->confLabinvent;
         $path = func_get_args();
         if($path[0] === null) {
@@ -48,13 +77,12 @@ class PagesController extends AppController
         }
         $this->myDebug($path);
-    	//Si l'utilisateur n'est pas connecté, on le redirige vers la page login.ctp
-    	//sauf si l'action demandé est 'about' ou si le mode install est activé
-    	if (!($this->LdapAuth->user($configuration->authentificationType_ldap)[0]) && $path[0] != 'about' && !($configuration->mode_install)) {
-    		return $this->redirect(['controller' => 'users', 'action' => 'login']);
-    	}
-    	
-
+        	//Si l'utilisateur n'est pas connecté, on le redirige vers la page login.ctp
+        	//sauf si l'action demandée est 'about' ou si le mode install est activé
+        	if (!($this->LdapAuth->user($configuration->authentificationType_ldap)[0]) && $path[0] != 'about' && !($configuration->mode_install)) {
+        		return $this->redirect(['controller' => 'users', 'action' => 'login']);
+        	}
+        	
         $count = count($path);
         if (!$count) {
             return $this->redirect('/');
@@ -67,6 +95,14 @@ class PagesController extends AppController
         if (!empty($path[1])) {
             $subpage = $path[1];
         }
+
+        if ($page == 'tools') {
+            // Autoriser seulement à partir du role ADMIN et +
+            if (! $this->userHasRoleAtLeast('Administration')) {
+                return $this->redirect('/');
+            }
+        }
+        
         $this->set(compact('page', 'subpage'));
         try {
@@ -35,7 +35,7 @@ class SitesController extends AppController
 			return true;
 		}
-		if($this->userHasRole('Administration Plus')) {
+		if($this->userHasRoleAtLeast('Administration Plus')) {
 			if($action != 'delete') return true;
 		}
@@ -38,7 +38,7 @@ class SousCategoriesController extends AppController
 			return true;
 		}
-		if($this->userHasRole('Administration Plus')) {
+		if($this->userHasRoleAtLeast('Administration Plus')) {
 			if($action != 'delete') return true;
 		}
@@ -25,7 +25,7 @@ class SuivisController extends AppController {
         ])
             ->first()['role'];
         $action = $this->request->getAttribute('params')['action'];
-        if ($this->userHasRole('Administration'))
+        if ($this->userHasRoleAtLeast('Administration'))
             return true;
         // Pour un "utilisateur"
         if (in_array($action, [
@@ -39,7 +39,7 @@ class SurCategoriesController extends AppController
 			return true;
 		}
-		if($this->userHasRole('Administration Plus')) {
+		if($this->userHasRoleAtLeast('Administration Plus')) {
 			if($action != 'delete') return true;
 		}
@@ -35,7 +35,7 @@ class TypeDocumentsController extends AppController
 			return true;
 		}
-		if($this->userHasRole('Administration Plus')) {
+		if($this->userHasRoleAtLeast('Administration Plus')) {
 			if($action != 'delete') return true;
 		}
@@ -35,7 +35,7 @@ class TypeSuivisController extends AppController
 			return true;
 		}
-		if($this->userHasRole('Administration Plus')) {
+		if($this->userHasRoleAtLeast('Administration Plus')) {
 			if($action != 'delete') return true;
 		}
@@ -34,7 +34,7 @@ class UnitesController extends AppController
 			return true;
 		}
-		if($this->userHasRole('Administration Plus')) {
+		if($this->userHasRoleAtLeast('Administration Plus')) {
 			if($action != 'delete') return true;
 		}
@@ -52,7 +52,9 @@ class ConfigurationsFixture extends TestFixture
     	'prix_inventaire_administratif' => ['type' => 'integer', 'length' => 10, 'null' => true, 'default' => 800, 'comment' => '', 'precision' => null],
     	'taille_max_doc' => ['type' => 'integer', 'length' => 15, 'null' => true, 'default' => 8000000, 'comment' => '', 'precision' => null],
     	'metrologie' => [ 'type' => 'boolean' , 'length' => null, 'null' => true, 'default' => null, 'comment' => '', 'precision' => null],
-		'aff_par_defaut' => ['type' => 'integer', 'length' => 6, 'null' => false, 'default' => 20, 'comment' => '', 'precision' => null],
+    'aff_par_defaut' => ['type' => 'integer', 'length' => 6, 'null' => false, 'default' => 20, 'comment' => '', 'precision' => null],
+    'procedure_sur_accueil' => [ 'type' => 'boolean' , 'length' => null, 'null' => true, 'default' => null, 'comment' => '', 'precision' => null],
+        
     	'_constraints' => [
             'primary' => ['type' => 'primary', 'columns' => ['id'], 'length' => []],
             'nom_UNIQUE' => ['type' => 'unique', 'columns' => ['nom'], 'length' => []],
@@ -102,7 +104,10 @@ class ConfigurationsFixture extends TestFixture
         	'emailGuest10' => 'Lorem ipsum dolor sit amet',
         	'test' => 1,
         	'prix_inventaire_administratif' => 800,
-			'metrologie' => 1,
+		
+        'metrologie' => 1,
+        'procedure_sur_accueil' => 1,
+            
         ],
     ];
 }
@@ -559,6 +559,7 @@ class MaterielsFixture extends TestFixture {
         [
             'id' => 1,
             'designation' => 'matos 1 USER',
+            //'designation' => 'Test 1',
             'sur_categorie_id' => 1,
             'categorie_id' => 1,
             'sous_categorie_id' => 1,
@@ -57,9 +57,11 @@ class UsersFixture extends TestFixture
             'id' => 1,
             //'nom' => 'test1 test2',
             'nom' => 'user1 SUPER',
+            //'username' => 'testa',
             'username' => 'user1_SUPER',
             'password' => '$2y$10$VtYdA8Evkc.K.VpvqmF9wui5hc9ep19f8ukWBeFBIlunXSHPqw.K2',
-            'email' => 'testa@test.fr',
+            //'email' => 'testa@test.fr',
+            'email' => 'user1_SUPER@test.fr',
             'role' => 'Super Administrateur',
             'groupes_metier_id' => 1,
             	'groupe_thematique_id' => 1
@@ -2,9 +2,11 @@
 namespace App\Test\TestCase\Controller;
 use Cake\TestSuite\IntegrationTestCase;
+use App\Controller\AppController;
+//use Cake\ORM\TableRegistry;
+
 //use App\Controller\MaterielsController;
 //use Cake\ORM\TableRegistry;
-//use phpDocumentor\Reflection\Types\Self_;
 /**
  * App\Controller\GeneralController Test Case
@@ -13,11 +15,13 @@ class General extends IntegrationTestCase {
     //TODO: definine this in a superclass
+    /*
     const PROFILE_USER = 1;
     const PROFILE_RESPONSABLE = 2;
     const PROFILE_ADMIN = 3;
     const PROFILE_ADMINPLUS = 4;
     const PROFILE_SUPERADMIN = 5;
+    */
     protected $ROLES = [
         'USER',
         'RESP',
@@ -25,11 +29,100 @@ class General extends IntegrationTestCase {
         'ADMINP',
         'SUPER'
     ];
+    
+    const PROFILES = AppController::PROFILES;
+    
+    public static function getRoleLevel($role) { return AppController::getRoleLevel($role); }
+    
+
+    /*
+    public $ControllerApp = null;
+    public function setUp() {
+        parent::setUp();
+        
+        /*
+        $config = TableRegistry::exists('Materiels') ? [] : [
+            'className' => 'App\Model\Table\MaterielsTable'
+        ];
+        $this->Materiels = TableRegistry::get('Materiels', $config);
+        
+        $config = TableRegistry::exists('Suivis') ? [] : [
+            'className' => 'App\Model\Table\SuivisTable'
+        ];
+        $this->Suivis = TableRegistry::get('Suivis', $config);
+        */ /*
+        $this->ControllerApp = new AppController();
+    }
+    
+    public function tearDown() {
+        //unset($this->Materiels);
+        //unset($this->Suivis);
+        unset($this->ControllerApp);
+        
+        parent::tearDown();
+    }
+    */
-    /* FONCTIONS UTILITAIRES UTILISÉES PAR LES TESTS */
+    
+    /* FONCTIONS UTILITAIRES UTILISÉES PAR (tous) LES TESTS */
+
+    public function USER_IS_ADMIN_AT_LEAST($role) {
+        return in_array($role, ['ADMIN','ADMINP','SUPER']);
+    }
+    
+    public function authAs($role) {
+        switch ($role) {
+            case 'USER': $this->authUtilisateur();
+            break;
+            case 'RESP': $this->authResponsable();
+            break;
+            case 'ADMIN': $this->authAdmin();
+            break;
+            case 'ADMINP': $this->authAdminPlus();
+            break;
+            case 'SUPER': $this->authSuperAdmin();
+            break;
+        }
+    }
+    
+    public function authUser($cn, $givenName='test1', $sn='test2') {
+        $user = [
+            'Auth' => [
+                'User' => [
+                    'sn' => [
+                        0 => $sn
+                    ],
+                    'mail' => [
+                        0 => 'testa@test.fr'
+                    ],
+                    'givenname' => [
+                        0 => $givenName
+                    ],
+                    'cn' => [
+                        0 => $cn
+                    ],
+                    'userpassword' => [
+                        0 => 'test'
+                    ]
+                ]
+            ]
+        ];
+        $this->session($user);
+        
+        $authType = [
+            'authType' => 'cn'
+        ];
+        $this->session($authType);
+    }
     public function authSuperAdmin() {
+        $this->authUser('user1_SUPER', 'user1', 'SUPER');
+        //$this->authUser('user1_SUPER', 'test1', 'test2');
+        //$this->authUser('testa', 'user1', 'SUPER');
+    }
+    /*
+    public function authSuperAdmin() {
         $user = [
             'Auth' => [
                 'User' => [
@@ -59,9 +152,13 @@ class General extends IntegrationTestCase {
         ];
         $this->session($authType);
     }
+    */
     public function authAdminPlus() {
-        $user = [
+        $this->authUser('user2_ADMINPLUS', 'test3', 'test4');
+    }
+    /*
+     $user = [
             'Auth' => [
                 'User' => [
                     'sn' => [
@@ -90,8 +187,12 @@ class General extends IntegrationTestCase {
         ];
         $this->session($authType);
     }
+    */
     public function authAdmin() {
+        $this->authUser('user3_ADMIN', 'test5', 'test6');
+    }
+    /*
         $user = [
             'Auth' => [
                 'User' => [
@@ -121,8 +222,12 @@ class General extends IntegrationTestCase {
         ];
         $this->session($authType);
     }
+    */
     public function authResponsable() {
+        $this->authUser('user4_RESP', 'test7', 'test8');
+    }
+    /*
         $user = [
             'Auth' => [
                 'User' => [
@@ -152,9 +257,13 @@ class General extends IntegrationTestCase {
         ];
         $this->session($authType);
     }
+    */
     public function authUtilisateur() {
-        $user = [
+        $this->authUser('user5_USER', 'test9', 'test0');
+    }
+    /*
+     $user = [
             'Auth' => [
                 'User' => [
                     'sn' => [
@@ -183,5 +292,30 @@ class General extends IntegrationTestCase {
         ];
         $this->session($authType);
     }
+    */
+    
+    //protected function assertResponseContainsIf($role, $condition, $content, $messageIfNot=null) {
+    //protected function assertResponseContainsIf(string $role, bool $condition, array $contents, $testOpposite=true) {
+    protected function assertResponseContainsIf($role, $condition, array $contents, $testOpposite=true) {
+        if ($condition) {
+            $messageOnFail = 'devrait avoir accès';
+            $assertMethod = 'assertResponseContains'; 
+        }
+        else {
+            if (!$testOpposite) return;
+            $messageOnFail = 'ne devrait PAS avoir accès';
+            $assertMethod = 'assertResponseNotContains';
+        }
+        foreach ($contents as $content=>$endmsg)
+            $this->$assertMethod($content, 'Le profil (rôle) '.$role.' '.$messageOnFail.' '.$endmsg);
+        /*
+        $this->assertResponseContains($content, 'Le profil (rôle) '.$role.' devrait avoir accès à '. $messageIfNot);
+        }
+        else {
+            $this->assertResponseNotContains($content, 'Le profil (rôle) '.$role.' ne devrait PAS avoir accès à '. $messageIfNot);
+        }
+        */
+    }
+    
 }
 \ No newline at end of file
@@ -13,6 +13,7 @@ use Cake\ORM\TableRegistry;
 class MaterielsControllerTest extends General {
 //class MaterielsControllerTest extends IntegrationTestCase {
+    
     /**
      * Fixtures
      *
@@ -39,13 +40,44 @@ class MaterielsControllerTest extends General {
     ];
+    /*
     private $statuses = [
         'CREATED',
         'VALIDATED',
         'TOBEARCHIVED',
         'ARCHIVED'
     ];
+    */
+    private $STATUSES = MaterielsController::allStatus;
+ 
+    const mandatoryFieldsForCreation = [
+        //'id',
+        'designation',
+        'sur_categorie_id',
+        'categorie_id',
+        //'materiel_administratif', 'materiel_technique',
+        //'status' => 'CREATED',
+        'date_acquisition'
+    ];
+    
+    private $newMaterielWithAllMandatoryFields = [
+        //'id' => 16,
+        'designation' => 'Test 15',
+        'sur_categorie_id' => 1,
+        'categorie_id' => 1,
+        'materiel_administratif' => 0,
+        'materiel_technique' => 1,
+        //'status' => 'CREATED',
+        'date_acquisition' => '19-04-2016',
+        /*
+         'nom_responsable' => 'Jacques Utilisateur',
+         'email_responsable' => 'Jacques.Utilisateur@irap.omp.eu'
+         'nom_createur' => 'Pallier Etienne',
+         'nom_modificateur' => 'Jean Administration',
+         */
+    ];
+        
     /* FONCTIONS UTILITAIRES UTILISÉES PAR LES TESTS */
@@ -201,7 +233,9 @@ class MaterielsControllerTest extends General {
     }
 */
+
+ 
     /**
      * setUp method
      *
@@ -239,92 +273,95 @@ class MaterielsControllerTest extends General {
     /*
-     * Tests organisés par ACTION
+     * Tests organisés par (CONTROLEUR puis par) ACTION
+     *  
+     * Ici, on teste les ACTIONS du controleur MaterielsController
+     * Voir https://docs.google.com/document/d/1-OhEeoi96j6ueUl5NQCQ9ZsTfbJTFw3ZVaWU2iYly_o/edit#heading=h.bxuswhw2zzwt
      * 
-     * Chaque test doit tester l'accès à cette ACTION pour chaque ROLE
+     * Chaque test d'une ACTION doit tester l'appel de cette action pour chaque ROLE
      * 
-     * Voir la liste des ACTIONS (à tester dans le même ordre ici) : 
-     * https://docs.google.com/document/d/1-OhEeoi96j6ueUl5NQCQ9ZsTfbJTFw3ZVaWU2iYly_o/edit#heading=h.u1v9mlc9wtqp
-     */
-    
-    /*
-     * ************ CONTROLEUR PAGES *************
-     */
-    
-    /*
-     * ACTION "pages/about" : aller à la page "A propos"
-     * +
-     * ACTION "pages/home" : aller à l'Accueil
-     * (avec ou sans connexion)
-     */
-    
-    /*
-     * ACTION "pages/tools" : aller aux Outils
-     */
-
-    
-    /*
-     * ************ CONTROLEUR USERS *************
-     */
-    
-    /*
-     * ACTION "users/login" : se connecter
-     * +
-     * ACTION "users/logout" : se déconnecter
-     * (avec ou sans connexion)
-     */
-
-
-    /*
-     * ************ CONTROLEUR MATERIELS *************
-     * Voir 
-     * https://docs.google.com/document/d/1-OhEeoi96j6ueUl5NQCQ9ZsTfbJTFw3ZVaWU2iYly_o/edit#heading=h.bxuswhw2zzwt
      */
-
-    /*
-     * I - ACTION READ
-     * (1) READ ALL : "materiels/index" : Voir la liste des matériels
+    /* 
+     * *****************************************************************************
+     * ACTION READ
+     * (1) READ ALL (index) : Voir la liste des matériels
+     * *****************************************************************************
      */
-
-    public function testActionIndexGroup() {
-    	$this->testIndexNoAuth();
-    	$this->testIndex();	
-    }
     /**
-     * Test index method if no authentified
-     *
-     * @return void
-     */
-    public function testIndexNoAuth() {
-        $this->get('/materiels/index');
-// 		$this->assertRedirect('/users/login', 'Problème : Accès à materiels/index SANS AUTHENTIFICATION');
-// Le changement est dû au changement de version de cakephp 3.2 vers 3.4
-        $this->assertRedirect('/users/login?redirect=%2Fmateriels%2Findex', 'Problème : Accès à materiels/index SANS AUTHENTIFICATION');
-    }
-
-    /**
      * Test index method
      *
      * @return void
      */
-    public function testIndex() {
-        $this->authSuperAdmin();
-
-        // 1) On doit pouvoir accéder à la page une fois authentifié
+    public function testMat20ReadAll() { // test INDEX action
+        foreach ($this->ROLES as $role) $this->_testMatReadAllAs($role);
+    }
+                
+    private function _testMatReadAllAs($role)
+    {
+        $this->setUp();
+        
+        // On doit pouvoir accéder à la page une fois authentifié
+        $this->authAs($role);
         $this->get('/materiels/index');
         $this->assertNoRedirect("Authentifié mais redirection vers /users/login.");
-        // 2) On doit avoir 7 matériels
-        $this->assertResponseContains("Liste des matériels (7)", "Le nombre de materiels renvoyé est incorrect.");
+        // Seul admin+ peut voir les materiels archivés et a accès à des filtres par statut + bouton exporter + cases à cocher
+        //if ( in_array($role, ['ADMIN','ADMINP','SUPER']) ) {
+        if ($this->USER_IS_ADMIN_AT_LEAST($role)) { 
+            $this->assertResponseContains('Liste des matériels (7)', 'Le profil '.$role.' devrait voir les matériels archivés.');
+            $this->assertResponseContains("A valider", 'Le profil '.$role.' devrait avoir accès à des filtres par statut.');
+            $this->assertResponseContains("A sortir", 'Le profil '.$role.' devrait avoir accès à des filtres par statut.');
+        }
+        else { 
+            $this->assertResponseContains("Liste des matériels (6)", 'Le profil '.$role.' ne devrait PAS voir les matériels archivés.');
+            $this->assertResponseNotContains("A valider", 'Le profil '.$role.' ne devrait PAS avoir accès à des filtres par statut.');
+        }
+        $this->assertResponseContainsIf($role, ($role != 'USER'), ["Exporter la liste complete"=>"un bouton Exporter"]);
+        
+        $this->get('/materiels/index/CREATED');
+        //TODO: il faudrait remplacer "false" par "true" dans ce test
+        $this->assertResponseContainsIf(
+            $role, 
+            //in_array($role,['ADMIN','ADMINP','SUPER']), 
+            $this->USER_IS_ADMIN_AT_LEAST($role),
+            [
+                "checkbox" => "à des checkboxes",
+                "Exporter la liste des matériels cochés" => "au bouton d\'exportation de liste",
+                "Valider les matériels cochés" => "au bouton de validation de liste"
+            ],
+            false
+        );
+        /*
+        if ( in_array($role, ['ADMIN','ADMINP','SUPER']) ) {
+            $this->assertResponseContains('checkbox', 'Le profil '.$role.' devrait avoir accès à des checkboxes');
+            $this->assertResponseContains("Exporter la liste des matériels cochés", 'Le profil '.$role.' devrait avoir accès au bouton d\'exportation de liste');
+            $this->assertResponseContains("Valider les matériels cochés", 'Le profil '.$role.' devrait avoir accès au bouton de validation de liste');
+        }
+        else {
+        }
+        */
+        
+        $this->tearDown();
     }
-
+    public function testMat21ReadAllAsAnonymous() { // test INDEX action
+        // On ne doit pas avoir accès sans authentification
+        $this->get('/materiels/index');
+        // 		$this->assertRedirect('/users/login', 'Problème : Accès à materiels/index SANS AUTHENTIFICATION');
+        // Le changement est dû au changement de version de cakephp 3.2 vers 3.4
+        $this->assertRedirect('/users/login?redirect=%2Fmateriels%2Findex', 'Problème : Accès à materiels/index SANS AUTHENTIFICATION');
+    }
+    
+    
+    
     /*
-     * I - ACTION READ
-     * (2) READ ONE : "materiels/view/id" : Voir le détail d'un matériel
+     * *****************************************************************************
+     * ACTION READ
+     * (2) READ ONE (view/id) : Voir le détail d'un matériel
+     * *****************************************************************************
      */
     /**
@@ -333,58 +370,155 @@ class MaterielsControllerTest extends General {
      * @group failing
      * @return void
      */
-    public function testView() {
-        $this->authSuperAdmin();
+    public function testMat10ReadOne() { // test VIEW action
+        foreach ($this->ROLES as $role) $this->_testMatReadOneAs($role);
+    }
+    private function _testMatReadOneAs($role)
+    {
+        $this->setUp();
+        //$this->authSuperAdmin();
+        $this->authAs($role);
         $this->get('/materiels/view/3');
         $this->assertResponseContains("Test 3", "Le matériel retourné n'est pas celui demandé.");
+        $this->assertResponseContains('alt="QrCode', "Le QRCode n'est pas sur la vue matériel.");
         $this->assertResponseContains("Suivi(s) du matériel (1)", "Le nb de suivis liés au matériel est incorrect.");
         $this->assertResponseContains("Emprunt(s) du matériel (1)", "Le nb d'emprunts liés au matériel est incorrect.");
         $this->assertResponseContains("Fichier(s) lié(s) au matériel (1)", "Le nb de fichiers liés au matériel est incorrect.");
+        // Only admin+ see admin section:
+        //if ( in_array($role, ['ADMIN','ADMINP','SUPER']) ) {
+        if ($this->USER_IS_ADMIN_AT_LEAST($role)) {
+            $this->assertResponseContains("Informations administratives");
+            $this->assertResponseContains("CentreFinancier/EOTP");
+        }
+        else $this->assertResponseNotContains("Informations administratives");
+        
+        $this->tearDown();
+        
     }
-
+    
+    
     /*
-     * II - ACTION CREATE
-     * ACTION "materiels/add" : Créer un matériel
+     * *****************************************************************************
+     * ACTION CREATE (add) : Créer un matériel
+     * *****************************************************************************
      */
     /**
-     * Test add method
+     * Test testMat30AccessCreateForm
      *
      * @return void
      */
-    public function testAdd() {
-        $this->authSuperAdmin();
-        
-        $data = [
-            'id' => 15,
-            'designation' => 'Test 15',
-            'sur_categorie_id' => 1,
-            'categorie_id' => 1,
-            'materiel_administratif' => 0,
-            'materiel_technique' => 1,
-            'status' => 'CREATED',
-            'date_acquisition' => '19-04-2016',
-            'nom_createur' => 'Pallier Etienne',
-            'nom_modificateur' => 'Jean Administration',
-            'nom_responsable' => 'Jacques Utilisateur',
-            'email_responsable' => 'Jacques.Utilisateur@irap.omp.eu'
-        ];
-        $this->post('/materiels/add', $data);
-        
-        $this->get('/materiels/index');
-        $this->assertResponseContains("Liste des matériels (8)", "Le matériel ne s'ajoute pas correctement.");
+    public function testMat30AccessCreateForm() {
+        foreach ($this->ROLES as $role) $this->_testMatAccessCreateFormAs($role);
+    }
+    
+    private function _testMatAccessCreateFormAs($role)
+    {
+        $this->setUp();
-        $this->assertResponseContains("Test 15", "Le matériel ne s'ajoute pas correctement.");
-        $this->assertResponseContains("TEST-2016-0015", "La génération du n°de labo n'est pas bonne.");
+        // On doit pouvoir accéder à la page une fois authentifié
+        $this->authAs($role);
+        $this->get('/materiels/add');
+        $this->assertResponseContains('Ajouter', 'La page n\'existe pas');
+        $this->assertResponseContainsIf(
+            $role, 
+            //in_array($role,['ADMIN', 'ADMINP', 'SUPER']), 
+            $this->USER_IS_ADMIN_AT_LEAST($role),
+            ["EOTP" => "à la partie administrative sur le formulaire add"]
+        );
+        /*
+        $this->assertResponseNotContains('EOTP', 'Le profil utilisateur a accès à la partie administrative sur le formulaire add.');
+        $this->assertResponseContains('EOTP', 'Le profil admin+ n\'a pas accès à la partie administrative sur le formulaire add.');
+        */
+    
+        $this->tearDown();
     }
+
     /**
-     * Test valueNeccessaryNotEmpty method
+     * Test testMat31Create
      *
      * @return void
      */
+    public function testMat31CreateAsSuper() { $this->_testMatCreateAs('SUPER'); }
+    public function testMat31CreateAsAdminp() { $this->_testMatCreateAs('ADMINP'); }
+    public function testMat31CreateAsAdmin() { $this->_testMatCreateAs('ADMIN'); }
+    public function testMat31CreateAsResp() { $this->_testMatCreateAs('RESP'); }
+    public function testMat31CreateAsUser() { $this->_testMatCreateAs('USER'); }
+    
+    public function testMat32CreateAdministratifOrTechnicalAsSuper() { $this->_testMatCreateAdministratifOrTechnicalAs('SUPER'); }
+    private function _testMatCreateAdministratifOrTechnicalAs($role) {
+        $newMateriel = $this->newMaterielWithAllMandatoryFields;
+        $fields = ['materiel_administratif','materiel_technique'];
+        // test with materiel_administratif and materiel_technique, all combinations from (0,0) to (1,1)
+        //for ($i=0,$j=0; $i<=1,$j<=1 ; $i++,$j++) {
+        for ($i=0; $i<=1 ; $i++) {
+            for ($j=0; $j<=1 ; $j++) {
+                $newMateriel["$fields[0]"] = $i;
+                $newMateriel["$fields[1]"] = $j;
+                $combination = [$i,$j];
+                if ($combination == [0,0]) $this->_testMatCreate1FailsAs($role, $newMateriel, implode(',',$combination));
+                if ($combination == [0,1]) {
+                    // cree un nouveau materiel
+                    $this->_testMatCreateAs($role, $newMateriel, implode(',',$combination));
+                    // supprimer un materiel pour avoir toujours le meme nombre
+                    //TODO: impossible de supprimer le matos 1, why ???
+                    //$this->post('/materiels/delete/1');
+                    $this->post('/materiels/delete/11');
+                    $this->get('/materiels/index');
+                    $nbmat = $this->USER_IS_ADMIN_AT_LEAST($role) ? 7 : 6;
+                    $this->assertResponseContains("Liste des matériels (".$nbmat.")", $role.','.implode($combination));
+                }
+                // Administratif : (1,0) et (1,1) : ok si prix > 800
+                // No need to test the case [1,1] because same as [1,0]
+                if ($combination == [1,0]) {
+                //if ($i == 1) {
+                     
+                     // prix null => fails
+                     $this->_testMatCreate1FailsAs($role, $newMateriel, implode(',',$combination));
+                     
+                     // prix < 800 => fails
+                     $newMateriel['prix_ht'] = 799;
+                     $this->_testMatCreate1FailsAs($role, $newMateriel, implode(',',$combination));
+                     
+                     // prix >= 800 => ok
+                     $newMateriel['prix_ht'] = 800;
+                     $this->_testMatCreateAs($role, $newMateriel, implode(',',$combination));
+                     /*
+                     // supprimer un materiel pour avoir toujours le meme nombre
+                     //TODO: impossible de supprimer le matos 1, why ???
+                     //$this->post('/materiels/delete/1');
+                     $this->post('/materiels/delete/2');
+                     $this->get('/materiels/index');
+                     $nbmat = $this->USER_IS_ADMIN_AT_LEAST($role) ? 7 : 6;
+                     $this->assertResponseContains("Liste des matériels (".$nbmat.")", $role.','.implode($combination));
+                     */
+                 }
+            }
+        }
+    }
+    
+    private function _testMatCreateAs($role, $materiel=null)
+    {
+        $materiel = $materiel ? $this->newMaterielWithAllMandatoryFields : $materiel;
+        //$this->setUp();
+        // On doit pouvoir accéder à la page une fois authentifié
+        $this->authAs($role);
+        $this->get('/materiels/index');
+        $nbmat = $this->USER_IS_ADMIN_AT_LEAST($role) ? 7 : 6;
+        $this->assertResponseContains("Liste des matériels (".$nbmat.")", $role);
+        $this->post('/materiels/add', $this->newMaterielWithAllMandatoryFields);
+        $this->get('/materiels/index');
+        $nbmat++;
+        $this->assertResponseContains("Liste des matériels (".$nbmat.")", $role." Le matériel ne s'ajoute pas correctement");
+        $this->assertResponseContains("Test 15", "Le matériel ne s'ajoute pas correctement.");
+        $this->assertResponseContains("TEST-2016-0015", "La génération du n°de labo n'est pas bonne.");
+        //$this->tearDown();
+    }
+    
+    /*    
     public function testValueNeccessaryNotEmpty() {
         $this->authSuperAdmin();
@@ -407,8 +541,61 @@ class MaterielsControllerTest extends General {
         $this->get('/materiels/index');
         $this->assertResponseContains("Liste des matériels (7)", "Le matériel s'ajoute alors que les champs obligatoires ne sont pas rempli.");
     }
+    */
+    
+    /**
+     * Test testMat32CreateFails
+     *
+     * @return void
+     */
+    public function testMat33CreateFailsAsSuper() { $this->_testMatCreateFailsAs('SUPER'); }
+    public function testMat33CreateFailsAsUser() { $this->_testMatCreateFailsAs('USER'); }
+    private function _testMatCreateFailsAs($role) {
+        // test with each mandatory field except materiel_administratif and materiel_technique
+        foreach (self::mandatoryFieldsForCreation as $mandatoryField) {
+            $newMaterielWithMissingMandatoryFields = $this->newMaterielWithAllMandatoryFields;
+            $newMaterielWithMissingMandatoryFields[$mandatoryField] = null;
+            $this->_testMatCreate1FailsAs($role, $newMaterielWithMissingMandatoryFields, $mandatoryField); 
+        }
+        // test with missing materiel_administratif AND materiel_technique null (equivalent to 0,0)
+        $newMaterielWithMissingMandatoryFields = $this->newMaterielWithAllMandatoryFields;
+        $fields = ['materiel_administratif','materiel_technique'];
+        foreach ($fields as $f) $newMaterielWithMissingMandatoryFields["$f"] = null;
+        $this->_testMatCreate1FailsAs($role, $newMaterielWithMissingMandatoryFields, implode(',',$fields));
+    }
+    private function _testMatCreate1FailsAs($role, $newMaterielWithMissingMandatoryFields, $mandatoryField) {
+        $this->setUp();
+        
+        //$newMaterielWithMissingMandatoryFields = $this->newMaterielWithAllMandatoryFields;
+        //$newMaterielWithMissingMandatoryFields['sur_categorie_id'] = null;
+        /* Mandatory fields :
+        'id' => 15,
+        'designation' => 'Test 15',
+        'sur_categorie_id' => 1,
+        'categorie_id' => 1,
+        'materiel_administratif' => 0,
+        'materiel_technique' => 1,
+        //'status' => 'CREATED',
+        'date_acquisition' => '19-04-2016',
+        */
+        //$this->setUp();
+        // On doit pouvoir accéder à la page une fois authentifié
+        $this->authAs($role);
+        $this->get('/materiels/index');
+        $nbmat = $this->USER_IS_ADMIN_AT_LEAST($role) ? 7 : 6;
+        $this->assertResponseContains("Liste des matériels (".$nbmat.")", $role.','.$mandatoryField);
+        $this->post('/materiels/add', $newMaterielWithMissingMandatoryFields);
+        $this->get('/materiels/index');
+        $this->assertResponseContains("Liste des matériels (".$nbmat.")", $role." Le matériel s'ajoute alors que le champ obligatoire ".$mandatoryField." n'est pas rempli");
+        
+        $this->tearDown();
+    }
+    
+    
+
+    
     /*
@@ -422,19 +609,8 @@ class MaterielsControllerTest extends General {
      */
     public function testUpdate() { // ACTION 'edit'
-        foreach ($this->ROLES as $role) {
-            switch ($role) {
-                case 'USER': $this->authUtilisateur();
-                    break;
-                case 'RESP': $this->authResponsable();
-                   break;
-                case 'ADMIN': $this->authAdmin();
-                   break;
-                case 'ADMINP': $this->authAdminPlus();
-                   break;
-                case 'SUPER': $this->authSuperAdmin();
-                   break;
-            }
+        foreach ($this->ROLES as $role) { 
+            $this->authAs($role);
             if ($role=='USER') continue;
             $this->_testUpdates($role);
         } // foreach
@@ -542,10 +718,13 @@ class MaterielsControllerTest extends General {
     public function testDelete() {
         $this->authSuperAdmin();
+        //TODO: impossible de supprimer le matos 1, why ???
+        //$this->post('/materiels/delete/1');
         $this->post('/materiels/delete/2');
         $this->get('/materiels/index');
         $this->assertResponseContains("Liste des matériels (6)", "Le matériel n'as pas été supprimé.");
         $this->assertResponseNotContains("Test 2", "Le matériel n'as pas été supprimé.");
+        //$this->assertResponseNotContains("matos 1 USER", "Le matériel n'as pas été supprimé.");
     }
     /*
@@ -886,102 +1065,8 @@ class MaterielsControllerTest extends General {
         $this->assertResponseNotContains('ARCHIVED', "La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
     }
-    /**
-     * Test PresenceQrCodeMateriel
-     *
-     * @return void
-     */
-    public function testPresenceQrCodeMateriel() {
-        $this->authSuperAdmin();
-
-        $this->get('/materiels/view/12');
-
-        $this->assertResponseContains('alt="QrCode', "Le QRCode n'est pas sur la vue matériel.");
-    }
-
-    /**
-     * Test ACLViewUtilisateur
-     *
-     * @return void
-     */
-    public function testACLViewUtilisateur() {
-        $this->authUtilisateur();
-
-        $this->get('/materiels/view/12');
-
-        $this->assertResponseContains('Test 12', 'Le profil utilisateur ne peut pas lire la view correctement.');
-
-        $this->assertResponseNotContains('EOTP', 'Le profil utilisateur a accès à la partie administrative sur la view.');
-    }
-
-    /**
-     * Test ACLViewAdmin
-     *
-     * @return void
-     */
-    public function testACLViewAdmin() {
-        $this->authAdmin();
-
-        $this->get('/materiels/view/12');
-
-        $this->assertResponseContains('Test 12', 'Le profil admin+ ne peut pas lire la view correctement.');
-
-        $this->assertResponseContains('EOTP', 'Le profil admin+ n\'a pas accès à la partie administrative sur la view.');
-    }
-
-    /**
-     * Test ACLIndexUtilisateur
-     *
-     * @return void
-     */
-    public function testACLIndexUtilisateur() {
-        $this->authUtilisateur();
-
-        $this->get('/materiels/index');
-
-        $this->assertResponseContains('Liste des matériels (6)', 'Le profil utilisateur a accès aux matériels archivés.');
-    }
-
-    /**
-     * Test ACLIndexAdmin
-     *
-     * @return void
-     */
-    public function testACLIndexAdmin() {
-        $this->authAdmin();
-
-        $this->get('/materiels/index');
-
-        $this->assertResponseContains('Liste des matériels (7)', 'Le profil admin n\'a pas accès aux matériels archivés.');
-    }
-
-    /**
-     * Test ACLAddUtilisateur
-     *
-     * @return void
-     */
-    public function testACLAddUtilisateur() {
-        $this->authUtilisateur();
-
-        $this->get('/materiels/add');
-
-        $this->assertResponseContains('Ajouter', 'La page n\'existe pas');
-        $this->assertResponseNotContains('EOTP', 'Le profil utilisateur a accès à la partie administrative sur le formulaire add.');
-    }
-
-    /**
-     * Test ACLAddAdmin
-     *
-     * @return void
-     */
-    public function testACLAddAdmin() {
-        $this->authAdmin();
-
-        $this->get('/materiels/add');
-
-        $this->assertResponseContains('EOTP', 'Le profil admin+ n\'a pas accès à la partie administrative sur le formulaire add.');
-    }
+ 
     /**
      * Test ACLEditUtilisateur
      *
@@ -29,7 +29,8 @@ class PagesControllerTest extends General
 	 * @var array
 	 */
 	public $fixtures = [
-			'app.configurations',
+	    'app.users',
+	    'app.configurations',
 	];
 	 /*
@@ -59,17 +60,69 @@ class PagesControllerTest extends General
      *
      * @return void
      */
-    public function testDisplay()
-    {
-        //$this->authUser();
-        $this->authSuperAdmin();
-        
-        $this->get('/pages/home');
-        $this->assertResponseOk();
-        $this->assertResponseContains('Accueil');
-    }
+	public function testPage10AccessHome() {
+	    foreach ($this->ROLES as $role) $this->subtestPageAccessHomeAs($role);
+	}
+	//public function testDisplay()
+	private function subtestPageAccessHomeAs($role)
+	{
+	    //$this->authUser();
+	    $this->authAs($role);
+	    
+	    $this->get('/pages/home');
+	    $this->assertResponseOk();
+	    $this->assertResponseContains('Accueil');
+	    $this->assertResponseContains('Voir mes matériels');
+        switch ($role) {
+            case 'USER':
+                $this->assertResponseContains('Utilisateur');
+                //$configuration->procedure_sur_accueil = TRUE;
+                $this->assertResponseContains('Procédure à suivre');
+            break;
+            case 'RESP':
+            break;
+            case 'ADMIN':
+                $this->assertResponseContains('Administration');
+                $this->assertResponseContains('Voir les matériels à valider');
+            break;
+            case 'ADMINP':
+            break;
+            case 'SUPER':
+            	    $this->assertResponseContains('Super Administrateur');
+            break;
+        }
+	        
+	}
+
+	
+	public function testPage20AccessTools() {
+	    foreach ($this->ROLES as $role) $this->subtestPageAccessToolsAs($role);
+	}
+	//public function testDisplay()
+	private function subtestPageAccessToolsAs($role)
+	{
+	    $this->authAs($role);
+	    $this->get('/pages/tools');
+	    
+	    if ( in_array($role, ['ADMIN','ADMINP','SUPER']) ) {
+	    //if ( $this->userHasRoleAtLeast('Administration') ) {
+	    //if ( $this->ControllerApp->userHasRoleAtLeast('Administration') ) {
+	       $this->assertResponseOk();
+	       $this->assertResponseContains('Outils', $role.' should be allowed to access the Tools page');
+	       $this->assertResponseContains('Gérer le contenu variable de ', $role);
+	       if ($role == 'SUPER') 
+        	       $this->assertResponseContains('Passer en mode DEBUG');
+	       else
+	           $this->assertResponseNotContains('Passer en mode DEBUG');
+	    }
+	    else {
+	        $this->assertResponseNotContains('Outils', $role.' should not be allowed to access the Tools page');
+	    }
+	}
+	
+    
     /**
      * Test that missing template renders 404 page in production
      *
@@ -54,30 +54,62 @@ class UsersControllerTest extends General
     */
+    
+    /**
+     * Test UNauthentified access to website
+     * 
+     * By default, must have ONLY access to /about page, or /login page  
+     *
+     * @return void
+     */
+    public function testUser10AccessAsAnonymous() {
+        $this->get('/pages/about');
+        $this->assertResponseContains('A PROPOS DU LOGICIEL', 'Un utilisateur NON authentifié devrait avoir accès à la page /about');
+        $this->get('/');
+        $this->assertRedirect('/users/login', 'Un utilisateur NON authentifié devrait etre redirigé sur la page de login');
+    }
+    
     /**
      * Test login method
      *
      * @return void
      */
-    public function testLogin()
+    public function testUser20LoginAsSuper()
     {
     	$this->post('/users/login', ['ldap' => 'user1_SUPER', 'password' => 'test']);
-        $this->assertRedirect('/pages/home', 'La connection de l\'utilisateur ne fonctionne pas correctement');
+        $this->assertRedirect('/pages/home', 'Un utilisateur privilégié devrait pouvoir se loguer');
     }
-
+    public function testUser21LoginAsUser()
+    {
+        $this->post('/users/login', ['ldap' => 'user5_USER', 'password' => 'test']);
+        $this->assertRedirect('/pages/home', 'Un utilisateur non privilégié devrait pouvoir se loguer');
+    }
+    
     /**
      * Test logout method
+     * TODO: ca marche pas, impossible de simuler un vrai logout, l'utilisateur est toujours connecté !!!
      *
      * @return void
      */
-    public function testLogout()
+    public function TODOtestUser30Logout()
     {
         //$this->authUser();
         $this->authSuperAdmin();
-        
-        $this->post('/users/logout');
-        $this->assertRedirect('/pages/home', 'La déconnection de l\'utilisateur ne fonctionne pas correctement');
+        $this->get('/pages/home');
+        //$this->assertSession(1, 'Auth.User.id'); // Check user is logged in
+        $this->assertSession('user1_SUPER', 'Auth.User.cn.0'); // Check user is logged in
+        $this->assertSession(['SUPER'], 'Auth.User.sn'); // Check user is logged in
+
+        $this->get('/users/logout');
+        $this->assertSession(null, 'Auth.User'); // Check user is logged out
+        //unset($this->session);
+        //$this->assertRedirect('/pages/home', 'Un utilisateur devrait pouvoir se déloguer');
+        //$this->get('/pages/home');
+        $this->assertRedirect('/users/login', 'Un utilisateur devrait pouvoir se déloguer et ainsi revenir sur la page login');
+        //$this->assertRedirect(['controller' => 'Users', 'action' => 'login']);
+        $this->get('/materiels');
+        $this->assertResponseNotContains('Liste des', 'On devrait normalement etre dé-logué !!! or on ne l\'est pas');
     }
     /**