diff --git a/src/wp-admin/includes/class-wp-privacy-data-removal-requests-list-table.php b/src/wp-admin/includes/class-wp-privacy-data-removal-requests-list-table.php index 716535160f12d..d9c2da1c2e3b8 100644 --- a/src/wp-admin/includes/class-wp-privacy-data-removal-requests-list-table.php +++ b/src/wp-admin/includes/class-wp-privacy-data-removal-requests-list-table.php @@ -91,11 +91,11 @@ public function column_email( $item ) { esc_attr( sprintf( /* translators: %s: Request email. */ - __( 'Mark export request for “%s” as completed.' ), + __( 'Mark erasure request for “%s” as completed.' ), $item->email ) ), - __( 'Complete request' ) + __( 'Close request' ) ); $complete_request_markup .= ''; } diff --git a/tests/phpunit/tests/privacy/wpPrivacyDataRemovalRequestsListTable.php b/tests/phpunit/tests/privacy/wpPrivacyDataRemovalRequestsListTable.php new file mode 100644 index 0000000000000..5cb75a50e0f90 --- /dev/null +++ b/tests/phpunit/tests/privacy/wpPrivacyDataRemovalRequestsListTable.php @@ -0,0 +1,62 @@ + 'erase-personal-data' ) ); + } + + /** + * @ticket 65735 + */ + public function test_column_email_close_request_link_text() { + $request = wp_get_user_request( self::$request_id ); + ob_start(); + echo self::$list_table->column_email( $request ); + $output = ob_get_clean(); + + $this->assertStringContainsString( 'Close request', $output, 'Row action should be labelled "Close request".' ); + $this->assertStringNotContainsString( 'Complete request', $output, 'Row action should not use the ambiguous "Complete request" label.' ); + } + + /** + * @ticket 65735 + */ + public function test_column_email_aria_label_references_erasure_not_export() { + $request = wp_get_user_request( self::$request_id ); + ob_start(); + echo self::$list_table->column_email( $request ); + $output = ob_get_clean(); + + $this->assertStringContainsString( 'erasure request', $output, 'Aria-label should reference an erasure request, not an export request.' ); + $this->assertStringNotContainsString( 'export request', $output, 'Aria-label should not reference an export request on the erasure screen.' ); + } +}