Change Details

PHPUnit 10 no longer invokes a static method called `suite` on test classes. Also, paratest (T50217) doesn't seem to work well with test classes that dynamically add more tests. For these reasons, Scribunto needs to stop relying on the `suite` method to define its test suites. Related: T345481 for removing the same deprecated pattern from core.

PHPUnit 10 no longer invokes a static method called `suite` on test classes. Also, paratest (T50217) doesn't seem to work well with test classes that dynamically add more tests. For these reasons, Scribunto needs to stop relying on the `suite` method to define its test suites. In general, the Scribunto test suite is quite "creative" and uses PHPUnit features that are marked as `@internal` in PHPUnit 10. An example issue this caused is T418217. All these should be avoided, in favour of a suite configuration that is supported by upstream. Related: T345481 for removing the same deprecated pattern from core.

PHPUnit 10 no longer invokes a static method called `suite` on test classes. Also, paratest (T50217) doesn't seem to work well with test classes that dynamically add more tests. For these reasons, Scribunto needs to stop relying on the `suite` method to define its test suites. In general, the Scribunto test suite is quite "creative" and uses PHPUnit features that are marked as `@internal` in PHPUnit 10. An example issue this caused is T418217. All these should be avoided, in favour of a suite configuration that is supported by upstream. Related: T345481 for removing the same deprecated pattern from core.