Trim fixed per-scenario overhead

Ouverte
#388 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
3/5
Temps estimé
1-2 jours
Accessibilité débutants
55/100
Type d'issue
Refactorisation
Clarté
Plutôt claire
Activité
Active
Stack technique
php, wordpress
Domaine
performance, testing

Piste de recherche

Find the code that deletes the scenario RUN_DIR and the step definition for I launch in the background in the PHP test framework. Read #384 to learn the baseline CI timing method, then open a draft PR and run the same CI jobs to record both results. Done means the directory cleanup runs in the background and the process check waits only as needed, with timings posted here.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

scope:testing status:unconfirmed

A few fixed costs are paid by every scenario, or by every scenario of a kind, without contributing to what is tested:

  1. Removing the scenario's RUN_DIR (about 3,800 files for a WordPress install) blocks the next scenario for about 50 ms. It can happen in the background once the directory is moved out of the way.
  2. Starting a background process (I launch in the background, the PHP built-in web server) always waits a full second before checking that the process is alive, even when it is ready much sooner.

Across all bundled commands, this adds up to roughly 2 to 3 minutes for each CI configuration.

Measuring: take the baseline from #384, then open a draft PR, run the same CI jobs against it and record both results here before we decide.

Langage dominant
PHP
Étoiles
42
Forks
29
Merge moyen
21 h 6 min
PR mergées (30 j)
24

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de wp-cli/wp-cli-tests

Toutes les issues de wp-cli/wp-cli-tests

Issues similaires

Plus d'issues PHP

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.