Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Commit a9b05ec

Browse filesBrowse files
EliahKaganEliahKagan
committed
Clarify a test helper docstring
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).
1 parent f0e15e8 commit a9b05ec
Copy full SHA for a9b05ec

File tree

1 file changed

+1
-1
lines changed
Filter options

1 file changed

+1
-1
lines changed

‎test/test_util.py

Copy file name to clipboardExpand all lines: test/test_util.py
+1-1Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ def __repr__(self):
5858

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

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.