Changeset 51628

Timestamp:

08/17/2021 08:01:15 PM (5 years ago)

Author:

SergeyBiryukov

Message:

Tests: Move loading the PO class to set_up_before_class().

This ensures that the class is loaded once before the first test of the test case class is run, and require_once() is not unnecessarily called for each test method individually.

Follow-up to [1106/tests].

See #53363.

File:

• trunk/tests/phpunit/tests/pomo/po.php (modified) (1 diff)

trunk/tests/phpunit/tests/pomo/po.php

@@ -5 +5 @@
  */
 class Tests_POMO_PO extends WP_UnitTestCase {
-        function set_up() {
+
+        public static function set_up_before_class() {
+                parent::set_up_before_class();
+
+                require_once ABSPATH . '/wp-includes/pomo/po.php';
+        }
+
+        public function set_up() {
                 parent::set_up();
-                require_once ABSPATH . '/wp-includes/pomo/po.php';
+
                 // Not so random wordpress.pot string -- multiple lines.
                 $this->mail    = 'Your new WordPress blog has been successfully set up at: