mirror of
https://gerrit.wikimedia.org/r/mediawiki/skins/Vector.git
synced 2024-11-29 01:45:00 +00:00
eea5373940
complete the migration to components. Changes: * shouldHideLanguages only used inside SkinVector22 so abstract method and legacy implementation removed. * Move isLanguagesInContent method to SkinVector22, updating one usage in SkinVectorLegacy which is always falsey (languages are always portals in legacy Vector) * Remove unused isLanguagesInContentAt method from SkinVectorLegacy and move getLanguagesCached, isULSExtensionEnabled, canHaveLanguages, isLanguagesInContentAt to SkinVector22 * runOnSkinTemplateNavigationHooks copied to both Vector skins * isResponsive moved to Vector22. In Legacy Vector it has no real use since legacy Vector has no responsive styles. * SkinVector class now (empty), is removed. Bug: T319355 Change-Id: Ib5df956cecb841b7c97264975d137af32e5da4ab
320 lines
9.3 KiB
PHP
320 lines
9.3 KiB
PHP
<?php
|
|
namespace MediaWiki\Skins\Vector\Tests\Integration;
|
|
|
|
use Exception;
|
|
use MediaWiki\MediaWikiServices;
|
|
use MediaWiki\Skins\Vector\SkinVector22;
|
|
use MediaWiki\Skins\Vector\SkinVectorLegacy;
|
|
use MediaWikiIntegrationTestCase;
|
|
use ReflectionMethod;
|
|
use RequestContext;
|
|
use Title;
|
|
use Wikimedia\TestingAccessWrapper;
|
|
|
|
/**
|
|
* Class VectorTemplateTest
|
|
* @package MediaWiki\Skins\Vector\Tests\Unit
|
|
* @group Vector
|
|
* @group Skins
|
|
*/
|
|
class SkinVectorTest extends MediaWikiIntegrationTestCase {
|
|
|
|
/**
|
|
* @return SkinVectorLegacy
|
|
*/
|
|
private function provideVectorTemplateObject() {
|
|
$skinFactory = MediaWikiServices::getInstance()->getSkinFactory();
|
|
$template = $skinFactory->makeSkin( 'vector' );
|
|
return $template;
|
|
}
|
|
|
|
/**
|
|
* @param string $nodeString an HTML of the node we want to verify
|
|
* @param string $tag Tag of the element we want to check
|
|
* @param string $attribute Attribute of the element we want to check
|
|
* @param string $search Value of the attribute we want to verify
|
|
* @return bool
|
|
*/
|
|
private function expectNodeAttribute( $nodeString, $tag, $attribute, $search ) {
|
|
$node = new \DOMDocument();
|
|
$node->loadHTML( $nodeString );
|
|
$element = $node->getElementsByTagName( $tag )->item( 0 );
|
|
if ( !$element ) {
|
|
return false;
|
|
}
|
|
|
|
$values = explode( ' ', $element->getAttribute( $attribute ) );
|
|
return in_array( $search, $values );
|
|
}
|
|
|
|
/**
|
|
* @covers \MediaWiki\Skins\Vector\SkinVectorLegacy::getTemplateData
|
|
*/
|
|
public function testGetTemplateData() {
|
|
$title = Title::newFromText( 'SkinVector' );
|
|
$context = RequestContext::newExtraneousContext( $title );
|
|
$context->setLanguage( 'fr' );
|
|
$vectorTemplate = $this->provideVectorTemplateObject();
|
|
$vectorTemplate->setContext( $context );
|
|
$this->setTemporaryHook( 'SkinTemplateNavigation::Universal', [
|
|
static function ( &$skinTemplate, &$content_navigation ) {
|
|
$content_navigation['actions'] = [
|
|
'action-1' => []
|
|
];
|
|
$content_navigation['namespaces'] = [
|
|
'ns-1' => []
|
|
];
|
|
$content_navigation['variants'] = [
|
|
[
|
|
'class' => 'selected',
|
|
'text' => 'Language variant',
|
|
'href' => '/url/to/variant',
|
|
'lang' => 'zh-hant',
|
|
'hreflang' => 'zh-hant',
|
|
]
|
|
];
|
|
$content_navigation['views'] = [];
|
|
$content_navigation['user-menu'] = [
|
|
'pt-1' => [ 'text' => 'pt1' ],
|
|
];
|
|
}
|
|
] );
|
|
$openVectorTemplate = TestingAccessWrapper::newFromObject( $vectorTemplate );
|
|
|
|
$props = $openVectorTemplate->getTemplateData()['data-portlets'];
|
|
$views = $props['data-views'];
|
|
$namespaces = $props['data-namespaces'];
|
|
|
|
// The mediawiki core specification might change at any time
|
|
// so let's limit the values we test to those we are aware of.
|
|
$keysToTest = [
|
|
'id', 'class', 'html-tooltip', 'html-items',
|
|
'html-after-portal', 'html-before-portal',
|
|
'label', 'heading-class', 'is-dropdown'
|
|
];
|
|
foreach ( $views as $key => $value ) {
|
|
if ( !in_array( $key, $keysToTest ) ) {
|
|
unset( $views[ $key] );
|
|
}
|
|
}
|
|
$this->assertSame(
|
|
[
|
|
// Provided by core
|
|
'id' => 'p-views',
|
|
'class' => 'mw-portlet mw-portlet-views emptyPortlet ' .
|
|
'vector-menu-tabs vector-menu-tabs-legacy',
|
|
'html-tooltip' => '',
|
|
'html-items' => '',
|
|
'html-after-portal' => '',
|
|
'html-before-portal' => '',
|
|
'label' => $context->msg( 'views' )->text(),
|
|
'heading-class' => '',
|
|
'is-dropdown' => false,
|
|
],
|
|
$views
|
|
);
|
|
|
|
$variants = $props['data-variants'];
|
|
$actions = $props['data-actions'];
|
|
$this->assertSame(
|
|
'mw-portlet mw-portlet-namespaces vector-menu-tabs vector-menu-tabs-legacy',
|
|
$namespaces['class']
|
|
);
|
|
$this->assertSame(
|
|
'mw-portlet mw-portlet-variants vector-menu-dropdown',
|
|
$variants['class']
|
|
);
|
|
$this->assertSame(
|
|
'mw-portlet mw-portlet-cactions vector-menu-dropdown',
|
|
$actions['class']
|
|
);
|
|
$this->assertSame(
|
|
'mw-portlet mw-portlet-personal vector-user-menu-legacy',
|
|
$props['data-personal']['class']
|
|
);
|
|
}
|
|
|
|
/**
|
|
* Standard config for Language Alert in Sidebar
|
|
* @return array
|
|
*/
|
|
private function enableLanguageAlertFeatureConfig(): array {
|
|
return [
|
|
'VectorLanguageInHeader' => [
|
|
'logged_in' => true,
|
|
'logged_out' => true
|
|
],
|
|
'VectorLanguageInMainPageHeader' => [
|
|
'logged_in' => false,
|
|
'logged_out' => false
|
|
],
|
|
'VectorLanguageAlertInSidebar' => [
|
|
'logged_in' => true,
|
|
'logged_out' => true
|
|
],
|
|
];
|
|
}
|
|
|
|
public function providerLanguageAlertRequirements() {
|
|
$testTitle = Title::makeTitle( NS_MAIN, 'Test' );
|
|
$testTitleMainPage = Title::makeTitle( NS_MAIN, 'MAIN_PAGE' );
|
|
return [
|
|
'When none of the requirements are present, do not show alert' => [
|
|
// Configuration requirements for language in header and alert in sidebar
|
|
[],
|
|
// Title instance
|
|
$testTitle,
|
|
// Cached languages
|
|
[],
|
|
// Is the language selector at the top of the content?
|
|
false,
|
|
// Should the language button be hidden?
|
|
false,
|
|
// Expected
|
|
false
|
|
],
|
|
'When the feature is enabled and languages should be hidden, do not show alert' => [
|
|
$this->enableLanguageAlertFeatureConfig(),
|
|
$testTitle,
|
|
[], true, true, false
|
|
],
|
|
'When the language alert feature is disabled, do not show alert' => [
|
|
[
|
|
'VectorLanguageInHeader' => [
|
|
'logged_in' => true,
|
|
'logged_out' => true
|
|
],
|
|
'VectorLanguageAlertInSidebar' => [
|
|
'logged_in' => false,
|
|
'logged_out' => false
|
|
]
|
|
],
|
|
$testTitle,
|
|
[ 'fr', 'en', 'ko' ], true, false, false
|
|
],
|
|
'When the language in header feature is disabled, do not show alert' => [
|
|
[
|
|
'VectorLanguageInHeader' => [
|
|
'logged_in' => false,
|
|
'logged_out' => false
|
|
],
|
|
'VectorLanguageAlertInSidebar' => [
|
|
'logged_in' => true,
|
|
'logged_out' => true
|
|
]
|
|
],
|
|
$testTitle,
|
|
[ 'fr', 'en', 'ko' ], true, false, false
|
|
],
|
|
'When it is a main page, feature is enabled, and there are no languages, do not show alert' => [
|
|
$this->enableLanguageAlertFeatureConfig(),
|
|
$testTitleMainPage,
|
|
[], true, true, false
|
|
],
|
|
'When it is a non-main page, feature is enabled, and there are no languages, do not show alert' => [
|
|
$this->enableLanguageAlertFeatureConfig(),
|
|
$testTitle,
|
|
[], true, true, false
|
|
],
|
|
'When it is a main page, header feature is disabled, and there are languages, do not show alert' => [
|
|
[
|
|
'VectorLanguageInHeader' => [
|
|
'logged_in' => false,
|
|
'logged_out' => false
|
|
],
|
|
'VectorLanguageAlertInSidebar' => [
|
|
'logged_in' => true,
|
|
'logged_out' => true
|
|
]
|
|
],
|
|
$testTitleMainPage,
|
|
[ 'fr', 'en', 'ko' ], true, true, false
|
|
],
|
|
'When it is a non-main page, alert feature is disabled, there are languages, do not show alert' => [
|
|
[
|
|
'VectorLanguageInHeader' => [
|
|
'logged_in' => true,
|
|
'logged_out' => true
|
|
],
|
|
'VectorLanguageAlertInSidebar' => [
|
|
'logged_in' => false,
|
|
'logged_out' => false
|
|
]
|
|
],
|
|
$testTitle,
|
|
[ 'fr', 'en', 'ko' ], true, true, false
|
|
],
|
|
'When most requirements are present but languages are not at the top, do not show alert' => [
|
|
$this->enableLanguageAlertFeatureConfig(),
|
|
$testTitle,
|
|
[ 'fr', 'en', 'ko' ], false, false, false
|
|
],
|
|
'When most requirements are present but languages should be hidden, do not show alert' => [
|
|
$this->enableLanguageAlertFeatureConfig(),
|
|
$testTitle,
|
|
[ 'fr', 'en', 'ko' ], true, true, false
|
|
],
|
|
'When it is a main page, features are enabled, and there are languages, show alert' => [
|
|
$this->enableLanguageAlertFeatureConfig(),
|
|
$testTitleMainPage,
|
|
[ 'fr', 'en', 'ko' ], true, false, true
|
|
],
|
|
'When all the requirements are present on a non-main page, show alert' => [
|
|
$this->enableLanguageAlertFeatureConfig(),
|
|
$testTitle,
|
|
[ 'fr', 'en', 'ko' ], true, false, true
|
|
],
|
|
];
|
|
}
|
|
|
|
/**
|
|
* @dataProvider providerLanguageAlertRequirements
|
|
* @covers \MediaWiki\Skins\Vector\SkinVector22::shouldLanguageAlertBeInSidebar
|
|
* @param array $requirements
|
|
* @param Title $title
|
|
* @param array $getLanguagesCached
|
|
* @param bool $isLanguagesInContentAt
|
|
* @param bool $shouldHideLanguages
|
|
* @param bool $expected
|
|
* @throws Exception
|
|
*/
|
|
public function testShouldLanguageAlertBeInSidebar(
|
|
array $requirements,
|
|
Title $title,
|
|
array $getLanguagesCached,
|
|
bool $isLanguagesInContentAt,
|
|
bool $shouldHideLanguages,
|
|
bool $expected
|
|
) {
|
|
$this->overrideConfigValues( array_merge( $requirements, [
|
|
'DefaultSkin' => 'vector-2022',
|
|
'VectorDefaultSkinVersion' => '2',
|
|
'VectorSkinMigrationMode' => true,
|
|
] ) );
|
|
|
|
$mockSkinVector = $this->getMockBuilder( SkinVector22::class )
|
|
->disableOriginalConstructor()
|
|
->onlyMethods( [ 'getTitle', 'getLanguagesCached','isLanguagesInContentAt', 'shouldHideLanguages' ] )
|
|
->getMock();
|
|
$mockSkinVector->method( 'getTitle' )
|
|
->willReturn( $title );
|
|
$mockSkinVector->method( 'getLanguagesCached' )
|
|
->willReturn( $getLanguagesCached );
|
|
$mockSkinVector->method( 'isLanguagesInContentAt' )->with( 'top' )
|
|
->willReturn( $isLanguagesInContentAt );
|
|
$mockSkinVector->method( 'shouldHideLanguages' )
|
|
->willReturn( $shouldHideLanguages );
|
|
|
|
$shouldLanguageAlertBeInSidebarMethod = new ReflectionMethod(
|
|
SkinVector22::class,
|
|
'shouldLanguageAlertBeInSidebar'
|
|
);
|
|
$shouldLanguageAlertBeInSidebarMethod->setAccessible( true );
|
|
|
|
$this->assertSame(
|
|
$expected,
|
|
$shouldLanguageAlertBeInSidebarMethod->invoke( $mockSkinVector )
|
|
);
|
|
}
|
|
}
|