This issue is the 7.x version of #2824366: Move tests from /src/tests to tests/src/functional and extend BrowserTestBase instead of WebTestBase. For 7.x it only concerns moving the files, there is no class change.

Move scheduler.test into tests/scheduler.test and update the line in .info to say files[] = tests/scheduler.test. The three files .info, .install and .module for the scheduler_test module should be moved down to tests/modules

CommentFileSizeAuthor
#2 2837008_1.move_test_files_7x.patch100.28 KBjonathan1055
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jonathan1055 created an issue. See original summary.

jonathan1055’s picture

jonathan1055 CreditAttribution: jonathan1055 as a volunteer commented
Status: Active » Needs review
Related issues: +#2824366: Move tests from /src/tests to tests/src/functional and extend BrowserTestBase instead of WebTestBase
FileSize
2837008_1.move_test_files_7x.patch100.28 KB

Here's a patch which moves the four files and amends .info

Status: Needs review » Needs work

The last submitted patch, 2: 2837008_1.move_test_files_7x.patch, failed testing.

jonathan1055’s picture

jonathan1055 CreditAttribution: jonathan1055 as a volunteer commented
Status: Needs work » Needs review

Retesting the patch now that #2837015: CI problem with permission when testing 7.x patches is fixed.

  • jonathan1055 committed 28b4475 on 7.x-1.x 
    Issue #2837008 by jonathan1055: Move scheduler.test into /tests and move...
jonathan1055’s picture

jonathan1055 CreditAttribution: jonathan1055 as a volunteer commented
Title: Move scheduler.test in /tests and move test module in /tests/modules » Move scheduler.test into /tests and move three test module into /tests/modules
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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