aicis / aicis/fresco

Surface recursive test cleanup failures

Open
#440 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
143
Forks
61
PR merge metrics
No merged PRs in 30d

Description

The recursive test cleanup helpers currently print an `IOException` and terminate their file walk when a file cannot be visited or a directory traversal fails. Because the exception is swallowed, test setup/teardown can continue with partially removed storage and obscure the actual cleanup failure.

I would like to change both duplicated visitors to rethrow the supplied `IOException`, allowing the caller and test runner to report the failure normally.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.