Changeset 49354

Timestamp:

10/28/2020 03:12:40 PM (6 years ago)

Author:

SergeyBiryukov

Message:

Themes: Move the test for post-formats theme support to a more appropriate place.

Follow-up to [49344].

See #51390.

Location:

trunk

Files:

• src/wp-includes/theme.php (modified) (1 diff)
• tests/phpunit/tests/post/formats.php (modified) (1 diff)
• tests/phpunit/tests/theme/support.php (modified) (1 diff)

trunk/src/wp-includes/theme.php

@@ -2482 +2482 @@
  * @since 5.5.0 The `core-block-patterns` feature was added and is enabled by default.
  * @since 5.5.0 The `custom-logo` feature now also accepts 'unlink-homepage-logo'.
- * @since 5.6.0 The `post-formats` feature now returns doing it wrong if no array is passed
+ * @since 5.6.0 The `post-formats` feature warns if no array is passed.
  *
  * @global array $_wp_theme_features

trunk/tests/phpunit/tests/post/formats.php

@@ -7 +7 @@
         function setUp() {
                 parent::setUp();
-        }
-
-        /**
-         * @ticket 51390
-         */
-        function test_post_format_doing_it_wrong() {
-                $this->setExpectedIncorrectUsage( "add_theme_support( 'post-formats' )" );
-
-                // The second parameter should be an array.
-                $this->assertFalse( add_theme_support( 'post-formats' ) );
         }
 

trunk/tests/phpunit/tests/theme/support.php

@@ -133 +133 @@
         }
 
+        /**
+         * @ticket 51390
+         *
+         * @expectedIncorrectUsage add_theme_support( 'post-formats' )
+         */
+        function test_supports_post_formats_doing_it_wrong() {
+                // The second parameter should be an array.
+                $this->assertFalse( add_theme_support( 'post-formats' ) );
+        }
+
         function supports_foobar( $yesno, $args, $feature ) {
                 if ( $args[0] === $feature[0] ) {