[FEATURE] Ajout de script pour supprimer des orga, dpo et tags associés (PIX-16091)

[bpetetot]

🥞 Problème

Suite à une mauvaise manipulation sur Pix admin, des organisations ont été ajoutées en double et il faudrait les retirer.

🥓 Proposition

Concevoir un script qui va parcourir le csv contenant tous les identifiants des organisations à supprimer et va successivement:

• Supprimer l'organisation de la table data-protection-officers,
• Supprimer l'association de l'organisation et de tous ces tags dans la table organization-tags,
• Supprimer l'organisation de la table organizations.

🧃 Remarques

Ce script n'a pas vocation à rester, mais il a permis d'ajouter les méthodes suivantes:

• Supprimer une dpo,
• Retirer tous les tags associés à une organisation sans supprimer les tags,
• Supprimer une organisation.

😋 Pour tester

Le teste peut être fait en RA, mais il est plus simple et confortable de le faire en local. La procédure ne diffère cependant pas beaucoup.

En local:

• Faire un reset de la db pour avoir le nouveau seed:

cd api
npm run db:reset

• Lancer l'api et pix-admin,
• Se connecter sur pix-admin avec le compte [email protected],
• Sur la page des organisations, constater que l'organisation "Accis est présente,
• Noter son identifiant,
• Créer un fichier .csv avec le contenu suivant:

"organization ID";
<identifiant de l'organisation>;

• Sauvegarder le fichier,
• Dans le dossier api, exécuter ce script:

node api/src/organizational-entities/scripts/delete-organizations-script.js --dryRun --file <chemin/vers/votre/fichier.csv>

L'option dryRun est une sécurité, elle n'efface pas les éléments. Un message de ce type sera affiché:
"Would delete {nombre} organizations.".

• Répéter la même commande en supprimant l'option --dryRun, soit:

node api/src/organizational-entities/scripts/delete-organizations-script.js --file <chemin/vers/votre/fichier.csv>

Cette fois, le message "Deleted {nombre} organizations." sera afiché.

• Retourner sur Pix admin et constater dans la liste des organisations que "Accis" a disparru.

En RA

• Se connecter sur pix-admin avec le compte [email protected],
• Sur la page des organisations, constater que l'organisation "Accis est présente,
• Noter son identifiant,
• Créer un fichier .csv avec le contenu suivant:

"organization ID";
<identifiant de l'organisation>;

• Sauvegarder le fichier,
• En ligne de commande, avec scalingo connecté, exécuter la commande suivante:

scalingo -a pix-api-review-pr11124 run --file <chemin/vers/votre/fichier.csv> /src/organizational-entities/scripts/delete-organizations-script.js --dryRun --file /tmp/uploads/<fichier.csv>

L'option dryRun est une sécurité, elle n'efface pas les éléments. Un message de ce type sera affiché "Would delete {nombre} organizations.".

• Répéter la même commande en supprimant l'option --dryRun, soit:

scalingo -a pix-api-review-pr11124 run --file <chemin/vers/votre/fichier.csv> /src/organizational-entities/scripts/delete-organizations-script.js --file /tmp/uploads/<fichier.csv>

Cette fois, le message "Deleted {nombre} organizations." sera afiché.

• Retourner sur Pix admin et constater dans la liste des organisations que "Accis" a disparru,
• Afin de permettre aux personnes suivante de tester, relancez les seeds avec cette commande:

scalingo -a pix-api-review-pr11124 run npm run db:seed

[pix-bot-github]

Une fois les applications déployées, elles seront accessibles via les liens suivants :

• App (.fr)
• App (.org)
• Orga (.fr)
• Orga (.org)
• Certif (.fr)
• Certif (.org)
• Junior
• Admin
• API
• Audit Logger

Les variables d'environnement seront accessibles via les liens suivants :

• scalingo front
• scalingo api
• scalingo audit-logger

[lego-technix]

Suggestion de nommage d'argument organizationId plus explicite et qui est plus la norme dans la base de code.

De plus le deleteCount ne sert jamais à rien dans le code de production (c'est à dire hors tests), en effet soit la suppression est un succès, soit il y a une erreur et une exception sera levée. Mettre à jour les tests pour ne pas chercher à utiliser la valeur de retour booléenne précédente. Enfin, les tests testent déjà bien que la suppression est effective.

Suggested change

	const deleteById = async function (id) {
	const deleteCount = await knex(ORGANIZATIONS_TABLE_NAME).where({ id }).del();
	return deleteCount === 1;
	};
	const deleteById = async function (organizationId) {
	await knex(ORGANIZATIONS_TABLE_NAME).where({ id: organizationId }).del();
	};