2019-05-01 11:39:45 +00:00
|
|
|
<?php
|
|
|
|
|
|
2020-10-28 20:51:19 +00:00
|
|
|
// phpcs:disable MediaWiki.Commenting.FunctionComment.ObjectTypeHintReturn
|
|
|
|
|
// phpcs:disable MediaWiki.Commenting.FunctionComment.ObjectTypeHintParam
|
|
|
|
|
|
2019-05-01 11:39:45 +00:00
|
|
|
/**
|
|
|
|
|
* Test that a factory class correctly forwards all arguments to the class it constructs. This is
|
|
|
|
|
* useful because sometimes a class' constructor will have more arguments added, and it's easy to
|
|
|
|
|
* accidentally have the factory's constructor fall out of sync.
|
|
|
|
|
*/
|
|
|
|
|
trait FactoryArgTestTrait {
|
|
|
|
|
/**
|
|
|
|
|
* @return string Name of factory class
|
|
|
|
|
*/
|
|
|
|
|
abstract protected static function getFactoryClass();
|
|
|
|
|
|
|
|
|
|
/**
|
|
|
|
|
* @return string Name of instance class
|
|
|
|
|
*/
|
|
|
|
|
abstract protected static function getInstanceClass();
|
|
|
|
|
|
|
|
|
|
/**
|
|
|
|
|
* @return int The number of arguments that the instance constructor receives but the factory
|
|
|
|
|
* constructor doesn't. Used for a simple argument count check. Override if this isn't zero.
|
|
|
|
|
*/
|
|
|
|
|
protected static function getExtraClassArgCount() {
|
|
|
|
|
return 0;
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
/**
|
|
|
|
|
* Override if your factory method name is different from newInstanceClassName.
|
|
|
|
|
*
|
|
|
|
|
* @return string
|
|
|
|
|
*/
|
|
|
|
|
protected function getFactoryMethodName() {
|
2021-03-26 22:56:39 +00:00
|
|
|
return 'new' . ( new ReflectionClass( $this->getInstanceClass() ) )->getShortName();
|
2019-05-01 11:39:45 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
/**
|
|
|
|
|
* Override if $factory->$method( ...$args ) isn't the right way to create an instance, where
|
|
|
|
|
* $method is returned from getFactoryMethodName(), and $args is constructed by applying
|
|
|
|
|
* getMockValueForParam() to the factory method's parameters.
|
|
|
|
|
*
|
2020-10-27 17:13:40 +00:00
|
|
|
* @param object $factory
|
2021-06-17 14:32:05 +00:00
|
|
|
* @return object
|
2019-05-01 11:39:45 +00:00
|
|
|
*/
|
|
|
|
|
protected function createInstanceFromFactory( $factory ) {
|
|
|
|
|
$methodName = $this->getFactoryMethodName();
|
|
|
|
|
$methodObj = new ReflectionMethod( $factory, $methodName );
|
|
|
|
|
$mocks = [];
|
|
|
|
|
foreach ( $methodObj->getParameters() as $param ) {
|
|
|
|
|
$mocks[] = $this->getMockValueForParam( $param );
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
return $factory->$methodName( ...$mocks );
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
public function testConstructorArgNum() {
|
|
|
|
|
$factoryClass = static::getFactoryClass();
|
|
|
|
|
$instanceClass = static::getInstanceClass();
|
|
|
|
|
$factoryConstructor = new ReflectionMethod( $factoryClass, '__construct' );
|
|
|
|
|
$instanceConstructor = new ReflectionMethod( $instanceClass, '__construct' );
|
|
|
|
|
$this->assertSame(
|
|
|
|
|
$instanceConstructor->getNumberOfParameters() - static::getExtraClassArgCount(),
|
|
|
|
|
$factoryConstructor->getNumberOfParameters(),
|
|
|
|
|
"$instanceClass and $factoryClass constructors have an inconsistent number of " .
|
|
|
|
|
' parameters. Did you add a parameter to one and not the other?' );
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
/**
|
|
|
|
|
* Override if getMockValueForParam doesn't produce suitable values for one or more of the
|
|
|
|
|
* parameters to your factory constructor or create method.
|
|
|
|
|
*
|
|
|
|
|
* @param ReflectionParameter $param One of the factory constructor's arguments
|
|
|
|
|
* @return array Empty to not override, or an array of one element which is the value to pass
|
|
|
|
|
* that will allow the object to be constructed successfully
|
|
|
|
|
*/
|
|
|
|
|
protected function getOverriddenMockValueForParam( ReflectionParameter $param ) {
|
|
|
|
|
return [];
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
/**
|
|
|
|
|
* Override if this doesn't produce suitable values for one or more of the parameters to your
|
|
|
|
|
* factory constructor or create method.
|
|
|
|
|
*
|
|
|
|
|
* @param ReflectionParameter $param One of the factory constructor's arguments
|
|
|
|
|
* @return mixed A value to pass that will allow the object to be constructed successfully
|
|
|
|
|
*/
|
|
|
|
|
protected function getMockValueForParam( ReflectionParameter $param ) {
|
|
|
|
|
$overridden = $this->getOverriddenMockValueForParam( $param );
|
|
|
|
|
if ( $overridden ) {
|
|
|
|
|
return $overridden[0];
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
$pos = $param->getPosition();
|
|
|
|
|
|
2019-11-22 08:00:47 +00:00
|
|
|
$type = $param->getType();
|
2020-09-25 20:02:03 +00:00
|
|
|
if ( !$type || $type->getName() === 'string' ) {
|
2019-11-22 08:00:47 +00:00
|
|
|
// Optimistically assume a string is okay
|
|
|
|
|
return "some unlikely string $pos";
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
$type = $type->getName();
|
2019-05-01 11:39:45 +00:00
|
|
|
|
2019-10-19 11:08:40 +00:00
|
|
|
if ( $type === 'array' || $type === 'iterable' ) {
|
2019-05-01 11:39:45 +00:00
|
|
|
return [ "some unlikely string $pos" ];
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
if ( class_exists( $type ) || interface_exists( $type ) ) {
|
|
|
|
|
return $this->createMock( $type );
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
$this->fail( "Unrecognized parameter type $type" );
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
/**
|
|
|
|
|
* Assert that the given $instance correctly received $val as the value for parameter $name. By
|
|
|
|
|
* default, checks that the instance has some member whose value is the same as $val.
|
|
|
|
|
*
|
|
|
|
|
* @param object $instance
|
|
|
|
|
* @param string $name Name of parameter to the factory object's constructor
|
|
|
|
|
* @param mixed $val
|
|
|
|
|
*/
|
|
|
|
|
protected function assertInstanceReceivedParam( $instance, $name, $val ) {
|
|
|
|
|
foreach ( ( new ReflectionObject( $instance ) )->getProperties() as $prop ) {
|
|
|
|
|
$prop->setAccessible( true );
|
|
|
|
|
if ( $prop->getValue( $instance ) === $val ) {
|
|
|
|
|
$this->assertTrue( true );
|
|
|
|
|
return;
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
|
2022-07-14 12:48:15 +00:00
|
|
|
$this->fail( "Param $name not received by " . static::getInstanceClass() );
|
2019-05-01 11:39:45 +00:00
|
|
|
}
|
|
|
|
|
|
2019-10-24 03:14:31 +00:00
|
|
|
/**
|
|
|
|
|
* Override to return a list of constructor parameters that are not stored
|
|
|
|
|
* in the instance properties directly, so should not be verified with
|
|
|
|
|
* assertInstanceReceivedParam.
|
|
|
|
|
* @return string[]
|
|
|
|
|
*/
|
|
|
|
|
protected function getIgnoredParamNames() {
|
|
|
|
|
return [ 'hookContainer' ];
|
|
|
|
|
}
|
|
|
|
|
|
2019-05-01 11:39:45 +00:00
|
|
|
public function testAllArgumentsWerePassed() {
|
|
|
|
|
$factoryClass = static::getFactoryClass();
|
|
|
|
|
|
|
|
|
|
$factoryConstructor = new ReflectionMethod( $factoryClass, '__construct' );
|
|
|
|
|
$mocks = [];
|
|
|
|
|
foreach ( $factoryConstructor->getParameters() as $param ) {
|
|
|
|
|
$mocks[$param->getName()] = $this->getMockValueForParam( $param );
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
$instance =
|
|
|
|
|
$this->createInstanceFromFactory( new $factoryClass( ...array_values( $mocks ) ) );
|
|
|
|
|
|
|
|
|
|
foreach ( $mocks as $name => $mock ) {
|
2019-10-24 03:14:31 +00:00
|
|
|
if ( in_array( $name, $this->getIgnoredParamNames() ) ) {
|
Hooks::run() call site migration
Migrate all callers of Hooks::run() to use the new
HookContainer/HookRunner system.
General principles:
* Use DI if it is already used. We're not changing the way state is
managed in this patch.
* HookContainer is always injected, not HookRunner. HookContainer
is a service, it's a more generic interface, it is the only
thing that provides isRegistered() which is needed in some cases,
and a HookRunner can be efficiently constructed from it
(confirmed by benchmark). Because HookContainer is needed
for object construction, it is also needed by all factories.
* "Ask your friendly local base class". Big hierarchies like
SpecialPage and ApiBase have getHookContainer() and getHookRunner()
methods in the base class, and classes that extend that base class
are not expected to know or care where the base class gets its
HookContainer from.
* ProtectedHookAccessorTrait provides protected getHookContainer() and
getHookRunner() methods, getting them from the global service
container. The point of this is to ease migration to DI by ensuring
that call sites ask their local friendly base class rather than
getting a HookRunner from the service container directly.
* Private $this->hookRunner. In some smaller classes where accessor
methods did not seem warranted, there is a private HookRunner property
which is accessed directly. Very rarely (two cases), there is a
protected property, for consistency with code that conventionally
assumes protected=private, but in cases where the class might actually
be overridden, a protected accessor is preferred over a protected
property.
* The last resort: Hooks::runner(). Mostly for static, file-scope and
global code. In a few cases it was used for objects with broken
construction schemes, out of horror or laziness.
Constructors with new required arguments:
* AuthManager
* BadFileLookup
* BlockManager
* ClassicInterwikiLookup
* ContentHandlerFactory
* ContentSecurityPolicy
* DefaultOptionsManager
* DerivedPageDataUpdater
* FullSearchResultWidget
* HtmlCacheUpdater
* LanguageFactory
* LanguageNameUtils
* LinkRenderer
* LinkRendererFactory
* LocalisationCache
* MagicWordFactory
* MessageCache
* NamespaceInfo
* PageEditStash
* PageHandlerFactory
* PageUpdater
* ParserFactory
* PermissionManager
* RevisionStore
* RevisionStoreFactory
* SearchEngineConfig
* SearchEngineFactory
* SearchFormWidget
* SearchNearMatcher
* SessionBackend
* SpecialPageFactory
* UserNameUtils
* UserOptionsManager
* WatchedItemQueryService
* WatchedItemStore
Constructors with new optional arguments:
* DefaultPreferencesFactory
* Language
* LinkHolderArray
* MovePage
* Parser
* ParserCache
* PasswordReset
* Router
setHookContainer() now required after construction:
* AuthenticationProvider
* ResourceLoaderModule
* SearchEngine
Change-Id: Id442b0dbe43aba84bd5cf801d86dedc768b082c7
2020-03-19 02:42:09 +00:00
|
|
|
continue;
|
|
|
|
|
}
|
2019-05-01 11:39:45 +00:00
|
|
|
$this->assertInstanceReceivedParam( $instance, $name, $mock );
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
}
|