As https://gerrit.wikimedia.org/r/#/c/382854 shows the phpunit tests are not being run on a per commit basis.
@Tgr any ideas to why?
This is blocking progress in the Collection extension.
See T177672#3671066 and T177672#3671894.
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Register PHPUnit tests via UnitTestsList hook | mediawiki/extensions/Collection | master | +14 -0 |
Why do you think so? There is a phpunit block at the end of https://integration.wikimedia.org/ci/job/mwext-testextension-hhvm-jessie/18577/console.
If something is wrong with that, --testsuite extensions probably runs https://github.com/wikimedia/mediawiki/blob/master/tests/phpunit/suites/ExtensionsTestSuite.php so that's where I'd look.
@Tgr our assumptions comes from the fact that while we worked on RenderBook class we changed the BookRenderer class a lot, we even changed the BookRenderer::renderBook() signature and tests didn't fail which is puzzling.
Example: Tests should fail as BookRenderer::renderBook() was expecting a single argument, a BookData instance, but tests were still passing three arrays (see https://gerrit.wikimedia.org/r/#/c/382609/3/includes/BookRenderer.php).
@Jdlrobson OK to formally say this task blocks T175856: Implement changes to article concatenation based on books requirements?
Why do you think so? There is a phpunit block at the end of https://integration.wikimedia.org/ci/job/mwext-testextension-hhvm-jessie/18577/console.
Tests are failing locally. It may be running but it doesn't seem to be locating the tests for some reason...
Looks like the test discovery relies on extension registration which Collection does not use. You'll probably have to implement the UnitTestsList hook manually.
Change 383350 had a related patch set uploaded (by Phuedx; owner: Phuedx):
[mediawiki/extensions/Collection@master] Register PHPUnit tests via UnitTestsList hook
@Tgr was correct. This is done in rECOL89f7386e303f: Register PHPUnit tests via UnitTestsList hook.
For the curious, [[ https://github.com/wikimedia/mediawiki/blob/master/tests/phpunit/suites/ExtensionsTestSuite.php#L14-L19 | the extension test path discovery mechanism is defined in tests/phpunit/suites/ExtensionsTestSuite.php ]]. Every extension registered with the extension registry has its tests added to the list and then the UnitTestsList hook is run to supplement that list.
Latest build output with the changes from T177801: Collection phpunit tests are failing for table of contents when run locally applied: https://integration.wikimedia.org/ci/job/mwext-testextension-hhvm-jessie/18862/console
I confirm the patch fixes the problem. I had no idea extension.json was needed for automatic test registration. The patch has been +2ed but it's merge is now blocked on T177801 which fixes the tests.
Change 383350 merged by jenkins-bot:
[mediawiki/extensions/Collection@master] Register PHPUnit tests via UnitTestsList hook
The tests are now run on a per-commit basis 🎉🎉🎉
This task skipped Needs QA because it's purely technical and was verified as part of merging the associated change and those associated with T177801: Collection phpunit tests are failing for table of contents when run locally.