WordPress.org

Make WordPress Core

Ticket #43280: 43280.3.diff

File 43280.3.diff, 1.4 KB (added by welcher, 15 months ago)

Adds some unit tests

  • src/wp-includes/widgets/class-wp-widget-media-image.php

    diff --git src/wp-includes/widgets/class-wp-widget-media-image.php src/wp-includes/widgets/class-wp-widget-media-image.php
    index 2b9bff221b..a0359bcbfd 100644
    class WP_Widget_Media_Image extends WP_Widget_Media { 
    261261                                $link .= sprintf( ' rel="%s"', esc_attr( $instance['link_rel'] ) );
    262262                        }
    263263                        if ( ! empty( $instance['link_target_blank'] ) ) {
     264                                if ( empty( $instance['link_rel'] ) ) {
     265                                        $link .= ' rel="noopener noreferrer"';
     266                                }
    264267                                $link .= ' target="_blank"';
    265268                        }
    266269                        $link .= '>';
  • tests/phpunit/tests/widgets/media-image-widget.php

    diff --git tests/phpunit/tests/widgets/media-image-widget.php tests/phpunit/tests/widgets/media-image-widget.php
    index e138f3e796..755a65c171 100644
    class Test_WP_Widget_Media_Image extends WP_UnitTestCase { 
    503503                                'link_type'         => 'custom',
    504504                                'link_url'          => 'https://example.org',
    505505                                'link_target_blank' => true,
     506                                'link_rel'          => false,
    506507                        )
    507508                );
    508509                $output = ob_get_clean();
    509510
    510511                $this->assertContains( '<a href="https://example.org"', $output );
    511512                $this->assertContains( 'target="_blank"', $output );
     513                $this->assertContains( 'rel="noopener noreferrer"', $output );
    512514
    513515                // Populate caption in attachment.
    514516                wp_update_post(