Skip to content

Commit

Permalink
Clarify a test helper docstring
Browse files Browse the repository at this point in the history
The wording was ambiguous before, because fixing a PermissionError
could mean addressing it successfully at runtime (which is the
intended meaning in that docstring) but it could also mean fixing
a bug or test case related to such an error (not intended).
  • Loading branch information
EliahKagan committed Oct 9, 2023
1 parent f0e15e8 commit a9b05ec
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/test_util.py
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ def __repr__(self):

@contextlib.contextmanager
def _tmpdir_to_force_permission_error():
"""Context manager to test permission errors in situations where we do not fix them."""
"""Context manager to test permission errors in situations where they are not overcome."""
if sys.platform == "cygwin":
raise SkipTest("Cygwin can't set the permissions that make the test meaningful.")
if sys.version_info < (3, 8):
Expand Down

0 comments on commit a9b05ec

Please sign in to comment.