Changeset 58912 for trunk/tests/phpunit/tests/privacy/wpPrivacySendErasureFulfillmentNotification.php

Timestamp:

08/18/2024 11:43:37 PM (7 weeks ago)

Author:

SergeyBiryukov

Message:

Tests: Bring some consistency to personal data email notification tests.

Includes:

• Adding a test for wp_privacy_send_personal_data_export_email() to verify the user_request post type.
• Reordering some pre-existing tests to check the request ID and post type first.

Follow-up to [43291], [43499], [44535].

Props garrett-eclipse, berubenic.
See #46560.

File:

• trunk/tests/phpunit/tests/privacy/wpPrivacySendErasureFulfillmentNotification.php (modified) (2 diffs)

trunk/tests/phpunit/tests/privacy/wpPrivacySendErasureFulfillmentNotification.php

@@ -100 +100 @@
 
     /**
+     * The function should not send an email when the request ID does not exist.
+     *
+     * @ticket 44234
+     */
+    public function test_should_not_send_email_when_not_a_valid_request_id() {
+        _wp_privacy_send_erasure_fulfillment_notification( 1234567890 );
+
+        $mailer = tests_retrieve_phpmailer_instance();
+
+        $this->assertEmpty( $mailer->mock_sent );
+    }
+
+    /**
+     * The function should not send an email when the ID passed does not correspond to a user request.
+     *
+     * @ticket 44234
+     */
+    public function test_should_not_send_email_when_not_a_user_request() {
+        $post_id = self::factory()->post->create(
+            array(
+                'post_type' => 'post', // Should be 'user_request'.
+            )
+        );
+
+        _wp_privacy_send_erasure_fulfillment_notification( $post_id );
+        $mailer = tests_retrieve_phpmailer_instance();
+
+        $this->assertEmpty( $mailer->mock_sent );
+    }
+
+    /**
+     * The function should not send an email when the request is not completed.
+     *
+     * @ticket 44234
+     */
+    public function test_should_not_send_email_when_request_not_completed() {
+        wp_update_post(
+            array(
+                'ID'          => self::$request_id,
+                'post_status' => 'request-confirmed', // Should be 'request-completed'.
+            )
+        );
+
+        _wp_privacy_send_erasure_fulfillment_notification( self::$request_id );
+
+        $mailer = tests_retrieve_phpmailer_instance();
+
+        $this->assertEmpty( $mailer->mock_sent );
+        $this->assertFalse( metadata_exists( 'post', self::$request_id, '_wp_user_notified' ) );
+    }
+
+    /**
      * The function should send an email when a valid request ID is passed.
      *
@@ -284 +336 @@
 
     /**
-     * The function should not send an email when the request ID does not exist.
-     *
-     * @ticket 44234
-     */
-    public function test_should_not_send_email_when_passed_invalid_request_id() {
-        _wp_privacy_send_erasure_fulfillment_notification( 1234567890 );
-
-        $mailer = tests_retrieve_phpmailer_instance();
-
-        $this->assertEmpty( $mailer->mock_sent );
-    }
-
-    /**
-     * The function should not send an email when the ID passed does not correspond to a user request.
-     *
-     * @ticket 44234
-     */
-    public function test_should_not_send_email_when_not_user_request() {
-        $post_id = self::factory()->post->create(
-            array(
-                'post_type' => 'post', // Should be 'user_request'.
-            )
-        );
-
-        _wp_privacy_send_erasure_fulfillment_notification( $post_id );
-        $mailer = tests_retrieve_phpmailer_instance();
-
-        $this->assertEmpty( $mailer->mock_sent );
-    }
-
-    /**
-     * The function should not send an email when the request is not completed.
-     *
-     * @ticket 44234
-     */
-    public function test_should_not_send_email_when_request_not_completed() {
-        wp_update_post(
-            array(
-                'ID'          => self::$request_id,
-                'post_status' => 'request-confirmed', // Should be 'request-completed'.
-            )
-        );
-
-        _wp_privacy_send_erasure_fulfillment_notification( self::$request_id );
-
-        $mailer = tests_retrieve_phpmailer_instance();
-
-        $this->assertEmpty( $mailer->mock_sent );
-        $this->assertFalse( metadata_exists( 'post', self::$request_id, '_wp_user_notified' ) );
-    }
-
-    /**
      * The function should respect the user locale settings when the site uses the default locale.
      *