Changeset 54403

Timestamp:

10/07/2022 01:07:28 AM (4 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';
         }