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

Related Objects

StatusAssignedTask
OpenNone
OpenNone
OpenNone
ResolvedLegoktm
ResolvedTgr
OpenPaladox
Resolved Spage
ResolvedPhysikerwelt
ResolvedNone
ResolvedGalorefitz
DeclinedNone
ResolvedMhutti1
OpenNone
OpenNone
ResolvedKarima
Resolvedraymondwu
ResolvedSamanthaNguyen
Resolvedashley
InvalidNone
DeclinedNone
ResolvedMainframe98
Resolved MarcoAurelio
ResolvedSamwilson
ResolvedYaron_Koren
ResolvedWelterkj
ResolvedMGChecker
OpenNone
ResolvedReception123
ResolvedSophivorus
OpenNone
ResolvedReception123
ResolvedReception123
OpenNone
ResolvedNone
OpenZoranzoki21
ResolvedLegoktm
Resolvedashley
Resolved MarcoAurelio
DeclinedNone
DeclinedNone
OpenMGChecker
Resolvedashley
ResolvedSkizzerz
DeclinedMainframe98
ResolvedSamwilson
DeclinedNone
Resolved MarcoAurelio
Resolved MarcoAurelio
Resolved MarcoAurelio
Resolved MarcoAurelio
ResolvedGabrielchihonglee
Resolved MarcoAurelio
InvalidNone
ResolvedLukBukkit
ResolvedGabrielchihonglee
OpenGabrielchihonglee
DeclinedNone
ResolvedLukBukkit
ResolvedLukBukkit
OpenLukBukkit
ResolvedMGChecker
ResolvedJayprakash12345
ResolvedJayprakash12345
ResolvedJayprakash12345
ResolvedJayprakash12345
Resolved MarcoAurelio
ResolvedMainframe98
ResolvedJayprakash12345
ResolvedJayprakash12345
ResolvedJayprakash12345
ResolvedJayprakash12345
ResolvedJayprakash12345
ResolvedJayprakash12345
OpenNone
ResolvedJayprakash12345
ResolvedJayprakash12345
Resolved MarcoAurelio
Resolved MarcoAurelio
OpenNone
ResolvedJayprakash12345
ResolvedJayprakash12345
OpenNone
Resolved MarcoAurelio
ResolvedJayprakash12345
ResolvedJayprakash12345
ResolvedJayprakash12345
ResolvedJayprakash12345
ResolvedJayprakash12345
ResolvedJayprakash12345
Resolved MarcoAurelio
OpenLukBukkit
OpenNone
OpenNone
Resolved MarcoAurelio
ResolvedJayprakash12345
ResolvedJayprakash12345
ResolvedJayprakash12345
ResolvedJayprakash12345
ResolvedJayprakash12345
ResolvedJayprakash12345
OpenJayprakash12345
OpenTacsipacsi
ResolvedJayprakash12345
OpenJayprakash12345
OpenJayprakash12345
ResolvedPwirth
ResolvedJayprakash12345
OpenZoranzoki21
OpenNone
OpenZoranzoki21
ResolvedMGChecker
OpenZoranzoki21
ResolvedMGChecker
OpenZoranzoki21
ResolvedPwirth
OpenZoranzoki21
DeclinedZoranzoki21
OpenNone
Resolved MarcoAurelio
Resolved MarcoAurelio
Resolved MarcoAurelio
OpenNone
DuplicateNone
OpenFlorian
OpenNone
OpenNone
Openntavares
Openntavares
InvalidZoranzoki21
OpenNone
OpenNone
OpenZoranzoki21
OpenNone
OpenNone
OpenZoranzoki21
OpenNone
ResolvedOsnard
OpenNone
OpenZoranzoki21
ResolvedZoranzoki21
DeclinedNone
OpenNone
OpenNone
OpenNone
OpenNone
ResolvedLokal_Profil
OpenNone
OpenNone
ResolvedZoranzoki21
ResolvedZoranzoki21
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
InvalidNone
ResolvedZoranzoki21
OpenZoranzoki21
OpenNone
OpenNone
OpenNone
OpenNone
OpenZoranzoki21
OpenZoranzoki21
ResolvedZoranzoki21
DeclinedNone
StalledNone
OpenNone
OpenZoranzoki21
OpenNone
OpenNone
OpenArasakumarars
OpenNone
ResolvedSumanth699
OpenSri_Lekha_Pathakamuri
OpenNone
OpenNone
OpenZoranzoki21
OpenZoranzoki21
OpenZoranzoki21
OpenZoranzoki21
OpenZoranzoki21

Event Timeline

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

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

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

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

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

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

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

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

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

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

D3r1ck01 updated the task description. (Show Details)Apr 19 2018, 7:32 PM

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

Reedy removed a subscriber: Reedy.Apr 20 2018, 8:01 PM

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

Gopavasanth updated the task description. (Show Details)Apr 21 2018, 3:05 PM

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

jeblad added a subscriber: jeblad.Dec 30 2018, 6:37 PM

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

Nikitrain updated the task description. (Show Details)Mar 31 2019, 9:04 AM