Changeset 52009 for trunk/tests/phpunit/includes/mock-fs.php

Timestamp:

11/04/2021 01:15:33 PM (3 years ago)

Author:

hellofromTonya

Message:

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

This commit adds the public visibility keyword to each method which did not have an explicit visibility keyword.

Why public?

With no visibility previously declared, these methods are implicitly public and available for use. As these are part of the WordPress testing framework (for Core and extenders), changing them to anything else would be a backwards-compatibility break.

Props costdev, jrf, hellofromTonya.
See #54177.

File:

• trunk/tests/phpunit/includes/mock-fs.php (modified) (11 diffs)

trunk/tests/phpunit/includes/mock-fs.php

@@ -14 +14 @@
     public $method  = 'MockFS';
 
-    function __construct() {}
-
-    function connect() {
+    public function __construct() {}
+
+    public function connect() {
         return true;
     }
 
     // Copy of core's function, but accepts a path.
-    function abspath( $path = false ) {
+    public function abspath( $path = false ) {
         if ( ! $path ) {
             $path = ABSPATH;
@@ -41 +41 @@
      * Can also be passed the initial filesystem to be setup which is passed to self::setfs()
      */
-    function init( $paths = '', $home_dir = '/' ) {
+    public function init( $paths = '', $home_dir = '/' ) {
         $this->fs     = new MockFS_Directory_Node( '/' );
         $this->fs_map = array(
@@ -54 +54 @@
      * "Bulk Loads" a filesystem into the internal virtual filesystem
      */
-    function setfs( $paths ) {
+    public function setfs( $paths ) {
         if ( ! is_array( $paths ) ) {
             $paths = explode( "\n", $paths );
@@ -94 +94 @@
     // Here starteth the WP_Filesystem functions.
 
-    function mkdir( $path, /* Optional args are ignored */ $chmod = false, $chown = false, $chgrp = false ) {
+    public function mkdir( $path, /* Optional args are ignored */ $chmod = false, $chown = false, $chgrp = false ) {
         $path = trailingslashit( $path );
 
@@ -115 +115 @@
     }
 
-    function put_contents( $path, $contents = '', $mode = null ) {
+    public function put_contents( $path, $contents = '', $mode = null ) {
         if ( ! $this->is_dir( dirname( $path ) ) ) {
             $this->mkdir( dirname( $path ) );
@@ -127 +127 @@
     }
 
-    function get_contents( $file ) {
+    public function get_contents( $file ) {
         if ( ! $this->is_file( $file ) ) {
             return false;
@@ -134 +134 @@
     }
 
-    function cwd() {
+    public function cwd() {
         return $this->cwd->path;
     }
 
-    function chdir( $path ) {
+    public function chdir( $path ) {
         if ( ! isset( $this->fs_map[ $path ] ) ) {
             return false;
@@ -147 +147 @@
     }
 
-    function exists( $path ) {
+    public function exists( $path ) {
         return isset( $this->fs_map[ $path ] ) || isset( $this->fs_map[ trailingslashit( $path ) ] );
     }
 
-    function is_file( $file ) {
+    public function is_file( $file ) {
         return isset( $this->fs_map[ $file ] ) && $this->fs_map[ $file ]->is_file();
     }
 
-    function is_dir( $path ) {
+    public function is_dir( $path ) {
         $path = trailingslashit( $path );
 
@@ -161 +161 @@
     }
 
-    function dirlist( $path = '.', $include_hidden = true, $recursive = false ) {
+    public function dirlist( $path = '.', $include_hidden = true, $recursive = false ) {
 
         if ( empty( $path ) || '.' === $path ) {
@@ -215 +215 @@
     public $path; // The full path to the entry.
 
-    function __construct( $path ) {
+    public function __construct( $path ) {
         $this->path = $path;
         $this->name = basename( $path );
     }
 
-    function is_file() {
+    public function is_file() {
         return 'f' === $this->type;
     }
 
-    function is_dir() {
+    public function is_dir() {
         return 'd' === $this->type;
     }
@@ -238 +238 @@
     public $contents = ''; // The contents of the file.
 
-    function __construct( $path, $contents = '' ) {
+    public function __construct( $path, $contents = '' ) {
         parent::__construct( $path );
         $this->contents = $contents;