Changeset 54403

Timestamp:

10/07/2022 01:07:28 AM (2 years ago)

Author:

SergeyBiryukov

Message:

Tests: Clean up test image for site icon in Tests_REST_Server on teardown.

This makes sure there are no leftover images after the test class is run.

Follow-up to [52080], [53463], [53920].

See #55652.

File:

• trunk/tests/phpunit/tests/rest-api/rest-server.php (modified) (2 diffs)

trunk/tests/phpunit/tests/rest-api/rest-server.php

@@ -13 +13 @@
     protected static $icon_id;
 
+    /**
+     * Called before setting up all tests.
+     */
+    public static function set_up_before_class() {
+        parent::set_up_before_class();
+
+        // Require files that need to load once.
+        require_once DIR_TESTROOT . '/includes/mock-invokable.php';
+    }
+
     public static function wpSetUpBeforeClass( WP_UnitTest_Factory $factory ) {
         $filename      = DIR_TESTDATA . '/images/test-image-large.jpg';
         self::$icon_id = $factory->attachment->create_upload_object( $filename );
+    }
+
+    public static function tear_down_after_class() {
+        wp_delete_attachment( self::$icon_id, true );
+
+        parent::tear_down_after_class();
     }
 
@@ -34 +50 @@
 
         parent::tear_down();
-    }
-
-    /**
-     * Called before setting up all tests.
-     */
-    public static function set_up_before_class() {
-        parent::set_up_before_class();
-
-        // Require files that need to load once.
-        require_once DIR_TESTROOT . '/includes/mock-invokable.php';
     }