mirror of
https://github.com/gosticks/wordpress-develop.git
synced 2025-10-16 12:05:38 +00:00
Removes the mutually exclusive directives check in `wp_robots()`, ie allow both `follow` and `nofollow` to be specified and for `archive` and `noarchive` to be specified. This fixes a bug in which WordPress would defer to the most permissive over the least permissive. When contradictory instructions are included, WordPress will defer to the search engine's or archivist's resolution policy: generally this is to observe the least, not most permissive. Props Cybr, flixos90. Fixes #52713. git-svn-id: https://develop.svn.wordpress.org/trunk@50566 602fd350-edb4-49c9-b593-d223f7449a82
161 lines
4.0 KiB
PHP
161 lines
4.0 KiB
PHP
<?php
|
|
/**
|
|
* Robots functions tests.
|
|
*
|
|
* @package WordPress
|
|
*/
|
|
|
|
/**
|
|
* Tests for robots template functions and filters.
|
|
*
|
|
* @group robots
|
|
*/
|
|
class Tests_Robots extends WP_UnitTestCase {
|
|
|
|
public function setUp() {
|
|
parent::setUp();
|
|
|
|
remove_all_filters( 'wp_robots' );
|
|
}
|
|
|
|
/**
|
|
* @ticket 51511
|
|
*/
|
|
public function test_wp_robots_renders_when_relevant() {
|
|
// Do not render robots meta tag when there are no directives.
|
|
$output = get_echo( 'wp_robots' );
|
|
$this->assertEmpty( $output );
|
|
|
|
// Render robots meta tag with noindex.
|
|
add_filter( 'wp_robots', array( $this, 'add_noindex_directive' ) );
|
|
$output = get_echo( 'wp_robots' );
|
|
$this->assertSame( "<meta name='robots' content='noindex' />\n", $output );
|
|
|
|
// Do not render robots meta tag when there are only false-y directives.
|
|
add_filter( 'wp_robots', array( $this, 'remove_noindex_directive' ), 11 );
|
|
$output = get_echo( 'wp_robots' );
|
|
$this->assertEmpty( $output );
|
|
}
|
|
|
|
/**
|
|
* @ticket 51511
|
|
*/
|
|
public function test_wp_robots_parses_directives_correctly() {
|
|
add_filter(
|
|
'wp_robots',
|
|
function( array $robots ) {
|
|
// Directives that should have values must use strings.
|
|
$robots['directive-with-value'] = 'yes';
|
|
$robots['directive-with-numeric-value'] = '1';
|
|
// Any non-string value will be evaluated as boolean.
|
|
// False-y directives will not be included.
|
|
$robots['directive-active-boolean'] = true;
|
|
$robots['directive-inactive-boolean'] = false;
|
|
$robots['directive-active-integer'] = 1;
|
|
$robots['directive-inactive-integer'] = 0;
|
|
return $robots;
|
|
}
|
|
);
|
|
|
|
$expected_directives_string = implode(
|
|
', ',
|
|
array(
|
|
'directive-with-value:yes',
|
|
'directive-with-numeric-value:1',
|
|
'directive-active-boolean',
|
|
'directive-active-integer',
|
|
)
|
|
);
|
|
|
|
$output = get_echo( 'wp_robots' );
|
|
$this->assertContains( "'{$expected_directives_string}'", $output );
|
|
}
|
|
|
|
/**
|
|
* @ticket 51511
|
|
*/
|
|
public function test_wp_robots_noindex() {
|
|
add_filter( 'wp_robots', 'wp_robots_noindex' );
|
|
|
|
update_option( 'blog_public', '1' );
|
|
$output = get_echo( 'wp_robots' );
|
|
$this->assertEmpty( $output );
|
|
|
|
update_option( 'blog_public', '0' );
|
|
$output = get_echo( 'wp_robots' );
|
|
$this->assertContains( "'noindex, nofollow'", $output );
|
|
}
|
|
|
|
/**
|
|
* @ticket 51511
|
|
*/
|
|
public function test_wp_robots_no_robots() {
|
|
add_filter( 'wp_robots', 'wp_robots_no_robots' );
|
|
|
|
update_option( 'blog_public', '1' );
|
|
$output = get_echo( 'wp_robots' );
|
|
$this->assertContains( "'noindex, follow'", $output );
|
|
|
|
update_option( 'blog_public', '0' );
|
|
$output = get_echo( 'wp_robots' );
|
|
$this->assertContains( "'noindex, nofollow'", $output );
|
|
}
|
|
|
|
/**
|
|
* @ticket 51511
|
|
*/
|
|
public function test_wp_robots_sensitive_page() {
|
|
add_filter( 'wp_robots', 'wp_robots_sensitive_page' );
|
|
|
|
$output = get_echo( 'wp_robots' );
|
|
$this->assertContains( "'noindex, noarchive'", $output );
|
|
}
|
|
|
|
/**
|
|
* @ticket 51511
|
|
*/
|
|
public function test_wp_robots_max_image_preview_large() {
|
|
add_filter( 'wp_robots', 'wp_robots_max_image_preview_large' );
|
|
|
|
update_option( 'blog_public', '1' );
|
|
$output = get_echo( 'wp_robots' );
|
|
$this->assertContains( "'max-image-preview:large'", $output );
|
|
|
|
update_option( 'blog_public', '0' );
|
|
$output = get_echo( 'wp_robots' );
|
|
$this->assertEmpty( $output );
|
|
}
|
|
|
|
/**
|
|
* @ticket 52457
|
|
*/
|
|
public function test_wp_robots_search_page() {
|
|
add_filter( 'wp_robots', 'wp_robots_noindex_search' );
|
|
$this->go_to( home_url( '?s=ticket+52457+core.trac.wordpress.org' ) );
|
|
|
|
$output = get_echo( 'wp_robots' );
|
|
$this->assertContains( 'noindex', $output );
|
|
}
|
|
|
|
/**
|
|
* @ticket 52457
|
|
*/
|
|
public function test_wp_robots_non_search_page() {
|
|
add_filter( 'wp_robots', 'wp_robots_noindex_search' );
|
|
$this->go_to( home_url() );
|
|
|
|
$output = get_echo( 'wp_robots' );
|
|
$this->assertNotContains( 'noindex', $output );
|
|
}
|
|
|
|
public function add_noindex_directive( array $robots ) {
|
|
$robots['noindex'] = true;
|
|
return $robots;
|
|
}
|
|
|
|
public function remove_noindex_directive( array $robots ) {
|
|
$robots['noindex'] = false;
|
|
return $robots;
|
|
}
|
|
}
|