(NFC) afform_mock - Updates for PHPUnit 9.x #30581
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
Small compatibility updates for pre-existing tests.
Before
phpunit.xml.dist
mockPublicForm.aff.html
quietly ignoredAfter
Comments
The XML update is just the regular automated update.
It wa strange that I had trouble running the test for
mockPublicForm
locally -- obviously, it worked at some point. I think the underlying issue is that phpunit9 introduced different behaviors re:file naming and class naming.Anyway, in theory, it just needed an extra XML directive to specify the formula (e.g.
<directory suffix=".test.php">./ang</directory>
). However, even with a suitable configuration, the usage is... flaky.phpunit
(no args) would run the test; butphpunit <FILENAME>
wouldn't; andphpunit9 --testyaddayadda <FILENAME>
would work. Of course,--testyaddayadda
is annoying.So the file gets renamed. #theNailThatSticksOutGetsHammeredDown