Problem/Motivation

simpletest_generate_file() is unused in core.

Proposed resolution

Deprecate simpletest_generate_file() formally.

Replaced by Drupal\Tests\TestFileCreationTrait.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#8 3028708_8.patch1.9 KBmile23

Comments

Mile23 created an issue. See original summary.

mile23’s picture

mile23
Primary language English
Location Seattle, WA
commented
Issue tags: +Needs change record
mile23’s picture

mile23
Primary language English
Location Seattle, WA
commented
Issue summary: View changes
mile23’s picture

mile23
Primary language English
Location Seattle, WA
commented
Issue summary: View changes
mile23’s picture

mile23
Primary language English
Location Seattle, WA
commented
Related issues: +#2738567: Add test trait for drupalGetTestFiles and drupalCompareFile

Version: 8.7.x-dev » 8.8.x-dev

Drupal 8.7.0-alpha1 will be released the week of March 11, 2019, which means new developments and disruptive changes should now be targeted against the 8.8.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

mile23’s picture

mile23
Primary language English
Location Seattle, WA
commented

CR for this issue and #2738567: Add test trait for drupalGetTestFiles and drupalCompareFile : https://www.drupal.org/node/3077768

mile23’s picture

mile23
Primary language English
Location Seattle, WA
commented
Status: Active » Needs review
Issue tags: -Needs change record
StatusFileSize
new 3028708_8.patch1.9 KB
mile23’s picture

mile23
Primary language English
Location Seattle, WA
commented
Related issues: +#3057420: [meta] How to deprecate Simpletest with minimal disruption
kim.pepper’s picture

kim.pepper
Primary language English
Location 🏄‍♂️🇦🇺Sydney, Australia
commented
Status: Needs review » Reviewed & tested by the community

LGTM 👍

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed 
+++ b/core/modules/simpletest/tests/src/Kernel/SimpletestDeprecationTest.php
@@ -82,4 +82,14 @@ public function testDeprecatedPhpUnitFunctions() {
+    $this->assertTrue(unlink($public_file));

Strictly this is unnecessary because we remove all files as far as I know. But can't hurt so /shrug.

Committed a99b54f and pushed to 8.8.x. Thanks!

  • alexpott committed a99b54f on 8.8.x 
    Issue #3028708 by Mile23: Deprecate simpletest_generate_file()

Status: Fixed » Closed (fixed)

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

quietone’s picture

quietone commented

publish the change record