[wp-trac] [WordPress Trac] #48265: The privacy export files cleanup can run unlink on directories throwing an error.

[WordPress Trac]

#48265: The privacy export files cleanup can run unlink on directories throwing an
error.
-----------------------------+------------------------------
 Reporter:  garrett-eclipse  |       Owner:  (none)
     Type:  defect (bug)     |      Status:  new
 Priority:  normal           |   Milestone:  Awaiting Review
Component:  Privacy          |     Version:  4.9.6
 Severity:  normal           |  Resolution:
 Keywords:  dev-feedback     |     Focuses:
-----------------------------+------------------------------

Comment (by garrett-eclipse):

 Relavent from the #51111 duplicate;
 > Background: #39975, #40856, #50664.

 > When running the test suite on Windows, occasionally I get some errors:
 {{{
 There were 4 errors:

 1)
 Tests_Privacy_WpPrivacyDeleteOldExportFiles::test_expired_files_should_be_deleted
 unlink(S:\home\wordpress.test\develop/build/wp-content/uploads/wp-
 personal-data-exports/test_contents/): Is a directory

 S:\home\wordpress.test\develop\build\wp-includes\functions.php:7324
 S:\home\wordpress.test\develop\tests\phpunit\tests\privacy\wpPrivacyDeleteOldExportFiles.php:120

 2)
 Tests_Privacy_WpPrivacyDeleteOldExportFiles::test_unexpired_files_should_not_be_deleted
 unlink(S:\home\wordpress.test\develop/build/wp-content/uploads/wp-
 personal-data-exports/test_contents/): Is a directory

 S:\home\wordpress.test\develop\build\wp-includes\functions.php:7324
 S:\home\wordpress.test\develop\tests\phpunit\tests\privacy\wpPrivacyDeleteOldExportFiles.php:131

 3)
 Tests_Privacy_WpPrivacyDeleteOldExportFiles::test_index_file_should_never_be_deleted
 unlink(S:\home\wordpress.test\develop/build/wp-content/uploads/wp-
 personal-data-exports/test_contents/): Is a directory

 S:\home\wordpress.test\develop\build\wp-includes\functions.php:7324
 S:\home\wordpress.test\develop\tests\phpunit\tests\privacy\wpPrivacyDeleteOldExportFiles.php:142

 4)
 Tests_Privacy_WpPrivacyDeleteOldExportFiles::test_filtered_expiration_time
 unlink(S:\home\wordpress.test\develop/build/wp-content/uploads/wp-
 personal-data-exports/test_contents/): Is a directory

 S:\home\wordpress.test\develop\build\wp-includes\functions.php:7324
 S:\home\wordpress.test\develop\tests\phpunit\tests\privacy\wpPrivacyDeleteOldExportFiles.php:155
 }}}

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/48265#comment:3>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform