Skip to content
Snippets Groups Projects
Verified Commit f6cb4b54 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

(cherry picked from commit 941b7e79)
parent 82bc1eed
Branches
Tags
25 merge requests!122353526426-warning-for-missing,!12212Issue #3445525 by alexpott, japerry, catch, mglaman, longwave: Add BC layer...,!11958Issue #3490507 by alexpott, smustgrave: Fix bogus mocking in...,!11769Issue #3517987: Add option to contextual filters to encode slashes in query parameter.,!11185Issue #3477324 by andypost, alexpott: Fix usage of str_getcsv() and fgetcsv() for PHP 8.4,!10602Issue #3438769 by vinmayiswamy, antonnavi, michelle, amateescu: Sub workspace does not clear,!10301Issue #3469309 by mstrelan, smustgrave, moshe weitzman: Use one-time login...,!10187Issue #3487488 by dakwamine: ExtensionMimeTypeGuesser::guessMimeType must support file names with "0" (zero) like foo.0.zip,!9944Issue #3483353: Consider making the createCopy config action optionally fail...,!9929Issue #3445469 by pooja_sharma, smustgrave: Add additional test coverage for...,!9787Resolve issue 3479427 - bootstrap barrio issue under Windows,!9742Issue #3463908 by catch, quietone: Split OptionsFieldUiTest into two,!9526Issue #3458177 by mondrake, catch, quietone, godotislate, longwave, larowlan,...,!8738Issue #3424162 by camilledavis, dineshkumarbollu, smustgrave: Claro...,!8704Make greek characters available in ckeditor5,!8597Draft: Issue #3442259 by catch, quietone, dww: Reduce time of Migrate Upgrade tests...,!8533Issue #3446962 by kim.pepper: Remove incorrectly added...,!8517Issue #3443748 by NexusNovaz, smustgrave: Testcase creates false positive,!8325Update file Sort.php,!8095Expose document root on install,!7930Resolve #3427374 "Taxonomytid viewsargumentdefault plugin",!7445Issue #3440169: When using drupalGet(), provide an associative array for $headers,!6502Draft: Resolve #2938524 "Plach testing issue",!38582585169-10.1.x,!3226Issue #2987537: Custom menu link entity type should not declare "bundle" entity key
Pipeline #154427 passed with warnings
Pipeline: drupal

#154497

    Pipeline: drupal

    #154483

      Pipeline: drupal

      #154458

        +1
        ......@@ -33,14 +33,6 @@ class HelpTest extends BrowserTestBase {
        '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}
        */
        ......@@ -77,7 +69,10 @@ protected function setUp(): void {
        public function testHelp() {
        // Log in the root user to ensure as many admin links appear as possible on
        // the module overview pages.
        $this->drupalLogin($this->rootUser);
        $this->drupalLogin($this->drupalCreateUser([
        'access help pages',
        'access administration pages',
        ]));
        $this->verifyHelp();
        // Log in the regular user.
        ......
        ......@@ -29,14 +29,6 @@ class HelpTopicSearchTest extends HelpTopicTranslatedTestBase {
        '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}
        */
        ......@@ -266,7 +258,10 @@ public function testUninstall() {
        \Drupal::service('module_installer')->uninstall(['help_topics_test']);
        // Ensure we can uninstall help_topics and use the help system without
        // breaking.
        $this->drupalLogin($this->rootUser);
        $this->drupalLogin($this->createUser([
        'administer modules',
        'access help pages',
        ]));
        $edit = [];
        $edit['uninstall[help]'] = TRUE;
        $this->drupalGet('admin/modules/uninstall');
        ......@@ -283,7 +278,10 @@ public function testUninstall() {
        public function testUninstallSearch() {
        // Ensure we can uninstall search and use the help system without
        // breaking.
        $this->drupalLogin($this->rootUser);
        $this->drupalLogin($this->createUser([
        'administer modules',
        'access help pages',
        ]));
        $edit = [];
        $edit['uninstall[search]'] = TRUE;
        $this->drupalGet('admin/modules/uninstall');
        ......
        ......@@ -29,14 +29,6 @@ class HelpTopicsSyntaxTest extends BrowserTestBase {
        '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}
        */
        ......@@ -46,7 +38,10 @@ class HelpTopicsSyntaxTest extends BrowserTestBase {
        * Tests that all Core help topics can be rendered and have good syntax.
        */
        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
        // will be defined.
        ......
        0% Loading or .
        You are about to add 0 people to the discussion. Proceed with caution.
        Please register or to comment