Skip to content
Snippets Groups Projects
Commit d9f9f161 authored by catch's avatar catch
Browse files

Issue #3281429 by tinto, nod_, deviantintegral, lauriii: Update Block...

Issue #3281429 by tinto, nod_, deviantintegral, lauriii: Update Block non-Migration tests to not use Bartik and Seven

(cherry picked from commit 763577f8)
parent bc5c3863
Branches
Tags
33 merge requests!8394[warning] array_flip(): Can only flip STRING and INTEGER values, when saving a non-revisionable custom content entity,!7780issue 3443822: fix for 'No route found for the specified format html. Supported formats: json, xml.',!7416Simplify the HTML of field.html.twig,!7150Revert "Issue #3137119 by munish.kumar, johnwebdev, Jaypan, jungle, xjm,...,!5013Issue #3071143: Table Render Array Example Is Incorrect,!4848Issue #1566662: Update module should send notifications on Thursdays,!4792Issue #2230689: Remove redundant "Italic" style,!4782Issue #2662898: "Links" field not displaying on custom view modes,!4220Issue #3368223: Link field > Access to internal links is not checked on display.,!4173Issue #2123543: Add string context and location filters to the translate interface,!3884Issue #3356842,!3870Issue #3087868,!3812Draft: Issue #3339373 by alexpott, andypost, mondrake:...,!3736Issue #3294005: Refactor Claro's form--password-confirm stylesheet,!3686Issue #3219967 against 9.5.x,!3683Issue #2939397: Clearing AliasManager cache with root path raises warning,!3543Issue #3344259: Allow ajax dialog to have focus configurable,!3356Issue #3209129: Scrolling problems when adding a block via layout builder,!2982Issue #3301562: Translate the default settings for this plugin (TimestampAgoFormatter),!2921Issue #1383696: Allow a custom HTML element to be selected for a grouping field,!2920Issue #3260175: Saving media entity without an owner crashes,!2857Issue #3314541: Remove unnecessary fill from SVG icon for the "Media Library" CKEditor 5 button — enabling dark mode support in contrib,!2841Resolve #3296811 "Resourceresponsetrait needs a",!2803Issue #3041402: Add option absolute url in formatter URL to image,!2280Issue #3280415: Metapackage Generator Breaks Under Composer --no-dev,!2205Quote all names in the regions section.,!2050Issue #3272969: Remove UnqiueField constraint.,!1956Issue #3268872: hook_views_invalidate_cache not called when a view is deleted,!1893Issue #3217260: Add a way to make media captions not editable in CKEditor,!1459Issue #3087632: menu_name max length is too long,!878Issue #3221534: throw an exception when IDs passed to loadMultiple() are badly formed,!866Issue #2845319: The highlighting of the 'Home' menu-link does not respect query strings and fragment identifiers,!204Issue #3040556: It is not possible to react to an entity being duplicated
......@@ -36,22 +36,22 @@ public function testAdminTheme() {
// Ensure that access to block admin page is denied when theme is not
// installed.
$this->drupalGet('admin/structure/block/list/bartik');
$this->drupalGet('admin/structure/block/list/olivero');
$this->assertSession()->statusCodeEquals(403);
// Install admin theme and confirm that tab is accessible.
\Drupal::service('theme_installer')->install(['bartik']);
$edit['admin_theme'] = 'bartik';
\Drupal::service('theme_installer')->install(['olivero']);
$edit['admin_theme'] = 'olivero';
$this->drupalGet('admin/appearance');
$this->submitForm($edit, 'Save configuration');
$this->drupalGet('admin/structure/block/list/bartik');
$this->drupalGet('admin/structure/block/list/olivero');
$this->assertSession()->statusCodeEquals(200);
}
/**
* Ensure contextual links are disabled in Seven theme.
* Ensure contextual links are disabled in Claro theme.
*/
public function testSevenAdminTheme() {
public function testClaroAdminTheme() {
// Create administrative user.
$admin_user = $this->drupalCreateUser([
'access administration pages',
......@@ -62,14 +62,14 @@ public function testSevenAdminTheme() {
$this->drupalLogin($admin_user);
// Install admin theme and confirm that tab is accessible.
\Drupal::service('theme_installer')->install(['seven']);
$edit['admin_theme'] = 'seven';
\Drupal::service('theme_installer')->install(['claro']);
$edit['admin_theme'] = 'claro';
$this->drupalGet('admin/appearance');
$this->submitForm($edit, 'Save configuration');
// Define our block settings.
$settings = [
'theme' => 'seven',
'theme' => 'claro',
'region' => 'header',
];
......
......@@ -44,9 +44,8 @@ public function testBlockDemo() {
// All available themes in core.
$available_themes = [
'bartik',
'olivero',
'seven',
'claro',
'stark',
];
......
......@@ -270,9 +270,8 @@ public function testBlock() {
public function testBlockThemeSelector() {
// Install all themes.
$themes = [
'bartik',
'olivero',
'seven',
'claro',
'stark',
];
\Drupal::service('theme_installer')->install($themes);
......@@ -530,14 +529,14 @@ public function testUninstallTheme() {
/** @var \Drupal\Core\Extension\ThemeInstallerInterface $theme_installer */
$theme_installer = \Drupal::service('theme_installer');
$theme_installer->install(['seven']);
$this->config('system.theme')->set('default', 'seven')->save();
$block = $this->drupalPlaceBlock('system_powered_by_block', ['theme' => 'seven', 'region' => 'help']);
$theme_installer->install(['claro']);
$this->config('system.theme')->set('default', 'claro')->save();
$block = $this->drupalPlaceBlock('system_powered_by_block', ['theme' => 'claro', 'region' => 'help']);
$this->drupalGet('<front>');
$this->assertSession()->pageTextContains('Powered by Drupal');
$this->config('system.theme')->set('default', 'stark')->save();
$theme_installer->uninstall(['seven']);
$theme_installer->uninstall(['claro']);
// Ensure that the block configuration does not exist anymore.
$this->assertNull(Block::load($block->id()));
......
......@@ -41,11 +41,11 @@ public function testNonDefaultBlockAdmin() {
'administer themes',
]);
$this->drupalLogin($admin_user);
$new_theme = 'bartik';
$new_theme = 'olivero';
\Drupal::service('theme_installer')->install([$new_theme]);
// Ensure that the Bartik tab is shown.
// Ensure that the Olivero tab is shown.
$this->drupalGet('admin/structure/block/list/' . $new_theme);
$this->assertSession()->pageTextContains('Bartik(active tab)');
$this->assertSession()->pageTextContains('Olivero(active tab)');
}
}
......@@ -28,7 +28,7 @@ class BlockAddTest extends WebDriverTestBase {
* Tests the AJAX for the theme selector.
*/
public function testBlockAddThemeSelector() {
\Drupal::service('theme_installer')->install(['seven']);
\Drupal::service('theme_installer')->install(['claro']);
$this->drupalLogin($this->drupalCreateUser([
'administer blocks',
......@@ -37,7 +37,7 @@ public function testBlockAddThemeSelector() {
$this->drupalGet('admin/structure/block/add/system_powered_by_block');
$assert_session = $this->assertSession();
// Pick a theme with a region that does not exist in another theme.
$assert_session->selectExists('Theme')->selectOption('seven');
$assert_session->selectExists('Theme')->selectOption('claro');
$assert_session->assertWaitOnAjaxRequest();
$assert_session->selectExists('Region')->selectOption('pre_content');
$assert_session->assertWaitOnAjaxRequest();
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment