Changeset 48127 for trunk/tests/phpunit/tests/privacy/wpPrivacyProcessPersonalDataExportPage.php

Timestamp:

06/22/2020 10:38:11 PM (6 years ago)

Author:

whyisjake

Message:

Privacy: Use relative paths for exported personal data.

Ensures back-compat while moving to paths off of the /exports directory.

Fixes: #44038.

Props: allendav, mrTall, desrosj, garrett-eclipse, cameronamcintyre, nmenescardi, xkon, whyisjake, davidbaumwald.

File:

• trunk/tests/phpunit/tests/privacy/wpPrivacyProcessPersonalDataExportPage.php (modified) (4 diffs)

trunk/tests/phpunit/tests/privacy/wpPrivacyProcessPersonalDataExportPage.php

@@ -45 +45 @@
 
         /**
-         * Export File Url.
-         *
-         * @since 5.2.0
+         * Export Url.
+         *
+         * @since 5.5.0
+         *
+         * @var string $export_url
+         */
+        protected static $export_url;
+
+        /**
+         * Export File Name.
+         *
+         * @since 5.5.0
+         *
+         * @var string $export_file_name
+         */
+        protected static $export_file_name;
+
+        /**
+         * Export File URL.
+         *
+         * @since 5.5.0
          *
          * @var string $export_file_url
@@ -132 +150 @@
         public static function wpSetUpBeforeClass( $factory ) {
                 self::$requester_email      = 'requester@example.com';
-                self::$export_file_url      = wp_privacy_exports_url() . 'wp-personal-data-file-Wv0RfMnGIkl4CFEDEEkSeIdfLmaUrLsl.zip';
+                self::$export_url           = wp_privacy_exports_url();
+                self::$export_file_name     = 'wp-personal-data-file-Wv0RfMnGIkl4CFEDEEkSeIdfLmaUrLsl.zip';
+                self::$export_file_url      = self::$export_url . self::$export_file_name;
                 self::$request_id           = wp_create_user_request( self::$requester_email, 'export_personal_data' );
                 self::$page_index_first     = 1;
@@ -503 +523 @@
          */
         public function test_return_response_with_export_file_url_when_not_sent_as_email_for_last_exporter_on_last_page() {
-                update_post_meta( self::$request_id, '_export_file_url', self::$export_file_url );
+                update_post_meta( self::$request_id, '_export_file_name', self::$export_file_name );
 
                 // Process data, given the last exporter, on the last page and not send as email.
@@ -529 +549 @@
          */
         public function test_return_response_without_export_file_url_when_sent_as_email_for_last_exporter_on_last_page() {
-                update_post_meta( self::$request_id, '_export_file_url', self::$export_file_url );
+                update_post_meta( self::$request_id, '_export_file_name', self::$export_file_name );
 
                 // Process data, given the last exporter, on the last page and send as email.