Phpunit: awaitExceptionMessageRegExp () wurde in PHPUnit 9 entfernt, ohne dass in PHPUnit 8 eine Warnung zur Ablehnung angezeigt wurde
| Q | EIN
| -------------------- | ---------------
| PHPUnit Version | 9
| PHP-Version | 7.3
| Installationsmethode | Komponist
Zusammenfassung
Das Commit, das expectExceptionMessageMatches einführt, besagt, dass expectExceptionMessageRegExp in 9 eine Warnung auslöst und in 10 entfernt wird.
In Version 9.0 wurde die Methode jedoch vollständig entfernt, anstatt eine Warnung hinzuzufügen. Dies bedeutet, dass die Methode entfernt wurde, ohne jemals eine Verfallswarnung zu melden, wenn Tests mit ihr ausgeführt wurden.
Und es gibt kein Änderungsprotokoll, das besagt, dass diese Methode veraltet ist (das PHPUnit 8-Änderungsprotokoll spricht nicht davon, expectExceptionMessageRegExp verwerfen, und das PHPUnit 9-Änderungsprotokoll sagt, dass es entfernt wurde).
stof
Alle 3 Kommentare
expectExceptionMessageMatches wurde erst in 8.4 hinzugefügt, weshalb in der PHPUnit 8-Ankündigung überhaupt nicht darüber gesprochen wird.
stof
am 6. März 2020
expectExceptionMessageRegExp() war in PHPUnit 8.4 veraltet. Die Tatsache, dass diese Ablehnung nicht gemeldet wird, wenn die Methode ärgerlich angewendet wird, stimme ich zu, aber ist es wirklich sinnvoll, dieses Verhalten in PHPUnit 8.5 hinzuzufügen?
sebastianbergmann
am 6. März 2020
Nun, das anfängliche Commit sollte die Verfallswarnung in 9 hinzufügen (siehe das verknüpfte Commit) und in 10 entfernen. Deshalb gibt es in 8.4 und 8.5 keine Warnung.
Das Problem ist, dass das Entfernen ohne Befolgung des ursprünglichen Plans und ohne Hinzufügen der Verfallswarnung in 8.x durchgeführt wurde, was bedeutet, dass überhaupt keine Warnung vorhanden ist.
Der Wert der Warnung besteht darin, dass ich beim Anzeigen der Warnung (und auf einmal) an der Abwertung arbeiten kann, anstatt den schwerwiegenden Fehler aufgrund der entfernten Methode beheben zu müssen, und dies für jeden Ort, an dem sie verwendet wird, wiederholen muss, bis ich es nicht mehr tue Es wird kein schwerwiegender Fehler mehr angezeigt (oder ich könnte mit meiner IDE nach expectExceptionMessageRegExp suchen, aber dann hilft mir PHPUnit nicht beim Upgrade, und das könnte es rechtfertigen, überhaupt keine Verfallswarnungen zu verwenden ...).
Wenn möglich, halte ich es für sinnvoll, die Warnung in 8.5 hinzuzufügen, ja.
stof
am 6. März 2020
Verwandte Themen
keradus
·
3Kommentare
sebastianbergmann
·
4Kommentare
TiMESPLiNTER
·
3Kommentare
edyan
·
4Kommentare
keradus
·
4Kommentare
Hilfreichster Kommentar
Nun, das anfängliche Commit sollte die Verfallswarnung in 9 hinzufügen (siehe das verknüpfte Commit) und in 10 entfernen. Deshalb gibt es in 8.4 und 8.5 keine Warnung.
Das Problem ist, dass das Entfernen ohne Befolgung des ursprünglichen Plans und ohne Hinzufügen der Verfallswarnung in 8.x durchgeführt wurde, was bedeutet, dass überhaupt keine Warnung vorhanden ist.
Der Wert der Warnung besteht darin, dass ich beim Anzeigen der Warnung (und auf einmal) an der Abwertung arbeiten kann, anstatt den schwerwiegenden Fehler aufgrund der entfernten Methode beheben zu müssen, und dies für jeden Ort, an dem sie verwendet wird, wiederholen muss, bis ich es nicht mehr tue Es wird kein schwerwiegender Fehler mehr angezeigt (oder ich könnte mit meiner IDE nach
expectExceptionMessageRegExpsuchen, aber dann hilft mir PHPUnit nicht beim Upgrade, und das könnte es rechtfertigen, überhaupt keine Verfallswarnungen zu verwenden ...).Wenn möglich, halte ich es für sinnvoll, die Warnung in 8.5 hinzuzufügen, ja.