WordPress.org
Get WordPress

Make WordPress Core

Changeset 54428


Ignore:
Timestamp:
10/09/2022 05:45:54 PM (4 years ago)
Author:
SergeyBiryukov
Message:

Tests: Minimize file copying in WP_REST_Attachments_Controller tests.

The tests use two images that were deleted on teardown and recreated on setup for every single test. This appears to be unnecessary, as the files can instead only be recreated if they are missing, and deleted after the test run is complete.

This commit reduces ~200 redundant file copying operations to ~5 when running this test class.

Follow-up to [38832], [48291], [54424].

See #55652.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/tests/phpunit/tests/rest-api/rest-attachments-controller.php

    r54226 r54428  
    2323         * @var string The path to a test file.
    2424         */
    25         private $test_file;
     25        private static $test_file;
    2626
    2727        /**
    2828         * @var string The path to a second test file.
    2929         */
    30         private $test_file2;
     30        private static $test_file2;
    3131
    3232        /**
    … …  
    6969
    7070        public static function wpTearDownAfterClass() {
     71                if ( file_exists( self::$test_file ) ) {
     72                        unlink( self::$test_file );
     73                }
     74                if ( file_exists( self::$test_file2 ) ) {
     75                        unlink( self::$test_file2 );
     76                }
     77
    7178                self::delete_user( self::$editor_id );
    7279                self::delete_user( self::$author_id );
    … …  
    8693
    8794                $orig_file       = DIR_TESTDATA . '/images/canola.jpg';
    88                 $this->test_file = get_temp_dir() . 'canola.jpg';
    89                 copy( $orig_file, $this->test_file );
     95                self::$test_file = get_temp_dir() . 'canola.jpg';
     96                if ( ! file_exists( self::$test_file ) ) {
     97                        copy( $orig_file, self::$test_file );
     98                }
     99
    90100                $orig_file2       = DIR_TESTDATA . '/images/codeispoetry.png';
    91                 $this->test_file2 = get_temp_dir() . 'codeispoetry.png';
    92                 copy( $orig_file2, $this->test_file2 );
     101                self::$test_file2 = get_temp_dir() . 'codeispoetry.png';
     102                if ( ! file_exists( self::$test_file2 ) ) {
     103                        copy( $orig_file2, self::$test_file2 );
     104                }
    93105
    94106                add_filter( 'rest_pre_dispatch', array( $this, 'wpSetUpBeforeRequest' ), 10, 3 );
    … …  
    107119
    108120        public function tear_down() {
    109                 if ( file_exists( $this->test_file ) ) {
    110                         unlink( $this->test_file );
    111                 }
    112                 if ( file_exists( $this->test_file2 ) ) {
    113                         unlink( $this->test_file2 );
    114                 }
    115 
    116121                $this->remove_added_uploads();
    117122
    … …  
    184189                // Single.
    185190                $attachment_id = self::factory()->attachment->create_object(
    186                         $this->test_file,
     191                        self::$test_file,
    187192                        0,
    188193                        array(
    … …  
    245250        public function test_registered_get_item_params() {
    246251                $id1      = self::factory()->attachment->create_object(
    247                         $this->test_file,
     252                        self::$test_file,
    248253                        0,
    249254                        array(
    … …  
    265270        public function test_allow_header_sent_on_options_request() {
    266271                $id1      = self::factory()->attachment->create_object(
    267                         $this->test_file,
     272                        self::$test_file,
    268273                        0,
    269274                        array(
    … …  
    293298                wp_set_current_user( 0 );
    294299                $id1            = self::factory()->attachment->create_object(
    295                         $this->test_file,
     300                        self::$test_file,
    296301                        0,
    297302                        array(
    … …  
    302307                $draft_post     = self::factory()->post->create( array( 'post_status' => 'draft' ) );
    303308                $id2            = self::factory()->attachment->create_object(
    304                         $this->test_file,
     309                        self::$test_file,
    305310                        $draft_post,
    306311                        array(
    … …  
    311316                $published_post = self::factory()->post->create( array( 'post_status' => 'publish' ) );
    312317                $id3            = self::factory()->attachment->create_object(
    313                         $this->test_file,
     318                        self::$test_file,
    314319                        $published_post,
    315320                        array(
    … …  
    333338                wp_set_current_user( self::$editor_id );
    334339                $id1            = self::factory()->attachment->create_object(
    335                         $this->test_file,
     340                        self::$test_file,
    336341                        0,
    337342                        array(
    … …  
    342347                $draft_post     = self::factory()->post->create( array( 'post_status' => 'draft' ) );
    343348                $id2            = self::factory()->attachment->create_object(
    344                         $this->test_file,
     349                        self::$test_file,
    345350                        $draft_post,
    346351                        array(
    … …  
    351356                $published_post = self::factory()->post->create( array( 'post_status' => 'publish' ) );
    352357                $id3            = self::factory()->attachment->create_object(
    353                         $this->test_file,
     358                        self::$test_file,
    354359                        $published_post,
    355360                        array(
    … …  
    371376        public function test_get_items_media_type() {
    372377                $id1      = self::factory()->attachment->create_object(
    373                         $this->test_file,
     378                        self::$test_file,
    374379                        0,
    375380                        array(
    … …  
    394399        public function test_get_items_mime_type() {
    395400                $id1      = self::factory()->attachment->create_object(
    396                         $this->test_file,
     401                        self::$test_file,
    397402                        0,
    398403                        array(
    … …  
    418423                $post_id        = self::factory()->post->create( array( 'post_title' => 'Test Post' ) );
    419424                $attachment_id  = self::factory()->attachment->create_object(
    420                         $this->test_file,
     425                        self::$test_file,
    421426                        $post_id,
    422427                        array(
    … …  
    426431                );
    427432                $attachment_id2 = self::factory()->attachment->create_object(
    428                         $this->test_file,
     433                        self::$test_file,
    429434                        0,
    430435                        array(
    … …  
    462467                wp_set_current_user( self::$editor_id );
    463468                self::factory()->attachment->create_object(
    464                         $this->test_file,
     469                        self::$test_file,
    465470                        0,
    466471                        array(
    … …  
    480485                wp_set_current_user( 0 );
    481486                $attachment_id1 = self::factory()->attachment->create_object(
    482                         $this->test_file,
     487                        self::$test_file,
    483488                        0,
    484489                        array(
    … …  
    504509                wp_set_current_user( 0 );
    505510                $attachment_id1 = self::factory()->attachment->create_object(
    506                         $this->test_file,
     511                        self::$test_file,
    507512                        0,
    508513                        array(
    … …  
    513518                );
    514519                $attachment_id2 = self::factory()->attachment->create_object(
    515                         $this->test_file,
     520                        self::$test_file,
    516521                        0,
    517522                        array(
    … …  
    549554        public function test_get_items_valid_date() {
    550555                $id1     = self::factory()->attachment->create_object(
    551                         $this->test_file,
     556                        self::$test_file,
    552557                        0,
    553558                        array(
    … …  
    558563                );
    559564                $id2     = self::factory()->attachment->create_object(
    560                         $this->test_file,
     565                        self::$test_file,
    561566                        0,
    562567                        array(
    … …  
    567572                );
    568573                $id3     = self::factory()->attachment->create_object(
    569                         $this->test_file,
     574                        self::$test_file,
    570575                        0,
    571576                        array(
    … …  
    600605        public function test_get_items_valid_modified_date() {
    601606                $id1 = self::factory()->attachment->create_object(
    602                         $this->test_file,
     607                        self::$test_file,
    603608                        0,
    604609                        array(
    … …  
    609614                );
    610615                $id2 = self::factory()->attachment->create_object(
    611                         $this->test_file,
     616                        self::$test_file,
    612617                        0,
    613618                        array(
    … …  
    618623                );
    619624                $id3 = self::factory()->attachment->create_object(
    620                         $this->test_file,
     625                        self::$test_file,
    621626                        0,
    622627                        array(
    … …  
    660665        public function test_get_items_with_empty_page_runs_count_query_after() {
    661666                self::factory()->attachment->create_object(
    662                         $this->test_file,
     667                        self::$test_file,
    663668                        0,
    664669                        array(
    … …  
    682687        public function test_get_item() {
    683688                $attachment_id = self::factory()->attachment->create_object(
    684                         $this->test_file,
     689                        self::$test_file,
    685690                        0,
    686691                        array(
    … …  
    702707        public function test_get_item_sizes() {
    703708                $attachment_id = self::factory()->attachment->create_object(
    704                         $this->test_file,
     709                        self::$test_file,
    705710                        0,
    706711                        array(
    … …  
    708713                                'post_excerpt'   => 'A sample caption',
    709714                        ),
    710                         $this->test_file
     715                        self::$test_file
    711716                );
    712717
    713718                add_image_size( 'rest-api-test', 119, 119, true );
    714                 wp_update_attachment_metadata( $attachment_id, wp_generate_attachment_metadata( $attachment_id, $this->test_file ) );
     719                wp_update_attachment_metadata( $attachment_id, wp_generate_attachment_metadata( $attachment_id, self::$test_file ) );
    715720
    716721                $request            = new WP_REST_Request( 'GET', '/wp/v2/media/' . $attachment_id );
    … …  
    733738        public function test_get_item_sizes_with_no_url() {
    734739                $attachment_id = self::factory()->attachment->create_object(
    735                         $this->test_file,
     740                        self::$test_file,
    736741                        0,
    737742                        array(
    … …  
    739744                                'post_excerpt'   => 'A sample caption',
    740745                        ),
    741                         $this->test_file
     746                        self::$test_file
    742747                );
    743748
    744749                add_image_size( 'rest-api-test', 119, 119, true );
    745                 wp_update_attachment_metadata( $attachment_id, wp_generate_attachment_metadata( $attachment_id, $this->test_file ) );
     750                wp_update_attachment_metadata( $attachment_id, wp_generate_attachment_metadata( $attachment_id, self::$test_file ) );
    746751
    747752                add_filter( 'wp_get_attachment_image_src', '__return_false' );
    … …  
    761766                $draft_post = self::factory()->post->create( array( 'post_status' => 'draft' ) );
    762767                $id1        = self::factory()->attachment->create_object(
    763                         $this->test_file,
     768                        self::$test_file,
    764769                        $draft_post,
    765770                        array(
    … …  
    775780        public function test_get_item_inherit_status_with_invalid_parent() {
    776781                $attachment_id = self::factory()->attachment->create_object(
    777                         $this->test_file,
     782                        self::$test_file,
    778783                        REST_TESTS_IMPOSSIBLY_HIGH_NUMBER,
    779784                        array(
    … …  
    792797        public function test_get_item_auto_status_with_invalid_parent_not_authenticated_returns_error() {
    793798                $attachment_id = self::factory()->attachment->create_object(
    794                         $this->test_file,
     799                        self::$test_file,
    795800                        REST_TESTS_IMPOSSIBLY_HIGH_NUMBER,
    796801                        array(
    … …  
    820825                $request->set_param( 'alt_text', 'Alt text is stored outside post schema.' );
    821826
    822                 $request->set_body( file_get_contents( $this->test_file ) );
     827                $request->set_body( file_get_contents( self::$test_file ) );
    823828                $response = rest_get_server()->dispatch( $request );
    824829                $data     = $response->get_data();
    … …  
    844849                        array(
    845850                                'file' => array(
    846                                         'file'     => file_get_contents( $this->test_file2 ),
     851                                        'file'     => file_get_contents( self::$test_file2 ),
    847852                                        'name'     => 'codeispoetry.png',
    848                                         'size'     => filesize( $this->test_file2 ),
    849                                         'tmp_name' => $this->test_file2,
     853                                        'size'     => filesize( self::$test_file2 ),
     854                                        'tmp_name' => self::$test_file2,
    850855                                ),
    851856                        )
    852857                );
    853                 $request->set_header( 'Content-MD5', md5_file( $this->test_file2 ) );
     858                $request->set_header( 'Content-MD5', md5_file( self::$test_file2 ) );
    854859                $response = rest_get_server()->dispatch( $request );
    855860                $this->assertSame( 201, $response->get_status() );
    … …  
    867872                        array(
    868873                                'file' => array(
    869                                         'file'     => file_get_contents( $this->test_file ),
     874                                        'file'     => file_get_contents( self::$test_file ),
    870875                                        'name'     => 'canola.jpg',
    871                                         'size'     => filesize( $this->test_file ),
    872                                         'tmp_name' => $this->test_file,
     876                                        'size'     => filesize( self::$test_file ),
     877                                        'tmp_name' => self::$test_file,
    873878                                ),
    874879                        )
    875880                );
    876                 $request->set_header( 'Content-MD5', md5_file( $this->test_file ) );
     881                $request->set_header( 'Content-MD5', md5_file( self::$test_file ) );
    877882                $response = rest_get_server()->dispatch( $request );
    878883                $this->assertSame( 201, $response->get_status() );
    … …  
    888893                        array(
    889894                                'file' => array(
    890                                         'file'     => file_get_contents( $this->test_file ),
     895                                        'file'     => file_get_contents( self::$test_file ),
    891896                                        'name'     => 'canola.jpg',
    892                                         'size'     => filesize( $this->test_file ),
    893                                         'tmp_name' => $this->test_file,
     897                                        'size'     => filesize( self::$test_file ),
     898                                        'tmp_name' => self::$test_file,
    894899                                ),
    895900                        )
    896901                );
    897                 $request->set_header( 'Content-MD5', md5_file( $this->test_file ) );
     902                $request->set_header( 'Content-MD5', md5_file( self::$test_file ) );
    898903                $response = rest_get_server()->dispatch( $request );
    899904                $this->assertSame( 201, $response->get_status() );
    … …  
    910915                wp_set_current_user( self::$author_id );
    911916                $request = new WP_REST_Request( 'POST', '/wp/v2/media' );
    912                 $request->set_body( file_get_contents( $this->test_file ) );
     917                $request->set_body( file_get_contents( self::$test_file ) );
    913918                $response = rest_get_server()->dispatch( $request );
    914919                $this->assertErrorResponse( 'rest_upload_no_content_type', $response, 400 );
    … …  
    919924                $request = new WP_REST_Request( 'POST', '/wp/v2/media' );
    920925                $request->set_header( 'Content-Type', 'image/jpeg' );
    921                 $request->set_body( file_get_contents( $this->test_file ) );
     926                $request->set_body( file_get_contents( self::$test_file ) );
    922927                $response = rest_get_server()->dispatch( $request );
    923928                $this->assertErrorResponse( 'rest_upload_no_content_disposition', $response, 400 );
    … …  
    930935                $request->set_header( 'Content-Disposition', 'attachment; filename=canola.jpg' );
    931936                $request->set_header( 'Content-MD5', 'abc123' );
    932                 $request->set_body( file_get_contents( $this->test_file ) );
     937                $request->set_body( file_get_contents( self::$test_file ) );
    933938                $response = rest_get_server()->dispatch( $request );
    934939                $this->assertErrorResponse( 'rest_upload_hash_mismatch', $response, 412 );
    … …  
    941946                        array(
    942947                                'file' => array(
    943                                         'file'     => file_get_contents( $this->test_file ),
     948                                        'file'     => file_get_contents( self::$test_file ),
    944949                                        'name'     => 'canola.jpg',
    945                                         'size'     => filesize( $this->test_file ),
    946                                         'tmp_name' => $this->test_file,
     950                                        'size'     => filesize( self::$test_file ),
     951                                        'tmp_name' => self::$test_file,
    947952                                ),
    948953                        )
    … …  
    990995                $request->set_header( 'Content-Type', 'image/jpeg' );
    991996                $request->set_header( 'Content-Disposition', 'attachment; filename=canola.jpg' );
    992                 $request->set_body( file_get_contents( $this->test_file ) );
     997                $request->set_body( file_get_contents( self::$test_file ) );
    993998                $request->set_param( 'post', $attachment_id );
    994999                $response = rest_get_server()->dispatch( $request );
    … …  
    10051010                $request->set_header( 'Content-Disposition', 'attachment; filename=canola.jpg' );
    10061011
    1007                 $request->set_body( file_get_contents( $this->test_file ) );
     1012                $request->set_body( file_get_contents( self::$test_file ) );
    10081013                $request->set_param( 'alt_text', 'test alt text' );
    10091014                $response   = rest_get_server()->dispatch( $request );
    … …  
    10201025                $request->set_header( 'Content-Type', 'image/jpeg' );
    10211026                $request->set_header( 'Content-Disposition', 'attachment; filename=canola.jpg' );
    1022                 $request->set_body( file_get_contents( $this->test_file ) );
     1027                $request->set_body( file_get_contents( self::$test_file ) );
    10231028                $request->set_param( 'alt_text', '<script>alert(document.cookie)</script>' );
    10241029                $response   = rest_get_server()->dispatch( $request );
    … …  
    10361041                $request->set_header( 'Content-Type', 'image/jpeg' );
    10371042                $request->set_header( 'Content-Disposition', 'attachment; filename=canola.jpg' );
    1038                 $request->set_body( file_get_contents( $this->test_file ) );
     1043                $request->set_body( file_get_contents( self::$test_file ) );
    10391044                $response   = rest_get_server()->dispatch( $request );
    10401045                $attachment = $response->get_data();
    … …  
    10451050                wp_set_current_user( self::$editor_id );
    10461051                $attachment_id = self::factory()->attachment->create_object(
    1047                         $this->test_file,
     1052                        self::$test_file,
    10481053                        0,
    10491054                        array(
    … …  
    10751080                $original_parent = self::factory()->post->create( array() );
    10761081                $attachment_id   = self::factory()->attachment->create_object(
    1077                         $this->test_file,
     1082                        self::$test_file,
    10781083                        $original_parent,
    10791084                        array(
    … …  
    10991104                wp_set_current_user( self::$author_id );
    11001105                $attachment_id = self::factory()->attachment->create_object(
    1101                         $this->test_file,
     1106                        self::$test_file,
    11021107                        0,
    11031108                        array(
    … …  
    11231128                wp_set_current_user( self::$editor_id );
    11241129                $attachment_id = self::factory()->attachment->create_object(
    1125                         $this->test_file,
     1130                        self::$test_file,
    11261131                        0,
    11271132                        array(
    … …  
    11441149                $parent_id     = self::factory()->post->create( array() );
    11451150                $attachment_id = self::factory()->attachment->create_object(
    1146                         $this->test_file,
     1151                        self::$test_file,
    11471152                        $parent_id,
    11481153                        array(
    … …  
    11671172                wp_set_current_user( self::$editor_id );
    11681173                $attachment_id = self::factory()->attachment->create_object(
    1169                         $this->test_file,
     1174                        self::$test_file,
    11701175                        0,
    11711176                        array(
    … …  
    11891194                $request->set_header( 'Content-Type', 'image/jpeg' );
    11901195                $request->set_header( 'Content-Disposition', 'attachment; filename=canola.jpg' );
    1191                 $request->set_body( file_get_contents( $this->test_file ) );
     1196                $request->set_body( file_get_contents( self::$test_file ) );
    11921197
    11931198                foreach ( $input as $name => $value ) {
    … …  
    14471452                wp_set_current_user( self::$editor_id );
    14481453                $attachment_id    = self::factory()->attachment->create_object(
    1449                         $this->test_file,
     1454                        self::$test_file,
    14501455                        0,
    14511456                        array(
    … …  
    14631468                wp_set_current_user( self::$editor_id );
    14641469                $attachment_id = self::factory()->attachment->create_object(
    1465                         $this->test_file,
     1470                        self::$test_file,
    14661471                        0,
    14671472                        array(
    … …  
    14881493                wp_set_current_user( self::$author_id );
    14891494                $attachment_id = self::factory()->attachment->create_object(
    1490                         $this->test_file,
     1495                        self::$test_file,
    14911496                        0,
    14921497                        array(
    … …  
    15031508        public function test_prepare_item() {
    15041509                $attachment_id = self::factory()->attachment->create_object(
    1505                         $this->test_file,
     1510                        self::$test_file,
    15061511                        0,
    15071512                        array(
    … …  
    15221527        public function test_prepare_item_limit_fields() {
    15231528                $attachment_id = self::factory()->attachment->create_object(
    1524                         $this->test_file,
     1529                        self::$test_file,
    15251530                        0,
    15261531                        array(
    … …  
    16131618
    16141619                $attachment_id = self::factory()->attachment->create_object(
    1615                         $this->test_file,
     1620                        self::$test_file,
    16161621                        0,
    16171622                        array(
    … …  
    16501655                wp_set_current_user( self::$editor_id );
    16511656                $attachment_id = self::factory()->attachment->create_object(
    1652                         $this->test_file,
     1657                        self::$test_file,
    16531658                        0,
    16541659                        array(
    … …  
    16761681        public function test_search_item_by_filename() {
    16771682                $id1 = self::factory()->attachment->create_object(
    1678                         $this->test_file,
     1683                        self::$test_file,
    16791684                        0,
    16801685                        array(
    … …  
    16831688                );
    16841689                $id2 = self::factory()->attachment->create_object(
    1685                         $this->test_file2,
     1690                        self::$test_file2,
    16861691                        0,
    16871692                        array(
    … …  
    16901695                );
    16911696
    1692                 $filename = wp_basename( $this->test_file2 );
     1697                $filename = wp_basename( self::$test_file2 );
    16931698
    16941699                $request = new WP_REST_Request( 'GET', '/wp/v2/media' );
    … …  
    18021807                                'file' => array(
    18031808                                        'error'    => '0',
    1804                                         'file'     => file_get_contents( $this->test_file ),
     1809                                        'file'     => file_get_contents( self::$test_file ),
    18051810                                        'name'     => 'canola.jpg',
    1806                                         'size'     => filesize( $this->test_file ),
    1807                                         'tmp_name' => $this->test_file,
     1811                                        'size'     => filesize( self::$test_file ),
     1812                                        'tmp_name' => self::$test_file,
    18081813                                ),
    18091814                        )
    … …  
    18111816                $request->set_param( 'title', 'My title is very cool' );
    18121817                $request->set_param( 'caption', 'This is a better caption.' );
    1813                 $request->set_header( 'Content-MD5', md5_file( $this->test_file ) );
     1818                $request->set_header( 'Content-MD5', md5_file( self::$test_file ) );
    18141819
    18151820                $response = rest_get_server()->dispatch( $request );
    … …  
    18301835                $request->set_header( 'Content-Type', 'image/jpeg' );
    18311836                $request->set_header( 'Content-Disposition', 'attachment; filename=canola.jpg' );
    1832                 $request->set_body( file_get_contents( $this->test_file ) );
     1837                $request->set_body( file_get_contents( self::$test_file ) );
    18331838                $request->set_param( 'title', 'My title is very cool' );
    18341839                $request->set_param( 'caption', 'This is a better caption.' );
    … …  
    18531858                                'file' => array(
    18541859                                        'error'    => '0',
    1855                                         'file'     => file_get_contents( $this->test_file ),
     1860                                        'file'     => file_get_contents( self::$test_file ),
    18561861                                        'name'     => 'canola.jpg',
    1857                                         'size'     => filesize( $this->test_file ),
    1858                                         'tmp_name' => $this->test_file,
     1862                                        'size'     => filesize( self::$test_file ),
     1863                                        'tmp_name' => self::$test_file,
    18591864                                ),
    18601865                        )
    … …  
    18621867                $request->set_param( 'title', 'My title is very cool' );
    18631868                $request->set_param( 'caption', 'This is a better caption.' );
    1864                 $request->set_header( 'Content-MD5', md5_file( $this->test_file ) );
     1869                $request->set_header( 'Content-MD5', md5_file( self::$test_file ) );
    18651870
    18661871                $response = rest_get_server()->dispatch( $request );
    … …  
    18811886                $request->set_header( 'Content-Type', 'image/jpeg' );
    18821887                $request->set_header( 'Content-Disposition', 'attachment; filename=canola.jpg' );
    1883                 $request->set_body( file_get_contents( $this->test_file ) );
     1888                $request->set_body( file_get_contents( self::$test_file ) );
    18841889                $request->set_param( 'title', 'My title is very cool' );
    18851890                $request->set_param( 'caption', 'This is a better caption.' );
    … …  
    19111916                $request->set_param( 'alt_text', 'Alt text is stored outside post schema.' );
    19121917
    1913                 $request->set_body( file_get_contents( $this->test_file ) );
     1918                $request->set_body( file_get_contents( self::$test_file ) );
    19141919                $response = rest_get_server()->dispatch( $request );
    19151920                $data     = $response->get_data();
    … …  
    19341939                wp_set_current_user( self::$editor_id );
    19351940                $attachment_id = self::factory()->attachment->create_object(
    1936                         $this->test_file,
     1941                        self::$test_file,
    19371942                        0,
    19381943                        array(
    … …  
    19721977                $request->set_param( 'meta', array( 'best_cannoli' => 'Chocolate-dipped, no filling' ) );
    19731978
    1974                 $request->set_body( file_get_contents( $this->test_file ) );
     1979                $request->set_body( file_get_contents( self::$test_file ) );
    19751980                $response = rest_get_server()->dispatch( $request );
    19761981                $data     = $response->get_data();
    … …  
    19931998         */
    19941999        public function test_edit_image_returns_error_if_logged_out() {
    1995                 $attachment = self::factory()->attachment->create_upload_object( $this->test_file );
     2000                $attachment = self::factory()->attachment->create_upload_object( self::$test_file );
    19962001
    19972002                $request = new WP_REST_Request( 'POST', "/wp/v2/media/{$attachment}/edit" );
    … …  
    20102015
    20112016                wp_set_current_user( $user->ID );
    2012                 $attachment = self::factory()->attachment->create_upload_object( $this->test_file );
     2017                $attachment = self::factory()->attachment->create_upload_object( self::$test_file );
    20132018
    20142019                $request = new WP_REST_Request( 'POST', "/wp/v2/media/{$attachment}/edit" );
    … …  
    20242029        public function test_edit_image_returns_error_if_cannot_edit() {
    20252030                wp_set_current_user( self::$uploader_id );
    2026                 $attachment = self::factory()->attachment->create_upload_object( $this->test_file );
     2031                $attachment = self::factory()->attachment->create_upload_object( self::$test_file );
    20272032
    20282033                $request = new WP_REST_Request( 'POST', "/wp/v2/media/{$attachment}/edit" );
    … …  
    20512056        public function test_edit_image_returns_error_if_unsupported_mime_type() {
    20522057                wp_set_current_user( self::$superadmin_id );
    2053                 $attachment = self::factory()->attachment->create_upload_object( $this->test_file );
     2058                $attachment = self::factory()->attachment->create_upload_object( self::$test_file );
    20542059                wp_update_post(
    20552060                        array(
    … …  
    20712076        public function test_edit_image_returns_error_if_no_edits() {
    20722077                wp_set_current_user( self::$superadmin_id );
    2073                 $attachment = self::factory()->attachment->create_upload_object( $this->test_file );
     2078                $attachment = self::factory()->attachment->create_upload_object( self::$test_file );
    20742079
    20752080                $request = new WP_REST_Request( 'POST', "/wp/v2/media/{$attachment}/edit" );
    … …  
    20852090        public function test_edit_image_rotate() {
    20862091                wp_set_current_user( self::$superadmin_id );
    2087                 $attachment = self::factory()->attachment->create_upload_object( $this->test_file );
     2092                $attachment = self::factory()->attachment->create_upload_object( self::$test_file );
    20882093
    20892094                $this->setup_mock_editor();
    … …  
    21102115        public function test_edit_image_crop() {
    21112116                wp_set_current_user( self::$superadmin_id );
    2112                 $attachment = self::factory()->attachment->create_upload_object( $this->test_file );
     2117                $attachment = self::factory()->attachment->create_upload_object( self::$test_file );
    21132118
    21142119                $this->setup_mock_editor();
    … …  
    21472152        public function test_edit_image() {
    21482153                wp_set_current_user( self::$superadmin_id );
    2149                 $attachment = self::factory()->attachment->create_upload_object( $this->test_file );
     2154                $attachment = self::factory()->attachment->create_upload_object( self::$test_file );
    21502155
    21512156                $params = array(
    … …  
    21742179        public function test_batch_edit_image() {
    21752180                wp_set_current_user( self::$superadmin_id );
    2176                 $attachment = self::factory()->attachment->create_upload_object( $this->test_file );
     2181                $attachment = self::factory()->attachment->create_upload_object( self::$test_file );
    21772182
    21782183                $params = array(
    … …  
    22172222        public function test_edit_image_returns_error_if_mismatched_src() {
    22182223                wp_set_current_user( self::$superadmin_id );
    2219                 $attachment_id_image1 = self::factory()->attachment->create_upload_object( $this->test_file );
    2220                 $attachment_id_image2 = self::factory()->attachment->create_upload_object( $this->test_file2 );
     2224                $attachment_id_image1 = self::factory()->attachment->create_upload_object( self::$test_file );
     2225                $attachment_id_image2 = self::factory()->attachment->create_upload_object( self::$test_file2 );
    22212226                $attachment_id_file   = self::factory()->attachment->create();
    22222227
Note: See TracChangeset for help on using the changeset viewer.