Changeset 51628

Timestamp:

08/17/2021 08:01:15 PM (3 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: