Skip to content
Snippets Groups Projects
Verified Commit 941b7e79 authored by Alex Pott's avatar Alex Pott
Browse files

Issue #3439871 by SolimanHarkas, vensires: Fix Help tests that rely on UID1's super user behavior

parent 1c914f2d
No related branches found
No related tags found
29 merge requests!12227Issue #3181946 by jonmcl, mglaman,!11131[10.4.x-only-DO-NOT-MERGE]: Issue ##2842525 Ajax attached to Views exposed filter form does not trigger callbacks,!9470[10.3.x-only-DO-NOT-MERGE]: #3331771 Fix file_get_contents(): Passing null to parameter,!8540Issue #3457061: Bootstrap Modal dialog Not closing after 10.3.0 Update,!8528Issue #3456871 by Tim Bozeman: Support NULL services,!8373Issue #3427374 by danflanagan8, Vighneshh: taxonomy_tid ViewsArgumentDefault...,!5423Draft: Resolve #3329907 "Test2",!3878Removed unused condition head title for views,!3818Issue #2140179: $entity->original gets stale between updates,!3742Issue #3328429: Create item list field formatter for displaying ordered and unordered lists,!3731Claro: role=button on status report items,!3651Issue #3347736: Create new SDC component for Olivero (header-search),!3531Issue #3336994: StringFormatter always displays links to entity even if the user in context does not have access,!3478Issue #3337882: Deleted menus are not removed from content type config,!3355Issue #3209129: Scrolling problems when adding a block via layout builder,!3154Fixes #2987987 - CSRF token validation broken on routes with optional parameters.,!3133core/modules/system/css/components/hidden.module.css,!2964Issue #2865710 : Dependencies from only one instance of a widget are used in display modes,!2812Issue #3312049: [Followup] Fix Drupal.Commenting.FunctionComment.MissingReturnType returns for NULL,!2378Issue #2875033: Optimize joins and table selection in SQL entity query implementation,!2062Issue #3246454: Add weekly granularity to views date sort,!1105Issue #3025039: New non translatable field on translatable content throws error,!1073issue #3191727: Focus states on mobile second level navigation items fixed,!10223132456: Fix issue where views instances are emptied before an ajax request is complete,!877Issue #2708101: Default value for link text is not saved,!617Issue #3043725: Provide a Entity Handler for user cancelation,!579Issue #2230909: Simple decimals fail to pass validation,!560Move callback classRemove outside of the loop,!555Issue #3202493
Pipeline #154534 canceled
Pipeline: drupal

#154536

    ...@@ -33,14 +33,6 @@ class HelpTest extends BrowserTestBase { ...@@ -33,14 +33,6 @@ class HelpTest extends BrowserTestBase {
    'history', 'history',
    ]; ];
    /**
    * {@inheritdoc}
    *
    * @todo Remove and fix test to not rely on super user.
    * @see https://www.drupal.org/project/drupal/issues/3437620
    */
    protected bool $usesSuperUserAccessPolicy = TRUE;
    /** /**
    * {@inheritdoc} * {@inheritdoc}
    */ */
    ...@@ -77,7 +69,10 @@ protected function setUp(): void { ...@@ -77,7 +69,10 @@ protected function setUp(): void {
    public function testHelp() { public function testHelp() {
    // Log in the root user to ensure as many admin links appear as possible on // Log in the root user to ensure as many admin links appear as possible on
    // the module overview pages. // the module overview pages.
    $this->drupalLogin($this->rootUser); $this->drupalLogin($this->drupalCreateUser([
    'access help pages',
    'access administration pages',
    ]));
    $this->verifyHelp(); $this->verifyHelp();
    // Log in the regular user. // Log in the regular user.
    ......
    ...@@ -29,14 +29,6 @@ class HelpTopicSearchTest extends HelpTopicTranslatedTestBase { ...@@ -29,14 +29,6 @@ class HelpTopicSearchTest extends HelpTopicTranslatedTestBase {
    'language', 'language',
    ]; ];
    /**
    * {@inheritdoc}
    *
    * @todo Remove and fix test to not rely on super user.
    * @see https://www.drupal.org/project/drupal/issues/3437620
    */
    protected bool $usesSuperUserAccessPolicy = TRUE;
    /** /**
    * {@inheritdoc} * {@inheritdoc}
    */ */
    ...@@ -266,7 +258,10 @@ public function testUninstall() { ...@@ -266,7 +258,10 @@ public function testUninstall() {
    \Drupal::service('module_installer')->uninstall(['help_topics_test']); \Drupal::service('module_installer')->uninstall(['help_topics_test']);
    // Ensure we can uninstall help_topics and use the help system without // Ensure we can uninstall help_topics and use the help system without
    // breaking. // breaking.
    $this->drupalLogin($this->rootUser); $this->drupalLogin($this->createUser([
    'administer modules',
    'access help pages',
    ]));
    $edit = []; $edit = [];
    $edit['uninstall[help]'] = TRUE; $edit['uninstall[help]'] = TRUE;
    $this->drupalGet('admin/modules/uninstall'); $this->drupalGet('admin/modules/uninstall');
    ...@@ -283,7 +278,10 @@ public function testUninstall() { ...@@ -283,7 +278,10 @@ public function testUninstall() {
    public function testUninstallSearch() { public function testUninstallSearch() {
    // Ensure we can uninstall search and use the help system without // Ensure we can uninstall search and use the help system without
    // breaking. // breaking.
    $this->drupalLogin($this->rootUser); $this->drupalLogin($this->createUser([
    'administer modules',
    'access help pages',
    ]));
    $edit = []; $edit = [];
    $edit['uninstall[search]'] = TRUE; $edit['uninstall[search]'] = TRUE;
    $this->drupalGet('admin/modules/uninstall'); $this->drupalGet('admin/modules/uninstall');
    ......
    ...@@ -29,14 +29,6 @@ class HelpTopicsSyntaxTest extends BrowserTestBase { ...@@ -29,14 +29,6 @@ class HelpTopicsSyntaxTest extends BrowserTestBase {
    'locale', 'locale',
    ]; ];
    /**
    * {@inheritdoc}
    *
    * @todo Remove and fix test to not rely on super user.
    * @see https://www.drupal.org/project/drupal/issues/3437620
    */
    protected bool $usesSuperUserAccessPolicy = TRUE;
    /** /**
    * {@inheritdoc} * {@inheritdoc}
    */ */
    ...@@ -46,7 +38,10 @@ class HelpTopicsSyntaxTest extends BrowserTestBase { ...@@ -46,7 +38,10 @@ class HelpTopicsSyntaxTest extends BrowserTestBase {
    * Tests that all Core help topics can be rendered and have good syntax. * Tests that all Core help topics can be rendered and have good syntax.
    */ */
    public function testHelpTopics() { public function testHelpTopics() {
    $this->drupalLogin($this->rootUser); $this->drupalLogin($this->createUser([
    'administer modules',
    'access help pages',
    ]));
    // Enable all modules and themes, so that all routes mentioned in topics // Enable all modules and themes, so that all routes mentioned in topics
    // will be defined. // will be defined.
    ......
    0% Loading or .
    You are about to add 0 people to the discussion. Proceed with caution.
    Please register or to comment