Skip to content

Commit 67f310e

Browse files
committed
Merge branch 'ab/test-cleanly-recreate-trash-directory'
Improve test framework around unwritable directories. * ab/test-cleanly-recreate-trash-directory: test-lib.sh: try to re-chmod & retry on failed trash removal
2 parents 97ab03b + 5d22e18 commit 67f310e

File tree

1 file changed

+13
-1
lines changed

1 file changed

+13
-1
lines changed

t/test-lib.sh

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1407,8 +1407,20 @@ HOME="$TRASH_DIRECTORY"
14071407
GNUPGHOME="$HOME/gnupg-home-not-used"
14081408
export HOME GNUPGHOME USER_HOME
14091409

1410+
# "rm -rf" existing trash directory, even if a previous run left it
1411+
# with bad permissions.
1412+
remove_trash_directory () {
1413+
dir="$1"
1414+
if ! rm -rf "$dir" 2>/dev/null
1415+
then
1416+
chmod -R u+rwx "$dir"
1417+
rm -rf "$dir"
1418+
fi
1419+
! test -d "$dir"
1420+
}
1421+
14101422
# Test repository
1411-
rm -fr "$TRASH_DIRECTORY" || {
1423+
remove_trash_directory "$TRASH_DIRECTORY" || {
14121424
GIT_EXIT_OK=t
14131425
echo >&5 "FATAL: Cannot prepare test area"
14141426
exit 1

0 commit comments

Comments
 (0)