Changeset 50230 for trunk/tests/phpunit/tests/privacy/wpCreateUserRequest.php

Timestamp:

02/05/2021 03:48:59 PM (5 years ago)

Author:

SergeyBiryukov

Message:

Privacy: Rename the $send_confirmation_email parameter of wp_create_user_request() to $status, for clarity.

Follow-up to [50159], [50165].

Props xkon, TimothyBlynJacobs.
Fixes #52430. See #43890.

File:

• trunk/tests/phpunit/tests/privacy/wpCreateUserRequest.php (modified) (3 diffs)

trunk/tests/phpunit/tests/privacy/wpCreateUserRequest.php

@@ -311 +311 @@
 
     /**
-     * Test that the request has a Pending status if a confirmation email is sent.
+     * Test that the request has a Pending status by default.
      *
      * @ticket 43890
      */
-    public function test_pending_status_with_default_wp_create_user_request_params() {
+    public function test_wp_create_user_request_default_pending_status() {
         $actual = wp_create_user_request( self::$non_registered_user_email, 'export_personal_data' );
         $post   = get_post( $actual );
@@ -323 +323 @@
 
     /**
-     * Test that the request has a Pending status if the $send_confirmation_email param is true.
+     * Test that the request has a Pending status if the $status param is 'pending'.
      *
      * @ticket 43890
      */
-    public function test_pending_status_with_true_send_confirmation_email() {
-        $request_data            = array();
-        $send_confirmation_email = true;
-
-        $actual = wp_create_user_request( self::$non_registered_user_email, 'export_personal_data', $request_data, $send_confirmation_email );
+    public function test_wp_create_user_request_pending_status() {
+        $actual = wp_create_user_request( self::$non_registered_user_email, 'export_personal_data', array(), 'pending' );
         $post   = get_post( $actual );
 
@@ -338 +335 @@
 
     /**
-     * Test that the request has a Completed status if the $send_confirmation_email param is false.
+     * Test that the request has a Confirmed status if the $status param is 'confirmed'.
      *
      * @ticket 43890
      */
-    public function test_pending_status_with_false_send_confirmation_email() {
-        $request_data            = array();
-        $send_confirmation_email = false;
-
-        $actual = wp_create_user_request( self::$non_registered_user_email, 'export_personal_data', $request_data, $send_confirmation_email );
+    public function test_wp_create_user_request_confirmed_status() {
+        $actual = wp_create_user_request( self::$non_registered_user_email, 'export_personal_data', array(), 'confirmed' );
         $post   = get_post( $actual );
 
-        $this->assertSame( 'request-completed', $post->post_status );
+        $this->assertSame( 'request-confirmed', $post->post_status );
+    }
+
+    /**
+     * Test that the request returns a WP_Error if $status isn't 'pending' or 'confirmed'.
+     *
+     * @ticket 43890
+     */
+    public function test_wp_create_user_request_wp_error_status() {
+        $actual = wp_create_user_request( self::$non_registered_user_email, 'export_personal_data', array(), 'wrong-status' );
+
+        $this->assertWPError( $actual );
     }
 }