php / php/php-src

PHPT test runner improvements

Offen
#16,623 4 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Category: Tests Feature Status: Needs Triage
Vorherrschende Sprache
C
Sterne
40.4k
Forks
8.1k
Ø Merge
2 T. 13 Std.
Gemergte PRs (30 T.)
96

Beschreibung

Description

Recently I have added a optimization to the PHPUnit PHPT testrunner (which was initially copied from run-tests.php) to avoid subprocess overhead when evaluating side-effect free —SKIPIF— conditions.

the implementation is in https://github.com/sebastianbergmann/phpunit/pull/5998
and I blogged about it (and other optimizations) in https://staabm.github.io/2024/10/19/phpunit-codesprint-munich.html

symfony reported a 9% perf improvement of their PHPT tests in https://github.com/symfony/symfony/pull/58680#issuecomment-2441035323

is this something which would be worth adding into run-tests.php ?
(It would require some code of my tiny lib https://github.com/staabm/side-effects-detector)

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne mit run-tests.php und vergleiche dessen PHPT-Test-Runner-Ablauf mit der Implementierung im verlinkten PHPUnit-Pull-Request. Überprüfe die Bibliothek side-effects-detector, um die referenzierte Abhängigkeit zu verstehen und festzustellen, ob ihr Ansatz zu diesem Runner passt. Als erledigt gilt die Festlegung des Umfangs für das Hinzufügen der Optimierung, einschließlich der Identifizierung aller erforderlichen Tests und Performance-Nachweise.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
php
Bereich
testing-qa
Issue-Typ
Feature
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.