diff options
author | toonn <toonn@toonn.io> | 2021-08-27 19:25:12 +0200 |
---|---|---|
committer | toonn <toonn@toonn.io> | 2021-08-27 19:27:57 +0200 |
commit | 40ddcf310ca8398000ab730e48042efe7b80847c (patch) | |
tree | a658d62f71e209170bb6b04c114032fa7977c045 /tests | |
parent | 53ccd22dbe37e0e42528e0c405a1a461514e0965 (diff) | |
download | ranger-40ddcf310ca8398000ab730e48042efe7b80847c.tar.gz |
tests: Pylint complains about open and literals
So let's use our new open23 to circumvent the lint. Updating the function to default to UTF-8. This is not the greatest solution but changing to `io.open` has performance repercussions and using `codecs.open` might change semantics so it would need thorough testing.
Diffstat (limited to 'tests')
-rwxr-xr-x | tests/manpage_completion_test.py | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/tests/manpage_completion_test.py b/tests/manpage_completion_test.py index f5e5c335..f6ce87de 100755 --- a/tests/manpage_completion_test.py +++ b/tests/manpage_completion_test.py @@ -6,6 +6,8 @@ import os.path import re import sys +from ranger.ext.open23 import open23 + # Add relevant ranger module to PATH... there surely is a better way to do this... sys.path.insert(0, os.path.join(os.path.dirname(__file__), '..')) @@ -26,14 +28,14 @@ def get_path_of_man_page(): def read_manpage(): path = get_path_of_man_page() - with open(path, 'r') as man_page: + with open23(path, 'r') as man_page: return man_page.read() def get_sections(): manpage = read_manpage() parts = manpage.split('=head1 ') - sections = dict() + sections = {} for part in parts: if '\n' in part: section_name, section_content = part.split('\n', 1) |