arginfo_zpp_mismatch.phpt and friends only tests statically built extensions on CI
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- C
- Sterne
- 40.4k
- Forks
- 8.2k
- Ø Merge
- 2 T. 13 Std.
- Gemergte PRs (30 T.)
- 96
Beschreibung
Description
There are a couple of tests (see below) which iterate over all declared classes, and instantiate them. However, these tests don't load any shared extensions on CI, so the respective classes are not tested. I don't think this is intentional, but rather an oversight (when we switched to loading shared extensions via dl()).
Test that there is no arginfo/zpp mismatch [.\Zend\tests\arginfo_zpp_mismatch.phpt]
Test that there is no arginfo/zpp mismatch in strict mode [.\Zend\tests\arginfo_zpp_mismatch_strict.phpt]
Try to instantiate all classes without arguments [.\Zend\tests\instantiate_all_classes.phpt]
Object free handler must call zend_weakrefs_notify [.\Zend\tests\weakrefs\notify.phpt]
PHP Version
PHP 8.1
Operating System
any
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit dem Lesen von Zend/tests/arginfo_zpp_mismatch.phpt, Zend/tests/arginfo_zpp_mismatch_strict.phpt, Zend/tests/instantiate_all_classes.phpt und Zend/tests/weakrefs/notify.phpt sowie der CI-Konfiguration, die Extensions lädt. Führe diese Tests in der aktuellen CI-Konfiguration aus und überprüfe, dass gemeinsam genutzte Extensions geladen und abgedeckt werden. Erledigt ist die Aufgabe, wenn die aufgeführten Tests sowohl statisch gebaute als auch gemeinsam genutzte Extensions in CI ausführen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- php
- Bereich
- ci-cd, testing
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100