Skip to content

Commit e1660c7

Browse files
committed
Fix coverage of test file itself
1 parent fc1125b commit e1660c7

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

python_packages/jupyter_lsp/jupyter_lsp/session.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -214,7 +214,7 @@ async def stop_process(self, timeout: int = 5):
214214
return
215215

216216
if sys.platform.startswith("win32"): # pragma: no cover
217-
# On Windows Process.kill() is an alias to Process.terminate so we cannot
217+
# On Windows Process.kill() is an alias to Process.terminate() so we cannot
218218
# force the process to stop. if you know of a better way to handle this on
219219
# Windows please consider contributing
220220
self.log.warning(

python_packages/jupyter_lsp/jupyter_lsp/tests/test_session.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -136,11 +136,12 @@ async def test_stop(handlers, timeout):
136136

137137
ws_handler.on_close()
138138

139-
if platform.startswith("win32"):
139+
if platform.startswith("win32"): # pragma: no cover
140140
# currently we cannot forcefully terminate the process on windows, so we just
141141
# give it a little more extra time to finish on its own
142142
await asyncio.sleep(timeout + 10)
143-
else: # linux and darwin
143+
else: # pragma: no cover
144+
# linux and darwin
144145
await asyncio.sleep(timeout + 2)
145146

146147
assert exists_process_with_pid(process_pid) is False

0 commit comments

Comments
 (0)