Skip to content

Tests: Cover rest_find_matching_pattern_property_schema(). #14819

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
ekamran wants to merge 1 commit into
base: trunk
Choose a base branch
from
Open

Tests: Cover rest_find_matching_pattern_property_schema(). #14819

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
129 changes: 129 additions & 0 deletions tests/phpunit/tests/rest-api/restFindMatchingPatternPropertySchema.php
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,129 @@
<?php

/**
* Tests for rest_find_matching_pattern_property_schema().
*
* @group restapi
*
* @covers ::rest_find_matching_pattern_property_schema
*/
class Tests_REST_API_RestFindMatchingPatternPropertySchema extends WP_UnitTestCase {

/**
* Tests that the schema of the matching pattern is returned.
*
* @ticket 65819
*
* @dataProvider data_matching_pattern_properties
*
* @param string|int $property Property name. Numeric names arrive as integers from array keys.
* @param array $pattern_properties Value of the patternProperties keyword.
* @param array $expected Expected schema.
*/
public function test_returns_schema_of_matching_pattern( $property, $pattern_properties, $expected ) {
$schema = array( 'patternProperties' => $pattern_properties );

$this->assertSame( $expected, rest_find_matching_pattern_property_schema( $property, $schema ) );
}

/**
* Data provider.
*
* @return array[]
*/
public static function data_matching_pattern_properties() {
return array(
'single pattern' => array(
'123',
array(
'^[0-9]+$' => array( 'type' => 'integer' ),
),
array( 'type' => 'integer' ),
),
'integer property name' => array(
123,
array(
'^[0-9]+$' => array( 'type' => 'integer' ),
),
array( 'type' => 'integer' ),
),
'pattern matches part of the name' => array(
'prop0',
array(
'[0-9]' => array( 'type' => 'integer' ),
),
array( 'type' => 'integer' ),
),
'match on a later pattern' => array(
'bar',
array(
'^a' => array( 'type' => 'integer' ),
'^b' => array( 'type' => 'string' ),
),
array( 'type' => 'string' ),
),
'pattern containing a hash' => array(
'a#b',
array(
'^a#b$' => array( 'type' => 'string' ),
),
array( 'type' => 'string' ),
),
'multibyte property name' => array(
'é',
array(
'^.$' => array( 'type' => 'string' ),
),
array( 'type' => 'string' ),
),
);
}

/**
* Tests that null is returned when no pattern matches.
*
* @ticket 65819
*
* @dataProvider data_no_matching_pattern_properties
*
* @param string $property Property name.
* @param array $schema Schema to search.
*/
public function test_returns_null_without_matching_pattern( $property, $schema ) {
$this->assertNull( rest_find_matching_pattern_property_schema( $property, $schema ) );
}

/**
* Data provider.
*
* @return array[]
*/
public static function data_no_matching_pattern_properties() {
return array(
'no patternProperties' => array(
'prop',
array( 'type' => 'object' ),
),
'empty patternProperties' => array(
'prop',
array( 'patternProperties' => array() ),
),
'no pattern matches' => array(
'prop',
array(
'patternProperties' => array(
'^[0-9]+$' => array( 'type' => 'integer' ),
),
),
),
'hash in pattern is matched literally' => array(
'ab',
array(
'patternProperties' => array(
'^a#b$' => array( 'type' => 'string' ),
),
),
),
);
}
}
Loading