Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Resources extension: TemporaryDirectory: Restore mint permissions before closing context to avoid AccessDeniedException #560

Open
jbduncan opened this issue Dec 12, 2021 · 0 comments

Comments

@jbduncan
Copy link
Contributor

Consider introducing this fix that the JUnit 5 team accepted for their own TempDir, to our own TemporaryDirectory class. It will allow tests that use our TemporaryDirectory to restore file permissions for temporary directories/files, so that they can be successfully removed after each test no matter what.

Note: we should consider applying this fix only for tests that use the local filesystem, as I have a feeling that other filesystems like jimfs (which we intend to support) might not support file permissions.

This issue is a follow-up to #348 | #491.

Relevant JUnit 5 issues:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant