WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
10/09/2022 05:45:54 PM (2 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.