Changeset 47270 for trunk/tests/phpunit/tests/user.php

Timestamp:

02/11/2020 06:57:51 PM (5 years ago)

Author:

SergeyBiryukov

Message:

Privacy: Introduce wp_privacy_additional_user_data filter to make it easier to include additional user meta in a personal data export.

Props pbiron, xkon, garrett-eclipse, azaozz.
Fixes #47509.

File:

• trunk/tests/phpunit/tests/user.php (modified) (1 diff)

trunk/tests/phpunit/tests/user.php

@@ -1795 +1795 @@
         $this->assertEquals( 'January 29, 2020 09:13 AM', $actual['data'][1]['data'][3]['value'] );
     }
+
+    /**
+     * Testing the `wp_privacy_additional_user_profile_data` filter works.
+     *
+     * @ticket 47509
+     */
+    function test_filter_wp_privacy_additional_user_profile_data() {
+        $test_user = new WP_User( self::$contrib_id );
+
+        add_filter( 'wp_privacy_additional_user_profile_data', array( $this, 'export_additional_user_profile_data' ) );
+
+        $actual = wp_user_personal_data_exporter( $test_user->user_email );
+
+        remove_filter( 'wp_privacy_additional_user_profile_data', array( $this, 'export_additional_user_profile_data' ) );
+
+        $this->assertTrue( $actual['done'] );
+
+        // Number of exported users.
+        $this->assertSame( 1, count( $actual['data'] ) );
+
+        // Number of exported user properties (the 11 core properties,
+        // plus 1 additional from the filter).
+        $this->assertSame( 12, count( $actual['data'][0]['data'] ) );
+
+        // Check that the item added by the filter was retained.
+        $this->assertSame(
+            1,
+            count(
+                wp_list_filter(
+                    $actual['data'][0]['data'],
+                    array(
+                        'name'  => 'Test Additional Data Name',
+                        'value' => 'Test Additional Data Value',
+                    )
+                )
+            )
+        );
+
+        // _doing_wrong() should be called because the filter callback
+        // adds a item with a 'name' that is the same as one generated by core.
+        $this->setExpectedIncorrectUsage( 'wp_user_personal_data_exporter' );
+        add_filter( 'wp_privacy_additional_user_profile_data', array( $this, 'export_additional_user_profile_data_with_dup_name' ) );
+
+        $actual = wp_user_personal_data_exporter( $test_user->user_email );
+
+        remove_filter( 'wp_privacy_additional_user_profile_data', array( $this, 'export_additional_user_profile_data' ) );
+
+        $this->assertTrue( $actual['done'] );
+
+        // Number of exported users.
+        $this->assertSame( 1, count( $actual['data'] ) );
+
+        // Number of exported user properties
+        // (the 11 core properties, plus 1 additional from the filter).
+        $this->assertSame( 12, count( $actual['data'][0]['data'] ) );
+
+        // Check that the duplicate 'name' => 'User ID' was stripped.
+        $this->assertSame(
+            1,
+            count(
+                wp_list_filter(
+                    $actual['data'][0]['data'],
+                    array(
+                        'name' => 'User ID',
+                    )
+                )
+            )
+        );
+
+        // Check that the item added by the filter was retained.
+        $this->assertSame(
+            1,
+            count(
+                wp_list_filter(
+                    $actual['data'][0]['data'],
+                    array(
+                        'name'  => 'Test Additional Data Name',
+                        'value' => 'Test Additional Data Value',
+                    )
+                )
+            )
+        );
+    }
+
+    /**
+     * Filter callback to add additional profile data to the User Group on Export Requests.
+     *
+     * @ticket 47509
+     *
+     * @return array $additional_profile_data The additional user data.
+     */
+    public function export_additional_user_profile_data() {
+        $additional_profile_data = array(
+            // This item should be retained and included in the export.
+            array(
+                'name'  => 'Test Additional Data Name',
+                'value' => 'Test Additional Data Value',
+            ),
+        );
+
+        return $additional_profile_data;
+    }
+
+    /**
+     * Filter callback to add additional profile data to the User Group on Export Requests.
+     *
+     * This callback should generate a `_doing_it_wrong()`.
+     *
+     * @ticket 47509
+     *
+     * @return array $additional_profile_data The additional user data.
+     */
+    public function export_additional_user_profile_data_with_dup_name() {
+        $additional_profile_data = array(
+            // This item should be stripped out by wp_user_personal_data_exporter()
+            // because it's 'name' duplicates one exported by core.
+            array(
+                'name'  => 'User ID',
+                'value' => 'Some User ID',
+            ),
+            // This item should be retained and included in the export.
+            array(
+                'name'  => 'Test Additional Data Name',
+                'value' => 'Test Additional Data Value',
+            ),
+        );
+
+        return $additional_profile_data;
+    }
 }