test suite now passes after OsPath conversion
authorJoey Hess <joeyh@joeyh.name>
Tue, 11 Feb 2025 16:37:09 +0000 (12:37 -0400)
committerJoey Hess <joeyh@joeyh.name>
Tue, 11 Feb 2025 16:44:09 +0000 (12:44 -0400)
commit5dbaaae299a6756b018a565260a79e95cacc473a
treeef1eca06b52479c86e75a15af9f278250ff6c551
parentc85d5a0dc8e97e56222f5f7ea1cc2744d4066ec9
test suite now passes after OsPath conversion

The test suite was failing because of a bug in the Database/* modules.
I had replaced doesPathExist with doesDirectoryExist, but it was
checking the database file.

I have audited commit f1ba21d698c908ad84c08bce24fbbc376190fe83 for
other changes to doesPathExist, and checked that doesDirectoryExist and
doesFileExist were used correctly.

The only change I found is in youtubeDl', where it used to return
directories that might have been created by youtube-dl. But it was
supposed to return media files, so changing it to use doesFileExist is
actually an improvement. Although only of theoretical benefit.

Note that it would actually be possible to keep using doesPathExist,
there is a version of that for OsPath as well. But the rest of these
changes seem safe.

Sponsored-by: Nicholas Golder-Manning
Database/ContentIdentifier.hs
Database/Export.hs
Database/Fsck.hs
Database/ImportFeed.hs
Database/Keys.hs
Database/RepoSize.hs