Changeset 47122 for trunk/tests/phpunit/tests/upload.php

Timestamp:

01/29/2020 12:43:23 AM (5 years ago)

Author:

SergeyBiryukov

Message:

Docs: Improve inline comments per the documentation standards.

Includes minor code layout fixes for better readability.

See #48303.

File:

• trunk/tests/phpunit/tests/upload.php (modified) (5 diffs)

trunk/tests/phpunit/tests/upload.php

@@ -14 +14 @@
 
     function _reset_options() {
-        // system defaults
+        // System defaults.
         update_option( 'upload_path', 'wp-content/uploads' );
         update_option( 'upload_url_path', '' );
@@ -21 +21 @@
 
     function test_upload_dir_default() {
-        // wp_upload_dir() with default parameters
+        // wp_upload_dir() with default parameters.
         $info   = wp_upload_dir();
         $subdir = gmstrftime( '/%Y/%m' );
@@ -32 +32 @@
 
     function test_upload_dir_relative() {
-        // wp_upload_dir() with a relative upload path that is not 'wp-content/uploads'
+        // wp_upload_dir() with a relative upload path that is not 'wp-content/uploads'.
         update_option( 'upload_path', 'foo/bar' );
         $info   = _wp_upload_dir();
@@ -49 +49 @@
         $path = '/tmp/wp-unit-test';
 
-        // wp_upload_dir() with an absolute upload path
+        // wp_upload_dir() with an absolute upload path.
         update_option( 'upload_path', $path );
 
-        // doesn't make sense to use an absolute file path without setting the url path
+        // Doesn't make sense to use an absolute file path without setting the url path.
         update_option( 'upload_url_path', '/baz' );
 
@@ -93 +93 @@
 
     function test_upload_dir_empty() {
-        // upload path setting is empty - it should default to 'wp-content/uploads'
+        // Upload path setting is empty - it should default to 'wp-content/uploads'.
         update_option( 'upload_path', '' );