tests leave behind unremovable file.

[gbaz]

After resolving #97 (i think), running the tests for directory leaves a file that cannot be removed except by root.

This caused the hackage builder to stall for roughly a week because it was unable to remove the following file:

/home/builder/builder-dir/build-cache/tmp-install/directory-1.3.9.0/dist/test-RemovePathForcibly.tmp/removePathForcibly.tmp/f

It would be good if the test suite were fixed to avoid this issue.

[Rufflewind]

Were the tests interrupted or did any of them fail? Under normal circumstances, RemovePathForcibly.hs is supposed to remove f successfully and there are assertions to verify the removal.

[gbaz]

they failed but the available logs don't explain why -- https://hackage.haskell.org/package/directory-1.3.9.0/reports/1

i do think its bad practice for a failing test suite to leave a file that needs to be fixed by hand.

[gbaz]

Reran on the builder and here are some lines in the log that seem suspicious:

*** LongPaths: exception: yeah_its_longyeah_its_longyeah_its_longyeah_its_longyeah_its_longyeah_its_longyeah_its_longyeah_its_long: removePathForcibly:openFdAt: permission denied (Permission denied)

*** RemovePathForcibly: exception: f: removePathForcibly:openFdAt: permission denied (Permission denied)

The failures appear to be precisely that removePathForcibly does not work!

[hasufell]

Do you have more information on the system that this runs on?

[gbaz]

Indeed, I can't remove the created file by hand as the builder user even with rm -f -- I need to change to superuser to do it.

[gbaz]

Linux hackage-docbuilder 5.10.0-14-amd64 #1 SMP Debian 5.10.113-1 (2022-04-29) x86_64 with a nonsuperuser account.