Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TECH] 🔧 Ajoute une tache npm pour executer uniquement les fichiers de test en cours de modification #11134

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

yaf
Copy link
Member

@yaf yaf commented Jan 16, 2025

🥞 Problème

Ce n'est pas toujours simple d'exécuté uniquement les tests qui ont été modifiés, surtout si on en modifie plusieurs par commit...

🥓 Proposition

npm run test:api:git-modified

Suite à la discussion slack https://1024pix.slack.com/archives/C038PTJ9UJE/p1736948910624919

  • @HEYGUL partage sa découverte de combinaison d'usage de git ls-files avec xargs et npm run test:api:path --
  • @laura-bergoens qui propos de mettre en place une tâche permettant pour exécuter uniquement les tests en cours de modification
  • @Steph0 qui précise la commande git ls-files pour qu'elle ne remonte QUE les fichiers de tests;

j'ai simplement pris la commande et ajouter une action pour permettre de lancer npm run test:api:git-modified :)

🧃 Remarques

RAS

😋 Pour tester

  1. Modifier des fichiers de test
  2. Exécuter la commande npm run test:api:git-modified
  3. Constater que seuls les fichiers de test modifiés et non commités sont lancés

@yaf yaf added 👀 Tech Review Needed cross-team Toutes les équipes de dev labels Jan 16, 2025
@yaf yaf self-assigned this Jan 16, 2025
@pix-bot-github
Copy link

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

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

@yaf
Copy link
Member Author

yaf commented Jan 16, 2025

Il y avait déjà une tache test:api:watch (merci @lego-technix), j'ai renommé en test:api:modified mais je trouve que ce n'est toujours pas super comme nom.

Copy link
Contributor

@lego-technix lego-technix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Lu et testé fonctionnellement avec succès. C'est génial 😃 Merci !

PS : Idéalement faire un squash des 2 commits pour n'en faire plus qu'un car sémantiquement il s'agit de la même chose.

@yaf yaf force-pushed the add-test-only-modified-test-task-in-api branch 2 times, most recently from 0639b10 to cd9658f Compare January 16, 2025 12:47
@yaf yaf force-pushed the add-test-only-modified-test-task-in-api branch from fdd2d7c to 77879d0 Compare January 17, 2025 08:43
@pix-service-auto-merge pix-service-auto-merge force-pushed the add-test-only-modified-test-task-in-api branch from 77879d0 to 74c422e Compare January 17, 2025 10:08
@lego-technix
Copy link
Contributor

lego-technix commented Jan 17, 2025

La PR la plus malchanceuse en CI de la terre 😭

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants