Problem/Motivation

As part of our plan to retire simpletest we try to split the UI from the rest of the functionality, so we can make decisions easier, see #3028663: Split up simpletest into simpletest and testing_ui to enable easier decisions for Drupal 9.
While doing so we want to also move tests to the right location.

We identified a couple of tests which should rather live in /tests

Proposed resolution

Here is a list of tests which should be moved to /core/tests/Drupal:

  • \Drupal\Tests\simpletest\Functional\FolderTest
  • \Drupal\Tests\simpletest\Functional\MailCaptureTest
  • \Drupal\Tests\simpletest\Functional\UserHelpersTest

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#2 3029661-2.patch2.51 KBLendude
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner created an issue. See original summary.

Lendude’s picture

Lendude
Primary language Dutch
Location Amsterdam
CreditAttribution: Lendude at ezCompany commented
Status: Active » Needs review
FileSize
3029661-2.patch2.51 KB

Here we go.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented
Status: Needs review » Reviewed & tested by the community

Looks fine to me.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Acro Commerce, Thunder commented
Version: 8.7.x-dev » 8.6.x-dev
Status: Reviewed & tested by the community » Fixed

Committed and pushed 185513febe to 8.7.x and 7a3b37ec5d to 8.6.x. Thanks!

Backported to 8.6.x to keep tests consistent.

  • alexpott committed 185513f on 8.7.x 
    Issue #3029661 by Lendude, dawehner: Move BrowserTestBase tests out of...

  • alexpott committed 7a3b37e on 8.6.x 
    Issue #3029661 by Lendude, dawehner: Move BrowserTestBase tests out of...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.