bugfix test date en cours...

Etienne Pallier
1 parent e5ded94f
Showing 4 changed files with 95 additions and 18 deletions Show diff stats
CHANGES.txt
README.md
install/update.sh
tests/TestCase/Controller/MaterielsControllerTest.php
@@ -74,6 +74,11 @@ Outre ces changements, voici d&#39;autres changements importants :
 ======= CHANGES =======
  
 -------
+30/07/2020	v3.7.9.78 (EP)
+	- (i) Meilleur résilience (reprise après crash) du script UPDATE avec fichier ftpf (files_to_process.txt file)
+	=> permet de s'assurer que les script sql non exécutés (à cause crash ou stop script) le seront à l'itération suivante 
+	
+-------
 29/07/2020	v3.7.9.77 (EP)
 	- Bugfix Page "Statistiques"
 	- (e) Nouveau champ recherche générale des matériels sur page liste des matériels (/materiels/index) : plus intuitif, plus pratique, plus efficace 
@@ -42,8 +42,8 @@ Logiciel testé et validé sur les configurations suivantes :
  
 --------------------------------------------------------------------------------------------
  
-Date: 29/07/2020
-Version: 3.7.9.77
+Date: 30/07/2020
+Version: 3.7.9.78
  
  
 HISTORIQUE DES CHANGEMENTS DE VERSION : voir le fichier CHANGES.txt (ou la page web /pages/changes)
@@ -48,7 +48,11 @@ echo
  
 # TEST only
 #[[ $TEST == 1 ]] && (rm ../database/update/test.txt ; rm ../database/update/db-update-2014-*.sh ; touch ../database/update/db-update-2014-08-23.sh)
-[[ $TEST == 1 ]] && (rm $db_update_scripts_folder/test.txt ; rm $db_update_scripts_folder/db-update-2014-*.sql ; touch $db_update_scripts_folder/db-update-2014-08-23.sql)
+if [[ $TEST == 1 ]] ; then
+	echo "(TEST MODE)"
+	echo "Suppression des fichiers de test créés précédemment (si besoin)"
+	(rm $db_update_scripts_folder/test.txt ; rm $db_update_scripts_folder/db-update-2014-*.sql ; touch $db_update_scripts_folder/db-update-2014-08-23.sql)
+fi
 #mv ../database/update/db-update-2019-01-12.sh /tmp/ 
  
 # ----------------------------------------
@@ -61,7 +65,6 @@ cp -fp ../composer.json /tmp/ || abort
 ######[[ $TEST == 1 ]] && cp ../composer.lock /tmp/composer.json
 composer_json_prev=/tmp/composer.json
  
-# b) Sauvegarde du dossier database/update/
 # b) Sauvegarde du dossier database/update/script_sql/
 #db_update_scripts_before=( "key1" "key2" "key3" "key4" "key5" "key6" "key7" "key8" "key9" "key10" )
 rm -rf $db_update_scripts_folder_tmp/ || abort
@@ -176,7 +179,52 @@ db_update_scripts_new=$( echo ${db_update_scripts_before[@]} ${db_update_scripts
 #echo ${#db_update_scripts_new[@]} ; echo ${db_update_scripts_new[@]}
 # Pas de nouveau script de mise à jour BD à exécuter => exit
 #[[ $db_update_scripts_new == '' ]] && echo "==> PAS NÉCESSAIRE" ; exit 0
+
+# On ajoute les anciens fichiers (de l'itération précédente) restant à traiter
+echo "- (a) Fichiers restant à traiter de l'itération précédente"
+#cat $ftpf
+if [[ $TEST == 1 ]] ; then
+	echo "db-update-2014-08-23.sql" > $ftpf 
+	echo "db-update-2014-08-24.sql" >> $ftpf 
+	echo "db-update-2014-08-26.sql" >> $ftpf
+	db_update_scripts_new+=("db-update-2014-08-10.sql")
+fi
+db_update_scripts_old=$(cat $ftpf)
+#echo $db_update_scripts_old
+echo ${db_update_scripts_old[*]}
+
+echo "- (b) Nouveaux fichiers à traiter (pour cette itération)"
+#echo ${db_update_scripts_new[@]}
+echo ${db_update_scripts_new[*]}
+
+# Ensemble des fichiers à traiter (anciens || nouveaux) : anciens UNION nouveaux (fusion sans doublon)"
+echo "- (c) Ensemble des fichiers à traiter (a + b)"
+declare -a db_update_scripts
+#[[ " ${a[@]} " =~ " ${val} " ]] && echo "true" || echo "false
+# foreach old script, add it to array db_update_scripts (only if not duplicate)
+for db_update_script_old in ${db_update_scripts_old[*]} ; do
+	if [[ ! "${db_update_scripts_new[*]}" =~ "${db_update_script_old}" ]] ; then
+		db_update_scripts_new+=($db_update_script_old) || abort
+	fi
+done
+db_update_scripts_new_tmp=$( echo ${db_update_scripts_new[*]} | tr ' ' '\n' | sort ) || abort
+db_update_scripts_new=(${db_update_scripts_new_tmp[*]})
+#echo ${db_update_scripts_new[*]}
+
+
+# On sauve les noms des nouveaux fichiers à traiter dans le fichier ftpf (MERGE)
+# D'abord, on le vide
+cat /dev/null > $ftpf
+# Puis, on le remplit
+for db_update_script_new in ${db_update_scripts_new[*]} ; do
+	###echo $db_update_script_new >> $ftpf
+	echo ""
+done
+
+# Si ftpf est vide, rien à faire, on stoppe
 if [[ $db_update_scripts_new == '' ]] ; then 
+#if [ ! -s $ftpf ] ; then
+	#echo "Le fichier ftpf est vide"
 	echo "==> PAS NÉCESSAIRE"
 	echo
 	exit 0
@@ -187,8 +235,10 @@ fi
  
 # b) Mise a jour de la BD 
  
-echo "Voici le(s) script(s) à exécuter :" 
-echo ${db_update_scripts_new[@]}
+#echo "Voici le(s) script(s) à exécuter :" 
+#cat $ftpf
+#echo $db_update_scripts_new
+echo ${db_update_scripts_new[*]}
 # Remove files that were DELETED by "git pull"
 for db_update_script_new in ${db_update_scripts_new[@]} ; do
 #	[[ ! -f ../database/update/$db_update_script_new ]] && echo "$db_update_script_new sera ignoré car supprimé par le 'git pull'" # TODO: supprimer cet element du tableau
@@ -196,20 +246,32 @@ for db_update_script_new in ${db_update_scripts_new[@]} ; do
 done
 # On va dans le dossier database/update/ :
 #cd $db_update_scripts_folder/ || abort
+#echo "* Exécuter ces scripts SQL (o/n) ? [o]"
+#read do_it ; [[ -z $do_it ]] && do_it="o"
+#echo $do_it
+#[[ $do_it != "o" ]] && exit 0
+
+#echo "go on"
+#exit
+
 cd $db_update_scripts_folder/../ || abort
 for db_update_script_new in ${db_update_scripts_new[@]} ; do
 	# Ignorer les fichiers supprimés par le "git pull"
 	#[[ ! -f $db_update_script_new ]] && continue
-	[[ ! -f script_sql/$db_update_script_new ]] && continue
-	echo ; echo
-	#echo "- Execution du script de mise à jour de la BD $db_update_scripts_folder/$db_update_script_new (o/n) ? [o]"
-	echo "- Execution du script de mise à jour de la BD $db_update_scripts_folder/$db_update_script_new"
-	#read do_it ; [[ -z $do_it ]] && do_it="o" ; echo $do_it
-	#if [[ $do_it == "o" ]] ; then
-	#ls -l ./$db_update_script_new_ || abort
-	#./$db_update_script_new || abort
-	./db-update.sh script_sql/$db_update_script_new || abort
-	#fi
+	#[[ ! -f script_sql/$db_update_script_new ]] && continue
+	if [[ -f script_sql/$db_update_script_new ]] ; then
+		echo ; echo
+		#echo "- Execution du script de mise à jour de la BD $db_update_scripts_folder/$db_update_script_new (o/n) ? [o]"
+		echo "- Execution du script de mise à jour de la BD $db_update_scripts_folder/$db_update_script_new"
+		#read do_it ; [[ -z $do_it ]] && do_it="o" ; echo $do_it
+		#if [[ $do_it == "o" ]] ; then
+		#ls -l ./$db_update_script_new_ || abort
+		#./$db_update_script_new || abort
+		./db-update.sh script_sql/$db_update_script_new || abort
+		#fi
+	fi
+	# On supprime le fichier traité de $ftpf
+	# TODO: 
 done
 #[[ $TEST == 1 ]] && (rm test.txt ; rm db-update-2014-*.sh)
 [[ $TEST == 1 ]] && (rm script_sql/test.txt ; rm script_sql/db-update-2014-*.sql)
@@ -2566,7 +2566,11 @@ class MaterielsControllerTest extends General {
         //debug($_SESSION['Flash']);
         $flash_message = isset($_SESSION['Flash']['flash']) ? $_SESSION['Flash']['flash'][0]['message'] : [];
         //debug($flash_message);
-        if ($SUCCESS && $expected_flash_message) $this->assertTrue($flash_message != [], "Pas de message flash alors qu'il devrait y en avoir un !");
+        if ($SUCCESS && $expected_flash_message) {
+            debug($expected_flash_message);
+            debug($flash_message);
+            $this->assertTrue($flash_message != [], "Pas de message flash alors qu'il devrait y en avoir un !");
+        }
         if ($flash_message) {
             // Assert a flash message in the 'flash' key.
             // Test message partiel
@@ -2596,7 +2600,7 @@ class MaterielsControllerTest extends General {
         //$full_action = "/materiels/$action_link";
         $full_action = "/$controller/$action_link";
         if ($id) $full_action .= "/$id";
-        //debug($full_action);
+        debug($full_action);
         //debug($data);
         $this->post($full_action, $data);
         //if ($action =='statusValidated') return;
@@ -2759,6 +2763,7 @@ class MaterielsControllerTest extends General {
             //$d2 = 'date_reception';
             $d1 = $d[0];
             $d2 = $d[1];
+            
             // - date (reception) < date acq => KO
             $date = $yesterday = new \DateTime('-1 days',$tz);
             /*
@@ -2771,6 +2776,7 @@ class MaterielsControllerTest extends General {
             ];
             $i++; debug("- test $i.a");
             $this->_doActionAndCheckResult($action, $id, FALSE, $new_data);
+            
             // - date (reception) = date acq => OK
             $date = $today;
             $new_data = [
@@ -2779,6 +2785,7 @@ class MaterielsControllerTest extends General {
             ];
             $i++; debug("- test $i.b");
             $this->_doActionAndCheckResult($action, $id, TRUE, $new_data);
+            
             // - date (reception) > date_acq (jqa MAX_DIFF - 1 jour) => OK...
             /*
             $far = new \DateTime('now',$tz);
@@ -2791,8 +2798,10 @@ class MaterielsControllerTest extends General {
                 $d1 => $today->format($format),
                 $d2 => $date->format($format)
             ];
+            debug($new_data);
             $i++; debug("- test $i.c");
             $this->_doActionAndCheckResult($action, $id, TRUE, $new_data);
+            
             // - ...mais pas trop loin dans le futur (MAX_DIFF) => KO
             $date = $far->add(new \DateInterval('P1D'));
             //debug($date);
@@ -2800,6 +2809,7 @@ class MaterielsControllerTest extends General {
                 $d1 => $today->format($format),
                 $d2 => $date->format($format)
             ];
+            debug($new_data);
             $i++; debug("- test $i.d");
             $this->_doActionAndCheckResult($action, $id, FALSE, $new_data);
         } // foreach $dates