Page MenuHomePhabricator

Remove UnitTestsList hook subscribers, move tests to be autodiscovered for extensions that have been converted to extension registration
Open, LowPublic

Description

Now https://gerrit.wikimedia.org/r/#/c/302944/ is merged, phpunit tests can be autodiscovered by moving them into the correct place. See https://gerrit.wikimedia.org/r/#/c/302958/ for example

Once this has been done, UnitTestsList subscribers can be removed

Ready to go list:

  • AbuseFilter
  • ActiveAbstract
  • AntiSpoof
  • ArticlePlaceholder
  • Babel
  • BaseHooks
  • BetaFeatures
  • BounceHandler
  • Capiunto
  • CentralAuth
  • Cognate
  • CollaborationKit
  • CommonsMetadata
  • ConfirmEdit
  • DonationInterface
  • Elastica
  • EventBus
  • ExtensionDistributor
  • Gadgets
  • GlobalCssJs
  • GoogleAPIClient
  • GuidedTour
  • Kartographer
  • LocalisationUpdate
  • LoginNotify
  • MaintenanceShell
  • MassMessage
  • Math
  • MathSearch
  • MobileFrontend
  • MolHandler
  • Newsletter
  • OAuthAuthentication
  • ORES
  • PageAssessments
  • PageForms
  • ParserFunctions
  • PhpTags
  • PhpTagsFunctions
  • PhpTagsWidgets
  • PhpTagsWiki
  • ProofreadPage
  • QuickSearchLookup
  • SendGrid
  • Sentry
  • TemplateData
  • TemplateStyles
  • TextExtracts
  • Thanks
  • TitleBlacklist
  • TwnMainPage
  • UploadWizard
  • UrlShortener
  • UserMerge
  • WikibaseMediaInfo
  • WikibaseQuality
  • Wikidata
  • WikidataPageBanner
  • WikimediaEvents
  • WikimediaIncubator

Details

Related Changes in Gerrit:
SubjectRepoBranchLines +/-
mediawiki/extensions/PhpTagsStoragemaster+0 -12
mediawiki/extensions/DonationInterfacemaster+15 -58
mediawiki/extensions/MediaWikiFarmmaster+0 -90
mediawiki/extensions/MultiMapsmaster+0 -18
mediawiki/extensions/SemanticSiftermaster+1 -8
mediawiki/extensions/googleAnalyticsmaster+0 -23
mediawiki/extensions/Acrolinxmaster+1 -14
mediawiki/extensions/PageFormsmaster+0 -17
mediawiki/extensions/BounceHandlermaster+0 -31
mediawiki/extensions/PagedTiffHandlermaster+0 -12
mediawiki/extensions/Mathmaster+0 -0
mediawiki/extensions/MathSearchmaster+0 -15
mediawiki/extensions/PhpTagsSMWmaster+1 -13
mediawiki/extensions/PhpTagsWidgetsmaster+4 -15
mediawiki/extensions/PhpTagsFunctionsmaster+1 -13
mediawiki/extensions/BlueSpiceFoundationmaster+0 -12
mediawiki/extensions/PhpTagsmaster+1 -13
mediawiki/extensions/PhpTagsWikimaster+1 -13
mediawiki/extensions/BlueSpicePageTemplatesmaster+1 -11
mediawiki/extensions/BlueSpiceReadersmaster+1 -13
mediawiki/extensions/BlueSpiceInterWikiLinksmaster+0 -15
mediawiki/extensions/BlueSpiceUserManagermaster+0 -14
mediawiki/extensions/BlueSpicePageAssignmentsmaster+1 -13
mediawiki/extensions/BlueSpiceSmartListmaster+0 -13
mediawiki/extensions/BlueSpiceNamespaceManagermaster+0 -12
mediawiki/extensions/MolHandlermaster+0 -17
mediawiki/extensions/NSFileRepomaster+1 -18
mediawiki/extensions/OAuthAuthenticationmaster+2 -12
mediawiki/extensions/NewUserActionsmaster+0 -14
mediawiki/extensions/MultiLanguageManagermaster+0 -11
mediawiki/extensions/BaseHooksmaster+1 -23
mediawiki/extensions/BlueSpiceAvatarsmaster+0 -13
mediawiki/extensions/BlueSpiceRSSFeedermaster+0 -11
mediawiki/extensions/BlueSpiceInsertMagicmaster+1 -13
mediawiki/extensions/BlueSpiceInsertFilemaster+0 -14
mediawiki/extensions/BlueSpiceChecklistmaster+1 -14
mediawiki/extensions/BlueSpiceGroupManagermaster+0 -15
mediawiki/extensions/BlueSpiceExtendedStatisticsmaster+1 -10
mediawiki/extensions/BlueSpiceExtendedSearchmaster+0 -11
mediawiki/extensions/BlueSpiceContextMenumaster+1 -13
mediawiki/extensions/BlueSpiceDashboardsmaster+1 -13
mediawiki/extensions/SendGridmaster+1 -28
mediawiki/extensions/QuickSearchLookupmaster+0 -14
mediawiki/extensions/Newslettermaster+0 -29
mediawiki/extensions/MaintenanceShellmaster+0 -8
mediawiki/extensions/CentralNoticemaster+6 -28
mediawiki/extensions/Scribuntomaster+5 -36
mediawiki/extensions/CirrusSearchmaster+1 -26
mediawiki/extensions/UserMergemaster+1 -30
mediawiki/extensions/VipsScalermaster+0 -16
mediawiki/extensions/TextExtractsmaster+0 -9
mediawiki/extensions/MassMessagemaster+2 -27
mediawiki/extensions/JsonConfigmaster+205 -166
mediawiki/extensions/LoginNotifymaster+1 -15
mediawiki/extensions/ArticlePlaceholdermaster+0 -11
mediawiki/extensions/Gadgetsmaster+0 -14
mediawiki/extensions/TemplateStylesmaster+0 -13
mediawiki/extensions/CollaborationKitmaster+0 -26
mediawiki/extensions/WikibaseMediaInfomaster+0 -25
mediawiki/extensions/ParserFunctionsmaster+0 -12
mediawiki/extensions/WikibaseQualitymaster+1 -18
mediawiki/extensions/TwnMainPagemaster+0 -12
mediawiki/extensions/Sentrymaster+0 -7
mediawiki/extensions/ProofreadPagemaster+6 -34
mediawiki/extensions/Mathmaster+0 -16
mediawiki/extensions/GlobalCssJsmaster+2 -23
mediawiki/extensions/Babelmaster+0 -8
mediawiki/extensions/GuidedTourmaster+0 -15
mediawiki/extensions/Capiuntomaster+0 -15
mediawiki/extensions/Elasticamaster+0 -42
mediawiki/extensions/CommonsMetadatamaster+0 -14
mediawiki/extensions/EventBusmaster+0 -11
mediawiki/extensions/ExtensionDistributormaster+0 -30
mediawiki/extensions/Kartographermaster+0 -16
mediawiki/extensions/ConfirmEditmaster+7 -36
mediawiki/extensions/WikimediaIncubatormaster+1 -11
mediawiki/extensions/WikimediaEventsmaster+0 -23
mediawiki/extensions/TitleBlacklistmaster+0 -14
mediawiki/extensions/WikidataPageBannermaster+0 -24
mediawiki/extensions/Thanksmaster+0 -18
mediawiki/extensions/PageAssessmentsmaster+0 -10
mediawiki/extensions/ORESmaster+0 -12
mediawiki/extensions/TemplateDatamaster+0 -12
mediawiki/extensions/LocalisationUpdatemaster+0 -21
mediawiki/extensions/RelatedArticlesmaster+0 -21
mediawiki/extensions/MobileFrontendmaster+0 -16
mediawiki/extensions/UploadWizardmaster+0 -12
mediawiki/extensions/UrlShortenermaster+0 -25
mediawiki/extensions/BetaFeaturesmaster+1 -14
mediawiki/extensions/ActiveAbstractmaster+2 -23
mediawiki/extensions/CentralAuthmaster+0 -14
mediawiki/extensions/AntiSpoofmaster+1 -27
mediawiki/extensions/AbuseFiltermaster+1 -21
Show related patches Customize query in gerrit

Related Objects

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes

Change 427739 merged by jenkins-bot:
[mediawiki/extensions/BlueSpiceInsertMagic@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/427739

Change 427733 merged by jenkins-bot:
[mediawiki/extensions/BlueSpiceRSSFeeder@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/427733

Change 427724 merged by jenkins-bot:
[mediawiki/extensions/BlueSpiceAvatars@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/427724

Change 303349 merged by jenkins-bot:
[mediawiki/extensions/BaseHooks@master] Move tests to tests/phpunit/ and remove hook

https://gerrit.wikimedia.org/r/303349

Change 427721 merged by jenkins-bot:
[mediawiki/extensions/NewUserActions@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/427721

Change 427725 merged by jenkins-bot:
[mediawiki/extensions/MultiLanguageManager@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/427725

Change 427716 merged by jenkins-bot:
[mediawiki/extensions/OAuthAuthentication@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/427716

Change 427741 merged by jenkins-bot:
[mediawiki/extensions/NSFileRepo@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/427741

Change 427726 merged by jenkins-bot:
[mediawiki/extensions/MolHandler@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/427726

Change 427736 merged by Umherirrender:
[mediawiki/extensions/BlueSpiceNamespaceManager@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/427736

Change 427731 merged by jenkins-bot:
[mediawiki/extensions/BlueSpiceSmartList@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/427731

Change 427735 merged by Umherirrender:
[mediawiki/extensions/BlueSpicePageAssignments@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/427735

Change 427730 merged by Umherirrender:
[mediawiki/extensions/BlueSpiceUserManager@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/427730

Change 427738 merged by Umherirrender:
[mediawiki/extensions/BlueSpiceInterWikiLinks@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/427738

Change 427732 merged by Umherirrender:
[mediawiki/extensions/BlueSpiceReaders@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/427732

Change 427734 merged by Umherirrender:
[mediawiki/extensions/BlueSpicePageTemplates@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/427734

Change 427706 merged by Umherirrender:
[mediawiki/extensions/PhpTagsWiki@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/427706

Change 427713 merged by Umherirrender:
[mediawiki/extensions/PhpTags@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/427713

Change 427749 merged by jenkins-bot:
[mediawiki/extensions/BlueSpiceFoundation@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/427749

Change 427712 merged by Umherirrender:
[mediawiki/extensions/PhpTagsFunctions@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/427712

Change 427708 merged by Umherirrender:
[mediawiki/extensions/PhpTagsWidgets@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/427708

Change 427711 merged by Umherirrender:
[mediawiki/extensions/PhpTagsSMW@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/427711

Change 428056 had a related patch set uploaded (by Gopavasanth; owner: Gopavasanth):
[mediawiki/extensions/PageForms@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/428056

Change 428057 had a related patch set uploaded (by Gopavasanth; owner: Gopavasanth):
[mediawiki/extensions/MathSearch@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/428057

Change 428059 had a related patch set uploaded (by Gopavasanth; owner: Gopavasanth):
[mediawiki/extensions/DonationInterface@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/428059

Change 428060 had a related patch set uploaded (by Physikerwelt; owner: Physikerwelt):
[mediawiki/extensions/Math@master] Move phpunit test files

https://gerrit.wikimedia.org/r/428060

Change 428057 merged by jenkins-bot:
[mediawiki/extensions/MathSearch@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/428057

Change 428060 merged by jenkins-bot:
[mediawiki/extensions/Math@master] Move phpunit test files

https://gerrit.wikimedia.org/r/428060

Change 427715 merged by jenkins-bot:
[mediawiki/extensions/PagedTiffHandler@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/427715

Change 302996 abandoned by Jforrester:
Move tests to tests/phpunit/ and remove hook

Reason:
Done in Iac91c86fc66ef99bbc838b7458e8efcdcd2bcfb8.

https://gerrit.wikimedia.org/r/302996

Please mark hooks as deprecated before such changes, and update the documentation accordingly.
This (in combo with a change in PHPunit) trigger some very strange bugs.

@jeblad: Since August 2016, https://phabricator.wikimedia.org/source/mediawiki/browse/master/maintenance/convertExtensionToRegistration.php$254-260 has

if ( $hookName === 'UnitTestsList' ) {
	$this->output( "Note: the UnitTestsList hook is no longer necessary as " .
		"long as your tests are located in the \"tests/phpunit/\" directory. " .
		"Please see <https://www.mediawiki.org/wiki/Manual:PHP_unit_testing/" .
	"Writing_unit_tests_for_extensions#Register_your_tests> for more details.\n"
	);
}

To clarify, which other kind of deprecation and documentation would you have expected and where?
The current policy is at https://www.mediawiki.org/wiki/Deprecation_policy

Change 428056 merged by jenkins-bot:
[mediawiki/extensions/PageForms@master] Remove 'UnitTestList' hook

https://gerrit.wikimedia.org/r/428056

Oh, I just wanted to close this, but there are a few usages which are left to be removed, per https://codesearch.wmcloud.org/search/?q=onUnitTestsList&i=nope&files=&excludeFiles=&repos=

Change #1163865 had a related patch set uploaded (by Hashar; author: Hashar):

[mediawiki/extensions/Acrolinx@master] Remove UnitTestsList in favor of autodiscovery

https://gerrit.wikimedia.org/r/1163865

Change #1163866 had a related patch set uploaded (by Hashar; author: Hashar):

[mediawiki/extensions/MediaWikiFarm@master] Remove UnitTestsList in favor of autodiscovery

https://gerrit.wikimedia.org/r/1163866

Change #1163867 had a related patch set uploaded (by Hashar; author: Hashar):

[mediawiki/extensions/googleAnalytics@master] Remove UnitTestsList in favor of autodiscovery

https://gerrit.wikimedia.org/r/1163867

Change #1163868 had a related patch set uploaded (by Hashar; author: Hashar):

[mediawiki/extensions/SemanticSifter@master] Remove UnitTestsList in favor of autodiscovery

https://gerrit.wikimedia.org/r/1163868

Change #1163869 had a related patch set uploaded (by Hashar; author: Hashar):

[mediawiki/extensions/PhpTagsStorage@master] Remove UnitTestsList in favor of autodiscovery

https://gerrit.wikimedia.org/r/1163869

Change #1163871 had a related patch set uploaded (by Hashar; author: Hashar):

[mediawiki/extensions/MultiMaps@master] Remove UnitTestsList in favor of autodiscovery

https://gerrit.wikimedia.org/r/1163871

DonationInterface and Wikibase would be addressed via the sibling task T142121

Change #1163865 merged by jenkins-bot:

[mediawiki/extensions/Acrolinx@master] Remove UnitTestsList in favor of autodiscovery

https://gerrit.wikimedia.org/r/1163865

Change #1163867 merged by jenkins-bot:

[mediawiki/extensions/googleAnalytics@master] Remove UnitTestsList in favor of autodiscovery

https://gerrit.wikimedia.org/r/1163867

Change #1163871 merged by jenkins-bot:

[mediawiki/extensions/MultiMaps@master] Remove UnitTestsList in favor of autodiscovery

https://gerrit.wikimedia.org/r/1163871

Change #1163868 merged by jenkins-bot:

[mediawiki/extensions/SemanticSifter@master] Remove UnitTestsList in favor of autodiscovery

https://gerrit.wikimedia.org/r/1163868

Change #1163869 merged by Umherirrender:

[mediawiki/extensions/PhpTagsStorage@master] Remove UnitTestsList in favor of autodiscovery

https://gerrit.wikimedia.org/r/1163869

Change #1163866 merged by jenkins-bot:

[mediawiki/extensions/MediaWikiFarm@master] Remove UnitTestsList in favor of autodiscovery

https://gerrit.wikimedia.org/r/1163866

Change #428059 abandoned by Hashar:

[mediawiki/extensions/DonationInterface@master] Remove 'UnitTestList' hook

Reason:

I did not notice that patch. It is superseded by I813b5107d72e434b71a6839140dbea72f8f400d8 and I9ac3c6543c06f07ea5270d28c56cf2048077f92c

https://gerrit.wikimedia.org/r/428059