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

SubjectRepoBranchLines +/-
mediawiki/extensions/DonationInterfacemaster+15 -58
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

View Standalone Graph
This task is connected to more than 200 other tasks. Only direct parents and subtasks are shown here. Use View Standalone Graph to show more of the graph.

Event Timeline

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

Change 427752 had a related patch set (by Jayprakash12345) published:
[mediawiki/extensions/BlueSpiceExtendedSearch@master] Remove 'UnitTestList' hook

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

Change 427753 had a related patch set (by Jayprakash12345) published:
[mediawiki/extensions/BlueSpiceDashboards@master] Remove 'UnitTestList' hook

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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=