Changeset 52010 for trunk/tests/phpunit/tests/pomo/mo.php

Timestamp:

11/04/2021 03:22:47 PM (5 years ago)

Author:

hellofromTonya

Message:

Coding Standards: Add visibility to methods in tests/phpunit/tests/.

Adds a public visibility to test fixtures, tests, data providers, and callbacks methods.

Adds a private visibility to helper methods within test classes.

Renames callbacks and helpers that previously started with a _ prefix. Why? For consistency and to leverage using the method visibility. Further naming standardizations is beyond the scope of this commit.

Props costdev, jrf, hellofromTonya.
Fixes #54177.

File:

• trunk/tests/phpunit/tests/pomo/mo.php (modified) (10 diffs)

trunk/tests/phpunit/tests/pomo/mo.php

@@ -6 +6 @@
 class Tests_POMO_MO extends WP_UnitTestCase {
 
-        function test_mo_simple() {
+        public function test_mo_simple() {
                 $mo = new MO();
                 $mo->import_from_file( DIR_TESTDATA . '/pomo/simple.mo' );
@@ -21 +21 @@
         }
 
-        function test_mo_plural() {
+        public function test_mo_plural() {
                 $mo = new MO();
                 $mo->import_from_file( DIR_TESTDATA . '/pomo/plural.mo' );
@@ -47 +47 @@
         }
 
-        function test_mo_context() {
+        public function test_mo_context() {
                 $mo = new MO();
                 $mo->import_from_file( DIR_TESTDATA . '/pomo/context.mo' );
@@ -74 +74 @@
         }
 
-        function test_translations_merge() {
+        public function test_translations_merge() {
                 $host = new Translations();
                 $host->add_entry( new Translation_Entry( array( 'singular' => 'pink' ) ) );
@@ -86 +86 @@
         }
 
-        function test_export_mo_file() {
+        public function test_export_mo_file() {
                 $entries              = array();
                 $entries[]            = new Translation_Entry(
@@ -144 +144 @@
         }
 
-        function test_export_should_not_include_empty_translations() {
+        public function test_export_should_not_include_empty_translations() {
                 $entries = array();
                 $mo      = new MO;
@@ -163 +163 @@
         }
 
-        function test_nplurals_with_backslashn() {
+        public function test_nplurals_with_backslashn() {
                 $mo = new MO();
                 $mo->import_from_file( DIR_TESTDATA . '/pomo/bad_nplurals.mo' );
@@ -171 +171 @@
         }
 
-        function disabled_test_performance() {
+        public function disabled_test_performance() {
                 $start = microtime( true );
                 $mo    = new MO();
@@ -178 +178 @@
         }
 
-        function test_overloaded_mb_functions() {
+        public function test_overloaded_mb_functions() {
                 if ( ( ini_get( 'mbstring.func_overload' ) & 2 ) === 0 ) {
                         $this->markTestSkipped( 'This test requires mbstring.func_overload to be enabled.' );
@@ -188 +188 @@
         }
 
-        function test_load_pot_file() {
+        public function test_load_pot_file() {
                 $mo = new MO();
                 $this->assertFalse( $mo->import_from_file( DIR_TESTDATA . '/pomo/mo.pot' ) );