WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
01/15/2025 12:44:43 PM (5 months ago)
Author:
swissspidy
Message:

REST API: Improve autosave and revision endpoints for templates and template parts.

Fixes those endpoints for file-based templates and template parts, as templates based on theme files can't be revisioned or autosaved.

Props antonvlasenko, swissspidy, spacedmonkey, kadamwhite.
Fixes #61970.

Location:
trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • trunk

  • trunk/tests/phpunit/tests/rest-api/wpRestTemplateAutosavesController.php

    r59073 r59605  
    2323     * @var string
    2424     */
    25     const PARENT_POST_TYPE = 'wp_template';
     25    const TEMPLATE_PART_NAME = 'my_template_part';
     26
     27    /**
     28     * @var string
     29     */
     30    const TEMPLATE_POST_TYPE = 'wp_template';
     31
     32    /**
     33     * @var string
     34     */
     35    const TEMPLATE_PART_POST_TYPE = 'wp_template_part';
    2636
    2737    /**
    … …  
    5363
    5464    /**
    55      * Create fake data before our tests run.
     65     * Template part post.
     66     *
     67     * @since 6.7.0
     68     *
     69     * @var WP_Post
     70     */
     71    private static $template_part_post;
     72
     73    /**
     74     * Create fake data before the tests run.
    5675     *
    5776     * @param WP_UnitTest_Factory $factory Helper that lets us create fake data.
    … …  
    7493        self::$template_post = $factory->post->create_and_get(
    7594            array(
    76                 'post_type'    => self::PARENT_POST_TYPE,
     95                'post_type'    => self::TEMPLATE_POST_TYPE,
    7796                'post_name'    => self::TEMPLATE_NAME,
    78                 'post_title'   => 'My Template',
     97                'post_title'   => 'My template',
    7998                'post_content' => 'Content',
    8099                'post_excerpt' => 'Description of my template',
    … …  
    87106        );
    88107        wp_set_post_terms( self::$template_post->ID, self::TEST_THEME, 'wp_theme' );
     108
     109        // Set up template part post.
     110        self::$template_part_post = $factory->post->create_and_get(
     111            array(
     112                'post_type'    => self::TEMPLATE_PART_POST_TYPE,
     113                'post_name'    => self::TEMPLATE_PART_NAME,
     114                'post_title'   => 'My template part',
     115                'post_content' => 'Content',
     116                'post_excerpt' => 'Description of my template part',
     117                'tax_input'    => array(
     118                    'wp_theme'              => array(
     119                        self::TEST_THEME,
     120                    ),
     121                    'wp_template_part_area' => array(
     122                        WP_TEMPLATE_PART_AREA_HEADER,
     123                    ),
     124                ),
     125            )
     126        );
     127        wp_set_post_terms( self::$template_part_post->ID, self::TEST_THEME, 'wp_theme' );
     128        wp_set_post_terms( self::$template_part_post->ID, WP_TEMPLATE_PART_AREA_HEADER, 'wp_template_part_area' );
    89129    }
    90130
    … …  
    118158
    119159    /**
     160     * @coversNothing
     161     * @ticket 56922
     162     */
     163    public function test_context_param() {
     164        // A proper data provider cannot be used because this method's signature must match the parent method.
     165        // Therefore, actual tests are performed in the test_context_param_with_data_provider method.
     166        $this->assertTrue( true );
     167    }
     168
     169    /**
     170     * @dataProvider data_context_param_with_data_provider
    120171     * @covers WP_REST_Template_Autosaves_Controller::get_context_param
    121172     * @ticket 56922
    122      */
    123     public function test_context_param() {
     173     *
     174     * @param string $rest_base   Base part of the REST API endpoint to test.
     175     * @param string $template_id Template ID to use in the test.
     176     */
     177    public function test_context_param_with_data_provider( $rest_base, $template_id ) {
    124178        // Collection.
    125         $request  = new WP_REST_Request( 'OPTIONS', '/wp/v2/templates/' . self::TEST_THEME . '/' . self::TEMPLATE_NAME . '/autosaves' );
     179        $request  = new WP_REST_Request( 'OPTIONS', '/wp/v2/' . $rest_base . '/' . $template_id . '/autosaves' );
    126180        $response = rest_get_server()->dispatch( $request );
    127181        $data     = $response->get_data();
    … …  
    145199
    146200        // Single.
    147         $request  = new WP_REST_Request( 'OPTIONS', '/wp/v2/templates/' . self::TEST_THEME . '/' . self::TEMPLATE_NAME . '/autosaves/1' );
     201        $request  = new WP_REST_Request( 'OPTIONS', '/wp/v2/' . $rest_base . '/' . $template_id . '/autosaves/1' );
    148202        $response = rest_get_server()->dispatch( $request );
    149203        $data     = $response->get_data();
    … …  
    166220
    167221    /**
     222     * Data provider for test_context_param_with_data_provider.
     223     *
     224     * @return array
     225     */
     226    public function data_context_param_with_data_provider() {
     227        return array(
     228            'templates'      => array( 'templates', self::TEST_THEME . '//' . self::TEMPLATE_NAME ),
     229            'template parts' => array( 'template-parts', self::TEST_THEME . '//' . self::TEMPLATE_PART_NAME ),
     230        );
     231    }
     232
     233    /**
     234     * @coversNothing
     235     * @ticket 56922
     236     */
     237    public function test_get_items() {
     238        // A proper data provider cannot be used because this method's signature must match the parent method.
     239        // Therefore, actual tests are performed in the test_get_items_with_data_provider method.
     240        $this->assertTrue( true );
     241    }
     242
     243    /**
     244     * @dataProvider data_get_items_with_data_provider
    168245     * @covers WP_REST_Template_Autosaves_Controller::get_items
    169246     * @ticket 56922
    170      */
    171     public function test_get_items() {
     247     *
     248     * @param string $parent_post_property_name  A class property name that contains the parent post object.
     249     * @param string $rest_base                  Base part of the REST API endpoint to test.
     250     * @param string $template_id                Template ID to use in the test.
     251     */
     252    public function test_get_items_with_data_provider( $parent_post_property_name, $rest_base, $template_id ) {
    172253        wp_set_current_user( self::$admin_id );
     254        // Cannot access this property in the data provider because it is not initialized at the time of execution.
     255        $parent_post      = self::$$parent_post_property_name;
    173256        $autosave_post_id = wp_create_post_autosave(
    174257            array(
    175258                'post_content' => 'Autosave content.',
    176                 'post_ID'      => self::$template_post->ID,
    177                 'post_type'    => self::PARENT_POST_TYPE,
     259                'post_ID'      => $parent_post->ID,
     260                'post_type'    => $parent_post->post_type,
    178261            )
    179262        );
    … …  
    181264        $request   = new WP_REST_Request(
    182265            'GET',
    183             '/wp/v2/templates/' . self::TEST_THEME . '/' . self::TEMPLATE_NAME . '/autosaves'
     266            '/wp/v2/' . $rest_base . '/' . $template_id . '/autosaves'
    184267        );
    185268        $response  = rest_get_server()->dispatch( $request );
    186269        $autosaves = $response->get_data();
     270        $this->assertSame( WP_Http::OK, $response->get_status(), 'Response is expected to have a status code of 200.' );
    187271
    188272        $this->assertCount(
    … …  
    198282        );
    199283        $this->assertSame(
    200             self::$template_post->ID,
     284            $parent_post->ID,
    201285            $autosaves[0]['parent'],
    202286            'Failed asserting that the parent ID of the autosave matches the template post ID.'
    … …  
    210294
    211295    /**
     296     * Data provider for test_get_items_with_data_provider.
     297     *
     298     * @return array
     299     */
     300    public function data_get_items_with_data_provider() {
     301        return array(
     302            'templates'      => array( 'template_post', 'templates', self::TEST_THEME . '//' . self::TEMPLATE_NAME ),
     303            'template parts' => array( 'template_part_post', 'template-parts', self::TEST_THEME . '//' . self::TEMPLATE_PART_NAME ),
     304        );
     305    }
     306
     307    /**
     308     * @dataProvider data_get_items_for_templates_based_on_theme_files_should_return_bad_response_status
     309     * @ticket 61970
     310     *
     311     * @param string $rest_base   Base part of the REST API endpoint to test.
     312     * @param string $template_id Template ID to use in the test.
     313     */
     314    public function test_get_items_for_templates_based_on_theme_files_should_return_bad_response_status( $rest_base, $template_id ) {
     315        wp_set_current_user( self::$admin_id );
     316        switch_theme( 'block-theme' );
     317
     318        $request = new WP_REST_Request( 'GET', '/wp/v2/' . $rest_base . '/' . $template_id . '/autosaves' );
     319
     320        $response = rest_get_server()->dispatch( $request );
     321        $this->assertErrorResponse(
     322            'rest_invalid_template',
     323            $response,
     324            WP_Http::BAD_REQUEST,
     325            sprintf( 'Response is expected to have a status code of %d.', WP_Http::BAD_REQUEST )
     326        );
     327    }
     328
     329    /**
     330     * Data provider for test_get_items_for_templates_based_on_theme_files_should_return_bad_response_status.
     331     *
     332     * @return array
     333     */
     334    public function data_get_items_for_templates_based_on_theme_files_should_return_bad_response_status() {
     335        return array(
     336            'templates'      => array( 'templates', self::TEST_THEME . '//page-home' ),
     337            'template parts' => array( 'template-parts', self::TEST_THEME . '//small-header' ),
     338        );
     339    }
     340
     341    /**
     342     * @dataProvider data_get_item_for_templates_based_on_theme_files_should_return_bad_response_status
     343     * @ticket 56922
     344     *
     345     * @param string $rest_base   Base part of the REST API endpoint to test.
     346     * @param string $template_id Template ID to use in the test.
     347     */
     348    public function test_get_item_for_templates_based_on_theme_files_should_return_bad_response_status( $rest_base, $template_id ) {
     349        wp_set_current_user( self::$admin_id );
     350        switch_theme( 'block-theme' );
     351
     352        $request = new WP_REST_Request( 'GET', '/wp/v2/' . $rest_base . '/' . $template_id . '/autosaves/1' );
     353
     354        $response = rest_get_server()->dispatch( $request );
     355        $this->assertErrorResponse(
     356            'rest_invalid_template',
     357            $response,
     358            WP_Http::BAD_REQUEST,
     359            sprintf( 'Response is expected to have a status code of %d.', WP_Http::BAD_REQUEST )
     360        );
     361    }
     362
     363    /**
     364     * Data provider for test_get_item_for_templates_based_on_theme_files_should_return_bad_response_status.
     365     *
     366     * @return array
     367     */
     368    public function data_get_item_for_templates_based_on_theme_files_should_return_bad_response_status() {
     369        return array(
     370            'templates'      => array( 'templates', self::TEST_THEME . '//page-home' ),
     371            'template parts' => array( 'template-parts', self::TEST_THEME . '//small-header' ),
     372        );
     373    }
     374
     375    /**
     376     * @coversNothing
     377     * @ticket 56922
     378     */
     379    public function test_get_item() {
     380        // A proper data provider cannot be used because this method's signature must match the parent method.
     381        // Therefore, actual tests are performed in the test_get_item_with_data_provider method.
     382        $this->assertTrue( true );
     383    }
     384
     385    /**
     386     * @dataProvider data_get_item_with_data_provider
    212387     * @covers WP_REST_Template_Autosaves_Controller::get_item
    213388     * @ticket 56922
    214      */
    215     public function test_get_item() {
     389     *
     390     * @param string  $parent_post_property_name  A class property name that contains the parent post object.
     391     * @param string  $rest_base                  Base part of the REST API endpoint to test.
     392     * @param string  $template_id                Template ID to use in the test.
     393     */
     394    public function test_get_item_with_data_provider( $parent_post_property_name, $rest_base, $template_id ) {
    216395        wp_set_current_user( self::$admin_id );
     396
     397        $parent_post = self::$$parent_post_property_name;
    217398
    218399        $autosave_post_id = wp_create_post_autosave(
    219400            array(
    220401                'post_content' => 'Autosave content.',
    221                 'post_ID'      => self::$template_post->ID,
    222                 'post_type'    => self::PARENT_POST_TYPE,
    223             )
    224         );
    225 
    226         $request  = new WP_REST_Request( 'GET', '/wp/v2/templates/' . self::TEST_THEME . '/' . self::TEMPLATE_NAME . '/autosaves/' . $autosave_post_id );
    227         $response = rest_get_server()->dispatch( $request );
     402                'post_ID'      => $parent_post->ID,
     403                'post_type'    => $parent_post->post_type,
     404            )
     405        );
     406
     407        $request  = new WP_REST_Request( 'GET', '/wp/v2/' . $rest_base . '/' . $template_id . '/autosaves/' . $autosave_post_id );
     408        $response = rest_get_server()->dispatch( $request );
     409
     410        $this->assertSame( WP_Http::OK, $response->get_status(), 'Response is expected to have a status code of 200.' );
    228411        $autosave = $response->get_data();
    229412
    … …  
    235418        );
    236419        $this->assertSame(
    237             self::$template_post->ID,
     420            $parent_post->ID,
    238421            $autosave['parent'],
    239422            sprintf(
    240423                'Failed asserting that the parent id of the autosave is the same as %s.',
    241                 self::$template_post->ID
    242             )
    243         );
    244     }
    245 
    246     /**
     424                $parent_post->ID
     425            )
     426        );
     427    }
     428
     429    /**
     430     * Data provider for test_get_item_with_data_provider.
     431     *
     432     * @return array
     433     */
     434    public function data_get_item_with_data_provider() {
     435        return array(
     436            'templates'      => array( 'template_post', 'templates', self::TEST_THEME . '//' . self::TEMPLATE_NAME ),
     437            'template parts' => array( 'template_part_post', 'template-parts', self::TEST_THEME . '//' . self::TEMPLATE_PART_NAME ),
     438        );
     439    }
     440
     441    /**
     442     * @coversNothing
     443     * @ticket 56922
     444     */
     445    public function test_prepare_item() {
     446        // A proper data provider cannot be used because this method's signature must match the parent method.
     447        // Therefore, actual tests are performed in the test_prepare_item_with_data_provider method.
     448        $this->assertTrue( true );
     449    }
     450
     451    /**
     452     * @dataProvider data_prepare_item_with_data_provider
    247453     * @covers WP_REST_Template_Autosaves_Controller::prepare_item_for_response
    248454     * @ticket 56922
    249      */
    250     public function test_prepare_item() {
     455     *
     456     * @param string $parent_post_property_name A class property name that contains the parent post object.
     457     * @param string $rest_base                 Base part of the REST API endpoint to test.
     458     * @param string $template_id               Template ID to use in the test.
     459     */
     460    public function test_prepare_item_with_data_provider( $parent_post_property_name, $rest_base, $template_id ) {
    251461        wp_set_current_user( self::$admin_id );
     462        $parent_post      = self::$$parent_post_property_name;
    252463        $autosave_post_id = wp_create_post_autosave(
    253464            array(
    254465                'post_content' => 'Autosave content.',
    255                 'post_ID'      => self::$template_post->ID,
    256                 'post_type'    => self::PARENT_POST_TYPE,
     466                'post_ID'      => $parent_post->ID,
     467                'post_type'    => $parent_post->post_type,
    257468            )
    258469        );
    259470        $autosave_db_post = get_post( $autosave_post_id );
    260         $template_id      = self::TEST_THEME . '//' . self::TEMPLATE_NAME;
    261         $request          = new WP_REST_Request( 'GET', '/wp/v2/templates/' . $template_id . '/autosaves/' . $autosave_db_post->ID );
    262         $controller       = new WP_REST_Template_Autosaves_Controller( self::PARENT_POST_TYPE );
     471        $request          = new WP_REST_Request( 'GET', '/wp/v2/' . $rest_base . '/' . $template_id . '/autosaves/' . $autosave_db_post->ID );
     472        $controller       = new WP_REST_Template_Autosaves_Controller( $parent_post->post_type );
    263473        $response         = $controller->prepare_item_for_response( $autosave_db_post, $request );
    264474        $this->assertInstanceOf(
    … …  
    276486        );
    277487        $this->assertSame(
    278             self::$template_post->ID,
     488            $parent_post->ID,
    279489            $autosave['parent'],
    280490            sprintf(
    281491                'Failed asserting that the parent id of the autosave is the same as %s.',
    282                 self::$template_post->ID
     492                $parent_post->ID
    283493            )
    284494        );
    … …  
    301511
    302512    /**
     513     * Data provider for test_prepare_item_with_data_provider.
     514     *
     515     * @return array
     516     */
     517    public function data_prepare_item_with_data_provider() {
     518        return array(
     519            'templates'      => array( 'template_post', 'templates', self::TEST_THEME . '//' . self::TEMPLATE_NAME ),
     520            'template parts' => array( 'template_part_post', 'template-parts', self::TEST_THEME . '//' . self::TEMPLATE_PART_NAME ),
     521        );
     522    }
     523
     524    /**
     525     * @coversNothing
     526     * @ticket 56922
     527     */
     528    public function test_get_item_schema() {
     529        // A proper data provider cannot be used because this method's signature must match the parent method.
     530        // Therefore, actual tests are performed in the test_prepare_item_with_data_provider method.
     531        $this->assertTrue( true );
     532    }
     533
     534    /**
     535     * @dataProvider data_get_item_schema_with_data_provider
    303536     * @covers WP_REST_Template_Autosaves_Controller::get_item_schema
    304537     * @ticket 56922
    305      */
    306     public function test_get_item_schema() {
    307         $request  = new WP_REST_Request( 'OPTIONS', '/wp/v2/templates/' . self::TEST_THEME . '/' . self::TEMPLATE_NAME . '/autosaves' );
     538     *
     539     * @param string $rest_base             Base part of the REST API endpoint to test.
     540     * @param string $template_id           Template ID to use in the test.
     541     * @param int    $properties_count      Number of properties to check for in the schema.
     542     * @param array  $additional_properties Additional properties to check for in the schema.
     543     */
     544    public function test_get_item_schema_with_data_provider( $rest_base, $template_id, $properties_count, $additional_properties = array() ) {
     545        $request  = new WP_REST_Request( 'OPTIONS', '/wp/v2/' . $rest_base . '/' . $template_id . '/autosaves' );
    308546        $response = rest_get_server()->dispatch( $request );
    309547        $data     = $response->get_data();
    … …  
    311549        $properties = $data['schema']['properties'];
    312550
    313         $this->assertCount( 19, $properties );
     551        $this->assertCount( $properties_count, $properties );
    314552        $this->assertArrayHasKey( 'id', $properties, 'ID key should exist in properties.' );
    315553        $this->assertArrayHasKey( 'slug', $properties, 'Slug key should exist in properties.' );
    … …  
    325563        $this->assertArrayHasKey( 'author', $properties, 'author key should exist in properties.' );
    326564        $this->assertArrayHasKey( 'modified', $properties, 'modified key should exist in properties.' );
    327         $this->assertArrayHasKey( 'is_custom', $properties, 'is_custom key should exist in properties.' );
    328565        $this->assertArrayHasKey( 'parent', $properties, 'Parent key should exist in properties.' );
    329566        $this->assertArrayHasKey( 'author_text', $properties, 'author_text key should exist in properties.' );
    330567        $this->assertArrayHasKey( 'original_source', $properties, 'original_source key should exist in properties.' );
    331         $this->assertArrayHasKey( 'plugin', $properties, 'plugin key should exist in properties.' );
    332     }
    333 
    334     /**
     568        foreach ( $additional_properties as $additional_property ) {
     569            $this->assertArrayHasKey( $additional_property, $properties, $additional_property . ' key should exist in properties.' );
     570        }
     571    }
     572
     573    /**
     574     * Data provider for test_get_item_schema_with_data_provider.
     575     *
     576     * @return array
     577     */
     578    public function data_get_item_schema_with_data_provider() {
     579        return array(
     580            'templates'      => array(
     581                'templates',
     582                self::TEST_THEME . '//' . self::TEMPLATE_NAME,
     583                19,
     584                array( 'is_custom', 'plugin' ),
     585            ),
     586            'template parts' => array(
     587                'template-parts',
     588                self::TEST_THEME . '//' . self::TEMPLATE_PART_NAME,
     589                18,
     590                array( 'area' ),
     591            ),
     592        );
     593    }
     594
     595    /**
     596     * @coversNothing
     597     * @ticket 56922
     598     */
     599    public function test_create_item() {
     600        // A proper data provider cannot be used because this method's signature must match the parent method.
     601        // Therefore, actual tests are performed in the test_create_item_with_data_provider method.
     602        $this->assertTrue( true );
     603    }
     604
     605    /**
     606     * @dataProvider data_create_item_with_data_provider
    335607     * @covers WP_REST_Template_Autosaves_Controller::create_item
    336608     * @ticket 56922
    337      */
    338     public function test_create_item() {
     609     *
     610     * @param string $rest_base   Base part of the REST API endpoint to test.
     611     * @param string $template_id Template ID to use in the test.
     612     */
     613    public function test_create_item_with_data_provider( $rest_base, $template_id ) {
    339614        wp_set_current_user( self::$admin_id );
    340615
    341         $template_id = self::TEST_THEME . '/' . self::TEMPLATE_NAME;
    342         $request     = new WP_REST_Request( 'POST', '/wp/v2/templates/' . $template_id . '/autosaves' );
     616        $request = new WP_REST_Request( 'POST', '/wp/v2/' . $rest_base . '/' . $template_id . '/autosaves' );
    343617        $request->add_header( 'Content-Type', 'application/x-www-form-urlencoded' );
    344618
    … …  
    354628        $response = rest_get_server()->dispatch( $request );
    355629
    356         $this->assertNotWPError( $response, 'The response from this request should not return a WP_Error object' );
     630        $this->assertNotWPError( $response, 'The response from this request should not return a WP_Error object.' );
    357631        $response = rest_ensure_response( $response );
    358632        $data     = $response->get_data();
    359633
    360         $this->assertArrayHasKey( 'content', $data, 'Response should contain a key called content' );
    361         $this->assertSame( $request_parameters['content'], $data['content']['raw'], 'Response data should match for field content' );
    362 
    363         $this->assertArrayHasKey( 'title', $data, 'Response should contain a key called title' );
    364         $this->assertSame( $request_parameters['title'], $data['title']['raw'], 'Response data should match for field title' );
    365     }
    366 
    367     /**
    368      * @covers WP_REST_Template_Autosaves_Controller::delete_item
    369      * @ticket 56922
    370      */
    371     public function test_create_item_incorrect_permission() {
     634        $this->assertArrayHasKey( 'content', $data, 'Response should contain a key called content.' );
     635        $this->assertSame( $request_parameters['content'], $data['content']['raw'], 'Response data should match for field content.' );
     636
     637        $this->assertArrayHasKey( 'title', $data, 'Response should contain a key called title.' );
     638        $this->assertSame( $request_parameters['title'], $data['title']['raw'], 'Response data should match for field title.' );
     639    }
     640
     641    /**
     642     * Data provider for test_create_item_with_data_provider.
     643     *
     644     * @return array
     645     */
     646    public function data_create_item_with_data_provider() {
     647        return array(
     648            'templates'     => array( 'templates', self::TEST_THEME . '//' . self::TEMPLATE_NAME ),
     649            'template part' => array( 'template-parts', self::TEST_THEME . '//' . self::TEMPLATE_PART_NAME ),
     650        );
     651    }
     652
     653    /**
     654     * @dataProvider data_create_item_incorrect_permission
     655     * @covers WP_REST_Template_Autosaves_Controller::create_item_permissions_check
     656     * @ticket 56922
     657     *
     658     * @param string $rest_base   Base part of the REST API endpoint to test.
     659     * @param string $template_id Template ID to use in the test.
     660     */
     661    public function test_create_item_incorrect_permission( $rest_base, $template_id ) {
    372662        wp_set_current_user( self::$contributor_id );
    373         $template_id = self::TEST_THEME . '/' . self::TEMPLATE_NAME;
    374         $request     = new WP_REST_Request( 'POST', '/wp/v2/templates/' . $template_id . '/autosaves' );
    375         $response    = rest_get_server()->dispatch( $request );
     663        $request  = new WP_REST_Request( 'POST', '/wp/v2/' . $rest_base . '/' . $template_id . '/autosaves' );
     664        $response = rest_get_server()->dispatch( $request );
    376665        $this->assertErrorResponse( 'rest_cannot_manage_templates', $response, WP_Http::FORBIDDEN );
    377666    }
    378667
    379668    /**
    380      * @covers WP_REST_Template_Autosaves_Controller::delete_item
    381      * @ticket 56922
    382      */
    383     public function test_create_item_no_permission() {
     669     * Data provider for test_create_item_incorrect_permission.
     670     *
     671     * @return array
     672     */
     673    public function data_create_item_incorrect_permission() {
     674        return array(
     675            'template'      => array( 'templates', self::TEST_THEME . '//' . self::TEMPLATE_NAME ),
     676            'template part' => array( 'template-parts', self::TEST_THEME . '//' . self::TEMPLATE_PART_NAME ),
     677        );
     678    }
     679
     680    /**
     681     * @dataProvider data_create_item_no_permission
     682     * @covers WP_REST_Template_Autosaves_Controller::create_item_permissions_check
     683     * @ticket 56922
     684     *
     685     * @param string $rest_base   Base part of the REST API endpoint to test.
     686     * @param string $template_id Template ID to use in the test.
     687     */
     688    public function test_create_item_no_permission( $rest_base, $template_id ) {
    384689        wp_set_current_user( 0 );
    385         $template_id = self::TEST_THEME . '/' . self::TEMPLATE_NAME;
    386         $request     = new WP_REST_Request( 'POST', '/wp/v2/templates/' . $template_id . '/autosaves' );
    387         $response    = rest_get_server()->dispatch( $request );
     690        $request  = new WP_REST_Request( 'POST', '/wp/v2/' . $rest_base . '/' . $template_id . '/autosaves' );
     691        $response = rest_get_server()->dispatch( $request );
    388692        $this->assertErrorResponse( 'rest_cannot_manage_templates', $response, WP_Http::UNAUTHORIZED );
     693    }
     694
     695    /**
     696     * Data provider for test_create_item_no_permission.
     697     *
     698     * @return array
     699     */
     700    public function data_create_item_no_permission() {
     701        return array(
     702            'template'      => array( 'templates', self::TEST_THEME . '//' . self::TEMPLATE_NAME ),
     703            'template part' => array( 'template-parts', self::TEST_THEME . '//' . self::TEMPLATE_PART_NAME ),
     704        );
    389705    }
    390706
Note: See TracChangeset for help on using the changeset viewer.